[PATCH v6 14/23] ACPI: property: Add support for cells property
Sunil V L
sunilvl at ventanamicro.com
Sun Jun 29 22:17:28 PDT 2025
On Mon, Jun 23, 2025 at 12:14:07PM +0300, Andy Shevchenko wrote:
> On Wed, Jun 18, 2025 at 05:43:49PM +0530, Anup Patel wrote:
> > From: Sunil V L <sunilvl at ventanamicro.com>
> >
> > Currently, ACPI doesn't support cells property when
> > fwnode_property_get_reference_args() is called. ACPI always expects
> > the number of arguments to be passed. However, the above mentioned
> > call being a common interface for OF and ACPI, it is better to have
> > single calling convention which works for both. Hence, add support
> > for cells property on the reference device to get the number of
> > arguments dynamically.
>
> ...
>
> > +static unsigned int acpi_fwnode_get_args_count(const struct acpi_device *device,
> > + const char *nargs_prop)
> > +{
> > + const union acpi_object *obj;
>
> > + if (!nargs_prop)
> > + return 0;
>
> This check is implied by the call. No need to duplicate.
>
> > + if (acpi_dev_get_property(device, nargs_prop, ACPI_TYPE_INTEGER, &obj))
> > + return 0;
> > +
> > + return obj->integer.value;
> > +}
>
> ...
>
> > + if (nargs_prop)
>
> Again, if you don't won't to reassign the existing value, it's better to have
> this data be collected in the temporary variable of the same semantics. Then
> you will choose one when it's needed, no need to have this dup check (again!).
>
Okay. Let me update in the next version.
Thanks!
Sunil
More information about the linux-riscv
mailing list