From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH v3 1/3] video: smscufx: Less checks in ufx_usb_probe() after error detection Date: Wed, 28 Mar 2018 15:46:32 +0200 Message-ID: <1923231.ySCFlVkifY@amdc3058> References: <03be4276-520e-1d22-1c6e-e35c8df2b1e8@users.sourceforge.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-reply-to: <03be4276-520e-1d22-1c6e-e35c8df2b1e8@users.sourceforge.net> Sender: linux-kernel-owner@vger.kernel.org To: SF Markus Elfring Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, Steve Glendinning , LKML , kernel-janitors@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org On Sunday, January 07, 2018 05:02:21 PM SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 7 Jan 2018 15:56:11 +0100 > > Up to four checks could be repeated by the ufx_usb_probe() function > during error handling even if the relevant properties can be determined > for the involved variables before by source code analysis. > > * Return directly after a call of the function "kzalloc" failed > at the beginning. > > * Adjust jump targets so that extra checks can be omitted at the end. > > * Delete initialisations for the variables "info" and "retval" > which became unnecessary with this refactoring. > > Signed-off-by: Markus Elfring Patch queued for 4.17, thanks. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics