linux-staging.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces
@ 2025-04-02 13:50 Gabriel Shahrouzi
  2025-04-02 14:36 ` Andy Shevchenko
  0 siblings, 1 reply; 5+ messages in thread
From: Gabriel Shahrouzi @ 2025-04-02 13:50 UTC (permalink / raw)
  To: linux-media
  Cc: andy, hdegoede, mchehab, sakari.ailus, gregkh, linux-kernel,
	linux-staging, gshahrouzi

Replace spaces with tab to comply with kernel coding style.

Signed-off-by: Gabriel Shahrouzi <gshahrouzi@gmail.com>
---
Changes in v2:
	- Resend using git send-email to fix formatting issues in email body.
---
 .../media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.host.c    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.host.c
index ece5e3da34ee..127f12ba2214 100644
--- a/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.host.c
+++ b/drivers/staging/media/atomisp/pci/isp/kernels/vf/vf_1.0/ia_css_vf.host.c
@@ -114,7 +114,7 @@ configure_dma(
 }
 
 int ia_css_vf_configure(const struct ia_css_binary *binary,
-		        const struct ia_css_frame_info *out_info,
+			const struct ia_css_frame_info *out_info,
 			struct ia_css_frame_info *vf_info,
 			unsigned int *downscale_log2)
 {
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces
  2025-04-02 13:50 [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces Gabriel Shahrouzi
@ 2025-04-02 14:36 ` Andy Shevchenko
  2025-04-03  0:20   ` Gabriel
  2025-04-03 12:36   ` Dan Carpenter
  0 siblings, 2 replies; 5+ messages in thread
From: Andy Shevchenko @ 2025-04-02 14:36 UTC (permalink / raw)
  To: Gabriel Shahrouzi
  Cc: linux-media, andy, hdegoede, mchehab, sakari.ailus, gregkh,
	linux-kernel, linux-staging

On Wed, Apr 2, 2025 at 4:50 PM Gabriel Shahrouzi <gshahrouzi@gmail.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.

-- 
With Best Regards,
Andy Shevchenko

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces
  2025-04-02 14:36 ` Andy Shevchenko
@ 2025-04-03  0:20   ` Gabriel
  2025-04-03 12:36   ` Dan Carpenter
  1 sibling, 0 replies; 5+ messages in thread
From: Gabriel @ 2025-04-03  0:20 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: linux-media, andy, hdegoede, mchehab, sakari.ailus, gregkh,
	linux-kernel, linux-staging

On Wed, Apr 2, 2025 at 10:37 AM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Wed, Apr 2, 2025 at 4:50 PM Gabriel Shahrouzi <gshahrouzi@gmail.com> wrote:
> >
> > Replace spaces with tab to comply with kernel coding style.
>
> Still 'tab'. please, capitalize it to be 'TAB'.
Whoops.Completely forgot about this one. I'll have to reread the
thread or write it down so I don't forgot in the future.
>
> But wait a bit, the driver maintainer might do that for you whilst applying.
Sounds good.
>
> --
> With Best Regards,
> Andy Shevchenko

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces
  2025-04-02 14:36 ` Andy Shevchenko
  2025-04-03  0:20   ` Gabriel
@ 2025-04-03 12:36   ` Dan Carpenter
  2025-04-03 13:26     ` Andy Shevchenko
  1 sibling, 1 reply; 5+ messages in thread
From: Dan Carpenter @ 2025-04-03 12:36 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Gabriel Shahrouzi, linux-media, andy, hdegoede, mchehab,
	sakari.ailus, gregkh, linux-kernel, linux-staging

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

regards,
dan carpenter


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces
  2025-04-03 12:36   ` Dan Carpenter
@ 2025-04-03 13:26     ` Andy Shevchenko
  0 siblings, 0 replies; 5+ messages in thread
From: Andy Shevchenko @ 2025-04-03 13:26 UTC (permalink / raw)
  To: Dan Carpenter
  Cc: Gabriel Shahrouzi, linux-media, hdegoede, mchehab, sakari.ailus,
	gregkh, linux-kernel, linux-staging

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



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-04-03 13:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-02 13:50 [PATCH v2] staging: media: Fix indentation to use tabs instead of spaces Gabriel Shahrouzi
2025-04-02 14:36 ` Andy Shevchenko
2025-04-03  0:20   ` Gabriel
2025-04-03 12:36   ` Dan Carpenter
2025-04-03 13:26     ` Andy Shevchenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).