Hi Jakub, On Fri, 6 Oct 2023 15:02:52 -0700 Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > On Thu, 28 Sep 2023 09:06:44 +0200 Herve Codina wrote: > > + for (i = 0; i < count; i++) { > > + (audio_devs + i)->name = "framer-codec"; > > + (audio_devs + i)->of_compatible = compatible; > > + (audio_devs + i)->id = i; > > Why not array notation? Will be change in the next iteration. > > > + } > > + > > + ret = mfd_add_devices(pef2256->dev, 0, audio_devs, count, NULL, 0, NULL); > > Should Lee be CCed for the MFD part? Will be added to the CC list. Best regards, Hervé