[PATCH 08/10] sun5i: add support for DIP detection to CHIP board

Köry Maincent kory.maincent at bootlin.com
Mon Feb 22 15:23:08 CET 2021


Hi Tom, Andre

Thanks for your reviews.

On Fri, 19 Feb 2021 17:29:26 +0000
Andre Przywara <andre.przywara at 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


More information about the U-Boot mailing list