diff mbox series

[1/2] dt-bindings: arm: Add i.MX8M Mini Gateworks GW7904 board

Message ID 20220902230500.2624739-1-tharvey@gateworks.com (mailing list archive)
State Changes Requested
Headers show
Series [1/2] dt-bindings: arm: Add i.MX8M Mini Gateworks GW7904 board | expand

Commit Message

Tim Harvey Sept. 2, 2022, 11:04 p.m. UTC
Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.

Signed-off-by: Tim Harvey <tharvey@gateworks.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: NXP Linux Team <linux-imx@nxp.com>
---
 Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski Sept. 5, 2022, 6:41 a.m. UTC | #1
On 03/09/2022 02:04, Tim Harvey wrote:
> Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
> 
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index 7431579ab0e8..ce89fac1898e 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -831,6 +831,7 @@ properties:
>                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
>                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
>                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
> +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board

Please put it ordered alphabetically, so before "gw".


Best regards,
Krzysztof
Tim Harvey Sept. 6, 2022, 7:08 p.m. UTC | #2
On Sun, Sep 4, 2022 at 11:41 PM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 03/09/2022 02:04, Tim Harvey wrote:
> > Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
> >
> > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: NXP Linux Team <linux-imx@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> > index 7431579ab0e8..ce89fac1898e 100644
> > --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> > @@ -831,6 +831,7 @@ properties:
> >                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
> >                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
> >                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
> > +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board
>
> Please put it ordered alphabetically, so before "gw".
>

Krzysztof,

Ok - will send a v2 after waiting for feedback on the dts patch.

Best Regards,

Tim
Rob Herring (Arm) Sept. 8, 2022, 9:19 p.m. UTC | #3
On Fri, Sep 02, 2022 at 04:04:59PM -0700, Tim Harvey wrote:
> Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
> 
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> Cc: Fabio Estevam <festevam@gmail.com>
> Cc: NXP Linux Team <linux-imx@nxp.com>
> ---
>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index 7431579ab0e8..ce89fac1898e 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -831,6 +831,7 @@ properties:
>                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
>                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
>                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
> +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board

A useful comment would be ones that distuiguish these boards. It's 
obvious from the compatible it's a i.MX8MM board from Gateworks.

>                - kontron,imx8mm-n801x-som  # i.MX8MM Kontron SL (N801X) SOM
>                - menlo,mx8menlo            # i.MX8MM Menlo board with Verdin SoM
>                - toradex,verdin-imx8mm     # Verdin iMX8M Mini Modules
> -- 
> 2.25.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
Tim Harvey Sept. 8, 2022, 9:44 p.m. UTC | #4
On Thu, Sep 8, 2022 at 2:19 PM Rob Herring <robh@kernel.org> wrote:
>
> On Fri, Sep 02, 2022 at 04:04:59PM -0700, Tim Harvey wrote:
> > Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
> >
> > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: NXP Linux Team <linux-imx@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> > index 7431579ab0e8..ce89fac1898e 100644
> > --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> > @@ -831,6 +831,7 @@ properties:
> >                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
> >                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
> >                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
> > +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board
>
> A useful comment would be ones that distuiguish these boards. It's
> obvious from the compatible it's a i.MX8MM board from Gateworks.

But isn't it clear that you need to go to the device-tree itself to
understand the details?

As far as basic features go sometimes there is very little difference
in these board models. It would be a struggle to list all the board
details (which I do in the dts commit) in a way that doesn't take up
too much space in fsl.yaml.

Best Regards,

Tim
Krzysztof Kozlowski Sept. 9, 2022, 8:02 a.m. UTC | #5
On 08/09/2022 23:44, Tim Harvey wrote:
> On Thu, Sep 8, 2022 at 2:19 PM Rob Herring <robh@kernel.org> wrote:
>>
>> On Fri, Sep 02, 2022 at 04:04:59PM -0700, Tim Harvey wrote:
>>> Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
>>>
>>> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
>>> Cc: Rob Herring <robh+dt@kernel.org>
>>> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
>>> Cc: Shawn Guo <shawnguo@kernel.org>
>>> Cc: Sascha Hauer <s.hauer@pengutronix.de>
>>> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
>>> Cc: Fabio Estevam <festevam@gmail.com>
>>> Cc: NXP Linux Team <linux-imx@nxp.com>
>>> ---
>>>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
>>> index 7431579ab0e8..ce89fac1898e 100644
>>> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
>>> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
>>> @@ -831,6 +831,7 @@ properties:
>>>                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
>>>                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
>>>                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
>>> +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board
>>
>> A useful comment would be ones that distuiguish these boards. It's
>> obvious from the compatible it's a i.MX8MM board from Gateworks.
> 
> But isn't it clear that you need to go to the device-tree itself to
> understand the details?
> 
> As far as basic features go sometimes there is very little difference
> in these board models. It would be a struggle to list all the board
> details (which I do in the dts commit) in a way that doesn't take up
> too much space in fsl.yaml.
> 

But then the comment you added is useless. So either add useful comment
or no comment. :)


Best regards,
Krzysztof
Tim Harvey Sept. 9, 2022, 3:25 p.m. UTC | #6
On Fri, Sep 9, 2022 at 1:03 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 08/09/2022 23:44, Tim Harvey wrote:
> > On Thu, Sep 8, 2022 at 2:19 PM Rob Herring <robh@kernel.org> wrote:
> >>
> >> On Fri, Sep 02, 2022 at 04:04:59PM -0700, Tim Harvey wrote:
> >>> Add DT compatible string for i.MX8M Mini based Gateworks GW7904 board.
> >>>
> >>> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> >>> Cc: Rob Herring <robh+dt@kernel.org>
> >>> Cc: Krzysztof Kozlowski <krzk+dt@kernel.org>
> >>> Cc: Shawn Guo <shawnguo@kernel.org>
> >>> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> >>> Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> >>> Cc: Fabio Estevam <festevam@gmail.com>
> >>> Cc: NXP Linux Team <linux-imx@nxp.com>
> >>> ---
> >>>  Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> >>> index 7431579ab0e8..ce89fac1898e 100644
> >>> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> >>> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> >>> @@ -831,6 +831,7 @@ properties:
> >>>                - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
> >>>                - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
> >>>                - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
> >>> +              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board
> >>
> >> A useful comment would be ones that distuiguish these boards. It's
> >> obvious from the compatible it's a i.MX8MM board from Gateworks.
> >
> > But isn't it clear that you need to go to the device-tree itself to
> > understand the details?
> >
> > As far as basic features go sometimes there is very little difference
> > in these board models. It would be a struggle to list all the board
> > details (which I do in the dts commit) in a way that doesn't take up
> > too much space in fsl.yaml.
> >
>
> But then the comment you added is useless. So either add useful comment
> or no comment. :)
>
>
> Best regards,
> Krzysztof

Krzysztof,

so are you saying that no comment is fine here as well? It seems to me
that most of the comments in that file look just like mine which I
agree are about just as descriptive as the compatible string.

For discussion purposes here is for example the commit log for the GW7904 dts:
    The GW7904 is based on the i.MX 8M Mini SoC featuring:
     - LPDDR4 DRAM
     - eMMC FLASH
     - microSD connector with UHS support
     - LIS2DE12 3-axis accelerometer
     - Gateworks System Controller
     - IMX8M FEC
     - 2x RS232 off-board connectors
     - PMIC
     - 10x bi-color LED's
     - 1x miniPCIe socket with PCIe and USB2.0
     - 802.3at Class 4 PoE
     - 10-30VDC input via barrel-jack

And the comit log for the very similar GW7903 dts:
    The GW7903 is based on the i.MX 8M Mini SoC featuring:
     - LPDDR4 DRAM
     - eMMC FLASH
     - microSD connector with UHS support
     - LIS2DE12 3-axis accelerometer
     - Gateworks System Controller
     - IMX8M FEC
     - software selectable RS232/RS485/RS422 serial transceiver
     - PMIC
     - 2x off-board bi-directional opto-isolated digital I/O
     - 1x M.2 A-E Key Socket and 1x MiniPCIe socket with USB2.0 and PCIe
       (resistor loading to route PCIe/USB2 between M.2 and MiniPCIe socket)

Best Regards,

Tim
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 7431579ab0e8..ce89fac1898e 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -831,6 +831,7 @@  properties:
               - gw,imx8mm-gw7901          # i.MX8MM Gateworks Board
               - gw,imx8mm-gw7902          # i.MX8MM Gateworks Board
               - gw,imx8mm-gw7903          # i.MX8MM Gateworks Board
+              - gateworks,imx8mm-gw7904   # i.MX8MM Gateworks Board
               - kontron,imx8mm-n801x-som  # i.MX8MM Kontron SL (N801X) SOM
               - menlo,mx8menlo            # i.MX8MM Menlo board with Verdin SoM
               - toradex,verdin-imx8mm     # Verdin iMX8M Mini Modules