Hi, Yes you are right on that regard. I did do that in my previous patches. It’s just something that I will have to remember now onwards. Thanks. Regards, Parth Sane > On 26-Apr-2016, at 8:14 PM, Luis de Bethencourt <lui...@osg.samsung.com> > wrote: > > On 26/04/16 15:33, Parth Sane wrote: >> Hi, >> Thanks for the feedback. I did find this issue with the assistance >> checkpatch.pl script. >> Regards, >> Parth Sane > > Normally people mention the checkpatch warning they are fixing. > > For example: > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=5f10ef7dce02d8e36665974aca0977cda58c1122 > > Verbosity in the commit message is a good thing :) > > I have tested that your patch still applies cleanly to Greg's staging-testing > branch. > > Thanks, > Luis > >>> On 26-Apr-2016, at 8:00 PM, Luis de Bethencourt <lui...@osg.samsung.com> >>> wrote: >>> >>> On 25/04/16 16:43, Parth Sane wrote: >>>> Fixed alignment to match open parenthesis. >>>> >>>> Signed-off-by: Parth Sane <laerdevstud...@gmail.com> >>>> >>>> --- >>>> Changes in v6: >>>> -Added line before Signed-off message >>>> >>> >>> This last version looks good to me. Did you find this issue with >>> checkpatch.pl? >>> >>> Thanks, >>> Luis >>> >>
signature.asc
Description: Message signed with OpenPGP using GPGMail
