On 20/11/2018 10:09, Jean-Jacques Hiblot wrote:
Hi Sven,

On 18/11/2018 21:46, Tom Rini wrote:
On Sun, Nov 18, 2018 at 08:05:11PM +0100, Sven Schwermer wrote:
This configuration doesn't use USB in the SPL, so we need to disable
driver model for USB in the SPL.

Signed-off-by: Sven Schwermer <s...@svenschwermer.de>
How many other boards will need such a patch ?
In this case, "all" am335x boards should as only am43xx and later offer ROM loads SPL from a USB mass storage device (rather than showing up as
a CDC ethernet device).
I actually tested this patch series on Travis CI and this was the only “offender”. See https://travis-ci.org/svenschwermer/u-boot/builds/456335517 <https://travis-ci.org/svenschwermer/u-boot/builds/456335517> Are the other boards not checked in Travis? If so, I’d go through them manually.
I mean functionally.  But if you can confirm that it's not being enabled
on other platforms, that would be good, thanks!

FYI I posted a series a few weeks ago that adds a tool to automate this kind of job. check out "[RFC PATCH v3 0/3] python tools to inspect configs"

Command lines would be:

"tools/configs2csv.py --rebuild -"  to build the database

and then

"tools/configs2csv.py CONFIG_SPL_DM_USB CONFIG_DM_USB > dm_usb_usage.csv"

more accurate (because some board don't use the same config for u-boot, spl and tpl)  and more readable (since we are interested only in the boards that activate the options) would be:

"tools/configs2csv.py CONFIG_SPL_DM_USB CONFIG_DM_USB --u-boot --spl --tpl --discard-empty > dm_usb_usage.csv"


JJ


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to