* [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).