Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-05-07 Thread Zhang, Helin
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Lu, Wenzhuo > Sent: Thursday, January 18, 2018 8:40 AM > To: Chas Williams > Cc: dev@dpdk.org; skh...@vmware.com; Charles (Chas) Williams; Chas > Williams > Subject: Re: [dpdk-dev] [PATCH] n

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-01-17 Thread Lu, Wenzhuo
re.com; Charles (Chas) Williams ; Chas Williams Subject: Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY I derived that value by observing what the ixgbe finds when loading on my platform. The docs for the platform say that the attached PHY is a Broadcom 54616. I co

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-01-15 Thread Chas Williams
dev@dpdk.org; skh...@vmware.com; Charles (Chas) Williams < > ciwil...@brocade.com>; Chas Williams > *Subject:* Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for > Broadcom 54616 PHY > > > > I derived that value by observing what the ixgbe finds when loading on my

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-01-14 Thread Lu, Wenzhuo
:3ch...@gmail.com] Sent: Friday, January 12, 2018 1:57 AM To: Lu, Wenzhuo Cc: dev@dpdk.org; skh...@vmware.com; Charles (Chas) Williams ; Chas Williams Subject: Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY I derived that value by observing what the ixgbe finds when

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-01-11 Thread Chas Williams
I derived that value by observing what the ixgbe finds when loading on my platform. The docs for the platform say that the attached PHY is a Broadcom 54616. I couldn't find any documentation on this PHY to determine what this value is (i.e. is it really some sort of ID string for that PHY).I

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2018-01-11 Thread Lu, Wenzhuo
Hi Chas, > -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Chas Williams > Sent: Thursday, December 7, 2017 7:55 AM > To: dev@dpdk.org > Cc: skh...@vmware.com; Charles (Chas) Williams ; > Chas Williams > Subject: [dpdk-dev] [PATCH] net/e1000

Re: [dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2017-12-06 Thread Chas Williams
Sorry, sent this to the wrong maintainer initially. I am cc'ing the Broadcom bnxt maintainers in hopes that they might comment about the 54616 behavior on reset. I couldn't find any programmer guide for this chip online but PHYs are typically well behaved. This patch does work on our test system

[dpdk-dev] [PATCH] net/e1000: add minimum support for Broadcom 54616 PHY

2017-12-06 Thread Chas Williams
From: "Charles (Chas) Williams" If we find a Broadcom 54616, handle as a e1000_phy_none assuming that the NIC reset has initialized the PHY to a sane state. Signed-off-by: Chas Williams --- drivers/net/e1000/base/e1000_82575.c | 5 + drivers/net/e1000/base/e1000_defines.h | 1 + 2 files