Hi Tom, Andre Thanks for your reviews.
On Fri, 19 Feb 2021 17:29:26 +0000 Andre Przywara <andre.przyw...@arm.com> wrote: > > > > And then based on my comment in the previous patch, similar to > > PINEPHONE_DT_SELECTION and other board-specific options, add a > > CHIP_DIP_SCAN option or similar. > > Yes, indeed, thanks Tom. > > The idea of making this extension scheme generic is great, it's just > that sunxi is using a different approach to board specific code here. > Normally we expect U-Boot-proper board specific code to be DT driven, > and where this does not really work, use those symbols that Tom pointed > to. For my extension_board_scan board specific function, would you prefer if I move to callback like below instead of Kconfig? if (of_machine_is_compatible("nextthing,chip")) extension_board_register_callback(chip_extension_board_scan); Regards, Köry