From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Wed, 1 Jul 2015 21:05:53 +0200 Subject: [U-Boot] [PATCH] sunxi: Select a bunch of CONFIG_foo options from ARCH_SUNXI In-Reply-To: <20150630204604.GG28577@bill-the-cat> References: <1435696397-1434-1-git-send-email-hdegoede@redhat.com> <20150630204604.GG28577@bill-the-cat> Message-ID: <55943A11.7000404@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > > 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.