| Message ID | 20250224205924.2861584-1-grzegorz.nitka@intel.com |
|---|---|
| State | Accepted |
| Delegated to: | Anthony Nguyen |
| Headers | show |
| Series | [iwl-next,v2] ice: Allow 100M speed for E825C SGMII device | expand |
On Mon, Feb 24, 2025 at 09:59:24PM +0100, Grzegorz Nitka wrote: > Add E825C 10GbE SGMII device to the list of devices supporting 100Mbit > link mode. Without that change, 100Mbit link mode is ignored in ethtool > interface. This change was missed while adding the support for E825C > devices family. > > Testing hints (please note, for previous version, 100baseT/Full entry > was missing): > [root@localhost]# ethtool eth3 > Settings for eth3: > Supported ports: [ TP ] > Supported link modes: 100baseT/Full > 1000baseT/Full > 10000baseT/Full > Supported pause frame use: Symmetric > Supports auto-negotiation: Yes > Supported FEC modes: None > Advertised link modes: 100baseT/Full > 1000baseT/Full > 10000baseT/Full > ... > > Fixes: f64e189442332 ("ice: introduce new E825C devices family") > Signed-off-by: Grzegorz Nitka <grzegorz.nitka@intel.com> > Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com> > Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de> Reviewed-by: Simon Horman <horms@kernel.org>
> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of Grzegorz Nitka > Sent: 25 February 2025 02:29 > To: intel-wired-lan@lists.osuosl.org > Cc: Loktionov, Aleksandr <aleksandr.loktionov@intel.com>; netdev@vger.kernel.org; Paul Menzel <pmenzel@molgen.mpg.de> > Subject: [Intel-wired-lan] [PATCH iwl-next v2] ice: Allow 100M speed for E825C SGMII device > > Add E825C 10GbE SGMII device to the list of devices supporting 100Mbit link mode. Without that change, 100Mbit link mode is ignored in ethtool interface. This change was missed while adding the support for E825C devices family. > > Testing hints (please note, for previous version, 100baseT/Full entry was missing): > [root@localhost]# ethtool eth3 > Settings for eth3: > Supported ports: [ TP ] > Supported link modes: 100baseT/Full > 1000baseT/Full > 10000baseT/Full > Supported pause frame use: Symmetric > Supports auto-negotiation: Yes > Supported FEC modes: None > Advertised link modes: 100baseT/Full > 1000baseT/Full > 10000baseT/Full > ... > > Fixes: f64e189442332 ("ice: introduce new E825C devices family") > Signed-off-by: Grzegorz Nitka <grzegorz.nitka@intel.com> > Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com> > Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de> > > --- > Changes in v2: > - improved commit meassege (added testing hint) > --- > drivers/net/ethernet/intel/ice/ice_common.c | 1 + > 1 file changed, 1 insertion(+) > Tested-by: Rinitha S <sx.rinitha@intel.com> (A Contingent worker at Intel)
diff --git a/drivers/net/ethernet/intel/ice/ice_common.c b/drivers/net/ethernet/intel/ice/ice_common.c index 7a2a2e8da8fa..caf3af2a32c3 100644 --- a/drivers/net/ethernet/intel/ice/ice_common.c +++ b/drivers/net/ethernet/intel/ice/ice_common.c @@ -3180,6 +3180,7 @@ bool ice_is_100m_speed_supported(struct ice_hw *hw) case ICE_DEV_ID_E822L_SGMII: case ICE_DEV_ID_E823L_1GBE: case ICE_DEV_ID_E823C_SGMII: + case ICE_DEV_ID_E825C_SGMII: return true; default: return false;
Add E825C 10GbE SGMII device to the list of devices supporting 100Mbit link mode. Without that change, 100Mbit link mode is ignored in ethtool interface. This change was missed while adding the support for E825C devices family. Testing hints (please note, for previous version, 100baseT/Full entry was missing): [root@localhost]# ethtool eth3 Settings for eth3: Supported ports: [ TP ] Supported link modes: 100baseT/Full 1000baseT/Full 10000baseT/Full Supported pause frame use: Symmetric Supports auto-negotiation: Yes Supported FEC modes: None Advertised link modes: 100baseT/Full 1000baseT/Full 10000baseT/Full ... Fixes: f64e189442332 ("ice: introduce new E825C devices family") Signed-off-by: Grzegorz Nitka <grzegorz.nitka@intel.com> Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com> Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de> --- Changes in v2: - improved commit meassege (added testing hint) --- drivers/net/ethernet/intel/ice/ice_common.c | 1 + 1 file changed, 1 insertion(+) base-commit: bf7e01518c9f3f77c220bf153411d9d3d2511f16