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 <hdego...@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.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to