Re: [PATCH V2 1/2] dt-bindings: arm: ti: Add bindings for J721E EAIK

From: Nishanth Menon <nm@ti.com>
To: Sinthu Raja M <sinthu.raja@mistralsolutions.com>
Cc: Tero Kristo <kristo@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Device Tree Mailing List <devicetree@vger.kernel.org>,
	Linux ARM Mailing List <linux-arm-kernel@lists.infradead.org>,
	Lokesh Vutla <lokeshvutla@ti.com>,
	Amarnath MB <amarnath.mb@ti.com>,
	Sinthu Raja <sinthu.raja@ti.com>
Subject: Re: [PATCH V2 1/2] dt-bindings: arm: ti: Add bindings for J721E EAIK
Date: Mon, 7 Jun 2021 15:20:02 -0500	[thread overview]
Message-ID: <20210607202002.b4ijmzoitwqdgnw6@sublease> (raw)
In-Reply-To: <CAEd-yTQVuGjftjQs7B83Y3fcgQtLj+DF_09Va8HjP0=qOwE8Ng@mail.gmail.com>

On 22:26-20210607, Sinthu Raja M wrote:
> On Mon, Jun 7, 2021 at 7:02 PM Nishanth Menon <nm@ti.com> wrote:
> >
> > On 15:03-20210607, Sinthu Raja wrote:
> > > From: Sinthu Raja <sinthu.raja@ti.com>
> > >
> > > J721E EdgeAI Kit (EAIK) is a low cost, small form factor board designed
> > > for TI’s J721E SoC.
> > > Add DT binding documentation for J721E EAIK
> >
> > Need a url for the board.
> The URL for the board is not available until the board launch.
> Probably should have mentioned in the cover letter.

Then, lets wait for the url to be available. k.org record will be for
ever, so, lets make sure that we have this information to aid people
looking to know more about the platform.

> >
> > >
> > > Signed-off-by: Amarnath MB <amarnath.mb@ti.com>
> > > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com>
> > > ---
> > > Change in V2:
> > > - Fix for dt_binding_check error.
> > >
> > >  Documentation/devicetree/bindings/arm/ti/k3.yaml                | 2 ++
> > >  .../devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml         | 2 +-
> > >  2 files changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> > > index c5aa362e4026..923dd7cf1dc6 100644
> > > --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml
> > > +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml
> > > @@ -29,6 +29,8 @@ properties:
> > >
> > >        - description: K3 J721E SoC
> > >          items:
> > > +          - enum:
> > > +              - ti,j721e-eaik
> > >            - const: ti,j721e
> > >
> > >        - description: K3 J7200 SoC
> > > diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > > index 6070456a7b67..464cee128811 100644
> > > --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > > +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > > @@ -135,7 +135,7 @@ examples:
> > >    - |
> > >      / {
> > >          model = "Texas Instruments K3 J721E SoC";
> > > -        compatible = "ti,j721e";
> > > +        compatible = "ti,j721e-eaik", "ti,j721e";
> > I see what we are attempting to do here.
> >
> > Documentation/devicetree/bindings/remoteproc/ti,k3-dsp-rproc.yaml
> > probably should strip off the board specifics out in the
> > examples. but, that belongs to a different patch.
> Will remove this change. But do we need to ignore the dt_binding
> _check for this patch series?

No, we cannot ignore dt_bindings_check. Hence the fixup needs to happen
before we consider this series.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

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