[U-Boot] [PATCH] sunxi: Select a bunch of CONFIG_foo options from ARCH_SUNXI

[Hans de Goede]

Hi,

On 30-06-15 22:46, Tom Rini wrote:
> On Tue, Jun 30, 2015 at 10:33:17PM +0200, Hans de Goede wrote:
>
>> Select a bunch of CONFIG_foo options from ARCH_SUNXI, instead of having
>> them set to y  in every single sunxi defconfig.
>>
>> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
>
> Ug, sorry, I was waiting for some build results to come out before
> posting my version of this.

No problem.

> To update the USB side:
> $ for F in `git grep -l SYS_EXTRA.*USB_EHCI`;do sed -i -e 's/,USB_EHCI//'
> -e 's/USB_EHCI,//' $F && echo CONFIG_USB_EHCI_HCD=y >> $F;done
>
> And was there a reason you didn't include a 'select DM_SERIAL' in yours?

Yes setting that for all boards breaks booting of boards which do not
have a serial port as drivers/serial/serial-uclass.c:
serial_find_console_or_panic() will then panic (runtime error,
yeah).

And we've one such board. I've a fix for this, next step is to merge
your and mine patches as both have some good things the other one
does not have (I like your work on setting CONFIG_USB_EHCI_HCD,
but you're not selecting CONFIG_USB).

I've run out of steam for today, so I will try to get this done
tomorrow.

> [snip]
>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> index 0829235..6c4d138 100644
>> --- a/arch/arm/Kconfig
>> +++ b/arch/arm/Kconfig
>> @@ -645,10 +645,13 @@ config ARCH_SOCFPGA
>>   config ARCH_SUNXI
>>   	bool "Support sunxi (Allwinner) SoCs"
>>   	select DM
>> +	select DM_ETH
>>   	select DM_GPIO
>> +	select DM_USB if !USB_MUSB_SUNXI
>>   	select OF_CONTROL
>>   	select OF_SEPARATE
>>   	select SPL_DISABLE_OF_CONTROL
>> +	select USB
>
> We really don't want to leave USB as a board choice?

All sunxi SoCs have 2 - 4 (*) usb controllers, and there
does not exist a single board which does not at least uses
one usb controller, and I like to keep things consistent
where ever possible to reduce the combinatorial explosion
problem all these Kconfig options give us.

Regards,

Hans


*) 3 - 6 if you count ehci/ohci seperately.