Commit 5c0acf3b authored by Adrian Hunter's avatar Adrian Hunter Committed by Rafael J. Wysocki

driver core: Add comments about returning array counts

The "read array" variants of the device property functions
can be used to return the number of values in an array.
Update the comments to reflect that.
Signed-off-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
Acked-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 06e5801b
...@@ -62,13 +62,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_present); ...@@ -62,13 +62,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_present);
* device_property_read_u8_array - return a u8 array property of a device * device_property_read_u8_array - return a u8 array property of a device
* @dev: Device to get the property of * @dev: Device to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Function reads an array of u8 properties with @propname from the device * Function reads an array of u8 properties with @propname from the device
* firmware description and stores them to @val if found. * firmware description and stores them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -85,13 +86,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u8_array); ...@@ -85,13 +86,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u8_array);
* device_property_read_u16_array - return a u16 array property of a device * device_property_read_u16_array - return a u16 array property of a device
* @dev: Device to get the property of * @dev: Device to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Function reads an array of u16 properties with @propname from the device * Function reads an array of u16 properties with @propname from the device
* firmware description and stores them to @val if found. * firmware description and stores them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -108,13 +110,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u16_array); ...@@ -108,13 +110,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u16_array);
* device_property_read_u32_array - return a u32 array property of a device * device_property_read_u32_array - return a u32 array property of a device
* @dev: Device to get the property of * @dev: Device to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Function reads an array of u32 properties with @propname from the device * Function reads an array of u32 properties with @propname from the device
* firmware description and stores them to @val if found. * firmware description and stores them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -131,13 +134,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u32_array); ...@@ -131,13 +134,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u32_array);
* device_property_read_u64_array - return a u64 array property of a device * device_property_read_u64_array - return a u64 array property of a device
* @dev: Device to get the property of * @dev: Device to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Function reads an array of u64 properties with @propname from the device * Function reads an array of u64 properties with @propname from the device
* firmware description and stores them to @val if found. * firmware description and stores them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -154,13 +158,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u64_array); ...@@ -154,13 +158,14 @@ EXPORT_SYMBOL_GPL(device_property_read_u64_array);
* device_property_read_string_array - return a string array property of device * device_property_read_string_array - return a string array property of device
* @dev: Device to get the property of * @dev: Device to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Function reads an array of string properties with @propname from the device * Function reads an array of string properties with @propname from the device
* firmware description and stores them to @val if found. * firmware description and stores them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO or %-EILSEQ if the property is not an array of strings, * %-EPROTO or %-EILSEQ if the property is not an array of strings,
...@@ -170,7 +175,8 @@ int device_property_read_string_array(struct device *dev, const char *propname, ...@@ -170,7 +175,8 @@ int device_property_read_string_array(struct device *dev, const char *propname,
const char **val, size_t nval) const char **val, size_t nval)
{ {
return IS_ENABLED(CONFIG_OF) && dev->of_node ? return IS_ENABLED(CONFIG_OF) && dev->of_node ?
of_property_read_string_array(dev->of_node, propname, val, nval) : (val ? of_property_read_string_array(dev->of_node, propname, val, nval)
: of_property_count_strings(dev->of_node, propname)) :
acpi_dev_prop_read(ACPI_COMPANION(dev), propname, acpi_dev_prop_read(ACPI_COMPANION(dev), propname,
DEV_PROP_STRING, val, nval); DEV_PROP_STRING, val, nval);
} }
...@@ -218,13 +224,14 @@ EXPORT_SYMBOL_GPL(device_property_read_string); ...@@ -218,13 +224,14 @@ EXPORT_SYMBOL_GPL(device_property_read_string);
* fwnode_property_read_u8_array - return a u8 array property of firmware node * fwnode_property_read_u8_array - return a u8 array property of firmware node
* @fwnode: Firmware node to get the property of * @fwnode: Firmware node to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Read an array of u8 properties with @propname from @fwnode and stores them to * Read an array of u8 properties with @propname from @fwnode and stores them to
* @val if found. * @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -243,13 +250,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u8_array); ...@@ -243,13 +250,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u8_array);
* fwnode_property_read_u16_array - return a u16 array property of firmware node * fwnode_property_read_u16_array - return a u16 array property of firmware node
* @fwnode: Firmware node to get the property of * @fwnode: Firmware node to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Read an array of u16 properties with @propname from @fwnode and store them to * Read an array of u16 properties with @propname from @fwnode and store them to
* @val if found. * @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -268,13 +276,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u16_array); ...@@ -268,13 +276,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u16_array);
* fwnode_property_read_u32_array - return a u32 array property of firmware node * fwnode_property_read_u32_array - return a u32 array property of firmware node
* @fwnode: Firmware node to get the property of * @fwnode: Firmware node to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Read an array of u32 properties with @propname from @fwnode store them to * Read an array of u32 properties with @propname from @fwnode store them to
* @val if found. * @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -293,13 +302,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u32_array); ...@@ -293,13 +302,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u32_array);
* fwnode_property_read_u64_array - return a u64 array property firmware node * fwnode_property_read_u64_array - return a u64 array property firmware node
* @fwnode: Firmware node to get the property of * @fwnode: Firmware node to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Read an array of u64 properties with @propname from @fwnode and store them to * Read an array of u64 properties with @propname from @fwnode and store them to
* @val if found. * @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of numbers, * %-EPROTO if the property is not an array of numbers,
...@@ -318,13 +328,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u64_array); ...@@ -318,13 +328,14 @@ EXPORT_SYMBOL_GPL(fwnode_property_read_u64_array);
* fwnode_property_read_string_array - return string array property of a node * fwnode_property_read_string_array - return string array property of a node
* @fwnode: Firmware node to get the property of * @fwnode: Firmware node to get the property of
* @propname: Name of the property * @propname: Name of the property
* @val: The values are stored here * @val: The values are stored here or %NULL to return the number of values
* @nval: Size of the @val array * @nval: Size of the @val array
* *
* Read an string list property @propname from the given firmware node and store * Read an string list property @propname from the given firmware node and store
* them to @val if found. * them to @val if found.
* *
* Return: %0 if the property was found (success), * Return: number of values if @val was %NULL,
* %0 if the property was found (success),
* %-EINVAL if given arguments are not valid, * %-EINVAL if given arguments are not valid,
* %-ENODATA if the property does not have a value, * %-ENODATA if the property does not have a value,
* %-EPROTO if the property is not an array of strings, * %-EPROTO if the property is not an array of strings,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment