Re: [PATCH v2 5/5] soc: renesas: Kconfig: Select POWER_SEQUENCING_RENESAS_PWRRDY for RZ/V2H and RZ/V2N

[Lad, Prabhakar]

Hi Bartosz,

Thank you for the review.

On Thu, Oct 15, 2026 at 10:45 AM Bartosz Golaszewski <brgl@xxxxxxxxxx> wrote:
>
> On Fri, 9 Oct 2026 18:57:10 +0200, Prabhakar <prabhakar.csengg@xxxxxxxxx> said:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> >
> > The RZ/V2H and RZ/V2N SYSC drivers register auxiliary devices for
> > power sequencing, which require the pwrseq-renesas-pwrrdy driver to bind
> > to them.
> >
> > Select POWER_SEQUENCING and POWER_SEQUENCING_RENESAS_PWRRDY from the
> > RZ/V2H and RZ/V2N SYSC configurations so the power sequencer driver is
> > built whenever SYSC support for these SoCs is enabled.
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
> > ---
> > v1->v2:
> > - No change
> > ---
> >  drivers/soc/renesas/Kconfig | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
> > index b7abca4559df..32cd0d9c8758 100644
> > --- a/drivers/soc/renesas/Kconfig
> > +++ b/drivers/soc/renesas/Kconfig
> > @@ -517,9 +517,13 @@ config SYS_R9A09G047
> >  config SYS_R9A09G056
> >       bool "Renesas System controller support for R9A09G056 (RZ/V2N)" if COMPILE_TEST
> >       select SYSC_RZ
> > +     select POWER_SEQUENCING
>
> That's implied by the symbol below though?
>
POWER_SEQUENCING_RENESAS_PWRRDY will only be visible when
POWER_SEQUENCING is enabled. If dropped, randconfig might generate the
following warning:

$ make menuconfig

WARNING: unmet direct dependencies detected for POWER_SEQUENCING_RENESAS_PWRRDY
  Depends on [n]: POWER_SEQUENCING [=n] && (SYSC_RZ [=y] || COMPILE_TEST [=y])
  Selected by [y]:
  - SYS_R9A09G056 [=y] && SOC_RENESAS [=y]

Cheers,
Prabhakar

> Bart
>
> > +     select POWER_SEQUENCING_RENESAS_PWRRDY
> >
> >  config SYS_R9A09G057
> >       bool "Renesas System controller support for R9A09G057 (RZ/V2H)" if COMPILE_TEST
> >       select SYSC_RZ
> > +     select POWER_SEQUENCING
> > +     select POWER_SEQUENCING_RENESAS_PWRRDY
> >
> >  endif # SOC_RENESAS
> > --
> > 2.55.0
> >
> >