On 4/1/25 13:10, Mattijs Korpershoek wrote:
> Hi Zixun,
>
> Thank you for the patch.
>
> On lun., mars 31, 2025 at 18:26, Zixun LI <ad...@hifiphile.com> wrote:
>
>> Add the USB gadget DT node for the sam9x60 SoC's.
>>
>> Signed-off-by: Zixun LI <ad...@hifiphile.com>
>
> Nitpick: title (subject) should not finish with a period (.)
>
> This can be fixed up by the maintainer that will apply the patches.
>
> Eugen, This is assigned to me on patchwork:
> https://patchwork.ozlabs.org/project/uboot/list/?series=450655
>
> Can I pick up all 3 patches or do you prefer to pick up [2/3] and [3/3] ?
It is better that I take these two, because there are some pending
changes on the same DT for the next merge window, so it would avoid
conflicts.
Reassign them to me in patchwork please.
>
> Reviewed-by: Mattijs Korpershoek <mkorpersh...@kernel.org>
>
>> ---
>> arch/arm/dts/sam9x60.dtsi | 13 +++++++++++++
>> 1 file changed, 13 insertions(+)
>>
>> diff --git a/arch/arm/dts/sam9x60.dtsi b/arch/arm/dts/sam9x60.dtsi
>> index 3b684fc63d5..ea2647e5fc9 100644
>> --- a/arch/arm/dts/sam9x60.dtsi
>> +++ b/arch/arm/dts/sam9x60.dtsi
>> @@ -69,6 +69,19 @@
>> #size-cells = <1>;
>> ranges;
>>
>> + usb0: gadget@500000 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + compatible = "microchip,sam9x60-udc";
>> + reg = <0x500000 0x100000>,
>> + <0xf803c000 0x400>;
>> + clocks = <&pmc PMC_TYPE_PERIPHERAL 23>, <&pmc
>> PMC_TYPE_CORE 8>;
>> + clock-names = "pclk", "hclk";
>> + assigned-clocks = <&pmc PMC_TYPE_CORE 8>;
>> + assigned-clock-rates = <480000000>;
>> + status = "disabled";
>> + };
>> +
>> usb1: usb@600000 {
>> compatible = "atmel,at91rm9200-ohci", "usb-ohci";
>> reg = <0x00600000 0x100000>;
>> --
>> 2.49.0
