Messages in this thread | | | Date | Fri, 27 Jan 2023 08:40:57 -0600 | Subject | Re: [PATCH] fsi: Add aliased device numbering | From | Eddie James <> |
| |
On 1/26/23 19:03, Jeremy Kerr wrote: > Hi Eddie, > >> The I2C and SPI subsystems can use an aliased name to number the >> device. Add similar support to the FSI subsystem for any device type. > Sounds good! > > However, the majority of this patch seems to be the change to a (const) > enum value, which you then convert to a string to generate the > non-number component of the device name. > > Does: > > dev_set_name(&scom->dev, "%s%d", fsi_get_dev_type_name(fsi_dev_scom), didx); > > have much of a benefit over: > > dev_set_name(&scom->dev, "scom%d", didx); > > ?
Hi Jeremy,
I agree, it's not a benefit but my objective here is to make sure the device name is definitely the same as what the alias lookup is looking for. Perhaps it doesn't matter so much, and I can drop that part of the patch...
Thanks for looking!
Eddie
> > Cheers, > > > Jeremy
| |