lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20170918114423.ewtuei2dpr7wcmdx@valkosipuli.retiisi.org.uk> Date: Mon, 18 Sep 2017 14:44:24 +0300 From: Sakari Ailus <sakari.ailus@....fi> To: Srishti Sharma <srishtishar@...il.com> Cc: mchehab@...nel.org, gregkh@...uxfoundation.org, linux-media@...r.kernel.org, devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org, outreachy-kernel@...glegroups.com Subject: Re: [PATCH] Staging: media: atomisp: Merge assignment with return On Tue, Sep 12, 2017 at 07:55:07PM +0530, Srishti Sharma wrote: > Merge the assignment and the return statements to return the value > directly. Done using the following semantic patch by coccinelle. > > @@ > local idexpression ret; > expression e; > @@ > > -ret = > +return > e; > -return ret; > > Signed-off-by: Srishti Sharma <srishtishar@...il.com> Hi Srishti, I've merged the two patches as they're trivial and the commit message is the same. It's entirely reasonable to have a patch per driver but you should mention that in the commit message (subject line) at least. This case is a bit special because the other driver is also specific to the atomisp staging driver. Thanks. -- Sakari Ailus e-mail: sakari.ailus@....fi
Powered by blists - more mailing lists