Marek, In the future if you're doing a large series like this please do a cover letter outlining what the series is trying to achieve.
Peter On Sun, 9 Feb 2025 at 12:04, Marek Vasut <[email protected]> wrote: > The init function does nothing, the bb_miiphy_init() already checks > whether the .init callback is assigned, and if not, skips calling it. > Remove the empty init function. The entire init callback will be > removed in follow up patches. > > Signed-off-by: Marek Vasut <[email protected]> > --- > Cc: Christian Marangi <[email protected]> > Cc: Ilias Apalodimas <[email protected]> > Cc: Jerome Forissier <[email protected]> > Cc: Jim Liu <[email protected]> > Cc: Joe Hershberger <[email protected]> > Cc: Mario Six <[email protected]> > Cc: Michal Simek <[email protected]> > Cc: Nobuhiro Iwamatsu <[email protected]> > Cc: Paul Barker <[email protected]> > Cc: Ramon Fried <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Sughosh Ganu <[email protected]> > Cc: Tom Rini <[email protected]> > Cc: [email protected] > --- > V2: Drop the = NULL assignment early in here. > --- > drivers/net/ravb.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/net/ravb.c b/drivers/net/ravb.c > index 7286ad19598..5df557da7da 100644 > --- a/drivers/net/ravb.c > +++ b/drivers/net/ravb.c > @@ -560,11 +560,6 @@ static int ravb_remove(struct udevice *dev) > return 0; > } > > -static int ravb_bb_init(struct bb_miiphy_bus *bus) > -{ > - return 0; > -} > - > static int ravb_bb_mdio_active(struct bb_miiphy_bus *bus) > { > struct ravb_priv *eth = bus->priv; > @@ -626,7 +621,6 @@ static int ravb_bb_delay(struct bb_miiphy_bus *bus) > struct bb_miiphy_bus bb_miiphy_buses[] = { > { > .name = "ravb", > - .init = ravb_bb_init, > .mdio_active = ravb_bb_mdio_active, > .mdio_tristate = ravb_bb_mdio_tristate, > .set_mdio = ravb_bb_set_mdio, > -- > 2.47.2 > >
