On Mon, Jun 27, 2016 at 06:19:28PM -0700, Florian Fainelli wrote: > 2016-06-27 18:05 GMT-07:00 Andrew Lunn <and...@lunn.ch>: > > On Mon, Jun 27, 2016 at 05:52:37PM -0700, Florian Fainelli wrote: > >> Hi all, > >> > >> This patch series adds support for platform data using the new code from > >> net/dsa/dsa2.c. The motivation behind this is that we have a bit of in tree > >> platforms (ar7, bcm47xx, x86, others) that could be benefiting from the new > >> dsa_register_switch() API model but do not support Device Tree, nor is > >> there a > >> plan to bring Device Tree to these platforms (time vs. benefits). > > > > Hi Florian > > > > Please could you convert an in tree device to actually use this. > > Sure, I don't think there are going to be in tree users who need the > dsa2_port_link information most of what we have is typically single > chip, and so in that case, we can even re-use the existing > dsa_platform_data.
O.K. Less is better. When going through the old code for the earlier re-structuring proposals, the code is not simple at times. I would prefer not adding more complexity to dsa2 than really is needed. Andrew