All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Documentation: networking: dsa: Remove platform device TODO
@ 2016-08-27 22:34 Florian Fainelli
  2016-08-28 15:30 ` Andrew Lunn
  2016-08-29  3:43 ` David Miller
  0 siblings, 2 replies; 3+ messages in thread
From: Florian Fainelli @ 2016-08-27 22:34 UTC (permalink / raw)
  To: netdev; +Cc: davem, andrew, vivien.didelot, Florian Fainelli

Since commit 83c0afaec7b7 ("net: dsa: Add new binding implementation"),
the shortcomings of the dsa platform device have been addressed, remove
that TODO item.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 Documentation/networking/dsa/dsa.txt | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/Documentation/networking/dsa/dsa.txt b/Documentation/networking/dsa/dsa.txt
index 44ed453ccf66..a4e55c76d371 100644
--- a/Documentation/networking/dsa/dsa.txt
+++ b/Documentation/networking/dsa/dsa.txt
@@ -587,26 +587,6 @@ of DSA, would be the its port-based VLAN, used by the associated bridge device.
 TODO
 ====
 
-The platform device problem
----------------------------
-DSA is currently implemented as a platform device driver which is far from ideal
-as was discussed in this thread:
-
-http://permalink.gmane.org/gmane.linux.network/329848
-
-This basically prevents the device driver model to be properly used and applied,
-and support non-MDIO, non-MMIO Ethernet connected switches.
-
-Another problem with the platform device driver approach is that it prevents the
-use of a modular switch drivers build due to a circular dependency, illustrated
-here:
-
-http://comments.gmane.org/gmane.linux.network/345803
-
-Attempts of reworking this has been done here:
-
-https://lwn.net/Articles/643149/
-
 Making SWITCHDEV and DSA converge towards an unified codebase
 -------------------------------------------------------------
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: networking: dsa: Remove platform device TODO
  2016-08-27 22:34 [PATCH] Documentation: networking: dsa: Remove platform device TODO Florian Fainelli
@ 2016-08-28 15:30 ` Andrew Lunn
  2016-08-29  3:43 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: Andrew Lunn @ 2016-08-28 15:30 UTC (permalink / raw)
  To: Florian Fainelli; +Cc: netdev, davem, vivien.didelot

On Sat, Aug 27, 2016 at 03:34:20PM -0700, Florian Fainelli wrote:
> Since commit 83c0afaec7b7 ("net: dsa: Add new binding implementation"),
> the shortcomings of the dsa platform device have been addressed, remove
> that TODO item.
> 
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Acked-by: Andrew Lunn <andrew@lunn.ch>

Thanks :-)

    Andrew

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Documentation: networking: dsa: Remove platform device TODO
  2016-08-27 22:34 [PATCH] Documentation: networking: dsa: Remove platform device TODO Florian Fainelli
  2016-08-28 15:30 ` Andrew Lunn
@ 2016-08-29  3:43 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2016-08-29  3:43 UTC (permalink / raw)
  To: f.fainelli; +Cc: netdev, andrew, vivien.didelot

From: Florian Fainelli <f.fainelli@gmail.com>
Date: Sat, 27 Aug 2016 15:34:20 -0700

> Since commit 83c0afaec7b7 ("net: dsa: Add new binding implementation"),
> the shortcomings of the dsa platform device have been addressed, remove
> that TODO item.
> 
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Applied.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-08-29  3:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-27 22:34 [PATCH] Documentation: networking: dsa: Remove platform device TODO Florian Fainelli
2016-08-28 15:30 ` Andrew Lunn
2016-08-29  3:43 ` David Miller

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.