From: Vladimir Oltean <olteanv@gmail.com>
To: "Rafał Miłecki" <zajec5@gmail.com>
Cc: "Andrew Lunn" <andrew@lunn.ch>,
"David S . Miller" <davem@davemloft.net>,
"Eric Dumazet" <edumazet@google.com>,
"Jakub Kicinski" <kuba@kernel.org>,
"Paolo Abeni" <pabeni@redhat.com>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Florian Fainelli" <florian.fainelli@broadcom.com>,
"Jonas Gorski" <jonas.gorski@gmail.com>,
netdev@vger.kernel.org, "Hauke Mehrtens" <hauke@hauke-m.de>,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
bcm-kernel-feedback-list@broadcom.com,
"Rafał Miłecki" <rafal@milecki.pl>,
"Semih Baskan" <strst.gs@gmail.com>
Subject: Re: [PATCH V2 net 1/2] b53: deal with Northstar devices using CPU port 5 or 7
Date: Wed, 12 Aug 2026 00:52:39 +0300 [thread overview]
Message-ID: <20260811215239.jogh366edojjo3mi@skbuf> (raw)
In-Reply-To: <20260811193658.14304-1-zajec5@gmail.com>
On Tue, Aug 11, 2026 at 09:36:57PM +0200, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> Northstar devices have 3 CPU ports. Ports 5 and 7 may be used but have
> their limitations. Warn user when CPU port 8 isn't used and add EAP
> setup workaround to avoid non-functional standalone ports.
>
> Fixes: 4227ea91e265 ("net: dsa: b53: prevent standalone from trying to forward to other ports")
> Cc: Semih Baskan <strst.gs@gmail.com>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> Tested-by: Semih Baskan <strst.gs@gmail.com>
> ---
> V2: Print warning on switch setup instead of on actual EAP mode change
>
> I added PATCH 2/2 as requested but I'm not sure what DT maintainers will
> say about it. I'd suggest to take this fix if there are not more
> objections on net changes so regression can be fixed. I believe we can
> take more time to improve Linux dealing with switch to be more optimal.
It's a good thing you added it, it created a better perspective into
what problem you are trying to fix.
Please look into implementing a solution based on commit b79d7c14f480
("net: dsa: introduce preferred_default_local_cpu_port and use on
MT7530").
pw-bot: cr
prev parent reply other threads:[~2026-08-11 21:52 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-11 19:36 [PATCH V2 net 1/2] b53: deal with Northstar devices using CPU port 5 or 7 Rafał Miłecki
2026-08-11 19:36 ` [PATCH V2 RFC 2/2] ARM: dts: BCM5301X: change Luxul XWR-3150 CPU port from 5 do 8 Rafał Miłecki
2026-08-11 19:46 ` sashiko-bot
2026-08-11 20:33 ` Andrew Lunn
2026-08-11 21:46 ` Vladimir Oltean
2026-08-11 21:52 ` Vladimir Oltean [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260811215239.jogh366edojjo3mi@skbuf \
--to=olteanv@gmail.com \
--cc=andrew@lunn.ch \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=florian.fainelli@broadcom.com \
--cc=hauke@hauke-m.de \
--cc=jonas.gorski@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=rafal@milecki.pl \
--cc=robh@kernel.org \
--cc=strst.gs@gmail.com \
--cc=zajec5@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.