2024-04-29 08:47:45

by Fabio Aiuto

[permalink] [raw]
Subject: [PATCH v7 1/3] dt-bindings: arm: fsl: add Engicam i.Core MX93 EDIMM 2.0 Starter Kit

i.Core MX93 is a NXP i.MX93 based EDIMM SoM by Engicam.

EDIMM 2.0 Starter Kit is an EDIMM 2.0 Form Factor Capacitive
Evaluation Board by Engicam.

i.Core MX93 needs to be mounted on top of EDIMM 2.0 Starter Kit
to get the full i.Core MX93 EDIMM 2.0 Starter Kit board.

Add bindings for this board.

Cc: Matteo Lisi <[email protected]>
Cc: Mirko Ardinghi <[email protected]>
Reviewed-by: Krzysztof Kozlowsky <[email protected]>
Acked-by: Conor Dooley <[email protected]>
Signed-off-by: Fabio Aiuto <[email protected]>
---
v3 ---> v7:
- no changes
v2 ---> v3:
- added {Reviewed,Acked}-by tags
v1 ---> v2:
- no changes

Documentation/devicetree/bindings/arm/fsl.yaml | 7 +++++++
1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 0027201e19f8..b497a01c7418 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -1265,6 +1265,13 @@ properties:
- fsl,imx93-11x11-evk # i.MX93 11x11 EVK Board
- const: fsl,imx93

+ - description: Engicam i.Core MX93 based Boards
+ items:
+ - enum:
+ - engicam,icore-mx93-edimm2 # i.MX93 Engicam i.Core MX93 EDIMM 2.0 Starter Kit
+ - const: engicam,icore-mx93 # i.MX93 Engicam i.Core MX93 Som
+ - const: fsl,imx93
+
- description: i.MXRT1050 based Boards
items:
- enum:
--
2.34.1



2024-04-29 08:53:25

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v7 1/3] dt-bindings: arm: fsl: add Engicam i.Core MX93 EDIMM 2.0 Starter Kit

On 29/04/2024 10:44, Fabio Aiuto wrote:
> i.Core MX93 is a NXP i.MX93 based EDIMM SoM by Engicam.
>
> EDIMM 2.0 Starter Kit is an EDIMM 2.0 Form Factor Capacitive
> Evaluation Board by Engicam.
>
> i.Core MX93 needs to be mounted on top of EDIMM 2.0 Starter Kit
> to get the full i.Core MX93 EDIMM 2.0 Starter Kit board.
>
> Add bindings for this board.
>
> Cc: Matteo Lisi <[email protected]>
> Cc: Mirko Ardinghi <[email protected]>
> Reviewed-by: Krzysztof Kozlowsky <[email protected]>

What the hell? Why are you changing tags?

This is corrupted since v3!

NAK.

Best regards,
Krzysztof


2024-04-29 09:02:07

by Fabio Aiuto

[permalink] [raw]
Subject: Re: [PATCH v7 1/3] dt-bindings: arm: fsl: add Engicam i.Core MX93 EDIMM 2.0 Starter Kit

Dear Krzysztof,

Il Mon, Apr 29, 2024 at 10:53:08AM +0200, Krzysztof Kozlowski ha scritto:
> On 29/04/2024 10:44, Fabio Aiuto wrote:
> > i.Core MX93 is a NXP i.MX93 based EDIMM SoM by Engicam.
> >
> > EDIMM 2.0 Starter Kit is an EDIMM 2.0 Form Factor Capacitive
> > Evaluation Board by Engicam.
> >
> > i.Core MX93 needs to be mounted on top of EDIMM 2.0 Starter Kit
> > to get the full i.Core MX93 EDIMM 2.0 Starter Kit board.
> >
> > Add bindings for this board.
> >
> > Cc: Matteo Lisi <[email protected]>
> > Cc: Mirko Ardinghi <[email protected]>
> > Reviewed-by: Krzysztof Kozlowsky <[email protected]>
>
> What the hell? Why are you changing tags?

I fear I don't understand your comment. Did you gave me
your Reviewed-by: tag, didn't you?

When Conor gave me his Acked-by in v2 told me to pick up
yours too...

that's all

kr,

fabio

>
> This is corrupted since v3!
>
> NAK.
>
> Best regards,
> Krzysztof
>

2024-04-29 09:05:53

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH v7 1/3] dt-bindings: arm: fsl: add Engicam i.Core MX93 EDIMM 2.0 Starter Kit

On 29/04/2024 11:01, Fabio Aiuto wrote:
> Dear Krzysztof,
>
> Il Mon, Apr 29, 2024 at 10:53:08AM +0200, Krzysztof Kozlowski ha scritto:
>> On 29/04/2024 10:44, Fabio Aiuto wrote:
>>> i.Core MX93 is a NXP i.MX93 based EDIMM SoM by Engicam.
>>>
>>> EDIMM 2.0 Starter Kit is an EDIMM 2.0 Form Factor Capacitive
>>> Evaluation Board by Engicam.
>>>
>>> i.Core MX93 needs to be mounted on top of EDIMM 2.0 Starter Kit
>>> to get the full i.Core MX93 EDIMM 2.0 Starter Kit board.
>>>
>>> Add bindings for this board.
>>>
>>> Cc: Matteo Lisi <[email protected]>
>>> Cc: Mirko Ardinghi <[email protected]>
>>> Reviewed-by: Krzysztof Kozlowsky <[email protected]>
>>
>> What the hell? Why are you changing tags?
>
> I fear I don't understand your comment. Did you gave me
> your Reviewed-by: tag, didn't you?
>
> When Conor gave me his Acked-by in v2 told me to pick up
> yours too...

I *DID NOT* give such tag. Why do you manually type people names? You
are ought to COPY the entire tag. Not amend it, not change it, not try
to write by yourself.

Best regards,
Krzysztof


2024-04-29 09:22:28

by Fabio Aiuto

[permalink] [raw]
Subject: Re: [PATCH v7 1/3] dt-bindings: arm: fsl: add Engicam i.Core MX93 EDIMM 2.0 Starter Kit

Dear Krzysztof,

Il Mon, Apr 29, 2024 at 11:04:03AM +0200, Krzysztof Kozlowski ha scritto:
> On 29/04/2024 11:01, Fabio Aiuto wrote:
> > Dear Krzysztof,
> >
> > Il Mon, Apr 29, 2024 at 10:53:08AM +0200, Krzysztof Kozlowski ha scritto:
> >> On 29/04/2024 10:44, Fabio Aiuto wrote:
> >>> i.Core MX93 is a NXP i.MX93 based EDIMM SoM by Engicam.
> >>>
> >>> EDIMM 2.0 Starter Kit is an EDIMM 2.0 Form Factor Capacitive
> >>> Evaluation Board by Engicam.
> >>>
> >>> i.Core MX93 needs to be mounted on top of EDIMM 2.0 Starter Kit
> >>> to get the full i.Core MX93 EDIMM 2.0 Starter Kit board.
> >>>
> >>> Add bindings for this board.
> >>>
> >>> Cc: Matteo Lisi <[email protected]>
> >>> Cc: Mirko Ardinghi <[email protected]>
> >>> Reviewed-by: Krzysztof Kozlowsky <[email protected]>
> >>
> >> What the hell? Why are you changing tags?
> >
> > I fear I don't understand your comment. Did you gave me
> > your Reviewed-by: tag, didn't you?
> >
> > When Conor gave me his Acked-by in v2 told me to pick up
> > yours too...
>
> I *DID NOT* give such tag. Why do you manually type people names? You
> are ought to COPY the entire tag. Not amend it, not change it, not try
> to write by yourself.

I apologize for that, will send a v8

kr,

fabio

>
> Best regards,
> Krzysztof
>