[PATCH v7 1/6] watchdog: core: dt: add support for the timeout-sec dt property
Wim Van Sebroeck
wim at iguana.be
Wed Feb 13 20:44:01 EST 2013
Hi Fabio,
> > I must admit that the different iterations of this patch only made it better.
> > I like the idea that you are proposing here (put the default timeout in the
> > watchdog_device struct and use a helper function to set the timeout parameter
> > value or the timeout-sec dt value). To be able to use other mechanism in the
> > future also, I think it make more sense to pass the device instead of the
> > device_node.
>
> If i understand correctly you want to use "struct platform_device"
> instead of "struct device_node",
> in the function watchdog_init_timeout?
No struct device instead of struct device_node.
> > I also detailed your documentation a bit more and incorporated above changes
> > in an adjusted patch. Can you have a look at it and if OK for you I will put
> > it in linux-watchdog-next as your v10 :-).
>
> That's good for sure :)
Will add it to the linux-watchdog-next tree today.
Kind regards,
Wim.
More information about the devicetree-discuss
mailing list