Clarification about dsa and ipq806x

[Adrian Schmutzler]

Hi,

> -----Original Message-----
> From: openwrt-devel [mailto:openwrt-devel-bounces at lists.openwrt.org]
> On Behalf Of ansuelsmth at gmail.com
> Sent: Montag, 1. März 2021 21:50
> To: 'Adrian Schmutzler' <mail at adrianschmutzler.de>; 'OpenWrt
> Development List' <openwrt-devel at lists.openwrt.org>
> Subject: R: Clarification about dsa and ipq806x
> 
> > > Hello,
> > > I just pushed a fist pr to the openwrt github repo for ipq806x port
> > > to kernel 5.10.
> > > The idea is to switch this target to dsa but there is a problem...
> > > Since kernel 5.10 is a testing kernel how should I change the base
> > > files to apply different network config?
> > > Kernel 5.4 would still use the old swconfig driver and the dsa
> > > driver still lacks vlan support in kernel 5.4.
> > > Do we have some way in openwrt to know if the system use dsa driver
> > > or swconfig driver?
> >
> > In my personal opinion, implementing a solution for a split config is
> > a waste of time. What would we gain?
> > If the aim is to switch to DSA, just convert to 5.10 with DSA. Then we
> > might need a bit more testing of the PR before merging it, but work
> > would be focused on what's the aim of the entire thing. IIRC that's
> > also how it was done for mt7621 as the initial DSA target back then.
> >
> 
> I backported the dsa patch to 5.4 and I converted the board and dts file to
> dsa.
> The testing kernel is ready... We just need some testing...
> Since some target still use the old driver, can we keep swconfig and the old
> driver compiled?

Well, I'd remove it from ipq806x, but obviously keep generic/other targets' stuff.
I do not see a point in keeping any remnants in ipq806x if we don't use it anymore there.

Best

Adrian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.openwrt.org/pipermail/openwrt-devel/attachments/20210301/14b66312/attachment.sig>