Hi Steen! Either this is a fix and it would need a Fixes: tag in addition to the Signed-off-by: one (you can have a look at the git history to see what is the format); or the patch is not a fix and then it should have [net-next] in its subject instead of [net].
Please have a look at the relevant documentation, https://www.kernel.org/doc/html/latest/networking/netdev-FAQ.html Thanks! Antoine Quoting Steen Hegelund (2020-11-11 10:55:11) > The MSCC PHYs selected for PTP and MACSec was not correct > > - PTP > - Add VSC8572 and VSC8574 > > - MACsec > - Removed VSC8575 > > The relevant datasheets can be found here: > - VSC8572: https://www.microchip.com/wwwproducts/en/VSC8572 > - VSC8574: https://www.microchip.com/wwwproducts/en/VSC8574 > - VSC8575: https://www.microchip.com/wwwproducts/en/VSC8575 > > Signed-off-by: Steen Hegelund <steen.hegel...@microchip.com> > --- > drivers/net/phy/mscc/mscc_macsec.c | 1 - > drivers/net/phy/mscc/mscc_ptp.c | 2 ++ > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/phy/mscc/mscc_macsec.c > b/drivers/net/phy/mscc/mscc_macsec.c > index 1d4c012194e9..72292bf6c51c 100644 > --- a/drivers/net/phy/mscc/mscc_macsec.c > +++ b/drivers/net/phy/mscc/mscc_macsec.c > @@ -981,7 +981,6 @@ int vsc8584_macsec_init(struct phy_device *phydev) > > switch (phydev->phy_id & phydev->drv->phy_id_mask) { > case PHY_ID_VSC856X: > - case PHY_ID_VSC8575: > case PHY_ID_VSC8582: > case PHY_ID_VSC8584: > INIT_LIST_HEAD(&vsc8531->macsec_flows); > diff --git a/drivers/net/phy/mscc/mscc_ptp.c b/drivers/net/phy/mscc/mscc_ptp.c > index b97ee79f3cdf..f0537299c441 100644 > --- a/drivers/net/phy/mscc/mscc_ptp.c > +++ b/drivers/net/phy/mscc/mscc_ptp.c > @@ -1510,6 +1510,8 @@ void vsc8584_config_ts_intr(struct phy_device *phydev) > int vsc8584_ptp_init(struct phy_device *phydev) > { > switch (phydev->phy_id & phydev->drv->phy_id_mask) { > + case PHY_ID_VSC8572: > + case PHY_ID_VSC8574: > case PHY_ID_VSC8575: > case PHY_ID_VSC8582: > case PHY_ID_VSC8584: > -- > 2.29.2 >
