[PATCH v2 5/5] net: sun8i-emac: Remove the SoC variant ID

Ramon Fried rfried.dev at gmail.com
Sat Feb 4 01:38:06 CET 2023


On Mon, Jan 23, 2023 at 7:24 PM Andre Przywara <andre.przywara at arm.com> wrote:
>
> On 22/01/2023 22:51, Samuel Holland wrote:
>
> Hi,
>
> > Now that all differences in functionality are covered by individual
> > flags, remove the enumeration of SoC variants.
>
> My favourite patch of this series ;-)
>
> > Signed-off-by: Samuel Holland <samuel at sholland.org>
>
> Reviewed-by: Andre Przywara <andre.przywara at arm.com>
>
> Cheers,
> Andre
>
>
> > ---
> >
> > Changes in v2:
> >   - New patch for v2
> >
> >   drivers/net/sun8i_emac.c | 14 --------------
> >   1 file changed, 14 deletions(-)
> >
> > diff --git a/drivers/net/sun8i_emac.c b/drivers/net/sun8i_emac.c
> > index 231aac19e3..04c3274fbe 100644
> > --- a/drivers/net/sun8i_emac.c
> > +++ b/drivers/net/sun8i_emac.c
> > @@ -127,16 +127,7 @@
> >
> >   DECLARE_GLOBAL_DATA_PTR;
> >
> > -enum emac_variant_id {
> > -     A83T_EMAC = 1,
> > -     H3_EMAC,
> > -     A64_EMAC,
> > -     R40_GMAC,
> > -     H6_EMAC,
> > -};
> > -
> >   struct emac_variant {
> > -     enum emac_variant_id    variant;
> >       uint                    syscon_offset;
> >       bool                    soc_has_internal_phy;
> >       bool                    support_rmii;
> > @@ -895,30 +886,25 @@ static int sun8i_emac_eth_of_to_plat(struct udevice *dev)
> >   }
> >
> >   static const struct emac_variant emac_variant_a83t = {
> > -     .variant                = A83T_EMAC,
> >       .syscon_offset          = 0x30,
> >   };
> >
> >   static const struct emac_variant emac_variant_h3 = {
> > -     .variant                = H3_EMAC,
> >       .syscon_offset          = 0x30,
> >       .soc_has_internal_phy   = true,
> >       .support_rmii           = true,
> >   };
> >
> >   static const struct emac_variant emac_variant_r40 = {
> > -     .variant                = R40_GMAC,
> >       .syscon_offset          = 0x164,
> >   };
> >
> >   static const struct emac_variant emac_variant_a64 = {
> > -     .variant                = A64_EMAC,
> >       .syscon_offset          = 0x30,
> >       .support_rmii           = true,
> >   };
> >
> >   static const struct emac_variant emac_variant_h6 = {
> > -     .variant                = H6_EMAC,
> >       .syscon_offset          = 0x30,
> >       .support_rmii           = true,
> >   };
>
Reviewed-by: Ramon Fried <rfried.dev at gmail.com>


More information about the U-Boot mailing list