Re: [PATCH] clocksource/drivers/renesas-ostm: Add support for RZ/V2L SoC

[Daniel Lezcano]

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@xxxxxxxxxx> 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@xxxxxxxxxxxxxx>
> > > ---
> > >    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


--
<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