mbox series

[v3,0/3] Make fwnode_property_get_reference_args accept NULL args

Message ID 20231102101316.1186215-1-sakari.ailus@linux.intel.com (mailing list archive)
Headers show
Series Make fwnode_property_get_reference_args accept NULL args | expand

Message

Sakari Ailus Nov. 2, 2023, 10:13 a.m. UTC
Hi all,

The of_parse_phandle_with_args() accepts NULL args but
fwnode_property_get_reference_args() does not currently, in its ACPI or
software node implementations. Fix this.

since v2:

- Correct fwnode_property_get_reference_args() name (was
  fwnode_property_get_reference()) in commit messages.

since v1:

- Applies to Rafael's devprop branch now (i.e. not on top of DisCo for
  Imaging set).

- Add Fixes: tags.

- Small documentation line wrap change in the first patch.

Sakari Ailus (3):
  acpi: property: Let args be NULL in __acpi_node_get_property_reference
  software node: Let args be NULL in software_node_get_reference_args
  device property: fwnode_property_get_reference_args allows NULL args
    now

 drivers/acpi/property.c | 7 +++++--
 drivers/base/property.c | 1 +
 drivers/base/swnode.c   | 3 +++
 3 files changed, 9 insertions(+), 2 deletions(-)

Comments

Andy Shevchenko Nov. 2, 2023, 1:13 p.m. UTC | #1
On Thu, Nov 02, 2023 at 12:13:13PM +0200, Sakari Ailus wrote:
> Hi all,
> 
> The of_parse_phandle_with_args() accepts NULL args but
> fwnode_property_get_reference_args() does not currently, in its ACPI or
> software node implementations. Fix this.

One nit in the first patch, otherwise LGTM,
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>