[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Zqn5sNLbg9pMD5LP@shredder.mtl.com>
Date: Wed, 31 Jul 2024 11:45:36 +0300
From: Ido Schimmel <idosch@...dia.com>
To: Krzysztof Olędzki <ole@....pl>
Cc: Tariq Toukan <tariqt@...dia.com>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
linux-rdma@...r.kernel.org, Dan Merillat <git@....merillat.org>,
Moshe Shemesh <moshe@...dia.com>, Michal Kubecek <mkubecek@...e.cz>,
Andrew Lunn <andrew@...n.ch>
Subject: Re: [PATCH v2 net-next] net/mlx4: Add support for EEPROM high pages
query for QSFP/QSFP+/QSFP28
On Tue, Jul 30, 2024 at 05:49:53PM -0700, Krzysztof Olędzki wrote:
> Enable reading additional EEPROM information from high pages such as
> thresholds and alarms on QSFP/QSFP+/QSFP28 modules.
>
> "This is similar to commit a708fb7b1f8d ("net/mlx5e: ethtool, Add
^ Nit: s/^\"//
> support for EEPROM high pages query") but given all the required logic
> already exists in mlx4_qsfp_eeprom_params_set() only s/_LEN/MAX_LEN/ is
> needed.
>
> Tested-by: Dan Merillat <git@....merillat.org>
> Signed-off-by: Krzysztof Piotr Oledzki <ole@....pl>
Reviewed-by: Ido Schimmel <idosch@...dia.com>
Powered by blists - more mailing lists