Hi Tom, Andre
Thanks for your reviews.
On Fri, 19 Feb 2021 17:29:26 +0000 Andre Przywara andre.przywara@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