Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>, netdev@vger.kernel.org
Cc:     devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        Rob Herring <robh+dt@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>, Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Prasanna Vengateshan <prasanna.vengateshan@microchip.com>,
        Ansuel Smith <ansuelsmth@gmail.com>,
        =?UTF-8?q?Alvin=20=C5=A0ipraga?= <alsi@bang-olufsen.dk>
Subject: [PATCH net-next 0/6] New RGMII delay DT bindings for the SJA1105 DSA driver
Date:   Thu, 14 Oct 2021 01:23:07 +0300
Message-Id: <20211013222313.3767605-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?L/6E3LJpqeyP7wQ3uVBWxfU6TPo7pgE3u6b91mbdcjQipUFy+UitfPPICmm+?=
 =?us-ascii?Q?RunrZirR/WEyYWCFaOd8u3u3xnpl9A+DmmctozOkPagxvK1ZYj8NcSWrXpCF?=
 =?us-ascii?Q?glLT1DqQKlGmhf4iHuW6Ecrew4iahIWRthJA/pVgOaPB+oyPWB0oPREBfU23?=
 =?us-ascii?Q?pB243q4aqvy0U2BOZvhgAVfHaNbG6yKEw/EswckugsXtNrdpKsD7AOB+ZyUW?=
 =?us-ascii?Q?hggSEVoKFaLJzQ5nBDjWk2tFv3suMbp2zhpR55TDwyjhxrxGbEIwfj3B0Bk3?=
 =?us-ascii?Q?ImtEUX9u34XEQCwv8iZF2h+nrtH1CTlOj9uIbb7ix+7FSPnkT/7ZSxsIf5gJ?=
 =?us-ascii?Q?748Bc3z5JRfZqs83AF7Qqxa9f13ufeAQmu83hd6JTGlGSIb3qFbdFbnbsw7i?=
 =?us-ascii?Q?k/twYvkLIjmJ9jH9nRwL3tGHx7Ojttx/x2qQEBAd3ttzG8e42CJIC8hIRb5X?=
 =?us-ascii?Q?jawX0h/Ghlj4k7Ex/PGC02fKFgJinSr+BtU4yRe0bEHvCgAwfJ67wZ8KvPKs?=
 =?us-ascii?Q?9d32w5EO+rxlcyh1CaOb0S0fmqBqd8ScSUuUKJe9Hgtr9Z1Z33zvZL7Wbk+o?=
 =?us-ascii?Q?zFuxlp0KW/KnGBaWiPIyKlnVzdudKUvY+RvqDWMx94emBLR5UJb0GbpDv389?=
 =?us-ascii?Q?Abtc9zqbTsmGxNfgqC/8OSII+g4v6RZf+TyY7LnYHvSMhTqMuJq0C52wD1Si?=
 =?us-ascii?Q?c1t4ypZotmS1iVIPytnD8ToCdjI2uHJ8IMpLu2c2GX5cn9tXiZpDlvGNhtbR?=
 =?us-ascii?Q?q6qMbE2aBZ3ArKtbVgSwvJy+XLItPlaLKt3FBle6SB5ed/TtT8aS5QbGqnT+?=
 =?us-ascii?Q?VN8ztg32pDLf9xs0EWBi78AIp3Ir3tKuYHQJtKqgoHPy6BLo1pZAntvoXO+D?=
 =?us-ascii?Q?YYp8gOkerRytmkR4c0szr7OmI6Z4Qb/wBpuuRBvI/9/oCc2Dgyd81n4zKrRu?=
 =?us-ascii?Q?b6iLYvVXjYLP2S/6qQ8rQwXxtCc4J5szKlj2iR7xZ8aYQouDZPj2i4l7+wXt?=
 =?us-ascii?Q?SPkICTt+ySxB0rJLeQhQGMSgBGMZtO02wRUAi1OQW7n+Uk1BBzHCIW+AMbsg?=
 =?us-ascii?Q?e41ouCEBKemio7y9yQcgjHKYZxqqL3DFyMI/7WE2qKzA62dVQXvVOE9KrKVW?=
 =?us-ascii?Q?prdqlB0mfm6dyO4e9y1wkihsczQWB78R8lWa5aBmjCHqTeANTsOIYo7zFEbr?=
 =?us-ascii?Q?BaLRapSj53GTejwzWlbGITaCxnkhpt1zG8LiNud57okMEadFt1awRUIoFrr4?=
 =?us-ascii?Q?g6ElyVqWAAdS9ukP7zYU4Fch5BC4PmaHhkOEvcgrNqKCv+PDArEhH9xgliKw?=
 =?us-ascii?Q?BhpbwJbxFrtqXcFmt6iTeV/f?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b9ccdfc6-1b91-459e-6aa5-08d98e9831a3
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Oct 2021 22:24:17.0491
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: d0Wbu7tLWgCbHQc1sEjpTaLZHCtL0Ms9sEc4+J3bhWyzz/5Y+sY6VTIR7yt3DQmvCWMcHNOnwELTXp6Coe9sBA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4816
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

During recent reviews I've been telling people that new MAC drivers
should adopt a certain DT binding format for RGMII delays in order to
avoid conflicting interpretations. Some suggestions were better received
than others, and it appears we are still far from a consensus.

Part of the problem seems to be that there are still drivers that apply
RGMII delays based on an incorrect interpretation of the device tree,
and these serve as a bad example for others.
I happen to maintain one of those drivers and I am able to test it, so I
figure that one of the ways in which I can make a change is to stop
providing a bad example.

Therefore, this series adds support for the "rx-internal-delay-ps" and
"tx-internal-delay-ps" properties inside sja1105 switch port DT nodes,
and if these are present, they will decide what RGMII delays will the
driver apply.

The in-tree device trees are also updated to follow the new format, as
well as the schema validator.

I assume it's okay to get all changes merged in through the same tree
(net-next). Although the DTS changes could be split, if needed - the
driver works with or without them. There is one more DTS which should be
changed, which is in Shawn's tree but not in net-next:
https://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git/tree/arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts?h=for-next
For that, I'd have to send a separate patch.

Vladimir Oltean (6):
  ARM: dts: imx6qp-prtwd3: update RGMII delays for sja1105 switch
  ARM: dts: ls1021a-tsn: update RGMII delays for sja1105 switch
  dt-bindings: net: dsa: sja1105: fix example so all ports have a
    phy-handle of fixed-link
  dt-bindings: net: dsa: inherit the ethernet-controller DT schema
  dt-bindings: net: dsa: sja1105: add {rx,tx}-internal-delay-ps
  net: dsa: sja1105: parse {rx,tx}-internal-delay-ps properties for
    RGMII delays

 .../devicetree/bindings/net/dsa/dsa.yaml      |  7 ++
 .../bindings/net/dsa/nxp,sja1105.yaml         | 43 +++++++++
 arch/arm/boot/dts/imx6qp-prtwd3.dts           |  2 +
 arch/arm/boot/dts/ls1021a-tsn.dts             |  2 +
 drivers/net/dsa/sja1105/sja1105.h             | 25 ++++-
 drivers/net/dsa/sja1105/sja1105_clocking.c    | 35 +++----
 drivers/net/dsa/sja1105/sja1105_main.c        | 94 ++++++++++++++-----
 7 files changed, 161 insertions(+), 47 deletions(-)

-- 
2.25.1