Received-SPF: pass (google.com: domain of linux-kernel+bounces-24851-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Fri, 12 Jan 2024 16:43:16 +0100
From: Marc Kleine-Budde <mkl@pengutronix.de>
To: "Kumar, Udit" <u-kumar1@ti.com>
Cc: Bhavya Kapoor <b-kapoor@ti.com>, linux-kernel@vger.kernel.org, 
	netdev@vger.kernel.org, linux-can@vger.kernel.org, mailhol.vincent@wanadoo.fr, 
	rcsekar@samsung.com, pabeni@redhat.com, kuba@kernel.org, edumazet@google.com, 
	davem@davemloft.net, wg@grandegger.com, vigneshr@ti.com
Subject: Re: [PATCH] net: can: Add support for aliases in CAN
Message-ID: <20240112-overreact-dwindling-2949267e8a02-mkl@pengutronix.de>
References: <20240102102949.138607-1-b-kapoor@ti.com>
 <20240102-chop-extending-b7dc1acaf5db-mkl@pengutronix.de>
 <8dee1738-0bde-48fb-bd0e-b8d06b609677@ti.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="uxczuahqgymwfz6l"
Content-Disposition: inline
In-Reply-To: <8dee1738-0bde-48fb-bd0e-b8d06b609677@ti.com>


--uxczuahqgymwfz6l
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On 12.01.2024 20:53:32, Kumar, Udit wrote:
> Hi Marc
>=20
> On 1/2/2024 4:43 PM, Marc Kleine-Budde wrote:
> > On 02.01.2024 15:59:49, Bhavya Kapoor wrote:
> > > When multiple CAN's are present, then names that are getting assigned
> > > changes after every boot even after providing alias in the device tre=
e.
> > > Thus, Add support for implementing CAN aliasing so that names or
> > > alias for CAN will now be provided from device tree.
> > NACK, please use udev or systemd-networkd to provide stable names for
> > CAN interfaces.
>=20
> Would you like to re-consider this NACK.

This is not a CAN device specific problem. If you want to change this,
talk/convince the networking people.

> From kernel side,
>=20
> IMO if aliasing is set in device tree then kernel should provide consiste=
nt
> baseline names.
>=20
> However, distributions may choose different or other stable naming,
>=20
> Also, if some distribution want to rely on kernel naming they still can d=
o.

regards,
Marc

--=20
Pengutronix e.K.                 | Marc Kleine-Budde          |
Embedded Linux                   | https://www.pengutronix.de |
Vertretung N=C3=BCrnberg              | Phone: +49-5121-206917-129 |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-9   |

--uxczuahqgymwfz6l
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEDs2BvajyNKlf9TJQvlAcSiqKBOgFAmWhXhAACgkQvlAcSiqK
BOjD0wgAriZqvA12Vu5nzdKCKAjSH8J55u83wid8H1TFwOTI44OzbAg0a6AakQua
JRGAbYx0pivUuwNf8emVJ1FVJMxyPH1lMPwQep6ZYtTVqoWfhOFDWk2m14xBYTVj
DguY9dlaK/uBU39GBDnk3b8dMd+l6QwSDJhymFTO5P5c87YLzVlpa4FkbSVE2Atz
wQVQZdMzIUJK4FwFaymKkT2SjLsLb5kZPM8t80L7oknZruvsJKGEAm2KF9TJYnV0
fjdBgfzdbdxFrJqJEd7YSOK5aN0mv2/gyb16rpGDYRsrWmBN2sffk4kIC1TmsSwS
Do9pEjkihZ2wEpAShuxai2OdqVMBMQ==
=709B
-----END PGP SIGNATURE-----

--uxczuahqgymwfz6l--