Re: [PATCH] staging: media: zoran: fix trailing whitespaces

From: Hans Verkuil
Date: Wed Apr 10 2019 - 06:44:09 EST


On 4/8/19 8:51 AM, Dan Carpenter wrote:
> On Mon, Apr 08, 2019 at 01:35:30AM -0300, Arthur Moraes do Lago wrote:
>> There were several form feeds(^L) in the file, this patch removes them
>> to conform with the coding style.
>>
>> Signed-off-by: Arthur Moraes do Lago <arthurmoraeslago@xxxxxxxxx>
>> ---
>> drivers/staging/media/zoran/videocodec.h | 10 +++++-----
>> 1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/staging/media/zoran/videocodec.h b/drivers/staging/media/zoran/videocodec.h
>> index 4946791fce0d..c37c0e4fb624 100644
>> --- a/drivers/staging/media/zoran/videocodec.h
>> +++ b/drivers/staging/media/zoran/videocodec.h
>> @@ -49,7 +49,7 @@
>> device dependent and vary between MJPEG/MPEG/WAVELET/... devices. (!!!!)
>> ----------------------------------------------------------------------------
>> */
>> -
>> +
>>
>
> Checkpatch also complains if you have two consecutive blank lines so
> really we should just delete the lines entirely... But this driver is
> scheduled for deletion in May. Let's not bother with cleaning it up.
>
> regards,
> dan carpenter
>

I will in fact post a pull request to remove this driver today.

Regards,

Hans