Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <95420387-3cde-dd6e-3cb1-169ca56268d2@tuxon.dev>
Date:   Thu, 27 Jul 2023 08:37:29 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH] ARM: dts: at91: ksz9477_evb: Add missing timer nodes
Content-Language: en-US
To:     Conor Dooley <conor@kernel.org>, Lukasz Majewski <lukma@denx.de>
Cc:     devicetree@vger.kernel.org,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        linux-kernel@vger.kernel.org, Conor Dooley <conor+dt@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Claudiu Beznea <claudiu.beznea@microchip.com>,
        linux-arm-kernel@lists.infradead.org
References: <20230712152111.3756211-1-lukma@denx.de>
 <20230726101902.769f6541@wsk> <20230726-igloo-science-96e926f086f6@spud>
From:   claudiu beznea <claudiu.beznea@tuxon.dev>
In-Reply-To: <20230726-igloo-science-96e926f086f6@spud>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 26.07.2023 21:33, Conor Dooley wrote:
> On Wed, Jul 26, 2023 at 10:19:02AM +0200, Lukasz Majewski wrote:
>> Dear Community,
>>
>>> Without this change the KSZ9477-EVB board hangs just after passing
>>> execution flow from u-boot to Linux kernel.
>>>
>>> This code has been copied from at91-sama5d3_xplained.dts.
>>>
>>> Test setup: Linux 6.5-rc1
>>> Config:     arch/arm/configs/sama5_defconfig
>>> Toolchain:  gcc-linaro-7.3.1-2018.05-x86_64_arm-linux-gnueabi
>>>
>>> Signed-off-by: Lukasz Majewski <lukma@denx.de>
>>> ---
>>>   .../boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts  | 12
>>> ++++++++++++ 1 file changed, 12 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts
>>> b/arch/arm/boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts index
>>> 14af1fd6d247..99cd6d15998b 100644 ---
>>> a/arch/arm/boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts +++
>>> b/arch/arm/boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts @@ -162,6
>>> +162,18 @@ };
>>>   };
>>>   
>>> +&tcb0 {
>>> +	timer0: timer@0 {
>>> +		compatible = "atmel,tcb-timer";
>>> +		reg = <0>;
>>> +	};
>>> +
>>> +	timer1: timer@1 {
>>> +		compatible = "atmel,tcb-timer";
>>> +		reg = <1>;
>>> +	};
>>> +};
>>> +
>>>   &usb0 {
>>>   	pinctrl-names = "default";
>>>   	pinctrl-0 = <&pinctrl_usba_vbus>;
>>
>> Gentle ping on this fix ...
> 
> Claudiu has left Microchip, so things might've got a bit lost on the
> way. I've added his new address, Claudiu, will you take a look/pick this
> up please? I can also do it if that is easier for you.

It's on my list.

Thanks,
Claudiu

> 
> Thanks,
> Conor.
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel