Re: [PATCH] net: sfp: add quirk for Finisar FTLF8536P4BCL

From: Vadym Kochan <vadym.kochan@plvision.eu>
To: "Russell King (Oracle)" <linux@armlinux.org.uk>
Cc: Paul Menzel <pmenzel@molgen.mpg.de>, Andrew Lunn <andrew@lunn.ch>,
	Heiner Kallweit <hkallweit1@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Jakub Kicinski <kuba@kernel.org>,
	Taras Chornyi <taras.chornyi@plvision.eu>,
	Vadym Kochan <vadym.kochan@plvision.eu>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] net: sfp: add quirk for Finisar FTLF8536P4BCL
Date: Mon, 18 Oct 2021 14:05:24 +0300	[thread overview]
Message-ID: <vrcxh2o87ma8zv.fsf@plvision.eu> (raw)
In-Reply-To: <YWhfsiA6Vngi/1l+@shell.armlinux.org.uk>

[-- Attachment #1: Type: text/plain, Size: 903 bytes --]

Hi Russel,

Russell King (Oracle) <linux@armlinux.org.uk> writes:

> On Wed, Oct 13, 2021 at 12:45:42PM +0200, Paul Menzel wrote:
>> From: Taras Chornyi <taras.chornyi@plvision.eu>
>> 
>> Finisar FTLF8536P4BCL can operate at 1000base-X and 10000base-SR, but
>> reports 25G & 100GBd SR in it's EEPROM.
>> 
>> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
>> Signed-off-by: Taras Chornyi <taras.chornyi@plvision.eu>
>> 
>> [Upstream from https://github.com/dentproject/dentOS/pull/133/commits/b87b10ef72ea4638e80588facf3c9c2c1be67b40]
>> 
>> Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
>
> Hi Paul,
>
> Please can you send me the file resulting from:
>
> ethtool -m ethX raw on > file
>
> please - it will be binary data, and that is exactly what I'm after.
> I would like to see what the EEPROM contains before making a decision
> on this patch.
>
> Thanks.

Attached output of ethtool:

[-- Attachment #2: FTLF8536P4BCL.bin --]
[-- Type: application/octet-stream, Size: 512 bytes --]

[-- Attachment #3: Type: text/plain, Size: 1327 bytes --]

Also adding hexdump version to have it in mail:

00000000  03 04 07 00 00 00 00 00  00 00 00 06 ff 00 00 00  |................|
00000010  02 00 0a 07 46 49 4e 49  53 41 52 20 43 4f 52 50  |....FINISAR CORP|
00000020  2e 20 20 20 02 00 90 65  46 54 4c 46 38 35 33 36  |.   ...eFTLF8536|
00000030  50 34 42 43 4c 20 20 20  41 20 20 20 03 52 00 b8  |P4BCL   A   .R..|
00000040  08 1a 70 00 55 55 48 32  30 46 4c 20 20 20 20 20  |..p.UUH20FL     |
00000050  20 20 20 20 31 36 30 32  31 32 20 20 68 f0 08 64  |    160212  h..d|
00000060  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
*
00000100  4b 00 fb 00 46 00 00 00  8c a0 75 30 88 b8 79 18  |K...F.....u0..y.|
00000110  17 70 01 f4 16 76 03 e8  3d e9 04 eb 31 2d 06 31  |.p...v..=...1-.1|
00000120  4d f1 00 64 3d e9 00 9e  00 00 00 00 00 00 00 00  |M..d=...........|
00000130  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
00000140  00 00 00 00 3f 80 00 00  00 00 00 00 00 01 00 00  |....?...........|
00000150  00 01 00 00 00 01 00 00  00 01 00 00 00 00 00 f4  |................|
00000160  26 f1 83 b6 00 51 00 00  0a c6 00 1e 24 4b b0 00  |&....Q......$K..|
00000170  05 28 00 00 05 28 00 00  00 00 00 00 00 00 00 01  |.(...(..........|
00000180  00 00 00 00 00 00 00 00  00 00 00 00 00 00 00 00  |................|
*
00000200