All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org>
To: Bert Lindner <bert-phGP8mJ4Mqk@public.gmane.org>
Cc: "Andreas Färber" <afaerber-l3A5Bk7waGM@public.gmane.org>,
	"Robin Murphy" <robin.murphy-5wv7dgnIgG8@public.gmane.org>,
	"Maxime Ripard"
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	"Giuseppe Cavallaro"
	<peppe.cavallaro-qxv4g6HH51o@public.gmane.org>
Subject: Re: 4.5.0 on sun7i-a20-olinuxino-lime2: libphy: PHY stmmac-0:ffffffff not found (regression from rc7)
Date: Thu, 17 Mar 2016 07:52:02 +0000	[thread overview]
Message-ID: <20160317075202.16b14ed6@arm.com> (raw)
In-Reply-To: <56E9F2B8.1030605-phGP8mJ4Mqk@public.gmane.org>

On Thu, 17 Mar 2016 00:56:40 +0100
Bert Lindner <bert-phGP8mJ4Mqk@public.gmane.org> wrote:

> On 2016-03-16 18:42, Marc Zyngier wrote:
> > On 16/03/16 15:10, Bert Lindner wrote:
> >> On 2016-03-16 14:10, Andreas Färber wrote:
> >>> Am 16.03.2016 um 13:09 schrieb Robin Murphy:
> >>>> On 16/03/16 11:39, Marc Zyngier wrote:
> >>>>> On 16/03/16 11:19, Bert Lindner wrote:
> >>>>>> Hopefully this is the correct place and way to report this.
> >>>
> >>> The main discussion is on netdev list actually, CC'ed.
> >>>
> >>>>>> For the board sun7i-a20-olinuxino-lime2, there seems to be a problem
> >>>>>> with the eth0 PHY in mainline kernel 4.5.0 that developed since
> >>>>>> 4.5.0-rc7. Ethernet does not work, although eth0 is reported:
> >>>>>>
> >>>>>> root@lime2-079f:~# ip a l eth0
> >>>>>> 2: eth0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group
> >>>>>> default qlen 1000
> >>>>>>         link/ether 02:c9:05:02:07:9f brd ff:ff:ff:ff:ff:ff
> >>>>>>
> >>>>>>      Difference reported in dmesg:
> >>>>>>
> >>>>>> 4.5.0-rc7:
> >>>>>> [    9.379279] NET: Registered protocol family 10
> >>>>>> [   10.217148]  RX IPC Checksum Offload disabled
> >>>>>> [   10.217195]  No MAC Management Counters available
> >>>>>> [   10.217627] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> >>>>>> [   15.206250] sun7i-dwmac 1c50000.ethernet eth0: Link is Up -
> >>>>>> 1Gbps/Full - flow control off
> >>>>>> [   15.206360] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> >>>>>>
> >>>>>> 4.5.0:
> >>>>>> [    9.767125] NET: Registered protocol family 10
> >>>>>> [   10.357405] libphy: PHY stmmac-0:ffffffff not found
> >>>>>> [   10.362382] eth0: Could not attach to PHY
> >>>>>> [   10.366557] stmmac_open: Cannot attach to PHY (error: -19)
> >>>>>>
> >>>>>> .config is identical for both, also after make oldconfig, apart from
> >>>>>> comment with version number. DTB file is also identical between the two
> >>>>>> versions.
> >>>>>>
> >>>>>> Kernels are compiled on the board itself. /proc/version string:
> >>>>>> Linux version 4.5.0-rc7 (root@lime2-079f) (gcc version 4.9.1
> >>>>>> (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #1 SMP Mon Mar 7 11:57:25 UTC 2016
> >>>>>> Linux version 4.5.0 (root@lime2-079f) (gcc version 4.9.1 (Ubuntu/Linaro
> >>>>>> 4.9.1-16ubuntu6) ) #1 SMP Tue Mar 15 11:39:01 UTC 2016
> >>>>>>
> >>>>>> Please let me know if more info is needed, if I should post complete
> >>>>>> .config, test compile with a particular config or patch, etc. Part of
> >>>>>> .config below.
> >>>>>
> >>>>> Can you please try reverting 88f8b1b ("stmmac: Fix 'eth0: No PHY found'
> >>>>> regression") and report whether or not this changes anything? This seems
> >>>>> to be the only stmac patch between -rc7 and release...
> >>>>
> >>>> Sounds like the same thing as the giant ongoing discussion thread here:
> >>>>
> >>>> http://thread.gmane.org/gmane.linux.drivers.devicetree/159007/focus=402830
> >>>
> >>> v4 fixes for 4.5 are here:
> >>>
> >>> https://patchwork.ozlabs.org/patch/598195/ (revert)
> >>> https://patchwork.ozlabs.org/patch/598196/
> >>>
> >>> v2 fixes for linux-next here:
> >>>
> >>> https://patchwork.ozlabs.org/patch/598331/ (revert)
> >>> https://patchwork.ozlabs.org/patch/598332/
> >>>
> >>> Please let Peppe know whether they work for you guys.
> >>
> >> Hi guys - I can confirm 4.5.0 minus 88f8b1b works for me:
> >>
> >> root@lime2-079f:~# cat /proc/version
> >> Linux version 4.5.0-minus-88f8b1b (root@lime2-079f) (gcc version 4.9.1
> >> (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #2 SMP Wed Mar 16 12:50:03 UTC 2016
> >>
> >>   From dmesg output:
> >> [    9.731730] NET: Registered protocol family 10
> >> [   10.516893]  RX IPC Checksum Offload disabled
> >> [   10.516948]  No MAC Management Counters available
> >> [   10.517374] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> >> [   15.505548] sun7i-dwmac 1c50000.ethernet eth0: Link is Up -
> >> 1Gbps/Full - flow control off
> >> [   15.505660] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> >>
> >> .. and connected over ethernet.
> >
> > Good to know, thanks. Could you also give the potential fix a go (as
> > mentioned by Andreas)? Just to make sure that whatever gets merged next
> > will actually fix the issue.
> 
> Yes sure, it took a while because I had to travel. Confirmed, the 
> v4-for-4.5 fix works well for me, on sun7i-a20-olinuxino-lime2:
> 
> root@lime2-079f:~# cat /proc/version
> Linux version 4.5.0-598195-598196-v4 (root@lime2-079f) (gcc version 
> 4.9.1 (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #1 SMP Wed Mar 16 16:44:22 UTC 2016
> 
> dmesg:
> [    8.245273] NET: Registered protocol family 10
> [    9.297406]  RX IPC Checksum Offload disabled
> [    9.297460]  No MAC Management Counters available
> [    9.297951] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> [   16.285658] sun7i-dwmac 1c50000.ethernet eth0: Link is Up - 
> 1Gbps/Full - flow control rx/tx
> [   16.285798] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> 
> The board is connected to my laptop rather than to a switch, so that 
> might be where the flow control message comes from (not sure). Anyway 
> ethernet works.

Cool, many thanks for taking the time to test and report.

Hopefully Giuseppe will get this merged quickly enough in mainline, and
it should then trickle into a 4.5-stable release (cc-ing stable on
these patches would probably be a good idea, BTW).

> I'm less familiar with linux-next - not sure what base to test against, 
> I understand it is being redefined almost daily.

You can consider -next as a daily view of what mainline could become.
An over-sized crystal ball, in a way... ;-) Given that the regression
exists in mainline, there was no real point testing with -next.

Thanks again,

	M.
-- 
Jazz is not dead. It just smells funny.

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

WARNING: multiple messages have this Message-ID (diff)
From: marc.zyngier@arm.com (Marc Zyngier)
To: linux-arm-kernel@lists.infradead.org
Subject: 4.5.0 on sun7i-a20-olinuxino-lime2: libphy: PHY stmmac-0:ffffffff not found (regression from rc7)
Date: Thu, 17 Mar 2016 07:52:02 +0000	[thread overview]
Message-ID: <20160317075202.16b14ed6@arm.com> (raw)
In-Reply-To: <56E9F2B8.1030605@a20.net>

On Thu, 17 Mar 2016 00:56:40 +0100
Bert Lindner <bert@a20.net> wrote:

> On 2016-03-16 18:42, Marc Zyngier wrote:
> > On 16/03/16 15:10, Bert Lindner wrote:
> >> On 2016-03-16 14:10, Andreas F?rber wrote:
> >>> Am 16.03.2016 um 13:09 schrieb Robin Murphy:
> >>>> On 16/03/16 11:39, Marc Zyngier wrote:
> >>>>> On 16/03/16 11:19, Bert Lindner wrote:
> >>>>>> Hopefully this is the correct place and way to report this.
> >>>
> >>> The main discussion is on netdev list actually, CC'ed.
> >>>
> >>>>>> For the board sun7i-a20-olinuxino-lime2, there seems to be a problem
> >>>>>> with the eth0 PHY in mainline kernel 4.5.0 that developed since
> >>>>>> 4.5.0-rc7. Ethernet does not work, although eth0 is reported:
> >>>>>>
> >>>>>> root at lime2-079f:~# ip a l eth0
> >>>>>> 2: eth0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN group
> >>>>>> default qlen 1000
> >>>>>>         link/ether 02:c9:05:02:07:9f brd ff:ff:ff:ff:ff:ff
> >>>>>>
> >>>>>>      Difference reported in dmesg:
> >>>>>>
> >>>>>> 4.5.0-rc7:
> >>>>>> [    9.379279] NET: Registered protocol family 10
> >>>>>> [   10.217148]  RX IPC Checksum Offload disabled
> >>>>>> [   10.217195]  No MAC Management Counters available
> >>>>>> [   10.217627] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> >>>>>> [   15.206250] sun7i-dwmac 1c50000.ethernet eth0: Link is Up -
> >>>>>> 1Gbps/Full - flow control off
> >>>>>> [   15.206360] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> >>>>>>
> >>>>>> 4.5.0:
> >>>>>> [    9.767125] NET: Registered protocol family 10
> >>>>>> [   10.357405] libphy: PHY stmmac-0:ffffffff not found
> >>>>>> [   10.362382] eth0: Could not attach to PHY
> >>>>>> [   10.366557] stmmac_open: Cannot attach to PHY (error: -19)
> >>>>>>
> >>>>>> .config is identical for both, also after make oldconfig, apart from
> >>>>>> comment with version number. DTB file is also identical between the two
> >>>>>> versions.
> >>>>>>
> >>>>>> Kernels are compiled on the board itself. /proc/version string:
> >>>>>> Linux version 4.5.0-rc7 (root at lime2-079f) (gcc version 4.9.1
> >>>>>> (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #1 SMP Mon Mar 7 11:57:25 UTC 2016
> >>>>>> Linux version 4.5.0 (root at lime2-079f) (gcc version 4.9.1 (Ubuntu/Linaro
> >>>>>> 4.9.1-16ubuntu6) ) #1 SMP Tue Mar 15 11:39:01 UTC 2016
> >>>>>>
> >>>>>> Please let me know if more info is needed, if I should post complete
> >>>>>> .config, test compile with a particular config or patch, etc. Part of
> >>>>>> .config below.
> >>>>>
> >>>>> Can you please try reverting 88f8b1b ("stmmac: Fix 'eth0: No PHY found'
> >>>>> regression") and report whether or not this changes anything? This seems
> >>>>> to be the only stmac patch between -rc7 and release...
> >>>>
> >>>> Sounds like the same thing as the giant ongoing discussion thread here:
> >>>>
> >>>> http://thread.gmane.org/gmane.linux.drivers.devicetree/159007/focus=402830
> >>>
> >>> v4 fixes for 4.5 are here:
> >>>
> >>> https://patchwork.ozlabs.org/patch/598195/ (revert)
> >>> https://patchwork.ozlabs.org/patch/598196/
> >>>
> >>> v2 fixes for linux-next here:
> >>>
> >>> https://patchwork.ozlabs.org/patch/598331/ (revert)
> >>> https://patchwork.ozlabs.org/patch/598332/
> >>>
> >>> Please let Peppe know whether they work for you guys.
> >>
> >> Hi guys - I can confirm 4.5.0 minus 88f8b1b works for me:
> >>
> >> root at lime2-079f:~# cat /proc/version
> >> Linux version 4.5.0-minus-88f8b1b (root at lime2-079f) (gcc version 4.9.1
> >> (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #2 SMP Wed Mar 16 12:50:03 UTC 2016
> >>
> >>   From dmesg output:
> >> [    9.731730] NET: Registered protocol family 10
> >> [   10.516893]  RX IPC Checksum Offload disabled
> >> [   10.516948]  No MAC Management Counters available
> >> [   10.517374] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> >> [   15.505548] sun7i-dwmac 1c50000.ethernet eth0: Link is Up -
> >> 1Gbps/Full - flow control off
> >> [   15.505660] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> >>
> >> .. and connected over ethernet.
> >
> > Good to know, thanks. Could you also give the potential fix a go (as
> > mentioned by Andreas)? Just to make sure that whatever gets merged next
> > will actually fix the issue.
> 
> Yes sure, it took a while because I had to travel. Confirmed, the 
> v4-for-4.5 fix works well for me, on sun7i-a20-olinuxino-lime2:
> 
> root at lime2-079f:~# cat /proc/version
> Linux version 4.5.0-598195-598196-v4 (root at lime2-079f) (gcc version 
> 4.9.1 (Ubuntu/Linaro 4.9.1-16ubuntu6) ) #1 SMP Wed Mar 16 16:44:22 UTC 2016
> 
> dmesg:
> [    8.245273] NET: Registered protocol family 10
> [    9.297406]  RX IPC Checksum Offload disabled
> [    9.297460]  No MAC Management Counters available
> [    9.297951] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> [   16.285658] sun7i-dwmac 1c50000.ethernet eth0: Link is Up - 
> 1Gbps/Full - flow control rx/tx
> [   16.285798] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> 
> The board is connected to my laptop rather than to a switch, so that 
> might be where the flow control message comes from (not sure). Anyway 
> ethernet works.

Cool, many thanks for taking the time to test and report.

Hopefully Giuseppe will get this merged quickly enough in mainline, and
it should then trickle into a 4.5-stable release (cc-ing stable on
these patches would probably be a good idea, BTW).

> I'm less familiar with linux-next - not sure what base to test against, 
> I understand it is being redefined almost daily.

You can consider -next as a daily view of what mainline could become.
An over-sized crystal ball, in a way... ;-) Given that the regression
exists in mainline, there was no real point testing with -next.

Thanks again,

	M.
-- 
Jazz is not dead. It just smells funny.

  parent reply	other threads:[~2016-03-17  7:52 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-16 11:19 4.5.0 on sun7i-a20-olinuxino-lime2: libphy: PHY stmmac-0:ffffffff not found (regression from rc7) Bert Lindner
2016-03-16 11:39 ` Marc Zyngier
2016-03-16 12:09   ` Robin Murphy
2016-03-16 13:06     ` Marc Zyngier
     [not found]     ` <56E94CF4.2060607-5wv7dgnIgG8@public.gmane.org>
2016-03-16 13:10       ` Andreas Färber
2016-03-16 13:10         ` Andreas Färber
     [not found]         ` <56E95B57.1090100-l3A5Bk7waGM@public.gmane.org>
2016-03-16 15:10           ` Bert Lindner
2016-03-16 15:10             ` Bert Lindner
     [not found]             ` <56E9776F.3070102-phGP8mJ4Mqk@public.gmane.org>
2016-03-16 17:42               ` Marc Zyngier
2016-03-16 17:42                 ` Marc Zyngier
     [not found]                 ` <56E99B0C.2070609-5wv7dgnIgG8@public.gmane.org>
2016-03-16 23:56                   ` Bert Lindner
2016-03-16 23:56                     ` Bert Lindner
     [not found]                     ` <56E9F2B8.1030605-phGP8mJ4Mqk@public.gmane.org>
2016-03-17  7:52                       ` Marc Zyngier [this message]
2016-03-17  7:52                         ` Marc Zyngier
2016-05-20  5:44                         ` Andre Heider
2016-05-20  5:44                           ` Andre Heider
     [not found]                           ` <CAHsu+b-t_r93XKLJTUcin6xjhOrFVn3_60vwHkbRt3iHK28CbA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-05-20  7:56                             ` Marc Zyngier
2016-05-20  7:56                               ` Marc Zyngier
     [not found]                               ` <573EC33D.2010209-5wv7dgnIgG8@public.gmane.org>
2016-05-20  8:14                                 ` Giuseppe CAVALLARO
2016-05-20  8:14                                   ` Giuseppe CAVALLARO
     [not found]                                   ` <573EC77A.2000800-qxv4g6HH51o@public.gmane.org>
2016-05-20 10:30                                     ` Andre Heider
2016-05-20 10:30                                       ` Andre Heider
2016-05-20 10:36                                       ` Marc Zyngier
2016-05-20 10:36                                         ` Marc Zyngier
     [not found]                                         ` <573EE8B4.1050701-5wv7dgnIgG8@public.gmane.org>
2016-05-20 10:43                                           ` Andre Heider
2016-05-20 10:43                                             ` Andre Heider
2016-05-21 15:47                                           ` Bert Lindner
2016-05-21 15:47                                             ` Bert Lindner
     [not found]                                             ` <574082F9.7010702-phGP8mJ4Mqk@public.gmane.org>
2016-05-22  9:30                                               ` Marc Zyngier
2016-05-22  9:30                                                 ` Marc Zyngier
     [not found]                                                 ` <20160522103056.2fa86293-5wv7dgnIgG8@public.gmane.org>
2016-05-23 13:28                                                   ` Giuseppe CAVALLARO
2016-05-23 13:28                                                     ` Giuseppe CAVALLARO
2016-05-23 13:28                                                     ` Giuseppe CAVALLARO

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160317075202.16b14ed6@arm.com \
    --to=marc.zyngier-5wv7dgnigg8@public.gmane.org \
    --cc=afaerber-l3A5Bk7waGM@public.gmane.org \
    --cc=bert-phGP8mJ4Mqk@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
    --cc=maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org \
    --cc=netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=peppe.cavallaro-qxv4g6HH51o@public.gmane.org \
    --cc=robin.murphy-5wv7dgnIgG8@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.