[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Z-6McoFVsJ8EiLPV@smile.fi.intel.com>
Date: Thu, 3 Apr 2025 16:26:10 +0300
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: Dan Carpenter <dan.carpenter@...aro.org>
Cc: Gabriel Shahrouzi <gshahrouzi@...il.com>, linux-media@...r.kernel.org,
hdegoede@...hat.com, mchehab@...nel.org,
sakari.ailus@...ux.intel.com, gregkh@...uxfoundation.org,
linux-kernel@...r.kernel.org, linux-staging@...ts.linux.dev
Subject: Re: [PATCH v2] staging: media: Fix indentation to use tabs instead
of spaces
On Thu, Apr 03, 2025 at 03:36:27PM +0300, Dan Carpenter wrote:
> On Wed, Apr 02, 2025 at 05:36:31PM +0300, Andy Shevchenko wrote:
> > On Wed, Apr 2, 2025 at 4:50 PM Gabriel Shahrouzi <gshahrouzi@...il.com> wrote:
> > >
> > > Replace spaces with tab to comply with kernel coding style.
> >
> > Still 'tab'. please, capitalize it to be 'TAB'.
> >
> > But wait a bit, the driver maintainer might do that for you whilst applying.
>
> It's not an acronym so it shouldn't be capitalized. It should be either
> "a tab" or "tabs".
Not everything that is capitalised is an acronym :-)
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists