Re: [PATCH net-next 6/7] net: dsa: mv88e6xxx: add number of database to info

2016-04-15 Thread Andrew Lunn
On Fri, Apr 15, 2016 at 04:24:26PM -0400, Vivien Didelot wrote: > Hi Andrew, > > Andrew Lunn writes: > > > On Fri, Apr 15, 2016 at 02:25:49PM -0400, Vivien Didelot wrote: > >> Move the number of databases to the info structure. > > > > Isn't the number of databases a property of

Re: [PATCH net-next 6/7] net: dsa: mv88e6xxx: add number of database to info

2016-04-15 Thread Vivien Didelot
Hi Andrew, Andrew Lunn writes: > On Fri, Apr 15, 2016 at 02:25:49PM -0400, Vivien Didelot wrote: >> Move the number of databases to the info structure. > > Isn't the number of databases a property of the family? No. We've seen [1] for instance that 6061 and 6065 are both part

Re: [PATCH net-next 6/7] net: dsa: mv88e6xxx: add number of database to info

2016-04-15 Thread Andrew Lunn
On Fri, Apr 15, 2016 at 02:25:49PM -0400, Vivien Didelot wrote: > Move the number of databases to the info structure. Isn't the number of databases a property of the family? I would add a table indexed by family. Andrew

[PATCH net-next 6/7] net: dsa: mv88e6xxx: add number of database to info

2016-04-15 Thread Vivien Didelot
Move the number of databases to the info structure. Signed-off-by: Vivien Didelot --- drivers/net/dsa/mv88e6123.c | 6 +++--- drivers/net/dsa/mv88e6131.c | 8 drivers/net/dsa/mv88e6171.c | 8 drivers/net/dsa/mv88e6352.c | 12 ++--