From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8CE2DC7EE23 for ; Mon, 22 May 2023 16:16:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229679AbjEVQQP (ORCPT ); Mon, 22 May 2023 12:16:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59338 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229555AbjEVQQN (ORCPT ); Mon, 22 May 2023 12:16:13 -0400 Received: from vps0.lunn.ch (vps0.lunn.ch [156.67.10.101]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A2EBAE9; Mon, 22 May 2023 09:16:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=1oYReIb21gx1lPn/zOspt867lqTrE9boyBC6cBnhc5M=; b=AV4hSZlfawpDM8oTuN7UkXIApw SF2R8kXwSp2pzbHldYbTZj6T+X71XjBjJEDHx1nLw22MMmUZxJwHoBfFi54awY/51YCekOG6qJjmN We9HBof4Sg/MUrD8PLSl4M1jPW8XX04Z5FhXZgCHAWKFY1pPhtHuQ22IM0t0nxkPbDbE=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1q18CS-00DZ3h-AB; Mon, 22 May 2023 18:15:56 +0200 Date: Mon, 22 May 2023 18:15:56 +0200 From: Andrew Lunn To: Francesco Dolcini Cc: Praneeth Bajjuri , Geet Modi , "David S. Miller" , Heiner Kallweit , Russell King , Eric Dumazet , Jakub Kicinski , Paolo Abeni , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Grygorii Strashko , Dan Murphy Subject: Re: DP83867 ethernet PHY regression Message-ID: <07037ce6-8d1c-4a1d-9fdd-2cd9e68c4594@lunn.ch> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 22, 2023 at 05:35:33PM +0200, Francesco Dolcini wrote: > On Mon, May 22, 2023 at 05:15:56PM +0200, Andrew Lunn wrote: > > On Mon, May 22, 2023 at 04:58:46PM +0200, Francesco Dolcini wrote: > > > Hello all, > > > commit da9ef50f545f ("net: phy: dp83867: perform soft reset and retain > > > established link") introduces a regression on my TI AM62 based board. > > > > > > I have a working DTS with Linux TI 5.10 downstream kernel branch, while > > > testing the DTS with v6.4-rc in preparation of sending it to the mailing > > > list I noticed that ethernet is working only on a cold poweron. > > > > Do you have more details about how it does not work. > > > > Please could you use: > > > > mii-tool -vvv ethX > > please see the attached files: > > working_da9ef50f545f_reverted.txt > this is on a v6.4-rc, with da9ef50f545f reverted > > not_working.txt > v6.4-rc not working > > working.txt > v6.4-rc working > > > It looks like, even on cold boot, it's not working in a reliable way. > Not sure the exact difference when it's working and when it's not. > Using SIOCGMIIPHY=0x8947 > eth0: negotiated 1000baseT-FD flow-control, link ok > registers for MII PHY 0: > 1140 796d 2000 a231 05e1 c5e1 006f 2001 > 5806 0200 3800 0000 0000 4007 0000 3000 > 5048 ac02 ec10 0004 2bc7 0000 0000 0040 > 6150 4444 0002 0000 0000 0000 0282 0000 > 1140 796d 2000 a231 05e1 c5e1 006d 2001 > 5806 0200 3800 0000 0000 4007 0000 3000 > 5048 af02 ec10 0000 2bc7 0000 0000 0040 > 6150 4444 0002 0000 0000 0000 0282 0000 Register 6: 006f vs 006d Register 17: ac02 vs 1f02 Register 19: 0004 vs 0000 Register 6 is MII_EXPANSION. Bit 1 is #define EXPANSION_LCWP 0x0002 /* Got new RX page code word */ So that is probably not relevant here. Register 17 is MII_DP83867_PHYSTS, and bits 8 and 9 are not documented in the driver. Do you have the datasheet? Register 19 is MII_DP83867_ISR. The interrupt bits are not documented in the driver either. This driver also uses C45 registers, which are not shown here. At some point, we might need to look at those. But first it would be good to understand what these differences mean. Andrew