Re: [PATCH] fsi: Add aliased device numbering

[Eddie James <eajames@linux.ibm.com>]

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