Hi Morimoto-san, Thanks for your patch! On Mon, Aug 17, 2020 at 6:51 AM Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> wrote: > From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> > > Some Renesas drivers are not organized, or organized by Chip number. They are sorted by SoC part number? > Because of it, menu table is not readable. > This patch sort these. > > This patch do > - Collect RZ/xx in one place > - Collect SH-Mobile xx in one place > - align chip number on each groups > > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx> So now we have: $ grep 'bool "' drivers/soc/renesas/Kconfig | sed -e 's/ / /g' bool "Renesas SoC driver support" if COMPILE_TEST && !ARCH_RENESAS bool "Emma Mobile EV2" bool "SH-Mobile AG5 (R8A73A00)" bool "R-Mobile APE6 (R8A73A40)" bool "R-Mobile A1 (R8A77400)" bool "RZ/A1H (R7S72100)" bool "RZ/A2 (R7S9210)" bool "RZ/G1H (R8A77420)" bool "RZ/G1M (R8A77430)" bool "RZ/G1N (R8A77440)" bool "RZ/G1E (R8A77450)" bool "RZ/G1C (R8A77470)" bool "RZ/N1D (R9A06G032)" bool "R-Car M1A (R8A77781)" bool "R-Car H1 (R8A77790)" bool "R-Car H2 (R8A77900)" bool "R-Car M2-W (R8A77910)" bool "R-Car V2H (R8A77920)" bool "R-Car M2-N (R8A7793)" bool "R-Car E2 (R8A77940)" What about changing all the above to match the below? I.e. "SoC Platform support for <foo>"? bool "SoC Platform support for RZ/G2M" bool "SoC Platform support for RZ/G2N" bool "SoC Platform support for RZ/G2E" bool "SoC Platform support for RZ/G2H" bool "SoC Platform support for R-Car H3 ES1.x" bool "SoC Platform support for R-Car H3 ES2.0+" bool "SoC Platform support for R-Car M3-W" bool "SoC Platform support for R-Car M3-W+" bool "SoC Platform support for R-Car M3-N" bool "SoC Platform support for R-Car V3M" bool "SoC Platform support for R-Car V3H" bool "SoC Platform support for R-Car E3" bool "SoC Platform support for R-Car D3" bool "System Controller support for RZ/G1H" if COMPILE_TEST bool "System Controller support for RZ/G1M" if COMPILE_TEST bool "System Controller support for RZ/G1E" if COMPILE_TEST bool "System Controller support for RZ/G1C" if COMPILE_TEST bool "System Controller support for RZ/G2M" if COMPILE_TEST bool "System Controller support for RZ/G2N" if COMPILE_TEST bool "System Controller support for RZ/G2E" if COMPILE_TEST bool "System Controller support for RZ/G2H" if COMPILE_TEST bool "System Controller support for R-Car H1" if COMPILE_TEST bool "System Controller support for R-Car H2" if COMPILE_TEST bool "System Controller support for R-Car M2-W/N" if COMPILE_TEST bool "System Controller support for R-Car V2H" if COMPILE_TEST bool "System Controller support for R-Car E2" if COMPILE_TEST bool "System Controller support for R-Car H3" if COMPILE_TEST bool "System Controller support for R-Car M3-W" if COMPILE_TEST bool "System Controller support for R-Car M3-W+" if COMPILE_TEST bool "System Controller support for R-Car M3-N" if COMPILE_TEST bool "System Controller support for R-Car V3M" if COMPILE_TEST bool "System Controller support for R-Car V3H" if COMPILE_TEST bool "System Controller support for R-Car E3" if COMPILE_TEST bool "System Controller support for R-Car D3" if COMPILE_TEST bool "Reset Controller support for R-Car" if COMPILE_TEST bool "System Controller support for R-Car" if COMPILE_TEST bool "System Controller support for R-Mobile" if COMPILE_TEST 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