On Wed, Mar 10, 2021 at 4:56 PM Ansuel Smith <[email protected]> wrote: > > > > > On Wed, Mar 10, 2021 at 4:19 PM Ansuel Smith <[email protected]> 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 > > > [email protected] > > > https://lists.openwrt.org/mailman/listinfo/openwrt-devel _______________________________________________ openwrt-devel mailing list [email protected] https://lists.openwrt.org/mailman/listinfo/openwrt-devel
