[PATCH] gpio: msc313: remove kcalloc

[Bartosz Golaszewski]

On Mon, Mar 9, 2026 at 11:00 PM Rosen Penev <rosenp at gmail.com> wrote:
>
> On Mon, Mar 9, 2026 at 2:03 AM Bartosz Golaszewski <brgl at kernel.org> wrote:
> >
> > On Mon, Mar 9, 2026 at 2:27 AM Rosen Penev <rosenp at gmail.com> wrote:
> > >
> > > On Sun, Mar 8, 2026 at 5:06 PM Linus Walleij <linusw at kernel.org> wrote:
> > > >
> > > > On Sun, Mar 8, 2026 at 3:15 AM Rosen Penev <rosenp at gmail.com> wrote:
> > > >
> > > > > Use a flexible array member to combine kzalloc and kcalloc.
> > > > >
> > > > > Signed-off-by: Rosen Penev <rosenp at gmail.com>
> > > > (...)
> > > > >  struct msc313_gpio {
> > > > >         void __iomem *base;
> > > > >         const struct msc313_gpio_data *gpio_data;
> > > >
> > > > Do you wanna add:
> > > >
> > > > const unsigned int saved_size;
> > > So in the code there's
> > >
> > > for (i = 0; i < gpio->gpio_data->num; i++)
> > >
> > > which is equivalent to match->num.
> > >
> > > __counted_by doesn't support pointers AFAIK.
> > > >
> >
> > This is a regular flexible array of u8. It should work fine with
> > __counted_by and I too am in favor of using it as it has become
> > standard for flexible arrays.
> That requires adding an extra variable in the struct.

That's ok, it's negligible and provides tangible advantages.

Please do the same for your other patches.

Bart