* [PATCH net-next 0/2] net: lan966x: some OF cleanups
@ 2025-08-27 21:50 Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 1/2] net: lan966x: use of_get_mac_address Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 2/2] net: lan966x: convert fwnode to of Rosen Penev
0 siblings, 2 replies; 6+ messages in thread
From: Rosen Penev @ 2025-08-27 21:50 UTC (permalink / raw)
To: netdev
Cc: Horatiu Vultur, maintainer:MICROCHIP LAN966X ETHERNET DRIVER,
Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Russell King
First commit basically adds NVMEM support and second one removes a lot
of fwnode usage.
Rosen Penev (2):
net: lan966x: use of_get_mac_address
net: lan966x: convert fwnode to of
.../ethernet/microchip/lan966x/lan966x_main.c | 41 ++++++++++---------
.../ethernet/microchip/lan966x/lan966x_main.h | 2 +-
2 files changed, 23 insertions(+), 20 deletions(-)
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH net-next 1/2] net: lan966x: use of_get_mac_address
2025-08-27 21:50 [PATCH net-next 0/2] net: lan966x: some OF cleanups Rosen Penev
@ 2025-08-27 21:50 ` Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 2/2] net: lan966x: convert fwnode to of Rosen Penev
1 sibling, 0 replies; 6+ messages in thread
From: Rosen Penev @ 2025-08-27 21:50 UTC (permalink / raw)
To: netdev
Cc: Horatiu Vultur, maintainer:MICROCHIP LAN966X ETHERNET DRIVER,
Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Russell King
As lan966x is an OF driver, switching to the OF version allows usage of
NVMEM to override the MAC address of the interface.
Handle EPROBE_DEFER in the case that NVMEM loads after lan966x.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
drivers/net/ethernet/microchip/lan966x/lan966x_main.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
index 7001584f1b7a..8bf28915c030 100644
--- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
+++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
@@ -1083,7 +1083,6 @@ static int lan966x_probe(struct platform_device *pdev)
{
struct fwnode_handle *ports, *portnp;
struct lan966x *lan966x;
- u8 mac_addr[ETH_ALEN];
int err;
lan966x = devm_kzalloc(&pdev->dev, sizeof(*lan966x), GFP_KERNEL);
@@ -1093,9 +1092,11 @@ static int lan966x_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, lan966x);
lan966x->dev = &pdev->dev;
- if (!device_get_mac_address(&pdev->dev, mac_addr)) {
- ether_addr_copy(lan966x->base_mac, mac_addr);
- } else {
+ err = of_get_mac_address(pdev->dev.of_node, lan966x->base_mac);
+ if (err == -EPROBE_DEFER)
+ return err;
+
+ if (err) {
pr_info("MAC addr was not set, use random MAC\n");
eth_random_addr(lan966x->base_mac);
lan966x->base_mac[5] &= 0xf0;
--
2.51.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH net-next 2/2] net: lan966x: convert fwnode to of
2025-08-27 21:50 [PATCH net-next 0/2] net: lan966x: some OF cleanups Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 1/2] net: lan966x: use of_get_mac_address Rosen Penev
@ 2025-08-27 21:50 ` Rosen Penev
2025-08-28 11:12 ` Daniel Machon
1 sibling, 1 reply; 6+ messages in thread
From: Rosen Penev @ 2025-08-27 21:50 UTC (permalink / raw)
To: netdev
Cc: Horatiu Vultur, maintainer:MICROCHIP LAN966X ETHERNET DRIVER,
Andrew Lunn, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, Russell King
This is a purely OF driver. There's no need for fwnode to handle any of
this, with the exception being phylik_create. Use of_fwnode_handle for
that.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
.../ethernet/microchip/lan966x/lan966x_main.c | 32 ++++++++++---------
.../ethernet/microchip/lan966x/lan966x_main.h | 2 +-
2 files changed, 18 insertions(+), 16 deletions(-)
diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
index 8bf28915c030..d778806dcfc6 100644
--- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
+++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
@@ -183,7 +183,7 @@ static int lan966x_port_open(struct net_device *dev)
ANA_PORT_CFG_PORTID_VAL,
lan966x, ANA_PORT_CFG(port->chip_port));
- err = phylink_fwnode_phy_connect(port->phylink, port->fwnode, 0);
+ err = phylink_of_phy_connect(port->phylink, port->dnode, 0);
if (err) {
netdev_err(dev, "Could not attach to PHY\n");
return err;
@@ -767,8 +767,8 @@ static void lan966x_cleanup_ports(struct lan966x *lan966x)
port->phylink = NULL;
}
- if (port->fwnode)
- fwnode_handle_put(port->fwnode);
+ if (port->dnode)
+ of_node_put(port->dnode);
}
disable_irq(lan966x->xtr_irq);
@@ -1081,7 +1081,7 @@ static int lan966x_reset_switch(struct lan966x *lan966x)
static int lan966x_probe(struct platform_device *pdev)
{
- struct fwnode_handle *ports, *portnp;
+ struct device_node *ports, *portnp;
struct lan966x *lan966x;
int err;
@@ -1179,7 +1179,7 @@ static int lan966x_probe(struct platform_device *pdev)
}
}
- ports = device_get_named_child_node(&pdev->dev, "ethernet-ports");
+ ports = of_get_child_by_name(pdev->dev.of_node, "ethernet-ports");
if (!ports)
return dev_err_probe(&pdev->dev, -ENODEV,
"no ethernet-ports child found\n");
@@ -1191,25 +1191,27 @@ static int lan966x_probe(struct platform_device *pdev)
lan966x_stats_init(lan966x);
/* go over the child nodes */
- fwnode_for_each_available_child_node(ports, portnp) {
+ for_each_available_child_of_node(ports, portnp) {
phy_interface_t phy_mode;
struct phy *serdes;
u32 p;
- if (fwnode_property_read_u32(portnp, "reg", &p))
+ if (of_property_read_u32(portnp, "reg", &p))
continue;
- phy_mode = fwnode_get_phy_mode(portnp);
- err = lan966x_probe_port(lan966x, p, phy_mode, portnp);
+ err = of_get_phy_mode(portnp, &phy_mode);
+ if (err)
+ goto cleanup_ports;
+
+ err = lan966x_probe_port(lan966x, p, phy_mode, of_fwnode_handle(portnp));
if (err)
goto cleanup_ports;
/* Read needed configuration */
lan966x->ports[p]->config.portmode = phy_mode;
- lan966x->ports[p]->fwnode = fwnode_handle_get(portnp);
+ lan966x->ports[p]->dnode = of_node_get(portnp);
- serdes = devm_of_phy_optional_get(lan966x->dev,
- to_of_node(portnp), NULL);
+ serdes = devm_of_phy_optional_get(lan966x->dev, portnp, NULL);
if (IS_ERR(serdes)) {
err = PTR_ERR(serdes);
goto cleanup_ports;
@@ -1222,7 +1224,7 @@ static int lan966x_probe(struct platform_device *pdev)
goto cleanup_ports;
}
- fwnode_handle_put(ports);
+ of_node_put(ports);
lan966x_mdb_init(lan966x);
err = lan966x_fdb_init(lan966x);
@@ -1255,8 +1257,8 @@ static int lan966x_probe(struct platform_device *pdev)
lan966x_fdb_deinit(lan966x);
cleanup_ports:
- fwnode_handle_put(ports);
- fwnode_handle_put(portnp);
+ of_node_put(ports);
+ of_node_put(portnp);
lan966x_cleanup_ports(lan966x);
diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
index 4f75f0688369..bafb8f5ee64d 100644
--- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
+++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
@@ -407,7 +407,7 @@ struct lan966x_port {
struct lan966x_port_config config;
struct phylink *phylink;
struct phy *serdes;
- struct fwnode_handle *fwnode;
+ struct device_node *dnode;
u8 ptp_tx_cmd;
bool ptp_rx_cmd;
--
2.51.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH net-next 2/2] net: lan966x: convert fwnode to of
2025-08-27 21:50 ` [PATCH net-next 2/2] net: lan966x: convert fwnode to of Rosen Penev
@ 2025-08-28 11:12 ` Daniel Machon
2025-08-28 20:17 ` Rosen Penev
0 siblings, 1 reply; 6+ messages in thread
From: Daniel Machon @ 2025-08-28 11:12 UTC (permalink / raw)
To: Rosen Penev; +Cc: netdev, horatiu.vultur
> This is a purely OF driver. There's no need for fwnode to handle any of
> this, with the exception being phylik_create. Use of_fwnode_handle for
> that.
>
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
> .../ethernet/microchip/lan966x/lan966x_main.c | 32 ++++++++++---------
> .../ethernet/microchip/lan966x/lan966x_main.h | 2 +-
> 2 files changed, 18 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> index 8bf28915c030..d778806dcfc6 100644
> --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> @@ -183,7 +183,7 @@ static int lan966x_port_open(struct net_device *dev)
> ANA_PORT_CFG_PORTID_VAL,
> lan966x, ANA_PORT_CFG(port->chip_port));
>
> - err = phylink_fwnode_phy_connect(port->phylink, port->fwnode, 0);
> + err = phylink_of_phy_connect(port->phylink, port->dnode, 0);
> if (err) {
> netdev_err(dev, "Could not attach to PHY\n");
> return err;
> @@ -767,8 +767,8 @@ static void lan966x_cleanup_ports(struct lan966x *lan966x)
> port->phylink = NULL;
> }
>
> - if (port->fwnode)
> - fwnode_handle_put(port->fwnode);
> + if (port->dnode)
> + of_node_put(port->dnode);
> }
>
> disable_irq(lan966x->xtr_irq);
> @@ -1081,7 +1081,7 @@ static int lan966x_reset_switch(struct lan966x *lan966x)
>
> static int lan966x_probe(struct platform_device *pdev)
> {
> - struct fwnode_handle *ports, *portnp;
> + struct device_node *ports, *portnp;
> struct lan966x *lan966x;
> int err;
>
> @@ -1179,7 +1179,7 @@ static int lan966x_probe(struct platform_device *pdev)
> }
> }
>
> - ports = device_get_named_child_node(&pdev->dev, "ethernet-ports");
> + ports = of_get_child_by_name(pdev->dev.of_node, "ethernet-ports");
> if (!ports)
> return dev_err_probe(&pdev->dev, -ENODEV,
> "no ethernet-ports child found\n");
> @@ -1191,25 +1191,27 @@ static int lan966x_probe(struct platform_device *pdev)
> lan966x_stats_init(lan966x);
>
> /* go over the child nodes */
> - fwnode_for_each_available_child_node(ports, portnp) {
> + for_each_available_child_of_node(ports, portnp) {
> phy_interface_t phy_mode;
> struct phy *serdes;
> u32 p;
>
> - if (fwnode_property_read_u32(portnp, "reg", &p))
> + if (of_property_read_u32(portnp, "reg", &p))
> continue;
>
> - phy_mode = fwnode_get_phy_mode(portnp);
> - err = lan966x_probe_port(lan966x, p, phy_mode, portnp);
> + err = of_get_phy_mode(portnp, &phy_mode);
> + if (err)
> + goto cleanup_ports;
> +
> + err = lan966x_probe_port(lan966x, p, phy_mode, of_fwnode_handle(portnp));
As I see it, you could change the signature of lan966x_probe_port() to accept a
struct device_node, and instead pass that. Then you can convert it to fwnode
for phylink_create, and ditch to_of_node().
Same goes for lan966x_port_parse_delays(), here you can change
fwnode_for_each_available_child_node() to for_each_available_child_of_node()
and fwnode_property_read_u32() to of_property_read_u32().
That will get rid of all the struct fwnode_handle uses and be more consistent.
> if (err)
> goto cleanup_ports;
>
> /* Read needed configuration */
> lan966x->ports[p]->config.portmode = phy_mode;
> - lan966x->ports[p]->fwnode = fwnode_handle_get(portnp);
> + lan966x->ports[p]->dnode = of_node_get(portnp);
>
> - serdes = devm_of_phy_optional_get(lan966x->dev,
> - to_of_node(portnp), NULL);
> + serdes = devm_of_phy_optional_get(lan966x->dev, portnp, NULL);
> if (IS_ERR(serdes)) {
> err = PTR_ERR(serdes);
> goto cleanup_ports;
> @@ -1222,7 +1224,7 @@ static int lan966x_probe(struct platform_device *pdev)
> goto cleanup_ports;
> }
>
> - fwnode_handle_put(ports);
> + of_node_put(ports);
>
> lan966x_mdb_init(lan966x);
> err = lan966x_fdb_init(lan966x);
> @@ -1255,8 +1257,8 @@ static int lan966x_probe(struct platform_device *pdev)
> lan966x_fdb_deinit(lan966x);
>
> cleanup_ports:
> - fwnode_handle_put(ports);
> - fwnode_handle_put(portnp);
> + of_node_put(ports);
> + of_node_put(portnp);
>
> lan966x_cleanup_ports(lan966x);
>
> diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> index 4f75f0688369..bafb8f5ee64d 100644
> --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> @@ -407,7 +407,7 @@ struct lan966x_port {
> struct lan966x_port_config config;
> struct phylink *phylink;
> struct phy *serdes;
> - struct fwnode_handle *fwnode;
> + struct device_node *dnode;
>
> u8 ptp_tx_cmd;
> bool ptp_rx_cmd;
> --
> 2.51.0
>
/Daniel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH net-next 2/2] net: lan966x: convert fwnode to of
2025-08-28 11:12 ` Daniel Machon
@ 2025-08-28 20:17 ` Rosen Penev
2025-09-01 11:38 ` Daniel Machon
0 siblings, 1 reply; 6+ messages in thread
From: Rosen Penev @ 2025-08-28 20:17 UTC (permalink / raw)
To: Daniel Machon; +Cc: netdev, horatiu.vultur
On Thu, Aug 28, 2025 at 4:12 AM Daniel Machon
<daniel.machon@microchip.com> wrote:
>
> > This is a purely OF driver. There's no need for fwnode to handle any of
> > this, with the exception being phylik_create. Use of_fwnode_handle for
> > that.
> >
> > Signed-off-by: Rosen Penev <rosenp@gmail.com>
> > ---
> > .../ethernet/microchip/lan966x/lan966x_main.c | 32 ++++++++++---------
> > .../ethernet/microchip/lan966x/lan966x_main.h | 2 +-
> > 2 files changed, 18 insertions(+), 16 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> > index 8bf28915c030..d778806dcfc6 100644
> > --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> > +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.c
> > @@ -183,7 +183,7 @@ static int lan966x_port_open(struct net_device *dev)
> > ANA_PORT_CFG_PORTID_VAL,
> > lan966x, ANA_PORT_CFG(port->chip_port));
> >
> > - err = phylink_fwnode_phy_connect(port->phylink, port->fwnode, 0);
> > + err = phylink_of_phy_connect(port->phylink, port->dnode, 0);
> > if (err) {
> > netdev_err(dev, "Could not attach to PHY\n");
> > return err;
> > @@ -767,8 +767,8 @@ static void lan966x_cleanup_ports(struct lan966x *lan966x)
> > port->phylink = NULL;
> > }
> >
> > - if (port->fwnode)
> > - fwnode_handle_put(port->fwnode);
> > + if (port->dnode)
> > + of_node_put(port->dnode);
> > }
> >
> > disable_irq(lan966x->xtr_irq);
> > @@ -1081,7 +1081,7 @@ static int lan966x_reset_switch(struct lan966x *lan966x)
> >
> > static int lan966x_probe(struct platform_device *pdev)
> > {
> > - struct fwnode_handle *ports, *portnp;
> > + struct device_node *ports, *portnp;
> > struct lan966x *lan966x;
> > int err;
> >
> > @@ -1179,7 +1179,7 @@ static int lan966x_probe(struct platform_device *pdev)
> > }
> > }
> >
> > - ports = device_get_named_child_node(&pdev->dev, "ethernet-ports");
> > + ports = of_get_child_by_name(pdev->dev.of_node, "ethernet-ports");
> > if (!ports)
> > return dev_err_probe(&pdev->dev, -ENODEV,
> > "no ethernet-ports child found\n");
> > @@ -1191,25 +1191,27 @@ static int lan966x_probe(struct platform_device *pdev)
> > lan966x_stats_init(lan966x);
> >
> > /* go over the child nodes */
> > - fwnode_for_each_available_child_node(ports, portnp) {
> > + for_each_available_child_of_node(ports, portnp) {
> > phy_interface_t phy_mode;
> > struct phy *serdes;
> > u32 p;
> >
> > - if (fwnode_property_read_u32(portnp, "reg", &p))
> > + if (of_property_read_u32(portnp, "reg", &p))
> > continue;
> >
> > - phy_mode = fwnode_get_phy_mode(portnp);
> > - err = lan966x_probe_port(lan966x, p, phy_mode, portnp);
> > + err = of_get_phy_mode(portnp, &phy_mode);
> > + if (err)
> > + goto cleanup_ports;
> > +
> > + err = lan966x_probe_port(lan966x, p, phy_mode, of_fwnode_handle(portnp));
>
> As I see it, you could change the signature of lan966x_probe_port() to accept a
> struct device_node, and instead pass that. Then you can convert it to fwnode
> for phylink_create, and ditch to_of_node().
Will fix.
>
> Same goes for lan966x_port_parse_delays(), here you can change
> fwnode_for_each_available_child_node() to for_each_available_child_of_node()
> and fwnode_property_read_u32() to of_property_read_u32().
I don't see this lan966x_port_parse_delays function.
>
> That will get rid of all the struct fwnode_handle uses and be more consistent.
>
> > if (err)
> > goto cleanup_ports;
> >
> > /* Read needed configuration */
> > lan966x->ports[p]->config.portmode = phy_mode;
> > - lan966x->ports[p]->fwnode = fwnode_handle_get(portnp);
> > + lan966x->ports[p]->dnode = of_node_get(portnp);
> >
> > - serdes = devm_of_phy_optional_get(lan966x->dev,
> > - to_of_node(portnp), NULL);
> > + serdes = devm_of_phy_optional_get(lan966x->dev, portnp, NULL);
> > if (IS_ERR(serdes)) {
> > err = PTR_ERR(serdes);
> > goto cleanup_ports;
> > @@ -1222,7 +1224,7 @@ static int lan966x_probe(struct platform_device *pdev)
> > goto cleanup_ports;
> > }
> >
> > - fwnode_handle_put(ports);
> > + of_node_put(ports);
> >
> > lan966x_mdb_init(lan966x);
> > err = lan966x_fdb_init(lan966x);
> > @@ -1255,8 +1257,8 @@ static int lan966x_probe(struct platform_device *pdev)
> > lan966x_fdb_deinit(lan966x);
> >
> > cleanup_ports:
> > - fwnode_handle_put(ports);
> > - fwnode_handle_put(portnp);
> > + of_node_put(ports);
> > + of_node_put(portnp);
> >
> > lan966x_cleanup_ports(lan966x);
> >
> > diff --git a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> > index 4f75f0688369..bafb8f5ee64d 100644
> > --- a/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> > +++ b/drivers/net/ethernet/microchip/lan966x/lan966x_main.h
> > @@ -407,7 +407,7 @@ struct lan966x_port {
> > struct lan966x_port_config config;
> > struct phylink *phylink;
> > struct phy *serdes;
> > - struct fwnode_handle *fwnode;
> > + struct device_node *dnode;
> >
> > u8 ptp_tx_cmd;
> > bool ptp_rx_cmd;
> > --
> > 2.51.0
> >
>
> /Daniel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH net-next 2/2] net: lan966x: convert fwnode to of
2025-08-28 20:17 ` Rosen Penev
@ 2025-09-01 11:38 ` Daniel Machon
0 siblings, 0 replies; 6+ messages in thread
From: Daniel Machon @ 2025-09-01 11:38 UTC (permalink / raw)
To: Rosen Penev; +Cc: netdev, horatiu.vultur
> > > @@ -1179,7 +1179,7 @@ static int lan966x_probe(struct platform_device *pdev)
> > > }
> > > }
> > >
> > > - ports = device_get_named_child_node(&pdev->dev, "ethernet-ports");
> > > + ports = of_get_child_by_name(pdev->dev.of_node, "ethernet-ports");
> > > if (!ports)
> > > return dev_err_probe(&pdev->dev, -ENODEV,
> > > "no ethernet-ports child found\n");
> > > @@ -1191,25 +1191,27 @@ static int lan966x_probe(struct platform_device *pdev)
> > > lan966x_stats_init(lan966x);
> > >
> > > /* go over the child nodes */
> > > - fwnode_for_each_available_child_node(ports, portnp) {
> > > + for_each_available_child_of_node(ports, portnp) {
> > > phy_interface_t phy_mode;
> > > struct phy *serdes;
> > > u32 p;
> > >
> > > - if (fwnode_property_read_u32(portnp, "reg", &p))
> > > + if (of_property_read_u32(portnp, "reg", &p))
> > > continue;
> > >
> > > - phy_mode = fwnode_get_phy_mode(portnp);
> > > - err = lan966x_probe_port(lan966x, p, phy_mode, portnp);
> > > + err = of_get_phy_mode(portnp, &phy_mode);
> > > + if (err)
> > > + goto cleanup_ports;
> > > +
> > > + err = lan966x_probe_port(lan966x, p, phy_mode, of_fwnode_handle(portnp));
> >
> > As I see it, you could change the signature of lan966x_probe_port() to accept a
> > struct device_node, and instead pass that. Then you can convert it to fwnode
> > for phylink_create, and ditch to_of_node().
> Will fix.
Thanks.
> >
> > Same goes for lan966x_port_parse_delays(), here you can change
> > fwnode_for_each_available_child_node() to for_each_available_child_of_node()
> > and fwnode_property_read_u32() to of_property_read_u32().
> I don't see this lan966x_port_parse_delays function.
Sorry, I was looking at an internal version that had this function. Disregard
this. The first comment still applies though.
/Daniel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-09-01 11:38 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-27 21:50 [PATCH net-next 0/2] net: lan966x: some OF cleanups Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 1/2] net: lan966x: use of_get_mac_address Rosen Penev
2025-08-27 21:50 ` [PATCH net-next 2/2] net: lan966x: convert fwnode to of Rosen Penev
2025-08-28 11:12 ` Daniel Machon
2025-08-28 20:17 ` Rosen Penev
2025-09-01 11:38 ` Daniel Machon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).