Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Michael Walle <michael@walle.cc>
CC:     "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        Shawn Guo <shawnguo@kernel.org>, Leo Li <leoyang.li@nxp.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH devicetree 0/3] NXP LS1028A DT changes for multiple switch
 CPU ports
Thread-Topic: [PATCH devicetree 0/3] NXP LS1028A DT changes for multiple
 switch CPU ports
Thread-Index: AQHYswuXtd5xxkc2eU+YiP2v4JjeM620vXCAgAABtgA=
Date:   Thu, 18 Aug 2022 14:55:56 +0000
Message-ID: <20220818145556.ieg37btfny3o2i4q@skbuf>
References: <20220818140519.2767771-1-vladimir.oltean@nxp.com>
 <8d3d96cdede2f1e40ed9ae5742a0468d@walle.cc>
In-Reply-To: <8d3d96cdede2f1e40ed9ae5742a0468d@walle.cc>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?0Du4K3v81bn6fsZk3QIMfNlD7DuvUBVIRknZA2e89so17QGmrrPlvqoG8CJx?=
 =?us-ascii?Q?s+5acmbmpTpKiWz8v/5aT0u10xEnDwy58l0EoV709T/ns+1I8AyHvmgn98eZ?=
 =?us-ascii?Q?2mba+LiZYvt6TrGjWTnSRtQagV57W0I904ACRo/1uGwmzOz/b9d2CWTJN9WA?=
 =?us-ascii?Q?/c20oewghlXQgfMU7ytsaiFiH73RY95iwch56U80wE6qW9aNC9yWi/wX+TsS?=
 =?us-ascii?Q?YkA+j7WfACxeHIGnZpKI9U+07RCgaPWJkvw4a7XpODEd3n3QIrlWKkJHf2of?=
 =?us-ascii?Q?IiJZTTba7a0kpNyZeTuL03ObGdRU1i6z+MtxTJ0ihMotip4RUNgSxbDF6P14?=
 =?us-ascii?Q?Cc/Ubj9OvsFqQJfBsfKnSXJ9O65n11LPbnYc8fk4ezMLIJpR2vn/dmBi18cc?=
 =?us-ascii?Q?JuYIAF3IzJrD7ImDKx7HYgNvHEQwOHzmfvWVXgOtULSmmmH7qQH/eaSf42yf?=
 =?us-ascii?Q?KPSZj0J+tNt43a0P08QGFa5cnq1SXg/O7uzbxrdKA5WINZ6qDmLWamX+O8WT?=
 =?us-ascii?Q?YAU1CLPistE7sax6LbD0Iwo6oJoD2/TNcLwdyR+2vdVo32RMHoYzV40lZEYu?=
 =?us-ascii?Q?ruqKhfZOeV9xlERjM9ghY4ACT2iPX2q/IvmFj10e99ZQhXJmWn9ukud+ilKg?=
 =?us-ascii?Q?jafpWMq2x8CwvLPFXI9+D9vZj7XmFac3xKVbJP7NFo8ClS2XvC2ImWkfO3Wg?=
 =?us-ascii?Q?6Sw8UrNIhUDITFniJ48XbmhDlC5kqdiH/wRwnEF1cQzAFEPJ2F9JhCQxaLcG?=
 =?us-ascii?Q?uTOB+ir+6ExhpDsZZUBR+A8BthTOnrvT3RVQUgq8KpyFb5KApFR3gMvm7vgb?=
 =?us-ascii?Q?313iH1rTmeqPdHzN+MNAaKxtRm21gwQ24vZ8rmrRNQY+aQB/zj2ZxlFOSuO6?=
 =?us-ascii?Q?g8lBlwxo5i/sQl8LFp+lq8edkrUCOJnjRF+RTXTTRORyJrWcdIUNoyVWdjK/?=
 =?us-ascii?Q?sTw9OKVXo+dARV4b9ixJX4jUsIzXTO3GTmwG4y2Pb/Z407sC3EO5EV3gQgA0?=
 =?us-ascii?Q?rrmrRDiQZtXzMTV9AtGPlgYFgbuQBfFCqCBvxHfM0K0tK9Z5BtSiOj76iDI3?=
 =?us-ascii?Q?Ye8sdmVpxkcgV2iO6c3NpkpAfrPa1nphtBw3UvMp6SQMpm4nIrJF8ejVt6+J?=
 =?us-ascii?Q?8bOZYhWDKrOYQsKR1vdMSF4gVnPFj8IClOD/dE7+a781f8HYTlQsxFFp99JJ?=
 =?us-ascii?Q?AKVuZaHsmPo5MoV22CSp+RTS6ECkaVktOYp5Mpv5lkDFOvNJIlyqvqT6q+hj?=
 =?us-ascii?Q?Md0wPHzAMMX01/w0ppncp424C9+spIptl54sI9mWNcbjwYefzBqPRzoocBk+?=
 =?us-ascii?Q?nPApjHy8VW8j2GvulTAmdFYNT7f8bO3hQwV8ZbMCbQZAsHIJ6XZwMK6io4H2?=
 =?us-ascii?Q?cowCXe/FAgvx1pFwC9i+Y02z7JpJ8Clqch2OLmzp6qRP3vWdm1aL62ns5Q8d?=
 =?us-ascii?Q?JtzQCdxaYbgckaFXBHu+cY5dchYpF/nL9Q3SBLPsOgwaUFoddZDvSEzLdHta?=
 =?us-ascii?Q?xKmiVnqZ4PCpK8/7at8rDBimav2+ITed4pq6tQN8o20KV5kNkt5TifcrD0m5?=
 =?us-ascii?Q?8qcL10B9OGvIWHtb1rqmWDiDHj9u6BTCPj7ShRnCD0CfTmbsMaDFpgwtyjTU?=
 =?us-ascii?Q?/g=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <62E4E7500C534240ABE3D0EF1365ED53@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2d5b3f4e-d7c8-4ae5-cdd3-08da8129c17c
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Aug 2022 14:55:56.6014
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 4g1FmRiNaVDmux81gd/CNHlISpHZp8i239LuCJBvNR7VGKSgd88OHA1S8c3DIiw87CTDLb7l9miv8I1RJ3ahSA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR04MB6028
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Aug 18, 2022 at 04:49:49PM +0200, Michael Walle wrote:
> Is it used automatically or does the userspace has to configure something=
?

DSA doesn't yet support multiple CPU ports, but even when it will, the
second DSA master still won't be used automatically. If you want more
details about the proposed UAPI to use the second CPU port, see here:
https://patchwork.kernel.org/project/netdevbpf/cover/20220523104256.3556016=
-1-olteanv@gmail.com/

> > Care has been taken that this change does not produce regressions when
> > using updated device trees with old kernels that do not support multipl=
e
> > DSA CPU ports. The only difference for old kernels will be the
> > appearance of a new net device (for &enetc_port3) which will not be ver=
y
> > useful for much of anything.
>=20
> Mh, I don't understand. Does it now cause regressions or not? I mean
> besides that there is a new unused interface?

It didn't cause regressions until kernel 5.13 when commit adb3dccf090b
("net: dsa: felix: convert to the new .change_tag_protocol DSA API")
happened, then commit 00fa91bc9cc2 ("net: dsa: felix: fix tagging
protocol changes with multiple CPU ports") fixed that regression and was
backported to the linux-5.15.y stable branch AFAIR. So at least kernels
5.15 and newer should work properly with the new device trees.

> I was just thinking of that systemready stuff where the u-boot might
> supply its (newer) device tree to an older kernel, i.e. an older debian
> or similar.
>=20
> -michael

Yeah, I hear you, I'm doing my best to make the driver work with a
one-size-fits-all device tree, both ways around.=