On 26/04/16 15:47, Parth Sane wrote: > 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 Sure. It isn't a big issue, just thought it was worth mentioning as a tip. Please keep submitting fixes. Appreciated. Luis >> On 26-Apr-2016, at 8:14 PM, Luis de Bethencourt <luisbg@xxxxxxxxxxxxxxx> 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 <luisbg@xxxxxxxxxxxxxxx> wrote: >>>> >>>> On 25/04/16 16:43, Parth Sane wrote: >>>>> Fixed alignment to match open parenthesis. >>>>> >>>>> Signed-off-by: Parth Sane <laerdevstudios@xxxxxxxxx> >>>>> >>>>> --- >>>>> 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 >>>> >>> > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
