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

From: Daniel Watson
Date: Tue Mar 21 2023 - 14:30:43 EST


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!