Re: [PATCH 1/7] arm64: dts: ti: k3-am62-main: Add eQEP nodes

[Judith Mendez]

Hi,

On 4/19/24 8:56 AM, Krzysztof Kozlowski wrote:
> On 19/04/2024 00:14, Judith Mendez wrote:
> > Add eQEP DT nodes 0-2 for AM625 SoC.
> >
> > Since external hardware was needed to test eQEP, the DT nodes
> > for eQEP were not included in the introductory commit. Now that
> > eQEP has been validated, add nodes to k3-am62-main.dtsi.
> >
> > Fixes: f1d17330a5be ("arm64: dts: ti: Introduce base support for AM62x SoC")
> > Signed-off-by: Judith Mendez <jm@xxxxxx>
> > ---
> >   arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 30 ++++++++++++++++++++++++
> >   1 file changed, 30 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> > index e9cffca073efc..0877899b90667 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
> > @@ -844,6 +844,36 @@ ecap2: pwm@23120000 {
> >   		status = "disabled";
> >   	};
> >   
> > +	eqep0: counter@23200000 {
> > +		compatible = "ti,am3352-eqep";
>
> That's k3-am62, not am3352. See writing-bindings (and numerous
> presentations telling you how to do it, e.g. from two previous EOSS/ELCE).

Thanks, will fix and send v2.

~ Judith

> Best regards,
> Krzysztof