From: Guenter Roeck <li...@roeck-us.net>

Return -ENXIO if device property array access functions don't find
a suitable firmware interface.

This lets drivers decide if they should use available platform data
instead.

Cc: Rafael J. Wysocki <rafael.j.wyso...@intel.com>
Signed-off-by: Guenter Roeck <li...@roeck-us.net>
Tested-by: Jeremy Linton <jeremy.lin...@arm.com>
Signed-off-by: David S. Miller <da...@davemloft.net>
(cherry picked from commit 4fa7508e9f1c64ae39516e40ee5495aaa4616ad7)
Signed-off-by: Voon, Weifeng <weifeng.v...@intel.com>
---
 drivers/base/property.c | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/drivers/base/property.c b/drivers/base/property.c
index 84a080b..33013b1 100644
--- a/drivers/base/property.c
+++ b/drivers/base/property.c
@@ -153,6 +153,7 @@ EXPORT_SYMBOL_GPL(fwnode_property_present);
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO if the property is not an array of numbers,
  *        %-EOVERFLOW if the size of the property is not as expected.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_u8_array(struct device *dev, const char *propname,
                                  u8 *val, size_t nval)
@@ -177,6 +178,7 @@ EXPORT_SYMBOL_GPL(device_property_read_u8_array);
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO if the property is not an array of numbers,
  *        %-EOVERFLOW if the size of the property is not as expected.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_u16_array(struct device *dev, const char *propname,
                                   u16 *val, size_t nval)
@@ -201,6 +203,7 @@ EXPORT_SYMBOL_GPL(device_property_read_u16_array);
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO if the property is not an array of numbers,
  *        %-EOVERFLOW if the size of the property is not as expected.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_u32_array(struct device *dev, const char *propname,
                                   u32 *val, size_t nval)
@@ -225,6 +228,7 @@ EXPORT_SYMBOL_GPL(device_property_read_u32_array);
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO if the property is not an array of numbers,
  *        %-EOVERFLOW if the size of the property is not as expected.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_u64_array(struct device *dev, const char *propname,
                                   u64 *val, size_t nval)
@@ -249,6 +253,7 @@ EXPORT_SYMBOL_GPL(device_property_read_u64_array);
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO or %-EILSEQ if the property is not an array of strings,
  *        %-EOVERFLOW if the size of the property is not as expected.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_string_array(struct device *dev, const char *propname,
                                      const char **val, size_t nval)
@@ -270,6 +275,7 @@ EXPORT_SYMBOL_GPL(device_property_read_string_array);
  *        %-EINVAL if given arguments are not valid,
  *        %-ENODATA if the property does not have a value,
  *        %-EPROTO or %-EILSEQ if the property type is not a string.
+ *        %-ENXIO if no suitable firmware interface is present.
  */
 int device_property_read_string(struct device *dev, const char *propname,
                                const char **val)
@@ -291,9 +297,11 @@ EXPORT_SYMBOL_GPL(device_property_read_string);
        else if (is_acpi_node(_fwnode_)) \
                _ret_ = acpi_dev_prop_read(to_acpi_node(_fwnode_), _propname_, \
                                           _proptype_, _val_, _nval_); \
-       else \
+       else if (is_pset(_fwnode_)) \
                _ret_ = pset_prop_read_array(to_pset(_fwnode_), _propname_, \
                                             _proptype_, _val_, _nval_); \
+       else \
+               _ret_ = -ENXIO; \
        _ret_; \
 })
 
@@ -431,9 +439,10 @@ int fwnode_property_read_string_array(struct fwnode_handle 
*fwnode,
        else if (is_acpi_node(fwnode))
                return acpi_dev_prop_read(to_acpi_node(fwnode), propname,
                                          DEV_PROP_STRING, val, nval);
-
-       return pset_prop_read_array(to_pset(fwnode), propname,
-                                   DEV_PROP_STRING, val, nval);
+       else if (is_pset(fwnode))
+               return pset_prop_read_array(to_pset(fwnode), propname,
+                                           DEV_PROP_STRING, val, nval);
+       return -ENXIO;
 }
 EXPORT_SYMBOL_GPL(fwnode_property_read_string_array);
 
-- 
1.9.1

-- 
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to