Hi Magnus,
On Fri, Oct 2, 2015 at 6:10 AM, Magnus Damm <[email protected]> wrote:
> On Fri, Oct 2, 2015 at 12:28 AM, Geert Uytterhoeven
> <[email protected]> wrote:
>> On Thu, Oct 1, 2015 at 4:37 PM, Magnus Damm <[email protected]> wrote:
>>> From: Geert Uytterhoeven <[email protected]>
>>> Add a driver for the Renesas Module Standby and Software Reset module
>>> found in several Renesas SoCs.
>>>
>>> This driver is based on the existing R-Car MSTP driver, and is intended
>>> to replace it.
>>> The existing driver is limited to Module Standby, and has bindings that
>>> are difficult to extend to more register sets for e.g. reset control.
>>>
>>> TODO:
>>> - Implement module reset handling,
>>> - Write binding documentation.
>>>
>>> Signed-off-by: Geert Uytterhoeven <[email protected]>
>>> Signed-off-by: Magnus Damm <[email protected]>
>>
>> Thanks, I'm happy you like it!
>
> Yeah, it looks good!
>
> I'm interested in learning how to tie in the reset functionality via
> DT though, do we need any integration update for that?
Yes, in addition to the "clocks = <&mssr R8A7795_CLK_xxx>" property, every
device node need a "resets = <&mssr R8A7795_CLK_xxx>" property (and an optional
reset-names property). In the case of e.g. DU, it needs multiple resets, just
like it needs multiple clocks.
Cfr. Documentation/devicetree/bindings/reset/reset.txt
Perhaps the R8A7795_CLK_xxx should be renamed to R8A7795_MOD_xxx?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
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
--
To unsubscribe from this list: send the line "unsubscribe linux-clk" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
