Messages in this thread | | | Date | Fri, 11 Dec 2015 11:30:17 +0100 | From | Johan Hovold <> | Subject | Re: [PATCH 1/2] usb: serial: remove redundant conditions |
| |
On Fri, Dec 11, 2015 at 07:25:51AM -0300, Geyslan G. Bem wrote: > 2015-12-11 7:13 GMT-03:00 Johan Hovold <johan@kernel.org>: > > On Fri, Dec 11, 2015 at 06:46:41AM -0300, Geyslan G. Bem wrote: > >> This patch removes redundant conditions. > >> > >> (!A || (A && B)) is the same as (!A || B). > >> > >> Tested by compilation only. > >> Caught by cppcheck. > >> > >> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com> > > > > You forgot to update the commit summary (to include the driver name) > > when splitting the original patch. I fixed that up and applied both for > > next. > Sorry. I got it: USB: mos7840: and USB: io_edgeport:. > > I'm still getting the hang of it. :-)
Looks like you got it. :)
> Thank you for apply them.
Thanks again for the patches.
Johan
| |