On Thu, Feb 1, 2024 at 7:26 PM Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> wrote: > > As we get a child node in the OF case, we should also clean up the > reference, add code to do so. ... > + if (is_of_node(fwnode)) { > + fwnode = fwnode_get_named_child_node(fwnode, "spi"); > + devm_add_action(priv->dev, cs42l43_release_of_node, fwnode); ret = ... if (ret) return ret; > + } With that being addressed, Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> -- With Best Regards, Andy Shevchenko