Re: [PATCH] net: Explicitly include correct DT includes

[...] > diff --git a/drivers/net/wireless/silabs/wfx/bus_sdio.c > b/drivers/net/wireless/silabs/wfx/bus_sdio.c > index 51a0d58a9070..909d5f346a01 100644 > --- a/drivers/net/wireless/silabs/wfx/bus_sdio.c > +++ b/drivers/net/wireless/silabs/wfx/bus_sdio.c > @@ -10,7 +10,7 @@ > #include > #include > #include > -#include > +#include > #include > #include > #include For 'drivers/net/wireless/silabs/wfx/bus_sdio.c': Reviewed-by: Jérôme Pouiller -- Jérôme Pouiller

Re: Nested function in drivers/of/of_mdio.c

ttempt? > > On Wed, Oct 7, 2009 at 9:11 AM, Grant Likely wrote: > > On Wed, Oct 7, 2009 at 9:15 AM, Jérôme Pouiller wrote: > >> Dear, > >> > >> I have a problem with commit > >> 8bc487d150b939e69830c39322df4ee486efe381 in file > >> driver

[PATCH] Remove nested function

Some toolchains dislike nested function definition, so we define function match outside of of_phy_find_device. Signed-off-by: Jérôme Pouiller --- drivers/of/of_mdio.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/of/of_mdio.c b/drivers/of/of_mdio.c

Nested function in drivers/of/of_mdio.c

- return dev_archdata_get_node(&dev->archdata) == phy_np; - } - if (!phy_np) return NULL; What do you think about it? Best regards, -- Jérôme Pouiller (jezz AT sysmic DOT org) ___ Linuxppc-dev mailing list