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,
        linux-kernel@vger.kernel.org
Cc:     devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
        Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        Vivien Didelot <vivien.didelot@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/4] New RGMII delay DT bindings for the SJA1105 DSA driver
Date:   Mon, 18 Oct 2021 22:29:48 +0300
Message-Id: <20211018192952.2736913-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?C620tpXhO0wLpRa3M91u9FuTsZ2/xIjrbQdsV+PcvjvE0r1yTqZVE66jdWPu?=
 =?us-ascii?Q?P+jrIRlGrKcFHpjaMeKGJWt+EXcvzQh9oLS8fEjPZJ+/VJ9mNgLkQ9UvUd74?=
 =?us-ascii?Q?l/PVArKPw0wKNQtS1H/ELHfn2RIqC6XLasyKwvONWNltHfT8wSHZtTjF94nd?=
 =?us-ascii?Q?N3/IJCPTXJkmKSDC0Yyf3pQ+sxdOz/umLsnG47BBsQi6k/v/7nFuiE7mhGrH?=
 =?us-ascii?Q?3BesNRz/zvBE6ljV1pn7W/uU+zw4w86Bu5HqltppkEF/M6XV5waeVd+NpQGO?=
 =?us-ascii?Q?x4kqECskgUsh4iBHPmzVKKcsNKDRdPHewlSkKChSohC925t2XVttaTsmH9Ey?=
 =?us-ascii?Q?kUjuY4q5tDbqiPaXhsrZK9Oq3ufhjFwixDZFdLIHy/pQVpjmBsrB4j2qCPQE?=
 =?us-ascii?Q?L3bXfhQMDyTHuEXGYoxTQ/WOATRnzjhmRv/g4rsGsf0GevkGZd4eJoxlQivg?=
 =?us-ascii?Q?HdtSi9GPywbQh5LTR3pBOZvkCAuTQcKOC9ysktUaPPkb51N8GJ/Cu1y628ec?=
 =?us-ascii?Q?3kqCQDlATIXdo3JerdcWAYxghE2BJpUf7aSjmn7cWxvdLQjCRtxjdlUTEUAh?=
 =?us-ascii?Q?sojWRjT3pAZKoDB1BaR8/wdhp78zOxe+1l6AnwKecqGoXe9poNnqUq9pweyf?=
 =?us-ascii?Q?ZKCN3TfV/ngsmThbThAaFaPnpfvdek5JpnJ1DWvO8cFXhDokEmoOM3+8zMb4?=
 =?us-ascii?Q?rmi0aCrm0eE5FUqMyS8AiLAFNuK+RZsJ8pN6T6J0uzKY0v67QLoEVjzrSMe9?=
 =?us-ascii?Q?GxWT+di0t6qhtZtZz0K6y1sgQ+lSFbPzmi+xrurT7mSX9OqAzHNzXGgABFxP?=
 =?us-ascii?Q?7TXLN5j0NAK5gaLeAwh2jPL/c1ihc4vfShGvtDTS6yZSzdx3uI2d9D0l55gR?=
 =?us-ascii?Q?FriBOhEb+iW29v71BiXqrE6Qn+MqYIdJuUY9Pk7zii8xPoX8cz2aCHP0EwtO?=
 =?us-ascii?Q?KFtY416+3vClj5N5iRxGtUGmNZkZBbzPkK+7oxNVC4WsWDiY91lgxE0qTdEf?=
 =?us-ascii?Q?5QKdas6mWZfGWF3V+PXZa8AkfNkcq+B/IUbJWkXbsqNE2TY02zfZClwlmcga?=
 =?us-ascii?Q?2zFG8pMK5/YK+TxNfdC2uKibKETly5sWfdRrHStZvwUiihYoHeuKATdtWdr+?=
 =?us-ascii?Q?g4PPvrtjDbBWHwi7eoH/13Qo8n7rXkGX1FCVGWFd1P4cN4Pz8xUmZMO6880h?=
 =?us-ascii?Q?mfRDCUUUaQ5j4dW6Fv+A/o7uRf3eHWkglXQkKXfAyLZVtsSvvUwbk+eRCaIM?=
 =?us-ascii?Q?vig76TG8W100D5tD/ZA2RseU0LZATOeSu/XvePuCTigdugJGuk6PIADv9yYc?=
 =?us-ascii?Q?oN3iV+ijsqOm6KgKE2NHE1YR?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b274cf11-6491-48a3-7154-08d9926dbce6
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2021 19:30:26.9814
 (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: Zai/2q71Cak9UBdUvGiVQctneVf1oQOIhYq5Y6HWKeg0tWcX88LHeGMe1ChMQBw7kDyr4KqK+7qOARyAym48jw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4430
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.

Changes in v2:
- removed devicetree patches, to be sent via Shawn Guo's tree
- fixed yamllint indentation warnings

Vladimir Oltean (4):
  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 +++++++++
 drivers/net/dsa/sja1105/sja1105.h             | 25 ++++-
 drivers/net/dsa/sja1105/sja1105_clocking.c    | 35 +++----
 drivers/net/dsa/sja1105/sja1105_main.c        | 94 ++++++++++++++-----
 5 files changed, 157 insertions(+), 47 deletions(-)

-- 
2.25.1