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:     devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org
Cc:     Rob Herring <robh+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
        Li Yang <leoyang.li@nxp.com>,
        Oleksij Rempel <o.rempel@pengutronix.de>
Subject: [PATCH devicetree 3/3] arm64: dts: lx2160abluebox3: update RGMII delays for sja1105 switch
Date:   Wed, 20 Oct 2021 14:36:13 +0300
Message-Id: <20211020113613.815210-4-vladimir.oltean@nxp.com>
In-Reply-To: <20211020113613.815210-1-vladimir.oltean@nxp.com>
References: <20211020113613.815210-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?mYZi5M8sKRvWZrZ57xcKu5sBBqp8cDUtuFnj2UpMtGsQHgoFlSJJdDIgloWp?=
 =?us-ascii?Q?5uNPuSXN2JSDhAUfqxEDSHvMLSPa2xUWjnGtokWvtvFiNLsPEfsCMXxAsR03?=
 =?us-ascii?Q?BLjLzrZPav3uCYEpDPVoCkkKW5X1Y6eR2gezcvZSQ8ab7sG4cRpWAJWcX0hx?=
 =?us-ascii?Q?lIYTOqqnPJbTu8Jyp+LwPq2X+q/n6S4fdA/0IKWZWPvJNUETueb+IgfYDFkp?=
 =?us-ascii?Q?WVSkhqTjm1PZxiR1cwj/PD2RDjj1NMckkboAHHk9fJp2Qz0d7Qyf9BrUGg7w?=
 =?us-ascii?Q?KYgZwwlNf+pOZuthCw+o+7am7oCLwFUvRbXsNsKFlESbG2LCgNmTEdRY11+C?=
 =?us-ascii?Q?KhLcosIeciG8A8r0Kxxva7mWucva/J2Y9wgEh9M4HDKDd7CYpScmopyHrEjQ?=
 =?us-ascii?Q?ncDoYjevijMw6WXnrhovvnjWk2kESjrmculgGCoLSdC3Tk9jCZbbbPLDxq0L?=
 =?us-ascii?Q?F5UJ1TLOxaxnDSYfQQVKt8Otc36UXKZkC58luqeJQoHnOHBWeFK3gY6mgRLC?=
 =?us-ascii?Q?WgDIoymH2Ii29fhDvmjPHvL/O508AmLxkPFarQzBMmrbowpYTyrUpMyye7s0?=
 =?us-ascii?Q?qJFeW9V73Of3pDhSbwtrOJ8VOObWPNQGON/3L4Cs6nS3nxfPU6tVFhBrhWXV?=
 =?us-ascii?Q?AcOqz5Z/VjANc95PPQDigX89eFuRDU0ftGC4isQluyB1xk2l4E8laf9KrrCO?=
 =?us-ascii?Q?c6jnZEnMR6bLxkAxJY+jGgRZ4fTglavkrTDr3RY0YiBwz1RWnN8wFpD9eX5Z?=
 =?us-ascii?Q?fWGuNo70bPQ5S/YmcGMxX0vPznVlYpDBZaryrwI99a1bMbYuvoUvtFuwYu9L?=
 =?us-ascii?Q?EVUye5+T+uuUbxDFlEvVn5wvsmkDlrSNE0JOsW0N/dhitF6bx5RW4n+ryeU9?=
 =?us-ascii?Q?sRE8mn9v1QkwdGz7oqNRxWn4ymVus2aY4QIHDJ2S6G09hF+1QazdIeU48wOP?=
 =?us-ascii?Q?HehSRSHX7CneUg4Xdac9gzmuzBrdULNqELg3FDYOJXz17r143z1BwkXghYIF?=
 =?us-ascii?Q?UR/m1CeLAF1mKdiN/YI16z2wV7A1MoxtBo0GeFSM9ZzcSKmSkHtlEk5KLTG/?=
 =?us-ascii?Q?D0kFHvW32UfXQ/wtlic/iHwZjaMwYDN3ESQ2zwtwkgKrefaGfVUTqFZcckM5?=
 =?us-ascii?Q?8eW4cHEd4Nsq+Qapar0iDbkOn6qPH2i8YyuLnHBW4zshpI9PPz48vTENTcnL?=
 =?us-ascii?Q?RjQJ2nwaKP7lZ+ZKBMg4zlAssvBCDq2gDIctPI55BU/b9yiswtbNagt87ytx?=
 =?us-ascii?Q?ZY8edWsji0GZ2pvMXM15AHL33hmCx8smzFy+yku0yvrh9hHcD7gzrfh9ZU6c?=
 =?us-ascii?Q?uKO6wspIqpp5BeQhWoBlmuvVvT8hPRzowga2BFT8RBFR9ctgB7QOMwp7zZ6Z?=
 =?us-ascii?Q?6ll95znGbCN9O2QJVDrUZa845uJt7LifNEpYQfz4EBqWpmGP/q95WPsy30Ys?=
 =?us-ascii?Q?ZxE/Mu1n7U5UzDVFRvsIpjiyNMBOkhL2EczC+BUx0ho7CSeGtpQj3DobmnQN?=
 =?us-ascii?Q?TTpbqkbOqOe7ZZRhlAb+P5GstjKZXooLNxcb9BLGuE0IBOAVF9Ut/qhAK9lj?=
 =?us-ascii?Q?P6XExD7Yoa6HJoMCMG/T4r7k0zSTO69WNIGThTskP40mgwEI3X1YhP0u7col?=
 =?us-ascii?Q?KDguZ73pxCQSfqqsV9I2FSo=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e382236e-71e1-44cc-5a23-08d993bdde9d
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Oct 2021 11:36:34.4680
 (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: Lboe29NJp4OucI2ko/fCJh90ior1TTGJuFqPVLNS861349wJ7uRyZO3r0cL2haZn2dgrURaet7GkYyWqhZ8ZMQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB4685
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

In the new behavior, the sja1105 driver expects there to be explicit
RGMII delays present on the fixed-link ports, otherwise it will complain
that it falls back to legacy behavior, which is to apply RGMII delays
incorrectly derived from the phy-mode string.

In this case, the legacy behavior of the driver is to apply both RX and
TX delays. To preserve that, add explicit 2 nanosecond delays, which are
identical with what the driver used to add (a 90 degree phase shift).
The delays from the phy-mode are ignored by new kernels (it's still
RGMII as long as it's "rgmii*" something), and the explicit
{rx,tx}-internal-delay-ps properties are ignored by old kernels, so the
change works both ways.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
 arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts b/arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts
index b21be03da0af..042c486bdda2 100644
--- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts
+++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3.dts
@@ -386,6 +386,8 @@ port@2 {
 				reg = <2>;
 				ethernet = <&dpmac17>;
 				phy-mode = "rgmii-id";
+				rx-internal-delay-ps = <2000>;
+				tx-internal-delay-ps = <2000>;
 
 				fixed-link {
 					speed = <1000>;
@@ -529,6 +531,8 @@ port@2 {
 				reg = <2>;
 				ethernet = <&dpmac18>;
 				phy-mode = "rgmii-id";
+				rx-internal-delay-ps = <2000>;
+				tx-internal-delay-ps = <2000>;
 
 				fixed-link {
 					speed = <1000>;
-- 
2.25.1