[PATCH v5 00/19] Add USB on SAM9X60, SAMA7G5 and SAMA5D2 boards

Marek Vasut marex at denx.de
Thu Dec 22 17:11:32 CET 2022


On 12/22/22 16:54, Sergiu.Moga at microchip.com wrote:
> On 22.12.2022 15:36, Marek Vasut wrote:
>> On 12/22/22 11:53, Sergiu Moga wrote:
>>> This series of patches is meant to add support for USB Mass Storage
>>> on SAM9X60, SAMA7G5 and SAMA5D2 boards and register ohci-at91 driver into
>>> Driver Model. In order for this to be achieved, the respective
>>> DT nodes have been added, the USB clock has been registered into CCF
>>> and the required defconfigs have been added to the boards' defconfig.
>>> What is more, in order for the VBUS to stay enabled, a `child_pre_probe`
>>> method has been added to overcome the DM core disabling it in
>>> `usb_scan_device`: when the generic `device_probe` method is called,
>>> the pinctrl is processed once again, undoing whatever changes have
>>> been made in our driver's probe method.
>>> In order to enable USB on SAMA7G5 the addition of RSTC and USB 2.0 PHY
>>> drivers were required.
>>
>> Please split the series into more manageable parts -- architecture and
>> DT bits, board, clock, usb .
> 
> Hi, I am sorry but I believe that they are ordered correctly and they
> all should be part of this series as it ensures usb across all boards
> and most of them depend on each other.
> I think the order in which they have been sent makes the most sense:
> adding what is needed by sam9x60, followed by sama7g5, sama5d2 and
> finally enabling them all in the defconfigs.
> 
> I am not sure that sending multiple series that are dependent on each
> other would make it easier.

The USB patches go through USB tree, they seem independent from the rest 
of the config changes and DTs. Please send them separately.


More information about the U-Boot mailing list