Hi Prabhakar, On Thu, May 5, 2022 at 9:32 PM Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > Instead of indexing the resets, stack them and instead create an id member > in struct rzg2l_reset to store the index. With this approach for every id > we will have to loop through the resets array to match the id. > > This in preparation to add support for Renesas RZ/Five CPG in > r9a07g043-cpg.c file where the resets array will be split up into three > i.e. common and two SoC specific. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> Thanks for your patch! An obvious alternative would be to allocate an array with pointers to the individual resets, like is done for clocks. Please see the suggestion in my reply to "[RFC PATCH 3/4] clk: renesas: r9a07g043: Split up core, module and resets array", which would make this patch unnecessary. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds