diff mbox

ethtool: Dump eeprom info for soldered on modules

Message ID 1439690718-10040-1-git-send-email-andrew@lunn.ch
State Not Applicable, archived
Delegated to: David Miller
Headers show

Commit Message

Andrew Lunn Aug. 16, 2015, 2:05 a.m. UTC
Modules which are soldered onto the motherboard may also use the sff8079
EEPROM format. Dump these in the same way as SFP modules.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
---
 sfpid.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrew Lunn Sept. 1, 2015, 8:52 a.m. UTC | #1
On Sun, Aug 16, 2015 at 04:05:18AM +0200, Andrew Lunn wrote:
> Modules which are soldered onto the motherboard may also use the sff8079
> EEPROM format. Dump these in the same way as SFP modules.

Hi Ben

I posted this patch two weeks ago. Since this is my first ethtool
patch, i've no idea what your response time usually is. Do i need to
repost? Send a gentle reminder? Or just wait a bit longer?

Thanks
	Andrew

> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> ---
>  sfpid.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sfpid.c b/sfpid.c
> index b1a5044..0b5cd62 100644
> --- a/sfpid.c
> +++ b/sfpid.c
> @@ -401,7 +401,7 @@ static void sff8079_show_options(const __u8 *id)
>  void sff8079_show_all(const __u8 *id)
>  {
>  	sff8079_show_identifier(id);
> -	if ((id[0] == 0x03) && (id[1] == 0x04)) {
> +	if (((id[0] == 0x02) || (id[0] == 0x03)) && (id[1] == 0x04)) {
>  		sff8079_show_ext_identifier(id);
>  		sff8079_show_connector(id);
>  		sff8079_show_transceiver(id);
> -- 
> 2.1.4
> 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ben Hutchings Sept. 5, 2015, 10:48 a.m. UTC | #2
On Sun, 2015-08-16 at 04:05 +0200, Andrew Lunn wrote:
> Modules which are soldered onto the motherboard may also use the 
> sff8079
> EEPROM format. Dump these in the same way as SFP modules.
> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
[...]

Applied, thanks.

Ben.
diff mbox

Patch

diff --git a/sfpid.c b/sfpid.c
index b1a5044..0b5cd62 100644
--- a/sfpid.c
+++ b/sfpid.c
@@ -401,7 +401,7 @@  static void sff8079_show_options(const __u8 *id)
 void sff8079_show_all(const __u8 *id)
 {
 	sff8079_show_identifier(id);
-	if ((id[0] == 0x03) && (id[1] == 0x04)) {
+	if (((id[0] == 0x02) || (id[0] == 0x03)) && (id[1] == 0x04)) {
 		sff8079_show_ext_identifier(id);
 		sff8079_show_connector(id);
 		sff8079_show_transceiver(id);