Hi Geert,
geert@xxxxxxxxxxxxxx wrote on Tue, 12 Apr 2022 10:12:41 +0200:
> Hi Miquel,
>
> On Tue, Apr 12, 2022 at 10:03 AM Miquel Raynal
> <miquel.raynal@xxxxxxxxxxx> wrote:
> > geert@xxxxxxxxxxxxxx wrote on Tue, 12 Apr 2022 09:52:25 +0200:
> > > On Tue, Apr 12, 2022 at 9:43 AM Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote:
> > > > geert@xxxxxxxxxxxxxx wrote on Tue, 12 Apr 2022 09:37:22 +0200:
> > > > > So far I've been rather terse in giving feedback on these series,
> > > > > as I'm in wait-and-see mode w.r.t. what else you've planned for the
> > > > > sysctrl DT node[1] and clock/sys controller code...
> > > > >
> > > > > [1] Did I say I'm not that fond of child nodes? But for the dmamux,
> > > > > it looks like a good solution to handle this.
> > > >
> > > > O:-)
> > > >
> > > > I plan in the coming days to write a proper reset controller driver
> > > > that will be queried by the rtc driver (as proposed by Alexandre).
> > >
> > > OK.
> > >
> > > > Which means I'll have to declare this reset controller as a child of
> > > > the systrl node. If you disagree with it, you may jump-in, see this
> > > > thread :
> > > >
> > > > Subject: Re: [PATCH 2/7] soc: renesas: rzn1-sysc: Export a
> > > > function to enable/disable the RTC
> > > > Date: Wed, 6 Apr 2022 10:32:31 +0200
> > >
> > > But do you need a child node for that? All(most all) other Renesas
> > > clock drivers provide reset functionality, and none of them use a
> > > child node for that.
> >
> > How do you "request" the reset handle from the consumer driver if it's
> > not described in the DT? Do you have examples to share?
>
> I didn't say it does not need to be described in DT ;-)
>
> Just add "#reset-cells = <1>" to the sysctrl node, and nodes can
> start referring to it using "resets = <&sysctrl N>".
> Currently, the sysctrl node is already a clock and power-domain provider.
>
> Documentation/devicetree/bindings/clock/renesas,cpg-mssr.yaml
> shows an R-Car CPG/MSSR node providing clock, power-domain, and
> reset functionalities.
Ok, works for me ;)
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
>
> 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
Thanks,
Miquèl
