[PATCH v2 0/2] Fix 'no USB device found' error.

[Julien Panis]

On 6/22/23 17:49, Tom Rini wrote:
> On Thu, Jun 22, 2023 at 04:34:34PM +0200, Julien Panis wrote:
>> This series fixes usb0 dr_mode for am335x-icev2
>> and am335x-evmsk. It must be set to 'peripheral'
>> in order to avoid 'no USB device found' error,
>> in usb_ether_init() function.
>>
>> Signed-off-by: Julien Panis <jpanis at baylibre.com>
>> ---
>> Changes in v2:
>> - Drop the modification made in arch/arm/mach-omap2/am33xx/board.c
>> - Configure usb0 dr_mode as peripheral in 'am335x-icev2-u-boot.dtsi'
>>    and 'am335x-evmsk-u-boot.dtsi' device trees.
>> - Link to v1: https://lore.kernel.org/r/20230621-fix_usb_ether_init-v1-1-2156923994a9@baylibre.com
>>
>> ---
>> Julien Panis (2):
>>        arm: dts: am335x-icev2-u-boot: Configure peripheral mode for usb0
>>        arm: dts: am335x-evmsk-u-boot: Configure peripheral mode for usb0
>>
>>   arch/arm/dts/am335x-evmsk-u-boot.dtsi | 4 ++++
>>   arch/arm/dts/am335x-icev2-u-boot.dtsi | 4 ++++
>>   2 files changed, 8 insertions(+)
> I'll ask the first question that Nishanth might also ask, which is why
> don't these belong in the kernel dts files?  Thanks!
>

That's a good question. :) Looping Nishanth...
usb0 dr_mode property is already overlayed for am335x-evm,
in 'am335x-evm-u-boot.dtsi'. So, it appeared more consistent
to me to do the same thing for am335x-icev2 and am335x-evmsk.
I guess that the goal is to configure usb0 as host by default at
kernel boot, since we do not necessarily want to use this usb0
as peripheral from userspace.
Is it the right explanation @Vignesh @Nishanth ?