From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Fri, 29 Dec 2017 18:43:00 +0000 Subject: Re: [0/5] video-SMSC UFX: Adjustments for five function implementations Message-Id: <265c7022-d12f-bcbe-c6f0-452ba3f73744@users.sourceforge.net> List-Id: References: <1736749.UIs06O4xc5@amdc3058> In-Reply-To: <1736749.UIs06O4xc5@amdc3058> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Bartlomiej Zolnierkiewicz , linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Cc: Steve Glendinning , LKML , kernel-janitors@vger.kernel.org >> Delete an error message for a failed memory allocation in two functions > > This patch removes the information about the device for which the allocation fails. Should a Linux allocation failure report be sufficient here? >> Less checks in ufx_usb_probe() after error detection > > This patch depends on the earlier patch (which is not being merged) Partly, yes. > so please re-base it if you want it to be applied. Would you dare to pick another change possibility up from a patch hunk? >> Adjust three checks for null pointers > > This patch contains unrelated change ({} braces addition) which should be dropped. I presented such an update approach so that warnings from the script “checkpatch.pl” will be fixed together. Regards, Markus