* [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY
@ 2011-01-19 9:09 Florian Fainelli
2011-01-20 6:51 ` Jeff Kirsher
0 siblings, 1 reply; 4+ messages in thread
From: Florian Fainelli @ 2011-01-19 9:09 UTC (permalink / raw)
To: netdev; +Cc: David Miller, Dirk Brandewie, Jeff Kirsher
From: Florian Fainelli <ffainelli@freebox.fr>
This patch adds support for Marvell Alask M88E188R PHY chips. Support for
other M88* PHYs is already there, so there is nothing more to add than its
PHY id.
Signed-off-by: Florian Fainelli <ffainelli@freebox.fr>
CC: Dirk Brandewie <dirk.j.brandewie@intel.com>
CC: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
---
diff --git a/drivers/net/e1000/e1000_hw.c b/drivers/net/e1000/e1000_hw.c
index aed223b..7501d97 100644
--- a/drivers/net/e1000/e1000_hw.c
+++ b/drivers/net/e1000/e1000_hw.c
@@ -124,6 +124,7 @@ static s32 e1000_set_phy_type(struct e1000_hw *hw)
case M88E1000_I_PHY_ID:
case M88E1011_I_PHY_ID:
case M88E1111_I_PHY_ID:
+ case M88E1118_E_PHY_ID:
hw->phy_type = e1000_phy_m88;
break;
case IGP01E1000_I_PHY_ID:
@@ -3222,7 +3223,8 @@ static s32 e1000_detect_gig_phy(struct e1000_hw *hw)
break;
case e1000_ce4100:
if ((hw->phy_id == RTL8211B_PHY_ID) ||
- (hw->phy_id == RTL8201N_PHY_ID))
+ (hw->phy_id == RTL8201N_PHY_ID) ||
+ (hw->phy_id == M88E1118_E_PHY_ID))
match = true;
break;
case e1000_82541:
diff --git a/drivers/net/e1000/e1000_hw.h b/drivers/net/e1000/e1000_hw.h
index 196eeda..c70b23d 100644
--- a/drivers/net/e1000/e1000_hw.h
+++ b/drivers/net/e1000/e1000_hw.h
@@ -2917,6 +2917,7 @@ struct e1000_host_command_info {
#define M88E1000_14_PHY_ID M88E1000_E_PHY_ID
#define M88E1011_I_REV_4 0x04
#define M88E1111_I_PHY_ID 0x01410CC0
+#define M88E1118_E_PHY_ID 0x01410E40
#define L1LXT971A_PHY_ID 0x001378E0
#define RTL8211B_PHY_ID 0x001CC910
--
1.7.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY
2011-01-19 9:09 [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY Florian Fainelli
@ 2011-01-20 6:51 ` Jeff Kirsher
2011-01-20 18:49 ` Dirk Brandewie
0 siblings, 1 reply; 4+ messages in thread
From: Jeff Kirsher @ 2011-01-20 6:51 UTC (permalink / raw)
To: Florian Fainelli; +Cc: netdev, David Miller, Dirk Brandewie
On Wed, Jan 19, 2011 at 01:09, Florian Fainelli <ffainelli@freebox.fr> wrote:
> From: Florian Fainelli <ffainelli@freebox.fr>
>
> This patch adds support for Marvell Alask M88E188R PHY chips. Support for
> other M88* PHYs is already there, so there is nothing more to add than its
> PHY id.
>
> Signed-off-by: Florian Fainelli <ffainelli@freebox.fr>
> CC: Dirk Brandewie <dirk.j.brandewie@intel.com>
> CC: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> ---
The patch itself looks fine. I am concerned about validation.
Dirk - is there a chance that the ce4100 will use this PHY? If so,
can you cover the validation?
For now I will add this to my net-next tree while I await word from Dirk.
--
Cheers,
Jeff
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY
2011-01-20 6:51 ` Jeff Kirsher
@ 2011-01-20 18:49 ` Dirk Brandewie
2011-01-21 16:27 ` Florian Fainelli
0 siblings, 1 reply; 4+ messages in thread
From: Dirk Brandewie @ 2011-01-20 18:49 UTC (permalink / raw)
To: Jeff Kirsher; +Cc: Florian Fainelli, netdev@vger.kernel.org, David Miller
On Wed, 2011-01-19 at 22:51 -0800, Jeff Kirsher wrote:
> On Wed, Jan 19, 2011 at 01:09, Florian Fainelli <ffainelli@freebox.fr> wrote:
> > From: Florian Fainelli <ffainelli@freebox.fr>
> >
> > This patch adds support for Marvell Alask M88E188R PHY chips. Support for
> > other M88* PHYs is already there, so there is nothing more to add than its
> > PHY id.
> >
> > Signed-off-by: Florian Fainelli <ffainelli@freebox.fr>
> > CC: Dirk Brandewie <dirk.j.brandewie@intel.com>
> > CC: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> > ---
>
> The patch itself looks fine. I am concerned about validation.
>
> Dirk - is there a chance that the ce4100 will use this PHY? If so,
> can you cover the validation?
Florian is working on a CE4100 based platform. It looks like they used
a different PHY from the Flacon Falls reference platform. I can't
directly test this patch since I don't have their hardware. I will be
testing .38-rc1 next week on falcon falls.
I think the best we can do without the hardware is to compare the data
sheet for the new PHY with the PHYs already supported and make sure they
are compatible. If the datasheets match up for the features the driver
is using this seems pretty low risk IMHO.
--Dirk
>
> For now I will add this to my net-next tree while I await word from Dirk.
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY
2011-01-20 18:49 ` Dirk Brandewie
@ 2011-01-21 16:27 ` Florian Fainelli
0 siblings, 0 replies; 4+ messages in thread
From: Florian Fainelli @ 2011-01-21 16:27 UTC (permalink / raw)
To: Dirk Brandewie; +Cc: Jeff Kirsher, netdev@vger.kernel.org, David Miller
Hello Dirk, Jeff,
On Thursday 20 January 2011 19:49:19 Dirk Brandewie wrote:
> On Wed, 2011-01-19 at 22:51 -0800, Jeff Kirsher wrote:
> > On Wed, Jan 19, 2011 at 01:09, Florian Fainelli <ffainelli@freebox.fr>
wrote:
> > > From: Florian Fainelli <ffainelli@freebox.fr>
> > >
> > > This patch adds support for Marvell Alask M88E188R PHY chips. Support
> > > for other M88* PHYs is already there, so there is nothing more to add
> > > than its PHY id.
> > >
> > > Signed-off-by: Florian Fainelli <ffainelli@freebox.fr>
> > > CC: Dirk Brandewie <dirk.j.brandewie@intel.com>
> > > CC: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> > > ---
> >
> > The patch itself looks fine. I am concerned about validation.
> >
> > Dirk - is there a chance that the ce4100 will use this PHY? If so,
> > can you cover the validation?
>
> Florian is working on a CE4100 based platform. It looks like they used
> a different PHY from the Flacon Falls reference platform. I can't
> directly test this patch since I don't have their hardware. I will be
> testing .38-rc1 next week on falcon falls.
Indeed, we use this PHY on our hardware.
>
> I think the best we can do without the hardware is to compare the data
> sheet for the new PHY with the PHYs already supported and make sure they
> are compatible. If the datasheets match up for the features the driver
> is using this seems pretty low risk IMHO.
As far as I could check, all M88E111* should behave the same for the setup
done in e1000.
--
Florian
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-01-21 16:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-01-19 9:09 [PATCH] e1000: add support for Marvell Alaska M88E1118R PHY Florian Fainelli
2011-01-20 6:51 ` Jeff Kirsher
2011-01-20 18:49 ` Dirk Brandewie
2011-01-21 16:27 ` Florian Fainelli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).