[PATCH] net: ethernet: mtk-star-emac: Use of_get_available_child_by_name()

Biju Das biju.das.jz at bp.renesas.com
Sun Feb 2 02:14:32 PST 2025


Hi Andrew,

> -----Original Message-----
> From: Andrew Lunn <andrew at lunn.ch>
> Sent: 01 February 2025 18:47
> Subject: Re: [PATCH] net: ethernet: mtk-star-emac: Use of_get_available_child_by_name()
> 
> On Sat, Feb 01, 2025 at 04:21:32PM +0000, Biju Das wrote:
> > Use the helper of_get_available_child_by_name() to simplify
> > mtk_star_mdio_init().
> >
> > Signed-off-by: Biju Das <biju.das.jz at bp.renesas.com>
> > ---
> > This patch is only compile tested and depend upon[1] [1]
> > https://lore.kernel.org/all/20250201093126.7322-1-biju.das.jz@bp.renes
> > as.com/
> > ---
> >  drivers/net/ethernet/mediatek/mtk_star_emac.c | 24
> > ++++---------------
> >  1 file changed, 5 insertions(+), 19 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/mediatek/mtk_star_emac.c
> > b/drivers/net/ethernet/mediatek/mtk_star_emac.c
> > index 25989c79c92e..beb0500fe9d5 100644
> > --- a/drivers/net/ethernet/mediatek/mtk_star_emac.c
> > +++ b/drivers/net/ethernet/mediatek/mtk_star_emac.c
> > @@ -1422,25 +1422,15 @@ static int mtk_star_mdio_init(struct
> > net_device *ndev)  {
> >  	struct mtk_star_priv *priv = netdev_priv(ndev);
> >  	struct device *dev = mtk_star_get_dev(priv);
> > -	struct device_node *of_node, *mdio_node;
> > -	int ret;
> > -
> > -	of_node = dev->of_node;
> > +	struct device_node *mdio_node _free(device_node) =
> > +		of_get_available_child_by_name(dev->of_node, "mdio");
> 
> Same comment as for you other similar patch.

Ok, Will send next version with readable code once
dependency patch hits on net-next.

Cheers,
Biju



More information about the linux-arm-kernel mailing list