On Mon, Mar 9, 2026 at 11:00 PM Rosen Penev <[email protected]> wrote: > > On Mon, Mar 9, 2026 at 2:03 AM Bartosz Golaszewski <[email protected]> wrote: > > > > On Mon, Mar 9, 2026 at 2:27 AM Rosen Penev <[email protected]> wrote: > > > > > > On Sun, Mar 8, 2026 at 5:06 PM Linus Walleij <[email protected]> wrote: > > > > > > > > On Sun, Mar 8, 2026 at 3:15 AM Rosen Penev <[email protected]> wrote: > > > > > > > > > Use a flexible array member to combine kzalloc and kcalloc. > > > > > > > > > > Signed-off-by: Rosen Penev <[email protected]> > > > > (...) > > > > > 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
