[Macchiato] SFP+ ports - any config needed?

[Steve McIntyre]

Hey Baruch,

On Wed, Dec 19, 2018 at 07:08:31AM +0200, Baruch Siach wrote:
>Steve McIntyre writes:
>>
>> Ah, but this is in the case where I've already loaded the marvell10g
>> module before the mvpp2 module. So if the MAC works fine with other
>> PHYs too (as you'd expect!), shouldn't the DTB be triggering
>> auto-loading of the PHY then? I'll admit I'm a bit hazy about the
>> exact logic of how to do that! :-)
>
>The DTB can't trigger the PHY driver load because there is no
>identification of the specific PHY in the DTB. There is only a
>"ethernet-phy-ieee802.3-c45" compatible string. The MAC detects the PHY
>at run time. This ID should be matched against the list given in an
>mdio_device_id struct array to the MODULE_DEVICE_TABLE()
>macro. Something is evidently still missing here because it does not
>currently work as expected.

ACK.

>>>What is it that you find odd in this output?
>>
>> On other machines with SFP+, I'm seeing "FIBRE" in the "Supported
>> Ports" list, is the main thing. Maybe "[ TP FIBRE ]" would be right?
>> The list for eth2 looks very bogus:
>>
>> # ethtool eth2
>> Settings for eth2:
>>         Supported ports: [ TP AUI BNC MII FIBRE Backplane ]
>>
>> I *really* don't think we have BNC! :-)
>
>One more thing that occurred to me only now. Do you see messages like
>this one from the sfp driver:
>
>[   90.186000] sfp sfp-cp1-eth0: module Intel Corp       FTLX8571D3BCV-IT rev A    sn INSSRI44328      dc 13-09-10
>
>If not, your SFP module has not been really detected.

No, no messages like that at all.

>Also, what do you get for 'ethtool -m eth0'?

root at mjolnir:/home/steve/linux.git# ethtool -m eth0
Cannot get module EEPROM information: Operation not supported

(All tested using 4.20.0-rc7)

-- 
Steve McIntyre, Cambridge, UK.                                steve at einval.com
"Arguing that you don't care about the right to privacy because you have
 nothing to hide is no different than saying you don't care about free
 speech because you have nothing to say."
   -- Edward Snowden