On Tue, 15 Aug 2023 09:49:56 +0300 Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > On Sat, Aug 12, 2023 at 07:57:41AM +0100, Biju Das wrote: > > Drop the unnecessary enum chip_ids by splitting the array > > yas5xx_chip_info_tbl[] as individual variables. > > In my opinion this is an unneeded churn. > I leave this to Jonathan to decide. > Agree with Andy. I don't care strongly about doing it one way or another. But as it's already done via an array - better to leave it like that and avoid the churn. Jonathan