On Thu, Apr 04, 2024 at 01:06:25PM +0200, Nuno Sa wrote: > Using dev_errp_probe() to simplify the code. ... > + if (IS_ERR(fwnode)) > + return dev_errp_probe(dev, PTR_ERR(fwnode), > + "Cannot get Firmware reference\n"); ERR_CAST() seems quite good candidate to have here. return dev_errp_probe(dev, fwnode, "Cannot get Firmware reference\n"); (Assuming dev_errp_probe() magically understands that, note you may have it as a macro and distinguish parameter type with _Generic() or so and behave differently: ERR_PTR() vs. ERR_CAST(), see acpi_dev_hid_uid_match() implementation, but also keep in mind that it doesn't distinguish NULL/0, there is a patch available in the mailing list to fix that, though.) -- With Best Regards, Andy Shevchenko