Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-24 Thread Andrew Lunn
On Tue, Jan 24, 2017 at 05:47:06PM +0100, Gregory CLEMENT wrote: > Hi David, > > On ven., janv. 20 2017, David Miller wrote: > > > From: Gregory CLEMENT > > Date: Thu, 19 Jan 2017 22:49:32 +0100 > > > >> I created a new family for this switch and filled the ops structure > >> by selecting whi

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-24 Thread Vivien Didelot
Hi Gregory, Gregory CLEMENT writes: > Hi David, > > On ven., janv. 20 2017, David Miller wrote: > >> From: Gregory CLEMENT >> Date: Thu, 19 Jan 2017 22:49:32 +0100 >> >>> I created a new family for this switch and filled the ops structure >>> by selecting which seems the more appropriate fu

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-24 Thread Gregory CLEMENT
Hi David, On ven., janv. 20 2017, David Miller wrote: > From: Gregory CLEMENT > Date: Thu, 19 Jan 2017 22:49:32 +0100 > >> I created a new family for this switch and filled the ops structure >> by selecting which seems the more appropriate functions. I rebased >> the series on net-next/master

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-20 Thread Andrew Lunn
> Actually I didn't find anything related to the temperature measurement > in the datasheet I have. For the 6390 there is a dedicated datsheet for > the PHY part for the 6352 it is part of the same datasheet. Hi Gregory The temperature sensor changes have landed in net-next. If you have time, ple

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-20 Thread David Miller
From: Gregory CLEMENT Date: Thu, 19 Jan 2017 22:49:32 +0100 > I created a new family for this switch and filled the ops structure > by selecting which seems the more appropriate functions. I rebased > the series on net-next/master which allowed me to benefit to the > eeprom functions introduced f

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-20 Thread Andrew Lunn
> What I wanted to do was to test 6390 and 6352 temperature related > functions and to see if one of them worked. That's how I realized it was > not possible to do it with dsa2. Hi Gregory Probably the best thing to do is wait until the temperature code moves into the PHY driver. It then becomes

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-20 Thread Gregory CLEMENT
Hi Andrew, On jeu., janv. 19 2017, Andrew Lunn wrote: >> While comparing the datasheet and the ops functions used, some >> question came to me. They should not prevent applying this series, >> but their answer would help me to have a better understanding of the >> dsa subsystem. >> >> - Are t

RE: [EXT] Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-19 Thread Jon Pannell
; Sebastian Hesselbarth ; Thomas Petazzoni ; linux-arm-ker...@lists.infradead.org; Nadav Haklai ; Wilson Ding ; Kostya Porotchkin ; Joe Zhou ; Jon Pannell Subject: [EXT] Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin External Email

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-19 Thread Vivien Didelot
Hi Gregory, Vivien Didelot writes: > The temperature is accessed via ethtool -e|-E. Oops, that was a silly mistake, no need to explain! ;-) Vivien

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-19 Thread Vivien Didelot
Hi Gregory, Gregory CLEMENT writes: > I created a new family for this switch and filled the ops structure by > selecting which seems the more appropriate functions. We don't create families, this information comes from the Marvell DSDT. If you don't know which switch family Marvell put the 6341

Re: [PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-19 Thread Andrew Lunn
> While comparing the datasheet and the ops functions used, some > question came to me. They should not prevent applying this series, > but their answer would help me to have a better understanding of the > dsa subsystem. > > - Are the temperature related operation still useful with dsa2 ? No. I'

[PATCH v5 0/2] Add support for the ethernet switch on the ESPRESSObin

2017-01-19 Thread Gregory CLEMENT
Hi, This set of patches adds support for the Marvell Ethernet switch 88E6341 which is found on the ESPRESSObin. With this series the network is usable on this board. >From now on, I am taking care of this series. As Andrew Lunn pointed this switch is not fully compatible with the 6352. However i