[PATCH v2 3/5] net: sun8i-emac: Add a flag for the internal PHY switch

[Andre Przywara]

On 22/01/2023 22:51, Samuel Holland wrote:
> Describe this feature instead of using the SoC ID.
>
> 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 | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/sun8i_emac.c b/drivers/net/sun8i_emac.c
> index f232b8f087..36cc2498b5 100644
> --- a/drivers/net/sun8i_emac.c
> +++ b/drivers/net/sun8i_emac.c
> @@ -137,6 +137,7 @@ enum emac_variant_id {
>
>   struct emac_variant {
>       enum emac_variant_id    variant;
> +     bool                    soc_has_internal_phy;
>       bool                    support_rmii;
>   };
>
> @@ -860,7 +861,7 @@ static int sun8i_emac_eth_of_to_plat(struct udevice *dev)
>       if (pdata->phy_interface == PHY_INTERFACE_MODE_NA)
>               return -EINVAL;
>
> -     if (priv->variant->variant == H3_EMAC) {
> +     if (priv->variant->soc_has_internal_phy) {
>               ret = sun8i_handle_internal_phy(dev, priv);
>               if (ret)
>                       return ret;
> @@ -906,6 +907,7 @@ static const struct emac_variant emac_variant_a83t = {
>
>   static const struct emac_variant emac_variant_h3 = {
>       .variant                = H3_EMAC,
> +     .soc_has_internal_phy   = true,
>       .support_rmii           = true,
>   };
>

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.