mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-07 11:28:32 +00:00
net: ethernet: cpsw-phy-sel: prefer phandle for phy sel
The cpsw-phy-sel device is not a child of the cpsw interconnect target module. It lives in the system control module. Let's fix this issue by trying to use cpsw-phy-sel phandle first if it exists and if not fall back to current usage of trying to find the cpsw-phy-sel child. That way the phy sel driver can be a child of the system control module where it belongs in the device tree. Without this fix, we cannot have a proper interconnect target module hierarchy in device tree for things like genpd. Note that deferred probe is mostly not supported by cpsw and this patch does not attempt to fix that. In case deferred probe support is needed, this could be added to cpsw_slave_open() and phy_connect() so they start handling and returning errors. For documenting it, looks like the cpsw-phy-sel is used for all cpsw device tree nodes. It's missing the related binding documentation, so let's also update the binding documentation accordingly. Cc: devicetree@vger.kernel.org Cc: Andrew Lunn <andrew@lunn.ch> Cc: Grygorii Strashko <grygorii.strashko@ti.com> Cc: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Murali Karicheri <m-karicheri2@ti.com> Cc: Rob Herring <robh+dt@kernel.org> Signed-off-by: Tony Lindgren <tony@atomide.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
10d7fac4c5
commit
18eb8aea7f
1 changed files with 6 additions and 3 deletions
|
@ -170,10 +170,13 @@ void cpsw_phy_sel(struct device *dev, phy_interface_t phy_mode, int slave)
|
|||
struct device_node *node;
|
||||
struct cpsw_phy_sel_priv *priv;
|
||||
|
||||
node = of_get_child_by_name(dev->of_node, "cpsw-phy-sel");
|
||||
node = of_parse_phandle(dev->of_node, "cpsw-phy-sel", 0);
|
||||
if (!node) {
|
||||
dev_err(dev, "Phy mode driver DT not found\n");
|
||||
return;
|
||||
node = of_get_child_by_name(dev->of_node, "cpsw-phy-sel");
|
||||
if (!node) {
|
||||
dev_err(dev, "Phy mode driver DT not found\n");
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
dev = bus_find_device(&platform_bus_type, NULL, node, match);
|
||||
|
|
Loading…
Reference in a new issue