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;
Date:   Wed, 26 Jul 2023 19:33:20 +0100
From:   Conor Dooley <conor@kernel.org>
To:     Lukasz Majewski <lukma@denx.de>
Cc:     Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Nicolas Ferre <nicolas.ferre@microchip.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Beznea <claudiu.beznea@microchip.com>,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: at91: ksz9477_evb: Add missing timer nodes
Message-ID: <20230726-igloo-science-96e926f086f6@spud>
References: <20230712152111.3756211-1-lukma@denx.de>
 <20230726101902.769f6541@wsk>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="KlkmgP3458lT5iDu"
Content-Disposition: inline
In-Reply-To: <20230726101902.769f6541@wsk>
Precedence: bulk


--KlkmgP3458lT5iDu
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 26, 2023 at 10:19:02AM +0200, Lukasz Majewski wrote:
> Dear Community,
>=20
> > Without this change the KSZ9477-EVB board hangs just after passing
> > execution flow from u-boot to Linux kernel.
> >=20
> > This code has been copied from at91-sama5d3_xplained.dts.
> >=20
> > 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
> >=20
> > Signed-off-by: Lukasz Majewski <lukma@denx.de>
> > ---
> >  .../boot/dts/microchip/at91-sama5d3_ksz9477_evb.dts  | 12
> > ++++++++++++ 1 file changed, 12 insertions(+)
> >=20
> > 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 @@ };
> >  };
> > =20
> > +&tcb0 {
> > +	timer0: timer@0 {
> > +		compatible =3D "atmel,tcb-timer";
> > +		reg =3D <0>;
> > +	};
> > +
> > +	timer1: timer@1 {
> > +		compatible =3D "atmel,tcb-timer";
> > +		reg =3D <1>;
> > +	};
> > +};
> > +
> >  &usb0 {
> >  	pinctrl-names =3D "default";
> >  	pinctrl-0 =3D <&pinctrl_usba_vbus>;
>=20
> 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.

Thanks,
Conor.

--KlkmgP3458lT5iDu
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZMFm8AAKCRB4tDGHoIJi
0gdvAQCHl+tV1VwtWJVltrks+jslsgaAFUe1PVWq7wPxcoKszwD9HgCxBTUQoQ4x
p812g3F0VCWepjWrWf10m5VLnBnmbgE=
=Br94
-----END PGP SIGNATURE-----

--KlkmgP3458lT5iDu--