On Wed, Jun 12, 2024 at 04:00:05PM +0100, Anatoly Burakov wrote: > From: Ian Stokes <ian.sto...@intel.com> > > The use of get link status PHY type for determining the media type can result > in > incorrect reporting the incorrect media type. This can happen for a couple > reasons: > > 1. The get link states current PHY type field can return non-PHY type values > (i.e. bit 127 indicates link state disabled, and bit 126 indicates PHY is in > auto-negotiation state) > > 2. Forced link configurations can result in incorrectly mapping on media type > (i.e. forcing optical link modes on copper media) > > Add the media type ICE_MEDIA_NONE. As part of this patch add the new PHY type > bits needed for 200G architecture. > > Signed-off-by: Paul Greenwalt <paul.greenw...@intel.com> Signed-off-by: Alice > Michael <alice.mich...@intel.com> > Signed-off-by: Ian Stokes <ian.sto...@intel.com> > ---
Nit: beware malformed sign-offs. No worries if any slip through, I can fix on apply of V3. Also, for any patches with titles starting "fix", consider if suitable for backport and CC stable if appropriate. [If not for backport, perhaps "fix" is not the best verb to use in title?] /Bruce
