Re: [PATCH] media: atomisp: coding style: Move trailing statements to next line

From: Sakari Ailus

Date: Wed Apr 22 2026 - 08:56:35 EST


On Mon, Apr 20, 2026 at 09:34:06AM +0300, Andy Shevchenko wrote:
> On Sun, Apr 19, 2026 at 3:46 PM Sakari Ailus <sakari.ailus@xxxxxx> wrote:
> > On Sun, Apr 19, 2026 at 11:32:45AM +0300, Andy Shevchenko wrote:
> > > On Sun, Apr 19, 2026 at 1:20 AM Sakari Ailus <sakari.ailus@xxxxxx> wrote:
> > > > On Wed, Nov 26, 2025 at 07:00:26PM +0000, Ayush Kumar wrote:
> > > > > Adhering to Linux kernel coding style guidelines (Chapter 3: Indentation).
> > > > >
> > > > > Signed-off-by: Ayush Kumar <ayushkr0s@xxxxxxxxx>
> > > >
> > > > This has been addressed by commit 003c2c39ec77c84d704ee8effe170fd8658fde0b
> > > > in my atomisp branch.
> > >
> > > Btw, can you rebase your branch and drop "staging" word from the
> > > Subject in all current patches?
> >
> > Good question. The staging: prefix has been used in the past for atomisp
> > patches and a lot of the rest appears to be doing that as well but
> > definitely not all. I don't really have an opinion but we should do that
> > consistently whichever is chosen.
>
> I believe you were Cc'ed to the (private) email response from Hans
> where he explained why it should be dropped. Let me find if I still
> have it and bounce to you.

AFAIU, those reasons haven't been applicable for a few years now.

Most patches to the staging tree have staging: prefix and "staging: media:
atomisp: " appears to be most common for atomisp based on a quick look.

--
Sakari Ailus