[PATCH V3 1/3] net: phy: dp83867: Explicitly check against sgmii

[Ramon Fried]

On Sat, Apr 15, 2023 at 1:06 AM Nishanth Menon <nm at ti.com> wrote:
>
> dp83867 driver only supports sgmii and not all the newer protocols.
> Drop the usage of the generic phy_interface_is_sgmii function and just
> matchup to the specific mode supported.
>
> Reported-by: Tom Rini <trini at konsulko.com>
> Suggested-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> Suggested-by: Marek Behún <kabel at kernel.org>
> Link: https://lore.kernel.org/all/b82ac325-4818-8e72-054b-640268dbf806@mailbox.org/
> Signed-off-by: Nishanth Menon <nm at ti.com>
> ---
> Changes since v2:
> * New patch.
>
>  drivers/net/phy/dp83867.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/phy/dp83867.c b/drivers/net/phy/dp83867.c
> index b861bf7cef36..7111e36aa0d0 100644
> --- a/drivers/net/phy/dp83867.c
> +++ b/drivers/net/phy/dp83867.c
> @@ -330,7 +330,7 @@ static int dp83867_config(struct phy_device *phydev)
>                               DP83867_RGMIIDCTL, delay);
>         }
>
> -       if (phy_interface_is_sgmii(phydev)) {
> +       if (phydev->interface == PHY_INTERFACE_MODE_SGMII) {
>                 if (dp83867->sgmii_ref_clk_en)
>                         phy_write_mmd(phydev, DP83867_DEVADDR, DP83867_SGMIICTL,
>                                       DP83867_SGMII_TYPE);
> --
> 2.40.0
>
Reviewed-by: Ramon Fried <rfried.dev at gmail.com>