Re: [PATCH v2 2/8] staging: greybus: audio_codec.c: Fix alignment should match open parenthesis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 17, 2018 at 3:42 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> On Wed, Jan 17, 2018 at 03:29:33PM +0200, Kamal Heib wrote:
>> Cleanup "Alignment should match open parenthesis" checkpatch.pl errors.
>>
>> Signed-off-by: Kamal Heib <kamalheib1@xxxxxxxxx>
>> ---
>>  drivers/staging/greybus/audio_codec.c | 14 +++++++-------
>>  1 file changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/staging/greybus/audio_codec.c b/drivers/staging/greybus/audio_codec.c
>> index fdb9e83cc34b..0dae31f39a2b 100644
>> --- a/drivers/staging/greybus/audio_codec.c
>> +++ b/drivers/staging/greybus/audio_codec.c
>> @@ -70,7 +70,7 @@ static int gbaudio_module_enable_tx(struct gbaudio_codec_info *codec,
>>               i2s_port = 0;   /* fixed for now */
>>               cportid = data->connection->hd_cport_id;
>>               ret = gb_audio_apbridgea_register_cport(data->connection,
>> -                                             i2s_port, cportid,
>> +                                                     i2s_port, cportid,
>>                                               AUDIO_APBRIDGEA_DIRECTION_TX);
>
> Now it doesn't line up with the next line.  The original was better.
>
> regards,
> dan carpenter
>

Thanks dan for you review, I'll send v3 soon.

Thanks,
Kamal
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux