[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <50B71290.2060200@sysgo.com>
Date: Thu, 29 Nov 2012 08:45:20 +0100
From: Alexander Sverdlin <asv@...go.com>
To: Stephen Warren <swarren@...dotorg.org>,
devicetree-discuss@...ts.ozlabs.org,
Rob Herring <rob.herring@...xeda.com>,
Grant Likely <grant.likely@...retlab.ca>
CC: alexander sverdlin <alexander.sverdlin.ext@....com>,
w.sang@...gutronix.de, Barry.Song@....com, netdev@...r.kernel.org
Subject: [PATCH 1/2] of_mdio: Honour "status=disabled" property of device
From: Alexander Sverdlin <alexander.sverdlin@...go.com>
of_mdio: Honour "status=disabled" property of device
Currently of_mdiobus_register() function registers all PHY devices,
independetly from their status property in device tree. According to
"ePAPR 1.1" spec, device should only be registered if there is no
"status" property, or it has "ok" (or "okay") value (see
of_device_is_available()). In case of "platform devices",
of_platform_device_create_pdata() checks for "status" and ensures
that disabled devices are not pupulated. But such check for MDIO buses
was missing until now. Fix it.
Signed-off-by: Alexander Sverdlin <alexander.sverdlin@...go.com>
---
--- linux.orig/drivers/of/of_mdio.c
+++ linux/drivers/of/of_mdio.c
@@ -53,7 +53,7 @@ int of_mdiobus_register(struct mii_bus *
return rc;
/* Loop over the child nodes and register a phy_device for each one */
- for_each_child_of_node(np, child) {
+ for_each_available_child_of_node(np, child) {
const __be32 *paddr;
u32 addr;
int len;
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists