AW: [PATCH 1/2] dt-bindings: arm/stm32: Add emtrion hardware emSBC-Argon

[Mueller, Reinhold]

Hi Rob,

please see the comments below.

Regards
Reinhold


Reinhold Mueller
Software engineer


emtrion GmbH
Am Hasenbiel 6 | 76297 Stutensee | Germany

Phone +49 7244 62694 20
Fax +49 7244 62694 19
Email Reinhold.Mueller@xxxxxxxxxx
Online www.emtrion.de




emtrion GmbH • Amtsgericht Mannheim • HRB 110 300 • Geschäftsführer: Ramona Maurer, Achmed Haddou • Umsatzsteueridentifikationsnummer:DE813694260 • Impressum: www.emtrion.de/de/impressum.html

HINWEIS: Personenbezogene Daten, die Sie per E-Mail an uns übermitteln, werden bei uns gespeichert und verarbeitet. Informationen zu unseren gesetzlichen Informationspflichten, zu uns und unseren Dienstleistungen finden Sie in unseren Datenschutzhinweisen.
Diese E-Mail kann vertrauliche und / oder rechtlich geschützte Informationen enthalten. Wenn Sie nicht der richtige Adressat sind, oder diese E-Mail irrtümlich erhalten haben, informieren Sie bitte den Absender und vernichten diese Mail. Das unerlaubte kopieren, sowie die unbefugte Weitergabe dieser Mail ist nicht gestattet.
> -----Ursprüngliche Nachricht-----
> Von: Rob Herring <robh@xxxxxxxxxx>
> Gesendet: Dienstag, 7. Dezember 2021 22:25
> An: Mueller, Reinhold <Reinhold.Mueller@xxxxxxxxxx>
> Cc: alexandre.torgue@xxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx
> Betreff: Re: [PATCH 1/2] dt-bindings: arm/stm32: Add emtrion hardware
> emSBC-Argon
>
> On Tue, Nov 30, 2021 at 09:16:53AM +0100, reinhold.mueller@xxxxxxxxxxx
> wrote:
> > From: Reinhold Mueller <reinhold.mueller@xxxxxxxxxxx>
> >
> > This patch presents the yaml patch for the emtrion GmbH
> > Argon board series.
> >
> > Signed-off-by: Reinhold Mueller <reinhold.mueller@xxxxxxxxxxx>
> > ---
> >  Documentation/devicetree/bindings/arm/stm32/stm32.yaml | 8
> ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> > index bcaf7be3ab37..710230847dcd 100644
> > --- a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> > +++ b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> > @@ -28,6 +28,14 @@ properties:
> >            - enum:
> >                - st,stm32mp153
> >                - st,stm32mp157
> > +
> > +      - description: emtrion STM32MP1 Argon based Boards
> > +        items:
> > +          - enum:
> > +              - emtrion,stm32mp157c-emsbc-argon
> > +          - enum:
> > +              - emtrion,stm32mp157c-emstamp-argon
>
> Is there going to be multiple entries here? If not, use 'const' instead
> of 'enum'.
>
currently no more entries, i will resent a modified patchset
> > +          - const: st,stm32mp157
> >        - items:
> >            - enum:
> >                - st,stm32f429i-disco
> > --
> > 2.20.1
> >
> >