Hi Daniel, Thank you for the patch.
On Mon, Nov 30, 2020 at 01:31:19PM +0000, Daniel Scally wrote: > Use the software_node_unregister_nodes() helper function to unwind this > array in a cleaner way. > > Acked-by: Petr Mladek <pmla...@suse.com> > Reviewed-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> > Suggested-by: Andy Shevchenko <andriy.shevche...@linux.intel.com> > Signed-off-by: Daniel Scally <djrsca...@gmail.com> Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > --- > Changes since RFC v3: > > Changed the called function name - didn't drop the tags since it's > such a trivial change, hope that's alright! > > lib/test_printf.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/lib/test_printf.c b/lib/test_printf.c > index 7ac87f18a10f..7d60f24240a4 100644 > --- a/lib/test_printf.c > +++ b/lib/test_printf.c > @@ -644,9 +644,7 @@ static void __init fwnode_pointer(void) > test(second_name, "%pfwP", software_node_fwnode(&softnodes[1])); > test(third_name, "%pfwP", software_node_fwnode(&softnodes[2])); > > - software_node_unregister(&softnodes[2]); > - software_node_unregister(&softnodes[1]); > - software_node_unregister(&softnodes[0]); > + software_node_unregister_nodes(softnodes); > } > > static void __init -- Regards, Laurent Pinchart