| Message ID | 1310113654-25887-6-git-send-email-shawn.guo@linaro.org |
|---|---|
| State | New |
| Headers | show |
Shawn, On 07/08/2011 03:27 AM, Shawn Guo wrote: > The patch adds empty functions of_property_read_u32 and > of_property_read_u32_array for non-dt build, so that drivers > migrating to dt can save some '#ifdef CONFIG_OF'. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > Cc: Rob Herring <rob.herring@calxeda.com> > Cc: Thomas Abraham <thomas.abraham@linaro.org> > Cc: Grant Likely <grant.likely@secretlab.ca> > --- > include/linux/of.h | 15 +++++++++++++++ > 1 files changed, 15 insertions(+), 0 deletions(-) > > diff --git a/include/linux/of.h b/include/linux/of.h > index 4761046..4ef636e 100644 > --- a/include/linux/of.h > +++ b/include/linux/of.h > @@ -255,5 +255,20 @@ static inline bool of_have_populated_dt(void) > return false; > } > > +static inline int of_property_read_u32_array(const struct device_node *np, > + char *propname, > + u32 *out_values, > + size_t sz) > +{ > + return -ENOSYS; > +} > + > +static inline int of_property_read_u32(const struct device_node *np, > + char *propname, > + u32 *out_value) > +{ > + return -ENOSYS; > +} > + > #endif /* CONFIG_OF */ > #endif /* _LINUX_OF_H */ You should include the string read function as well for completeness. Rob
On Fri, Jul 08, 2011 at 04:27:33PM +0800, Shawn Guo wrote: > The patch adds empty functions of_property_read_u32 and > of_property_read_u32_array for non-dt build, so that drivers > migrating to dt can save some '#ifdef CONFIG_OF'. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > Cc: Rob Herring <rob.herring@calxeda.com> > Cc: Thomas Abraham <thomas.abraham@linaro.org> > Cc: Grant Likely <grant.likely@secretlab.ca> Applied, thanks, and added the string version. > --- > include/linux/of.h | 15 +++++++++++++++ > 1 files changed, 15 insertions(+), 0 deletions(-) > > diff --git a/include/linux/of.h b/include/linux/of.h > index 4761046..4ef636e 100644 > --- a/include/linux/of.h > +++ b/include/linux/of.h > @@ -255,5 +255,20 @@ static inline bool of_have_populated_dt(void) > return false; > } > > +static inline int of_property_read_u32_array(const struct device_node *np, > + char *propname, > + u32 *out_values, > + size_t sz) > +{ > + return -ENOSYS; > +} > + > +static inline int of_property_read_u32(const struct device_node *np, > + char *propname, > + u32 *out_value) > +{ > + return -ENOSYS; > +} > + > #endif /* CONFIG_OF */ > #endif /* _LINUX_OF_H */ > -- > 1.7.4.1 >
diff --git a/include/linux/of.h b/include/linux/of.h index 4761046..4ef636e 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -255,5 +255,20 @@ static inline bool of_have_populated_dt(void) return false; } +static inline int of_property_read_u32_array(const struct device_node *np, + char *propname, + u32 *out_values, + size_t sz) +{ + return -ENOSYS; +} + +static inline int of_property_read_u32(const struct device_node *np, + char *propname, + u32 *out_value) +{ + return -ENOSYS; +} + #endif /* CONFIG_OF */ #endif /* _LINUX_OF_H */
The patch adds empty functions of_property_read_u32 and of_property_read_u32_array for non-dt build, so that drivers migrating to dt can save some '#ifdef CONFIG_OF'. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> Cc: Rob Herring <rob.herring@calxeda.com> Cc: Thomas Abraham <thomas.abraham@linaro.org> Cc: Grant Likely <grant.likely@secretlab.ca> --- include/linux/of.h | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-)