| Message ID | 20240309021831.264018-3-marex@denx.de |
|---|---|
| State | Superseded |
| Delegated to: | Patrice Chotard |
| Headers | show |
| Series | net: dwc_eth_qos: Clean up STM32 glue code and add STM32MP13xx support | expand |
On 3/9/24 03:11, Marek Vasut wrote: > The current glue code is specific to STM32MP15xx, the upcoming STM32MP13xx > will introduce another entry specific to the STM32MP13xx. Rename the current > entry to eqos_stm32mp15_config in preparation for STM32MP13xx addition. No > functional change. > > Signed-off-by: Marek Vasut <marex@denx.de> > --- > Cc: Christophe Roullier <christophe.roullier@st.com> > Cc: Joe Hershberger <joe.hershberger@ni.com> > Cc: Patrice Chotard <patrice.chotard@foss.st.com> > Cc: Patrick Delaunay <patrick.delaunay@foss.st.com> > Cc: Ramon Fried <rfried.dev@gmail.com> > Cc: u-boot@dh-electronics.com > Cc: uboot-stm32@st-md-mailman.stormreply.com > --- > drivers/net/dwc_eth_qos.c | 2 +- > drivers/net/dwc_eth_qos.h | 2 +- > drivers/net/dwc_eth_qos_stm32.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c > index 533c2bf070b..203bfc0848c 100644 > --- a/drivers/net/dwc_eth_qos.c > +++ b/drivers/net/dwc_eth_qos.c > @@ -1507,7 +1507,7 @@ static const struct udevice_id eqos_ids[] = { > #if IS_ENABLED(CONFIG_DWC_ETH_QOS_STM32) > { > .compatible = "st,stm32mp1-dwmac", > - .data = (ulong)&eqos_stm32_config > + .data = (ulong)&eqos_stm32mp15_config > }, > #endif > #if IS_ENABLED(CONFIG_DWC_ETH_QOS_IMX) > diff --git a/drivers/net/dwc_eth_qos.h b/drivers/net/dwc_eth_qos.h > index a6087f191ab..bafd0d339fb 100644 > --- a/drivers/net/dwc_eth_qos.h > +++ b/drivers/net/dwc_eth_qos.h > @@ -290,5 +290,5 @@ int eqos_null_ops(struct udevice *dev); > extern struct eqos_config eqos_imx_config; > extern struct eqos_config eqos_rockchip_config; > extern struct eqos_config eqos_qcom_config; > -extern struct eqos_config eqos_stm32_config; > +extern struct eqos_config eqos_stm32mp15_config; > extern struct eqos_config eqos_jh7110_config; > diff --git a/drivers/net/dwc_eth_qos_stm32.c b/drivers/net/dwc_eth_qos_stm32.c > index cfda757133e..fd29a604987 100644 > --- a/drivers/net/dwc_eth_qos_stm32.c > +++ b/drivers/net/dwc_eth_qos_stm32.c > @@ -184,7 +184,7 @@ static struct eqos_ops eqos_stm32_ops = { > .eqos_get_tick_clk_rate = eqos_get_tick_clk_rate_stm32 > }; > > -struct eqos_config __maybe_unused eqos_stm32_config = { > +struct eqos_config __maybe_unused eqos_stm32mp15_config = { > .reg_access_always_ok = false, > .mdio_wait = 10000, > .swr_wait = 50, Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com> Thanks Patrice
> On 3/9/24 03:11, Marek Vasut wrote: >> The current glue code is specific to STM32MP15xx, the upcoming >> STM32MP13xx will introduce another entry specific to the STM32MP13xx. >> Rename the current entry to eqos_stm32mp15_config in preparation for >> STM32MP13xx addition. No functional change. >> >> Signed-off-by: Marek Vasut<marex@denx.de> >> --- >> Cc: Christophe Roullier<christophe.roullier@st.com> >> Cc: Joe Hershberger<joe.hershberger@ni.com> >> Cc: Patrice Chotard<patrice.chotard@foss.st.com> >> Cc: Patrick Delaunay<patrick.delaunay@foss.st.com> >> Cc: Ramon Fried<rfried.dev@gmail.com> >> Cc:u-boot@dh-electronics.com >> Cc:uboot-stm32@st-md-mailman.stormreply.com >> --- >> drivers/net/dwc_eth_qos.c | 2 +- >> drivers/net/dwc_eth_qos.h | 2 +- >> drivers/net/dwc_eth_qos_stm32.c | 2 +- >> 3 files changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c >> index 533c2bf070b..203bfc0848c 100644 >> --- a/drivers/net/dwc_eth_qos.c >> +++ b/drivers/net/dwc_eth_qos.c >> @@ -1507,7 +1507,7 @@ static const struct udevice_id eqos_ids[] = { >> #if IS_ENABLED(CONFIG_DWC_ETH_QOS_STM32) >> { >> .compatible = "st,stm32mp1-dwmac", >> - .data = (ulong)&eqos_stm32_config >> + .data = (ulong)&eqos_stm32mp15_config >> }, >> #endif >> #if IS_ENABLED(CONFIG_DWC_ETH_QOS_IMX) >> diff --git a/drivers/net/dwc_eth_qos.h b/drivers/net/dwc_eth_qos.h >> index a6087f191ab..bafd0d339fb 100644 >> --- a/drivers/net/dwc_eth_qos.h >> +++ b/drivers/net/dwc_eth_qos.h >> @@ -290,5 +290,5 @@ int eqos_null_ops(struct udevice *dev); extern >> struct eqos_config eqos_imx_config; extern struct eqos_config >> eqos_rockchip_config; extern struct eqos_config eqos_qcom_config; >> -extern struct eqos_config eqos_stm32_config; >> +extern struct eqos_config eqos_stm32mp15_config; >> extern struct eqos_config eqos_jh7110_config; diff --git >> a/drivers/net/dwc_eth_qos_stm32.c b/drivers/net/dwc_eth_qos_stm32.c >> index cfda757133e..fd29a604987 100644 >> --- a/drivers/net/dwc_eth_qos_stm32.c >> +++ b/drivers/net/dwc_eth_qos_stm32.c >> @@ -184,7 +184,7 @@ static struct eqos_ops eqos_stm32_ops = { >> .eqos_get_tick_clk_rate = eqos_get_tick_clk_rate_stm32 }; >> >> -struct eqos_config __maybe_unused eqos_stm32_config = { >> +struct eqos_config __maybe_unused eqos_stm32mp15_config = { >> .reg_access_always_ok = false, >> .mdio_wait = 10000, >> .swr_wait = 50, > Reviewed-by: Christophe Roullier<christophe.roullier@foss.st.com> >
diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c index 533c2bf070b..203bfc0848c 100644 --- a/drivers/net/dwc_eth_qos.c +++ b/drivers/net/dwc_eth_qos.c @@ -1507,7 +1507,7 @@ static const struct udevice_id eqos_ids[] = { #if IS_ENABLED(CONFIG_DWC_ETH_QOS_STM32) { .compatible = "st,stm32mp1-dwmac", - .data = (ulong)&eqos_stm32_config + .data = (ulong)&eqos_stm32mp15_config }, #endif #if IS_ENABLED(CONFIG_DWC_ETH_QOS_IMX) diff --git a/drivers/net/dwc_eth_qos.h b/drivers/net/dwc_eth_qos.h index a6087f191ab..bafd0d339fb 100644 --- a/drivers/net/dwc_eth_qos.h +++ b/drivers/net/dwc_eth_qos.h @@ -290,5 +290,5 @@ int eqos_null_ops(struct udevice *dev); extern struct eqos_config eqos_imx_config; extern struct eqos_config eqos_rockchip_config; extern struct eqos_config eqos_qcom_config; -extern struct eqos_config eqos_stm32_config; +extern struct eqos_config eqos_stm32mp15_config; extern struct eqos_config eqos_jh7110_config; diff --git a/drivers/net/dwc_eth_qos_stm32.c b/drivers/net/dwc_eth_qos_stm32.c index cfda757133e..fd29a604987 100644 --- a/drivers/net/dwc_eth_qos_stm32.c +++ b/drivers/net/dwc_eth_qos_stm32.c @@ -184,7 +184,7 @@ static struct eqos_ops eqos_stm32_ops = { .eqos_get_tick_clk_rate = eqos_get_tick_clk_rate_stm32 }; -struct eqos_config __maybe_unused eqos_stm32_config = { +struct eqos_config __maybe_unused eqos_stm32mp15_config = { .reg_access_always_ok = false, .mdio_wait = 10000, .swr_wait = 50,
The current glue code is specific to STM32MP15xx, the upcoming STM32MP13xx will introduce another entry specific to the STM32MP13xx. Rename the current entry to eqos_stm32mp15_config in preparation for STM32MP13xx addition. No functional change. Signed-off-by: Marek Vasut <marex@denx.de> --- Cc: Christophe Roullier <christophe.roullier@st.com> Cc: Joe Hershberger <joe.hershberger@ni.com> Cc: Patrice Chotard <patrice.chotard@foss.st.com> Cc: Patrick Delaunay <patrick.delaunay@foss.st.com> Cc: Ramon Fried <rfried.dev@gmail.com> Cc: u-boot@dh-electronics.com Cc: uboot-stm32@st-md-mailman.stormreply.com --- drivers/net/dwc_eth_qos.c | 2 +- drivers/net/dwc_eth_qos.h | 2 +- drivers/net/dwc_eth_qos_stm32.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-)