Re: [PATCH v7 26/30] net: wan: framer: Add support for the Lantiq PEF2256 framer

From: Herve Codina
Date: Tue Oct 10 2023 - 04:30:20 EST


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é