| Message ID | 20230309125421.3900962-1-lukma@denx.de (mailing list archive) |
|---|---|
| Headers | show |
| Series | dsa: marvell: Add support for mv88e6071 and 6020 switches | expand |
Hello Lukasz, On Thu, Mar 09, 2023 at 01:54:14PM +0100, Lukasz Majewski wrote: > This patch set provides following changes: > > - Provide support for mv88e6020 and mv88e6071 switch circuits (the > "Link Street" family of products including added earlier to this > driver mv88e6250 and mv88e6220). > > - Add the max_frame size variable to specify the buffer size for the > maximal frame size > > - The above change required adjusting all supported devices in the > mv88e6xxx driver, as the current value assignment is depending > on the set of provided callbacks for each switch circuit - i.e. > until now the value was not explicitly specified. > > - As the driver for Marvell's mv88e6xxx switches was rather complicated > the intermediate function (removed by the end of this patch set) > has been introduced. It was supposed to both validate the provided > values deduced from the code and leave a trace of the exact > methodology used. The problem with MTU 1492 has been resolved as commit 7e9517375a14 ("net: dsa: mv88e6xxx: fix max_mtu of 1492 on 6165, 6191, 6220, 6250, 6290"), is present in net-next, and as such, you are free to resubmit this patchset on top of the current net-next.gi/main whenever you feel like it, no need to wait for a merge window.
This patch set provides following changes: - Provide support for mv88e6020 and mv88e6071 switch circuits (the "Link Street" family of products including added earlier to this driver mv88e6250 and mv88e6220). - Add the max_frame size variable to specify the buffer size for the maximal frame size - The above change required adjusting all supported devices in the mv88e6xxx driver, as the current value assignment is depending on the set of provided callbacks for each switch circuit - i.e. until now the value was not explicitly specified. - As the driver for Marvell's mv88e6xxx switches was rather complicated the intermediate function (removed by the end of this patch set) has been introduced. It was supposed to both validate the provided values deduced from the code and leave a trace of the exact methodology used. Lukasz Majewski (6): dsa: marvell: Provide per device information about max frame size net: dsa: mv88e6xxx: add support for MV88E6071 switch dsa: marvell: Define .set_max_frame_size() function for mv88e6250 SoC family dsa: marvell: Add helper function to validate the max_frame_size variable dsa: marvell: Correct value of max_frame_size variable after validation dsa: marvell: Modify get max MTU callback to use per switch provided value Matthias Schiffer (1): net: dsa: mv88e6xxx: add support for MV88E6020 switch drivers/net/dsa/mv88e6xxx/chip.c | 83 ++++++++++++++++++++++++++++++-- drivers/net/dsa/mv88e6xxx/chip.h | 8 +++ drivers/net/dsa/mv88e6xxx/port.h | 2 + 3 files changed, 88 insertions(+), 5 deletions(-)