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:     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>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Heiko Thiery <heiko.thiery@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>
Subject: Re: [PATCH] Revert "arm64: dts: ls1028a: sl28: use ocelot-8021q
 tagging by default"
Thread-Topic: [PATCH] Revert "arm64: dts: ls1028a: sl28: use ocelot-8021q
 tagging by default"
Thread-Index: AQHY6ffdJD/5Oq16AkSmf/jVUKUJTq4iJNOAgAAGMICAAAOMgA==
Date:   Thu, 27 Oct 2022 12:40:09 +0000
Message-ID: <20221027124009.mxfcyuayfv2snqf5@skbuf>
References: <20221027113248.420216-1-michael@walle.cc>
 <20221027120519.7f3xun66l4lamcq6@skbuf>
 <20221027122727.fhs35eqtzmeen6x4@skbuf>
In-Reply-To: <20221027122727.fhs35eqtzmeen6x4@skbuf>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Hno4tj8wSUa98lpN4Q/186dWaOLmyy1+Q/3th1UTe0focea/Yl2SapNFKVcz?=
 =?us-ascii?Q?4ipGvOE+nRgL3f1n+HEvtxucKW4ZxiMoWsLfdxxOAur1wM2xEYVp+sN/Twea?=
 =?us-ascii?Q?rrO0mifkQD40R4m7C21/KpXb0YFcd3wkqaDPl3e7E741S/Cd+JeTS6tZ5IDk?=
 =?us-ascii?Q?SG2yDYEoXNTT4o0SISWl1CeogNRkW3mEVV0oZ13kgE/CHc01BtVSKaNom2az?=
 =?us-ascii?Q?HWBQswa+Yp6Asc8+DkXJ4ZOjZO22jxqy/EWzjpMkxhXagDzAoWU8e3UcEUIg?=
 =?us-ascii?Q?eT/hORn0yxrg8JiFQDoIUmCGV0jUD7qL11kp1WobHUnjtW7FRSrQ88CIOFyI?=
 =?us-ascii?Q?WcuYDeaS67x6lMvZvZP0OFobrcFWJN0kphro/WoZkGKKfAjNFU9HwzWL3ze9?=
 =?us-ascii?Q?nQgr6vf+7PDvJYoFD5pHSSVfMT6+sckQT2Vp+MhnXt6eEGsW12yinYNcqLlG?=
 =?us-ascii?Q?J0FS5Yyo1M8GkE8PULv9f5xYvZt3Cck8736gi8CSv+J3Xu9iZjzLtQ/+YjS0?=
 =?us-ascii?Q?sDwm0WIBedJ9UFhqjOZhPuETZnYsHdSuQ9EMsEciJW4+YqTmoEX6GQUMy+c9?=
 =?us-ascii?Q?jNZE5wZ3Q6bIuIfrfPTraxP8Tcs5nth3SOy6RHCqtqdDJh2BU+rqN1wvnWT2?=
 =?us-ascii?Q?jb4ONIrBz4KVeN0K3tPGmFpVhE2blmCXRkOGQX/WzOR0zL9dPQyd/CSExW+X?=
 =?us-ascii?Q?PwopK6LSGP4gsThLRP0QI46G3RnmZmSiCIOR9U1pnWOFOZtwBm+JYETXpYCt?=
 =?us-ascii?Q?9rJ8GcrYDkbFp6FlFj6bZFefyqU14Elo5jYtbBf30Cvb1t6+NR/NqjnnrYBB?=
 =?us-ascii?Q?qmiKH6bdLhW1wkkwYNgen9fuFoq85tS78nTQ94tt2QXwsDt4gDdJW4vgFVy3?=
 =?us-ascii?Q?dGIr1J9/t9YcUE/VwQdhQ2BIHfPzrV8j+Oiwg02s10KvwEbVuvVIFcg9XoWK?=
 =?us-ascii?Q?kKBSOrPoNhMO/5G/pvQFR5GSZ5APuSTGyY5X8GAeB73aAK5/HjDh9dSR0Zp3?=
 =?us-ascii?Q?D2RcP6IC5bUB+8gVczqSnAkrJLtfMUHytMEQPkThErF0f/GKrxjesQfwR0uw?=
 =?us-ascii?Q?059uUEqgKrTzcKYnHl5saytEHyMDATW+7+x6G7bTzMxWyNTj0nyV+enKTxZ1?=
 =?us-ascii?Q?TMFfZzrpGRthBa+SdRWgrkSUa36tlJ/08dFQw/UqBzSrimBs04JoyS9grMTS?=
 =?us-ascii?Q?rokwBxEx8hRpRlRT5WYbcrlVQdMPxOrYl9dvZwxnndkEDqvTv3l661Uj5EAo?=
 =?us-ascii?Q?ZUa6oA7n0nYdf2XR2R6yBeux4q7yvHxOunIWuT4OQkGZeJKoRL6zcywpZvBX?=
 =?us-ascii?Q?fqMGK/sDh2q3ughAkd8F+zFR3qhbgYbOjRMl8jb2PpYwJsuyNikwzWELjuhi?=
 =?us-ascii?Q?ZCxuE3QaD8U2mCuPasZRn/wlsAJUbTMh6mzlhbkV0ViVAZRY2nyUiHO9Jslm?=
 =?us-ascii?Q?p940ajcYm0SmladH5WJ7ryw5rS+8VZVA/8UrJcA3uf6KHF0ww1KTyPrLYLNB?=
 =?us-ascii?Q?vb0050O9hO5/ipDZB++6Xrv4eBMioHdPby9gCWrvyzaLYZmmiH+ViX2hF24O?=
 =?us-ascii?Q?OE/2yjVxnMTZrU+ulXE72UTFudkfM2tQDUN0ITVp85FahPGWA/JuXWSmij2+?=
 =?us-ascii?Q?lg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <9F1EE8B1A8079D4FAEE34596FCCD5638@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: b49a7f1f-30db-490f-053b-08dab818627b
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Oct 2022 12:40:09.6676
 (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: dCtx3zAe+61ED7hJQ2Gp9Oh7J0DrNtMsX7BRp5ugnGVUPMjQcFaTTs9KbRUckqER2xGAr8MDrdTu+FMUzB15Tw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9400
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 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, Oct 27, 2022 at 03:27:27PM +0300, Vladimir Oltean wrote:
> I think we should treat what we committed to in terms of dt-bindings
> with utmost respect, so I would consider your proposed revert as the
> absolute last option. Reverting a device tree change doesn't mean that
> the device trees without the revert will disappear from circulation.
>=20
> So far we have 3 options for fixing this within the kernel
>=20
> - make tag_ocelot.o and tag_ocelot_8021q.o link into the same
>   tag_ocelot.ko
>=20
> - change the MODULE_ALIAS() of all tagging protocol driver modules from
>   "dsa_tag-<number" to something containing their string name - what you
>   proposed. I don't know why the current MODULE_ALIAS() is formatted the
>   way it is. Maybe Andrew can comment on whether this is feasible.
>   I think there isn't any backwards compatibility concern, since only
>   modules compiled for a certain kernel version are expected to be
>   loaded.
>=20
> - put a translation table between string and MODULE_ALIAS() inside
>   dsa_core.ko, which potentially duplicates code. Maybe if we
>   auto-generate it somehow?

Sorry for sending so many emails. I think the problem we should fix
first and foremost is that, if there's a user protocol specified in the
device tree but the kernel fails to load it, it should simply stick with
the default tagging protocol, instead of failing to probe. Everything
else can be dealt with as a future refinement.=