Re: [PATCH v2] staging/bcm2835-audio: Fix alignment of function arguments - Style

From: Greg Gallagner
Date: Wed Aug 08 2018 - 08:54:56 EST


On 08/08/2018 08:26 AM, Greg Kroah-Hartman wrote:
> On Tue, Aug 07, 2018 at 09:40:47PM -0400, Greg Gallagher wrote:
>> Fix warnings generated by checkpatch.pl for alignment should match open
>> parenthesis. This is a coding style change only.
>>
>> Signed-off-by: Greg Gallagher <greg@xxxxxxxxxxxxxxxx>
>> ---
>> Changes in v2:
>> - Clean up all the alignment warnings in the file
>> ---
>> .../staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 18 +++++++++---------
>> 1 file changed, 9 insertions(+), 9 deletions(-)
> You sent 2 patches that did the same thing, with different subjects :(
>
> Please fix up and send only the "correct" one.
>
> greg k-h
Sorry, the first one was a mistake and I should have sent something saying
to ignore it. I will double check the second one is correct. Is there
a best
practice when I'd like a patch ignored?

thanks

Greg