| Message ID | 20220406072417.14185-1-prabhakar.mahadev-lad.rj@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | clocksource/drivers/renesas-ostm: Add support for RZ/V2L SoC | expand |
On 06/04/2022 09:24, Lad Prabhakar wrote: > The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead > of adding dependency for each SoC's add dependency on ARCH_RZG2L. The > ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and > ARCH_R9A07G054. > > With the above change OSTM will be enabled on RZ/V2L SoC. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > --- > drivers/clocksource/renesas-ostm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clocksource/renesas-ostm.c b/drivers/clocksource/renesas-ostm.c > index 21d1392637b8..8da972dc1713 100644 > --- a/drivers/clocksource/renesas-ostm.c > +++ b/drivers/clocksource/renesas-ostm.c > @@ -224,7 +224,7 @@ static int __init ostm_init(struct device_node *np) > > TIMER_OF_DECLARE(ostm, "renesas,ostm", ostm_init); > > -#ifdef CONFIG_ARCH_R9A07G044 > +#ifdef CONFIG_ARCH_RZG2L I'm not finding this option anywhere > static int __init ostm_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev;
Hi Daniel, On Wed, Apr 6, 2022 at 9:03 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > On 06/04/2022 09:24, Lad Prabhakar wrote: > > The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead > > of adding dependency for each SoC's add dependency on ARCH_RZG2L. The > > ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and > > ARCH_R9A07G054. > > > > With the above change OSTM will be enabled on RZ/V2L SoC. > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > > --- > > drivers/clocksource/renesas-ostm.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/clocksource/renesas-ostm.c b/drivers/clocksource/renesas-ostm.c > > index 21d1392637b8..8da972dc1713 100644 > > --- a/drivers/clocksource/renesas-ostm.c > > +++ b/drivers/clocksource/renesas-ostm.c > > @@ -224,7 +224,7 @@ static int __init ostm_init(struct device_node *np) > > > > TIMER_OF_DECLARE(ostm, "renesas,ostm", ostm_init); > > > > -#ifdef CONFIG_ARCH_R9A07G044 > > +#ifdef CONFIG_ARCH_RZG2L > > I'm not finding this option anywhere > ARCH_RZG2L config option is part of v5.18-rc1 [0] [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/soc/renesas/Kconfig?h=v5.18-rc1#n43 Cheers, Prabhakar > > > static int __init ostm_probe(struct platform_device *pdev) > > { > > struct device *dev = &pdev->dev; > > > -- > <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs > > Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | > <http://twitter.com/#!/linaroorg> Twitter | > <http://www.linaro.org/linaro-blog/> Blog
Hi Lad, On 06/04/2022 10:16, Lad, Prabhakar wrote: > Hi Daniel, > > On Wed, Apr 6, 2022 at 9:03 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: >> >> On 06/04/2022 09:24, Lad Prabhakar wrote: >>> The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead >>> of adding dependency for each SoC's add dependency on ARCH_RZG2L. The >>> ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and >>> ARCH_R9A07G054. >>> >>> With the above change OSTM will be enabled on RZ/V2L SoC. >>> >>> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> >>> --- >>> drivers/clocksource/renesas-ostm.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/clocksource/renesas-ostm.c b/drivers/clocksource/renesas-ostm.c >>> index 21d1392637b8..8da972dc1713 100644 >>> --- a/drivers/clocksource/renesas-ostm.c >>> +++ b/drivers/clocksource/renesas-ostm.c >>> @@ -224,7 +224,7 @@ static int __init ostm_init(struct device_node *np) >>> >>> TIMER_OF_DECLARE(ostm, "renesas,ostm", ostm_init); >>> >>> -#ifdef CONFIG_ARCH_R9A07G044 >>> +#ifdef CONFIG_ARCH_RZG2L >> >> I'm not finding this option anywhere >> > ARCH_RZG2L config option is part of v5.18-rc1 [0] > > [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/soc/renesas/Kconfig?h=v5.18-rc1#n43 Ok, thanks -- Daniel > Cheers, > Prabhakar >> >>> static int __init ostm_probe(struct platform_device *pdev) >>> { >>> struct device *dev = &pdev->dev; >> >> >> -- >> <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs >> >> Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | >> <http://twitter.com/#!/linaroorg> Twitter | >> <http://www.linaro.org/linaro-blog/> Blog
On Wed, Apr 6, 2022 at 9:31 AM Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> wrote: > The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead > of adding dependency for each SoC's add dependency on ARCH_RZG2L. The > ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and > ARCH_R9A07G054. > > With the above change OSTM will be enabled on RZ/V2L SoC. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> 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
Hi Daniel, On Wed, Apr 6, 2022 at 8:24 AM Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> wrote: > > The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead > of adding dependency for each SoC's add dependency on ARCH_RZG2L. The > ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and > ARCH_R9A07G054. > > With the above change OSTM will be enabled on RZ/V2L SoC. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> > --- > drivers/clocksource/renesas-ostm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > This patch has not been merged yet. Do you want me to resend it? Cheers, Prabhakar
diff --git a/drivers/clocksource/renesas-ostm.c b/drivers/clocksource/renesas-ostm.c index 21d1392637b8..8da972dc1713 100644 --- a/drivers/clocksource/renesas-ostm.c +++ b/drivers/clocksource/renesas-ostm.c @@ -224,7 +224,7 @@ static int __init ostm_init(struct device_node *np) TIMER_OF_DECLARE(ostm, "renesas,ostm", ostm_init); -#ifdef CONFIG_ARCH_R9A07G044 +#ifdef CONFIG_ARCH_RZG2L static int __init ostm_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev;
The OSTM block is identical on Renesas RZ/G2L and RZ/V2L SoC's, so instead of adding dependency for each SoC's add dependency on ARCH_RZG2L. The ARCH_RZG2L config option is already selected by ARCH_R9A07G044 and ARCH_R9A07G054. With the above change OSTM will be enabled on RZ/V2L SoC. Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> --- drivers/clocksource/renesas-ostm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)