[U-Boot] [PATCH v2 4/4] usb: am335x_evm: Disable CONFIG_SPL_DM_USB

[Jean-Jacques Hiblot]

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 <sven at 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 at lists.denx.de
>> https://lists.denx.de/listinfo/u-boot
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot