On Mon, Mar 07, 2016 at 08:53:56AM +0100, Geert Uytterhoeven wrote: > Hi Simon, > > Oops, seems I dropped all CCs in my earlier reply. Fixing up... > > On Mon, Mar 7, 2016 at 2:28 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > > On Thu, Mar 03, 2016 at 09:40:07AM +0100, Geert Uytterhoeven wrote: > >> On Thu, Mar 3, 2016 at 2:52 AM, Simon Horman <horms+renesas@xxxxxxxxxxxx> wrote: > >> > Make use of ARCH_RENESAS in place of ARCH_SHMOBILE. > >> > > >> > This is part of an ongoing process to migrate from ARCH_SHMOBILE to > >> > ARCH_RENESAS the motivation for which being that RENESAS seems to be a more > >> > appropriate name than SHMOBILE for the majority of Renesas ARM based SoCs. > >> > > >> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > >> > >> > --- a/drivers/media/platform/soc_camera/Kconfig > >> > +++ b/drivers/media/platform/soc_camera/Kconfig > >> > @@ -36,7 +36,7 @@ config VIDEO_PXA27x > >> > config VIDEO_RCAR_VIN > >> > tristate "R-Car Video Input (VIN) support" > >> > depends on VIDEO_DEV && SOC_CAMERA > >> > - depends on ARCH_SHMOBILE || COMPILE_TEST > >> > + depends on ARCH_RENESAS || COMPILE_TEST > >> > >> OK > >> > >> >> config VIDEO_SH_MOBILE_CEU > >> > tristate "SuperH Mobile CEU Interface driver" > >> > depends on VIDEO_DEV && SOC_CAMERA && HAS_DMA && HAVE_CLK > >> > - depends on ARCH_SHMOBILE || SUPERH || COMPILE_TEST > >> > + depends on ARCH_RENESAS || SUPERH || COMPILE_TEST > >> > >> I think dropping the SUPERH dependency is the right approach here, as all > >> SuperH platforms using this driver select ARCH_SHMOBILE. > >> > >> "sh_mobile_ceu" is used on SH_AP325RXA, SH_ECOVEC, SH_KFR2R09, SH_MIGOR, > >> and SH_7724_SOLUTION_ENGINE, which depend on either CPU_SUBTYPE_SH7722, > >> CPU_SUBTYPE_SH7723, or > >> CPU_SUBTYPE_SH7724, and all three select ARCH_SHMOBILE. > > > > Dropping SUPERH seems fine to me. But in that case I think the following > > would be best as I would like to stop selecting ARCH_SHMOBILE for > > the ARM SoCs. > > > > depends on ARCH_RENESAS || ARCH_SHMOBILE || COMPILE_TEST > > Do we need this driver with ARCH_RENESAS? It does not support DT. > > R8a7740 has the sh_mobile_ceu hardware, but support for it was dropped with > r8a7740/armadillo legacy removal. Silly me, no it is not needed.
