Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 15 May 2023 14:48:09 +0300
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Maxime Chevallier <maxime.chevallier@bootlin.com>
Cc:     Mark Brown <broonie@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        rafael@kernel.org, Colin Foster <colin.foster@in-advantage.com>,
        Lee Jones <lee@kernel.org>, davem@davemloft.net,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
        thomas.petazzoni@bootlin.com
Subject: Re: [RFC 7/7] net: pcs: Drop the TSE PCS driver
Message-ID: <20230515114809.d4jzhiazymfqajbj@skbuf>
References: <20230324093644.464704-1-maxime.chevallier@bootlin.com>
 <20230324093644.464704-8-maxime.chevallier@bootlin.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230324093644.464704-8-maxime.chevallier@bootlin.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?+ChVDA8VAwRu/I5pFzIZCizKMXbxwHZLnWSBbOddPV4OlEFOhAhuVrE4K8yh?=
 =?us-ascii?Q?ixtOK2ioQDEEUvTbCVtxianyvRMFPMoyGmgxdBgnyM5Ygjx1r08fv6nl2Ul/?=
 =?us-ascii?Q?B0PW/IZRSWYGGmWLxs1q64nfDEpBpmmfgsd5E4oSkfR3cAcyaV5nTPOFxWnA?=
 =?us-ascii?Q?naLTskLQJ2+0XYIL4ljfAAWgpehqSEKbQTJa1CyDF6jWnUQR8+dGjDu3FBqE?=
 =?us-ascii?Q?zL55t1qwfeEeWnXMyYKsoyF70OZ+wq6I789AtF9JEwcjKMf40J3bhMcQKT5D?=
 =?us-ascii?Q?GiPn/FX7BqLtS7L3h+fvRYo/WsCGjTaVK3DZdffzKQRmg2H4wCauWR3Jr1at?=
 =?us-ascii?Q?mlsSQklRHSedfTctr5epUXgiuekiSgKKc98Xf3kwCG6951XD83ysQrN4PTRV?=
 =?us-ascii?Q?RjmrrQUQ29QmXdaCzmocnht96SdIUmMIEW3kd2s7CHDWn6aXmtJ6Yt1bBttz?=
 =?us-ascii?Q?urz7mkism+9OTtzZUsbd5pjVe820GHOIjLZExxkMzO8OhPiF2rLO+1tdaLQu?=
 =?us-ascii?Q?kEdsIM0OO3Yf1jjqH2TQlPPufkHuMaFBsC1g09VGwK/6e8/WvDQIKNXjj4NS?=
 =?us-ascii?Q?e7rpQT7MkyO0r5+bihYD5WXHjlALxYt+LxHVZwwQAG2aBmVtGdiwNcqeqjt8?=
 =?us-ascii?Q?Ouvtg8mpg6jz3ouUcO3Mcm49+9666v0yjx6Uyr04gncKJFqz8HFH54/iu6RL?=
 =?us-ascii?Q?H/iujplbVSJWDFx6jUiz/tlMBvQhrqDBKmvAqjT+Ue4wZpe5DN7Zk3LjDG/Z?=
 =?us-ascii?Q?HtNhxhgKVFFgIzHVEksMJ74p5bfGeXJS+pPyIKSlzL91s5zBGskHkyLrtHIK?=
 =?us-ascii?Q?jikzP7OtjUwnz7skdT9toU3/XeiZhhwIFFGXPxd0sL8UHiv2SZkvBViTu4hL?=
 =?us-ascii?Q?rAiflUIDS+3iL7th61iC7tTMnXlIMSvge9wQ/5SIPwhNaoU3l4rWbrmSSouR?=
 =?us-ascii?Q?N9qtDwtJB4CmXq+i9ugYjcqYYMbUeUXH93/JTDaBOeUlkDHzUgJFDfP23nNE?=
 =?us-ascii?Q?tleMZXw8l2SgSiBhuMYZZDaiHde9DNhOWIICHHsSBT/kqKGIqTKpZebd1Vph?=
 =?us-ascii?Q?vvHmzzOlO1WSHqvqHW9aBog59lxDpDklx2dUc7zlfkxTA1oplpWn01Wf/tDj?=
 =?us-ascii?Q?bt5jt2fSNOqXIbECn95crgAjj4yi4wbMDUATnlRaMkfDzwWZP2+Z4G68962/?=
 =?us-ascii?Q?JDaz5n8F6hTd/8p/Q3BtX8J9m/IPm1UT8/e/b/iLyLHv7hbCXBU/Pmxz/GaP?=
 =?us-ascii?Q?x8VWC0bv7mVCUrPX/k1orb9hV3Fp3am0DTg7c8glf1r/c93sZjYMy4iu70Vd?=
 =?us-ascii?Q?qnI/AQ+Y4EPwidJJBBloJ1omGISuOhJSx0a67nQvgCB87hpr9nyL395UnKGx?=
 =?us-ascii?Q?RFH0zBwRitX9Q4LQLdywl42nMuEzHxwn7LkP8gp/4hC8r4OOwKsfzdz+r9NX?=
 =?us-ascii?Q?EXXATQFaLn7yfyPBSlYlGK0d4umVF6G1HJtx1eZbkrizRDitiexdCsG5jxge?=
 =?us-ascii?Q?vQ5C+/pY/ov+MSmpEuPzoSHNjqJ8xV2fTTcpcKfh+3MHuMsGlOJMpuK/WWA8?=
 =?us-ascii?Q?kFeiBo5S0NQHVhd584o4/Vtk7xrpnI9S7kjezvV7RMGf/5aAHO61s3r48uMm?=
 =?us-ascii?Q?PA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 88e399e5-dfb3-49a0-9385-08db553a434c
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 May 2023 11:48:13.0406
 (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: ELUwMl18mkZyFHApPDwHdDGbT0FNtQAAbz9jW+dAMbrmDj0c4Iw0GQOxZ4GgHEQIIDycZzQMB7HWO9cz+AAOsw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB9977
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Maxime,

On Fri, Mar 24, 2023 at 10:36:44AM +0100, Maxime Chevallier wrote:
> Now that we can easily create a mdio-device that represents a
> memory-mapped device that exposes an MDIO-like register layout, we don't
> need the Altera TSE PCS anymore, since we can use the Lynx PCS instead.
> 
> Signed-off-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
> ---
> -static int tse_pcs_reset(struct altera_tse_pcs *tse_pcs)
> -{
> -	u16 bmcr;
> -
> -	/* Reset PCS block */
> -	bmcr = tse_pcs_read(tse_pcs, MII_BMCR);
> -	bmcr |= BMCR_RESET;
> -	tse_pcs_write(tse_pcs, MII_BMCR, bmcr);
> -
> -	return read_poll_timeout(tse_pcs_read, bmcr, (bmcr & BMCR_RESET),
> -				 10, SGMII_PCS_SW_RESET_TIMEOUT, 1,
> -				 tse_pcs, MII_BMCR);
> -}

I just noticed this difference between the Lynx PCS and Altera TSE PCS
drivers. The Lynx driver doesn't reset the PCS, and if it did, it would
wait until the opposite condition from yours would be true: BMCR_RESET
should clear: "!(bmcr & BMCR_RESET)".

Is your reset procedure correct, I wonder?