On 05/11/2012 08:47 AM, Bjørn Mork wrote:
CONFIG_OF_MDIO is tristate and will be m if PHYLIB is m. Use IS_ENABLED macro to prevent build error: ERROR: "of_mdio_find_bus" [drivers/net/phy/mdio-mux.ko] undefined! Reported-by: Randy Dunlap<rdunlap@xxxxxxxxxxxx> Cc: David Daney<david.daney@xxxxxxxxxx> Signed-off-by: Bjørn Mork<bjorn@xxxxxxx>
I was able to reproduce the failure, and this patch both fixes it and seems correct, so...
Acked-by: David Daney<david.daney@xxxxxxxxxx> Sorry about this failure.
--- I wonder if this could be as banal as this? Not even build tested... Should be wrapped into commit 25106022 if it works, to ensure bisectability. Bjørn drivers/net/phy/mdio_bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/mdio_bus.c b/drivers/net/phy/mdio_bus.c index 83d5c9f..683ef1c 100644 --- a/drivers/net/phy/mdio_bus.c +++ b/drivers/net/phy/mdio_bus.c @@ -88,7 +88,7 @@ static struct class mdio_bus_class = { .dev_release = mdiobus_release, }; -#ifdef CONFIG_OF_MDIO +#if IS_ENABLED(CONFIG_OF_MDIO) /* Helper function for of_mdio_find_bus */ static int of_mdio_bus_match(struct device *dev, void *mdio_bus_np) {
-- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html