Thread (8 messages) 8 messages, 4 authors, 2015-12-29

Re: [PATCH 0/3] drivers: net: cpsw: phy-handle fixes

From: David Rivshin (Allworx) <hidden>
Date: 2015-12-23 21:51:50
Also in: linux-arm-kernel, linux-devicetree, linux-omap

On Wed, 23 Dec 2015 19:35:37 +0100
Markus Brunner [off-list ref] wrote:
On Wednesday 23 December 2015 12:04:25 David Miller wrote:
quoted
From: "David Rivshin (Allworx)" <redacted>
Date: Tue, 22 Dec 2015 19:36:31 -0500
quoted
Testing by anyone who has real hardware using phy-handle or
dual_emac with fixed-link would be appreciated.
I'm going to wait for such testing before applying this series.

Thanks.
Successfully tested the following 3 configurations.
1. emac0 with phy_id and emac1 with fixed phy
2. emac0 with phy-handle and emac1 with fixed phy
3. emac0 with fixed phy and emac1 with fixed phy
Great, thanks for testing. Using the same technique
for the phy-handle case as you, I also just tested:
 - (EVMSK) dual emac, phy-handle property in both slaves

I think that covers all the interesting cases.

Dave,
 I actually just received a note off-list reporting a problem  
with this series on the dm8148-t410 board. So please hold off  
applying this series for now. If it turns out to be a real  
problem I'll have a v2. 

[...]
&davinci_mdio {
	status = "okay";
	phy0: ethernet-phy@0 {
		reg = <5>;
	};
};
I was unaware that the davinci-mdio driver creates PHY devices
from child nodes. The davinci-mdio.txt binding documentation 
makes no mention of that. By comparison the emac_rockchip.txt 
file does talk about it.

Now that I take a closer look at the code, it looks like that 
capability was added in commit 0a0ea0687281 ("net: davinci_mdio: 
allow to create phys from dt"), but it didn't update the binding.

Grygorii, was that just an oversight, or capability that's not
supposed to be used? 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help