diff mbox series

[3/7] dt-bindings: net: dsa: sja1105: fix wrong indentation

Message ID 20210715082536.1882077-4-aisheng.dong@nxp.com
State Superseded, archived
Headers show
Series dt-bindings: imx8mp: fix dt schema check errors | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 1 warnings, 10 lines checked
robh/dt-meta-schema success
robh/dtbs-check success

Commit Message

Dong Aisheng July 15, 2021, 8:25 a.m. UTC 
This patch fixes the following error:
Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml:70:17: [warning] wrong indentation: expected 18 but found 16 (indentation)

Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Vivien Didelot <vivien.didelot@gmail.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Vladimir Oltean <olteanv@gmail.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: netdev@vger.kernel.org
Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
---
 Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Vladimir Oltean July 15, 2021, 10:04 a.m. UTC | #1 
Hi Aisheng,

On Thu, Jul 15, 2021 at 04:25:32PM +0800, Dong Aisheng wrote:
> This patch fixes the following error:
> Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml:70:17: [warning] wrong indentation: expected 18 but found 16 (indentation)
> 
> Cc: Andrew Lunn <andrew@lunn.ch>
> Cc: Vivien Didelot <vivien.didelot@gmail.com>
> Cc: Florian Fainelli <f.fainelli@gmail.com>
> Cc: Vladimir Oltean <olteanv@gmail.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: netdev@vger.kernel.org
> Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
> ---

Thank you for preparing and sending the patch.
It looks like Rob already applied another version of this change
yesterday:
https://lore.kernel.org/netdev/20210622113327.3613595-1-thierry.reding@gmail.com/
I wasn't copied on that patch, I noticed it rather by coincidence.
Dong Aisheng July 15, 2021, 10:43 a.m. UTC | #2 
On Thu, Jul 15, 2021 at 6:05 PM Vladimir Oltean <olteanv@gmail.com> wrote:
>
> Hi Aisheng,
>
> On Thu, Jul 15, 2021 at 04:25:32PM +0800, Dong Aisheng wrote:
> > This patch fixes the following error:
> > Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml:70:17: [warning] wrong indentation: expected 18 but found 16 (indentation)
> >
> > Cc: Andrew Lunn <andrew@lunn.ch>
> > Cc: Vivien Didelot <vivien.didelot@gmail.com>
> > Cc: Florian Fainelli <f.fainelli@gmail.com>
> > Cc: Vladimir Oltean <olteanv@gmail.com>
> > Cc: "David S. Miller" <davem@davemloft.net>
> > Cc: Rob Herring <robh+dt@kernel.org>
> > Cc: netdev@vger.kernel.org
> > Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com>
> > ---
>
> Thank you for preparing and sending the patch.
> It looks like Rob already applied another version of this change
> yesterday:
> https://lore.kernel.org/netdev/20210622113327.3613595-1-thierry.reding@gmail.com/
> I wasn't copied on that patch, I noticed it rather by coincidence.

Got it, then this one can be dropped.

Regards
Aisheng
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml b/Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml
index 0b8a05dd52e6..f978f8719d8e 100644
--- a/Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/nxp,sja1105.yaml
@@ -67,8 +67,8 @@  properties:
           reg:
             oneOf:
               - enum:
-                - 0
-                - 1
+                  - 0
+                  - 1
 
         required:
           - compatible