Hi Sato-san, On Tue, Jan 9, 2024 at 9:23 AM Yoshinori Sato <ysato@xxxxxxxxxxxxxxxxxxxx> wrote: > Targets that support OF should be treated as one board. > > Signed-off-by: Yoshinori Sato <ysato@xxxxxxxxxxxxxxxxxxxx> Thanks for your patch! > --- a/arch/sh/Kconfig > +++ b/arch/sh/Kconfig > @@ -710,6 +710,7 @@ choice > prompt "Kernel command line" > optional > default CMDLINE_OVERWRITE > + depends on !OF || USE_BUILTIN_DTB This is still useful in the generic OF case. I think it would be good to model this similar to what arm/arm64/riscv are using (from bootloader / extend / force). > help > Setting this option allows the kernel command line arguments > to be set. > diff --git a/arch/sh/boards/Kconfig b/arch/sh/boards/Kconfig > index 109bec4dad94..e7e52779ef62 100644 > --- a/arch/sh/boards/Kconfig > +++ b/arch/sh/boards/Kconfig > @@ -19,16 +19,9 @@ config SH_DEVICE_TREE > select TIMER_OF > select COMMON_CLK > select GENERIC_CALIBRATE_DELAY > - > -config SH_JCORE_SOC > - bool "J-Core SoC" > - select SH_DEVICE_TREE > - select CLKSRC_JCORE_PIT > - select JCORE_AIC > - depends on CPU_J2 > - help > - Select this option to include drivers core components of the > - J-Core SoC, including interrupt controllers and timers. > + select GENERIC_IRQ_CHIP > + select SYS_SUPPORTS_PCI > + select GENERIC_PCI_IOMAP if PCI > > config SH_SOLUTION_ENGINE > bool "SolutionEngine" > @@ -293,6 +286,7 @@ config SH_LANDISK > bool "LANDISK" > depends on CPU_SUBTYPE_SH7751R > select HAVE_PCI > + select SYS_SUPPORTS_PCI > help > I-O DATA DEVICE, INC. "LANDISK Series" support. > > @@ -369,6 +363,16 @@ config SH_APSH4AD0A > help > Select AP-SH4AD-0A if configuring for an ALPHAPROJECT AP-SH4AD-0A. > > +config SH_OF_BOARD > + bool "General Open Firmware boards" > + select SH_DEVICE_TREE > + select CLKSRC_JCORE_PIT if CPU_J2 > + select JCORE_AIC if CPU_J2 Please move these selects to CPU_J2 instead... > + select HAVE_PCI if CPU_SUBTYPE_SH7751R ... and this to CPU_SUBTYPE_SH7751R, else it will become a long unmaintainable list soon... > + help > + This board means general OF supported targets. > + > + 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