On Wed, Oct 10, 2012 at 4:29 PM, Rob Herring <[email protected]> wrote: > On 10/09/2012 07:04 AM, Fabio Porcedda wrote: >> The first user of this function is the watchdog framework. > > Who is the 2nd user because I don't see any others. I don't see this > being a widely used property.
Hi Rob, thanks for reviewing. There is only one use of this function in my patchset. I added this function after Jean-Christophe repeatedly suggested to add it. On Fri, Oct 5, 2012 at 1:16 PM, Jean-Christophe PLAGNIOL-VILLARD <[email protected]> wrote: > why I bother to comment > *make this of generic* this is not watchdog specific other driver can use it Maybe the idea was to have the helper ready when the dt support it's added to a drivers that use a "timeout" parameter. If it's for better i can just get rid of this function. I don't mind. Best regards > >> >> Signed-off-by: Fabio Porcedda <[email protected]> >> Cc: Grant Likely <[email protected]> >> Cc: Rob Herring <[email protected]> >> --- >> include/linux/of.h | 11 +++++++++++ >> 1 file changed, 11 insertions(+) >> >> diff --git a/include/linux/of.h b/include/linux/of.h >> index 72843b7..5870818 100644 >> --- a/include/linux/of.h >> +++ b/include/linux/of.h >> @@ -470,4 +470,15 @@ static inline int of_property_read_u32(const struct >> device_node *np, >> return of_property_read_u32_array(np, propname, out_value, 1); >> } >> >> +/** >> + * of_get_timeout_sec() - Helper to read the timeout_sec property >> + * @np: device node from which the property value is to be >> read. >> + * @timeout: adress of the output value >> + */ >> +static inline int of_get_timeout_sec(const struct device_node *np, >> + u32 *timeout) >> +{ >> + return of_property_read_u32(np, "timeout-sec", timeout); >> +} >> + >> #endif /* _LINUX_OF_H */ >> > -- Fabio Porcedda _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
