Re: [PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-06 Thread Florian Fainelli
On 06/05/2016 08:19 PM, Andrew Lunn wrote: >> How much support do we want to have for the old binding for in tree >> platforms? Is the plan to migrate them all to the new binding? > > I think there are three cases to consider. > > 1) There are some old boards using setup.c files which have a

Re: [PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-05 Thread Andrew Lunn
> How much support do we want to have for the old binding for in tree > platforms? Is the plan to migrate them all to the new binding? I think there are three cases to consider. 1) There are some old boards using setup.c files which have a platform device, platform data, etc. I've never used

Re: [PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-05 Thread Florian Fainelli
Le 04/06/2016 13:00, Andrew Lunn a écrit : >> -static int dsa_cpu_parse(struct device_node *port, u32 index, >> - struct dsa_switch_tree *dst, >> - struct dsa_switch *ds) >> +static int _dsa_cpu_parse(struct dsa_switch_tree *dst, >> +

Re: [PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-04 Thread Andrew Lunn
> -static int dsa_cpu_parse(struct device_node *port, u32 index, > - struct dsa_switch_tree *dst, > - struct dsa_switch *ds) > +static int _dsa_cpu_parse(struct dsa_switch_tree *dst, > +struct dsa_switch *ds, > +

Re: [PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-04 Thread Andrew Lunn
> @@ -626,8 +736,8 @@ static int _dsa_register_switch(struct dsa_switch *ds, > struct device_node *np) > { > struct device_node *ports = dsa_get_ports(ds, np); > > - if (IS_ERR(ports)) > - return PTR_ERR(ports); > + if (IS_ERR(ports) && PTR_ERR(ports) == -EINVAL) > +

[PATCH net-next 2/9] net: dsa: Add support for parsing the old binding

2016-06-03 Thread Florian Fainelli
Extend dsa2.c to support parsing for the old binding, which mostly means looking for more or less the same properties in different places. Signed-off-by: Florian Fainelli --- net/dsa/dsa2.c | 142 ++--- 1 file changed,