Dsa multi-cpu port problem

[Rosen Penev]

On Wed, Mar 10, 2021 at 4:56 PM Ansuel Smith <ansuelsmth at gmail.com> wrote:
>
> >
> > On Wed, Mar 10, 2021 at 4:19 PM Ansuel Smith <ansuelsmth at gmail.com> wrote:
> > >
> > > I'm working on the conversion of ipq806x to dsa.
> > > It's been 2 years and still dsa doesn't support multi-cpu port.
> > > Online there are many patches that add support for this but they were
> > > never accepted upstream since there isn't a solution that would work
> > > for all the switches. Fact is that openwrt is different and have some
> > > flexibility about this kind problem.
> > > There is actually a patchset (rejected) that adds multi-cpu support
> > > defined in the dts that can be used and doesn't look that bad.
> > >
> > > So here is the question:
> > > Considering the fact that at least 3 target have disabled port (mtk,
> > > mvebu and upcoming ipq806x) why not use the proposed patch
> > > while upstream a final solution is found for this problem?
> > > Turris already use a variant of that patch and also mediatek use
> > > it.
> > I closed https://github.com/openwrt/openwrt/pull/3655 since 21.02 was
> > branched. I would use that patchset as the Turris people are working
> > on upstreaming a bunch of stuff.
>
> Didn't know there was already a port of the variant. So in your
> opinion the round-robin should be the right solution.
> Turris and mediatek from what i found online used the original
> implementation with the cpu port hardcoded in the dts.
> I can only find rfc of the round-robin patch online, do you have more
> discussion about that? Curious to know if upstream they finally take a
> decision instead of keep staying in the limbo.
This patchset was taken from
https://gitlab.nic.cz/turris/turris-build/-/blob/hbd/patches/openwrt/wip/0005-mvebu-initial-support-for-Omnia-on-5.4-kernel.patch

Limbo so far. The developer needs to resend it. I believe upstream had
no issue with this patchset.
>
> > >
> > > The idea would be to place the patch as a hack in the generic
> > > target dir and targets can add patches to modify the dsa driver
> > > accordingly (if needed).
> > >
> > > (the multi-cpu variant i'm talking about it the one that statically
> > > link a port to a dedicated cpu port in the dts)
> > >
> > > _______________________________________________
> > > openwrt-devel mailing list
> > > openwrt-devel at lists.openwrt.org
> > > https://lists.openwrt.org/mailman/listinfo/openwrt-devel