[U-Boot] [PATCH] net/phy: support the mv88e6352 switch

Joe Hershberger joe.hershberger at gmail.com
Fri Sep 21 21:05:25 CEST 2012


Hi Valentin,

On Fri, Sep 21, 2012 at 9:00 AM, Valentin Longchamp
<valentin.longchamp at keymile.com> wrote:
> On 08/17/2012 09:59 AM, Prafulla Wadaskar wrote:
>>
>>
>>> -----Original Message-----
>>> From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
>>> Sent: 16 August 2012 16:47
>>> To: u-boot at lists.denx.de
>>> Cc: Valentin Longchamp; Holger Brunck; Prafulla Wadaskar; Joe
>>> Hershberger
>>> Subject: [PATCH] net/phy: support the mv88e6352 switch
>>>
>>> This patch add support for the configuration of an external switch
>>> from
>>> the 88E6xxx series from Marvell trough an MDIO link using indirect
>>> adressing. This can be used if we do not want to use an EEPROM for the
>>> configuration.
>>>
>>> This driver is not generic and was not tested on other switches than
>>> the
>>> 88e6352. This is proposed as a first implementation that is somewhat
>>> limited but works and that can be used as a basis for further
>>> developments for this switch family.
>>>
>>> Signed-off-by: Valentin Longchamp <valentin.longchamp at keymile.com>
>>> cc: Holger Brunck <holger.brunck at keymile.com>
>>> cc: Prafulla Wadaskar <prafulla at marvell.com>
>>> cc: Joe Hershberger <joe.hershberger at gmail.com>
>>> ---
>>>  drivers/net/phy/Makefile    |    1 +
>>>  drivers/net/phy/mv88e6352.c |  318
>>> +++++++++++++++++++++++++++++++++++++++++++
>>>  include/mv88e6352.h         |   92 +++++++++++++
>>>  3 files changed, 411 insertions(+), 0 deletions(-)
>>>  create mode 100644 drivers/net/phy/mv88e6352.c
>>>  create mode 100644 include/mv88e6352.h
>>
>> Hi Valentin
>>
>> I do not have any specific feedback for this patch,
>> To me this looks okay.
>>
>> Joe may pull it if he finds everything okay.
>>
>> Regards...
>> Prafulla . . .
>>
>
>
> Hi Joe,
>
> We have never received any feedback from you regarding this patch. Could you
> please consider review it and if applicable apply it since it is a dependency
> for another series in the Marvell tree.

Sorry... I'm a bit behind on applying patches and testing.  It is
assigned to me in patchwork.  I'll get to it as soon as I can.

Cheers,
-Joe


More information about the U-Boot mailing list