Re: [PATCH] media: av7110: put spaces around operators

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

 



On Tue, Mar 21, 2023 at 04:08:30PM +0100, Hans Verkuil wrote:
> Hi Daniel,
> 
Hi!

> I'm not sure what the point it of these two fixes since this driver is
> full of such things.
these fixes were the only ones where checkpatch said "spaces required
around that ..." for this file.

> In any case, I'm rejecting this. Just leave this driver alone, it's not
> worth the effort.
ok.

> If you are doing this as part of some mentorship program, then skip the
i'm doing this on my own.  i read that newbies should do about 10 minor
patches to get familiar with the process before doing something more
substantial.

> drivers/staging/media directory, it's not suitable for that.
ok, i'll try other drivers in staging.  i'll also see about getting a
mentor.

thanks again!




[Index of Archives]     [Linux Driver Development]     [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