Re: [PATCH] igb: Explicitly select page 0 at initialization

From: Sergei Shtylyov
Date: Thu Nov 24 2016 - 05:37:05 EST


On 11/24/2016 1:32 PM, Matwey V. Kornilov wrote:

The functions igb_read_phy_reg_gs40g/igb_write_phy_reg_gs40g (which were
removed in 2a3cdea) explicitly selected the required page at every phy_reg

This is not the way to cite a commit -- you need to specify at least 12 digits and follow them with the commit subject enclosed into ("").

access. Currently, igb_get_phy_id_82575 relays on the fact that page 0 is
already selected. The assumption is not fulfilled for my Lex 3I380CW
motherboard with integrated dual i211 based gigabit ethernet. This leads to igb
initialization failure and network interfaces are not working:

igb: Intel(R) Gigabit Ethernet Network Driver - version 5.4.0-k
igb: Copyright (c) 2007-2014 Intel Corporation.
igb: probe of 0000:01:00.0 failed with error -2
igb: probe of 0000:02:00.0 failed with error -2

In order to fix it, we explicitly select page 0 before first access to phy

See also:
See also:

Fixes: 2a3cdea ("igb: Remove GS40G specific defines/functions")

Again, at least 12 digits.

Cc: <stable@xxxxxxxxxxxxxxx> # 4.5+
Signed-off-by: Matwey V. Kornilov <matwey@xxxxxxxxxx>

MBR, Sergei