Hi Andy, Zijun, On Tue, Feb 25, 2025 at 04:20:34PM +0200, Andy Shevchenko wrote: > On Tue, Feb 25, 2025 at 09:58:07PM +0800, Zijun Hu wrote: > > From: Zijun Hu <quic_zijuhu@xxxxxxxxxxx> > > > > Macro NR_FWNODE_REFERENCE_ARGS defines the maximal argument count > > for firmware node reference, and MAX_PHANDLE_ARGS defines the maximal > > argument count for DT node reference, both have the same value now. > > > > To void argument count inconsistency between firmware and DT, simply > > align both macros by '#define MAX_PHANDLE_ARGS NR_FWNODE_REFERENCE_ARGS'. > > I would add here that the of.h includes fwnode.h already, so it doesn't > add any new compile time dependency. It's included via property.h but I think that's fine. > > Both patches LGTM, > Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> -- Sakari Ailus
