[PATCH rfc v0 0/9] DSA: Move parts of inband signalling into the DSA
flat view
STALE1476d
From: Andrew Lunn <andrew@lunn.ch>
Date: 2022-09-19 22:19:13
Revision v0 of 2 in this series.
Revisions (2)
- v0 current
- v2 [diff vs current]
This is an RFC patchset. Mattias Forsblad proposal for adding some core helpers to DSA for inband signalling is going in a good direction, but there are a couple of things which i think can be better. This patchset offs an alternative to patch 2/7: net: dsa: Add convenience functions for frame handling and patch 7/7 net: dsa: qca8k: Use new convenience functions This patchset takes the abstraction further, putting more into the core. It also makes the qca8k fully use the abstraction unlike 7/7. The end result has a slightly different structure, in that there is a struct dsa_inband of which qca8k has two instances of this. Doing this avoids the custom completion code. If qca8k can have multiple parallel request/replies in flight, it seems likely other devices can as well, so this should be part of the abstraction. Since i don't have the qck8 hardware, i hope that lots of small patches make the review work easier, and finding the introduced bugs is quicker. The MIB handling of the qc8k is somewhat odd. It would be nice to work on that further and try to make it better fit the model used here. That work can be done later, and probably is more invasive than the step by step approach taken here. Another aim has been to make it easy to merge Mattias mv88e6xxx patches with this patchset. The basic API is the same, so i think it should be possible. These are compile tested only.... Andrew Lunn (9): net: dsa: qca8k: Fix inconsistent use of jiffies vs milliseconds net: dsa: qca8k: Move completion into DSA core net: dsa: qca8K: Move queuing for request frame into the core net: dsa: qca8k: dsa_inband_request: More normal return values net: dsa: qca8k: Move request sequence number handling into core net: dsa: qca8k: Refactor sequence number mismatch to use error code net: dsa: qca8k: Pass error code from reply decoder to requester net: dsa: qca8k: Pass response buffer via dsa_rmu_request net: dsa: qca8k: Move inband mutex into DSA core drivers/net/dsa/qca/qca8k-8xxx.c | 234 ++++++++----------------------- drivers/net/dsa/qca/qca8k.h | 8 +- include/net/dsa.h | 25 ++++ net/dsa/dsa.c | 90 ++++++++++++ 4 files changed, 172 insertions(+), 185 deletions(-) -- 2.37.2