RE: [PATCH v3 2/4] drm: rcar-du: Fix typo
Hi Laurent, > Subject: Re: [PATCH v3 2/4] drm: rcar-du: Fix typo > > Hello, > > On Fri, Apr 22, 2022 at 09:38:00AM +, Biju Das wrote: > > > Subject: Re: [PATCH v3 2/4] drm: rcar-du: Fix typo On Thu, Apr 21, > > > 2022 at 6:31 PM Biju Das wrote: > > > > Fix typo rcar_du_vsp.h->rcar_du_vsp.c > > > > > > > > Signed-off-by: Biju Das > > > > > > Thanks for your patch! > > > > > > > --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > > > +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > > > @@ -1,6 +1,6 @@ > > > > // SPDX-License-Identifier: GPL-2.0+ > > > > /* > > > > - * rcar_du_vsp.h -- R-Car Display Unit VSP-Based Compositor > > > > + * rcar_du_vsp.c -- R-Car Display Unit VSP-Based Compositor > > > > > > Perhaps drop the file name completely instead? > > > > Currently all the R-Car DU files have file name. May be A single patch > > to remove all off them if we are planning to do. > > > > Laurent, please share your thoughts on this. > > Geert has a point, the file names cause issues and don't add much value. > Would you like to send a patch to drop them all, to replace this one ? I > can also handle it myself if you prefer. OK, I don't have any preference. I Just noticed that issue. Feel free to post the patch. Cheers, Biju
Re: [PATCH v3 2/4] drm: rcar-du: Fix typo
Hello, On Fri, Apr 22, 2022 at 09:38:00AM +, Biju Das wrote: > > Subject: Re: [PATCH v3 2/4] drm: rcar-du: Fix typo > > On Thu, Apr 21, 2022 at 6:31 PM Biju Das wrote: > > > Fix typo rcar_du_vsp.h->rcar_du_vsp.c > > > > > > Signed-off-by: Biju Das > > > > Thanks for your patch! > > > > > --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > > +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > > @@ -1,6 +1,6 @@ > > > // SPDX-License-Identifier: GPL-2.0+ > > > /* > > > - * rcar_du_vsp.h -- R-Car Display Unit VSP-Based Compositor > > > + * rcar_du_vsp.c -- R-Car Display Unit VSP-Based Compositor > > > > Perhaps drop the file name completely instead? > > Currently all the R-Car DU files have file name. May be > A single patch to remove all off them if we are planning to do. > > Laurent, please share your thoughts on this. Geert has a point, the file names cause issues and don't add much value. Would you like to send a patch to drop them all, to replace this one ? I can also handle it myself if you prefer. -- Regards, Laurent Pinchart
RE: [PATCH v3 2/4] drm: rcar-du: Fix typo
Hi Geert, Thanks for the feedback. > Subject: Re: [PATCH v3 2/4] drm: rcar-du: Fix typo > > Hi Biju, > > On Thu, Apr 21, 2022 at 6:31 PM Biju Das > wrote: > > Fix typo rcar_du_vsp.h->rcar_du_vsp.c > > > > Signed-off-by: Biju Das > > Thanks for your patch! > > > --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c > > @@ -1,6 +1,6 @@ > > // SPDX-License-Identifier: GPL-2.0+ > > /* > > - * rcar_du_vsp.h -- R-Car Display Unit VSP-Based Compositor > > + * rcar_du_vsp.c -- R-Car Display Unit VSP-Based Compositor > > Perhaps drop the file name completely instead? Currently all the R-Car DU files have file name. May be A single patch to remove all off them if we are planning to do. Laurent, please share your thoughts on this. Cheers, Biju
Re: [PATCH v3 2/4] drm: rcar-du: Fix typo
Hi Biju,
On Thu, Apr 21, 2022 at 6:31 PM Biju Das wrote:
> Fix typo rcar_du_vsp.h->rcar_du_vsp.c
>
> Signed-off-by: Biju Das
Thanks for your patch!
> --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
> +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c
> @@ -1,6 +1,6 @@
> // SPDX-License-Identifier: GPL-2.0+
> /*
> - * rcar_du_vsp.h -- R-Car Display Unit VSP-Based Compositor
> + * rcar_du_vsp.c -- R-Car Display Unit VSP-Based Compositor
Perhaps drop the file name completely instead?
> *
> * Copyright (C) 2015 Renesas Electronics Corporation
> *
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
[PATCH v3 2/4] drm: rcar-du: Fix typo
Fix typo rcar_du_vsp.h->rcar_du_vsp.c Signed-off-by: Biju Das --- v3: * New patch --- drivers/gpu/drm/rcar-du/rcar_du_vsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c index 8eb9b2b097ae..9c4d1d1be1d4 100644 --- a/drivers/gpu/drm/rcar-du/rcar_du_vsp.c +++ b/drivers/gpu/drm/rcar-du/rcar_du_vsp.c @@ -1,6 +1,6 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * rcar_du_vsp.h -- R-Car Display Unit VSP-Based Compositor + * rcar_du_vsp.c -- R-Car Display Unit VSP-Based Compositor * * Copyright (C) 2015 Renesas Electronics Corporation * -- 2.25.1
