* [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message
@ 2019-10-24 13:09 Geert Uytterhoeven
2019-10-24 16:07 ` Kieran Bingham
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2019-10-24 13:09 UTC (permalink / raw)
To: Kieran Bingham, Mauro Carvalho Chehab
Cc: Laurent Pinchart, linux-media, linux-renesas-soc, Geert Uytterhoeven
The official name is "R-Car M3-N", not "R-Car M3N".
Fixes: 4e8c120de9268fc2 ("media: fdp1: Support M3N and E3 platforms")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/media/platform/rcar_fdp1.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/rcar_fdp1.c b/drivers/media/platform/rcar_fdp1.c
index cb93a13e1777a53e..97bed45360f088d0 100644
--- a/drivers/media/platform/rcar_fdp1.c
+++ b/drivers/media/platform/rcar_fdp1.c
@@ -2369,7 +2369,7 @@ static int fdp1_probe(struct platform_device *pdev)
dprintk(fdp1, "FDP1 Version R-Car H3\n");
break;
case FD1_IP_M3N:
- dprintk(fdp1, "FDP1 Version R-Car M3N\n");
+ dprintk(fdp1, "FDP1 Version R-Car M3-N\n");
break;
case FD1_IP_E3:
dprintk(fdp1, "FDP1 Version R-Car E3\n");
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message
2019-10-24 13:09 [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message Geert Uytterhoeven
@ 2019-10-24 16:07 ` Kieran Bingham
2019-10-25 8:46 ` Geert Uytterhoeven
0 siblings, 1 reply; 4+ messages in thread
From: Kieran Bingham @ 2019-10-24 16:07 UTC (permalink / raw)
To: Geert Uytterhoeven, Mauro Carvalho Chehab
Cc: Laurent Pinchart, linux-media, linux-renesas-soc
Hi Geert,
On 24/10/2019 14:09, Geert Uytterhoeven wrote:
> The official name is "R-Car M3-N", not "R-Car M3N".
>
> Fixes: 4e8c120de9268fc2 ("media: fdp1: Support M3N and E3 platforms")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> drivers/media/platform/rcar_fdp1.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/rcar_fdp1.c b/drivers/media/platform/rcar_fdp1.c
> index cb93a13e1777a53e..97bed45360f088d0 100644
> --- a/drivers/media/platform/rcar_fdp1.c
> +++ b/drivers/media/platform/rcar_fdp1.c
> @@ -2369,7 +2369,7 @@ static int fdp1_probe(struct platform_device *pdev)
> dprintk(fdp1, "FDP1 Version R-Car H3\n");
> break;
> case FD1_IP_M3N:
Should the FD1_IP_M3N naming also be updated accordingly?
I guess that's not so important.
Either way, up to you.
Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
> - dprintk(fdp1, "FDP1 Version R-Car M3N\n");
> + dprintk(fdp1, "FDP1 Version R-Car M3-N\n");
> break;
> case FD1_IP_E3:
> dprintk(fdp1, "FDP1 Version R-Car E3\n");
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message
2019-10-24 16:07 ` Kieran Bingham
@ 2019-10-25 8:46 ` Geert Uytterhoeven
2019-10-25 10:20 ` Kieran Bingham
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2019-10-25 8:46 UTC (permalink / raw)
To: Kieran Bingham
Cc: Geert Uytterhoeven, Mauro Carvalho Chehab, Laurent Pinchart,
Linux Media Mailing List, Linux-Renesas
Hi Kieran,
On Thu, Oct 24, 2019 at 6:08 PM Kieran Bingham
<kieran.bingham+renesas@ideasonboard.com> wrote:
> On 24/10/2019 14:09, Geert Uytterhoeven wrote:
> > The official name is "R-Car M3-N", not "R-Car M3N".
> >
> > Fixes: 4e8c120de9268fc2 ("media: fdp1: Support M3N and E3 platforms")
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > ---
> > drivers/media/platform/rcar_fdp1.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/media/platform/rcar_fdp1.c b/drivers/media/platform/rcar_fdp1.c
> > index cb93a13e1777a53e..97bed45360f088d0 100644
> > --- a/drivers/media/platform/rcar_fdp1.c
> > +++ b/drivers/media/platform/rcar_fdp1.c
> > @@ -2369,7 +2369,7 @@ static int fdp1_probe(struct platform_device *pdev)
> > dprintk(fdp1, "FDP1 Version R-Car H3\n");
> > break;
> > case FD1_IP_M3N:
>
> Should the FD1_IP_M3N naming also be updated accordingly?
> I guess that's not so important.
>
> Either way, up to you.
Na, you can't use hyphens in preprocessor definitions ;-)
FD1_IP_M3N is consistent with FD1_IP_M3W, and not user-visible.
> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message
2019-10-25 8:46 ` Geert Uytterhoeven
@ 2019-10-25 10:20 ` Kieran Bingham
0 siblings, 0 replies; 4+ messages in thread
From: Kieran Bingham @ 2019-10-25 10:20 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Geert Uytterhoeven, Mauro Carvalho Chehab, Laurent Pinchart,
Linux Media Mailing List, Linux-Renesas
Hi Geert,
On 25/10/2019 09:46, Geert Uytterhoeven wrote:
> Hi Kieran,
>
> On Thu, Oct 24, 2019 at 6:08 PM Kieran Bingham
> <kieran.bingham+renesas@ideasonboard.com> wrote:
>> On 24/10/2019 14:09, Geert Uytterhoeven wrote:
>>> The official name is "R-Car M3-N", not "R-Car M3N".
>>>
>>> Fixes: 4e8c120de9268fc2 ("media: fdp1: Support M3N and E3 platforms")
>>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>>> ---
>>> drivers/media/platform/rcar_fdp1.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/media/platform/rcar_fdp1.c b/drivers/media/platform/rcar_fdp1.c
>>> index cb93a13e1777a53e..97bed45360f088d0 100644
>>> --- a/drivers/media/platform/rcar_fdp1.c
>>> +++ b/drivers/media/platform/rcar_fdp1.c
>>> @@ -2369,7 +2369,7 @@ static int fdp1_probe(struct platform_device *pdev)
>>> dprintk(fdp1, "FDP1 Version R-Car H3\n");
>>> break;
>>> case FD1_IP_M3N:
>>
>> Should the FD1_IP_M3N naming also be updated accordingly?
>> I guess that's not so important.
>>
>> Either way, up to you.
>
> Na, you can't use hyphens in preprocessor definitions ;-)> FD1_IP_M3N is consistent with FD1_IP_M3W, and not user-visible.
Hahaha - of course :-D , I meant with M3_N but as we already have IP_M3W
I think we're fine.
>> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
>
> Thanks!
No worries
--
KB
>
> Gr{oetje,eeting}s,
>
> Geert
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-25 10:20 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-24 13:09 [PATCH] media: fdp1: Fix R-Car M3-N naming in debug message Geert Uytterhoeven
2019-10-24 16:07 ` Kieran Bingham
2019-10-25 8:46 ` Geert Uytterhoeven
2019-10-25 10:20 ` Kieran Bingham
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.