Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] staging: media: zoran: fix trailing whitespaces
To:     Dan Carpenter <dan.carpenter@oracle.com>,
        Arthur Moraes do Lago <arthurmoraeslago@gmail.com>
Cc:     mchehab@kernel.org, gregkh@linuxfoundation.org,
        mjpeg-users@lists.sourceforge.net, linux-media@vger.kernel.org,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
        lkcamp@lists.libreplanetbr.org
References: <20190408043530.21594-1-arthurmoraeslago@gmail.com>
 <20190408065103.GA6095@kadam>
From:   Hans Verkuil <hverkuil@xs4all.nl>
Message-ID: <65b52cdf-740c-6bc5-ef95-781ddd15fd02@xs4all.nl>
Date:   Wed, 10 Apr 2019 12:44:04 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.5.1
MIME-Version: 1.0
In-Reply-To: <20190408065103.GA6095@kadam>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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@gmail.com>
>> ---
>>  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