Re: [PATCH 2/3] v4l: rcar_fdp1: Enable compilation on Gen2 platforms
Hi Geert, On Monday, 23 April 2018 10:58:13 EEST Geert Uytterhoeven wrote: > On Sun, Apr 22, 2018 at 12:28 PM, Laurent Pinchart wrote: > > Commit 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") fixed > > a compilation breakage when the optional VIDEO_RENESAS_FCP dependency is > > compiled as a module while the rcar_fdp1 driver is built in. As a side > > effect it disabled compilation on Gen2 by disallowing the valid > > combination ARCH_RENESAS && !VIDEO_RENESAS_FCP. Fix it by handling the > > dependency the same way the vsp1 driver did in commit 199946731fa4 > > ("[media] vsp1: clarify FCP dependency"). > > > > Fixes: 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") > > Signed-off-by: Laurent Pinchart > >> > --- > > > > drivers/media/platform/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/media/platform/Kconfig > > b/drivers/media/platform/Kconfig index 621d63b2001d..81c3ab95c050 100644 > > --- a/drivers/media/platform/Kconfig > > +++ b/drivers/media/platform/Kconfig > > @@ -397,7 +397,7 @@ config VIDEO_RENESAS_FDP1 > > tristate "Renesas Fine Display Processor" > > depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA > > depends on ARCH_RENESAS || COMPILE_TEST > > - depends on (!ARCH_RENESAS && !VIDEO_RENESAS_FCP) || > > VIDEO_RENESAS_FCP > > + depends on (!ARM64 && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP > > s/!ARM64/ARCH_RCAR_GEN2/? That would work too, but is it any better ? It would only matter is COMPILE_TEST is set, and thus won't really make a difference. > > select VIDEOBUF2_DMA_CONTIG > > select V4L2_MEM2MEM_DEV -- Regards, Laurent Pinchart
Re: [PATCH 2/3] v4l: rcar_fdp1: Enable compilation on Gen2 platforms
Hi Laurent, On Sun, Apr 22, 2018 at 12:28 PM, Laurent Pinchartwrote: > Commit 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") fixed > a compilation breakage when the optional VIDEO_RENESAS_FCP dependency is > compiled as a module while the rcar_fdp1 driver is built in. As a side > effect it disabled compilation on Gen2 by disallowing the valid > combination ARCH_RENESAS && !VIDEO_RENESAS_FCP. Fix it by handling the > dependency the same way the vsp1 driver did in commit 199946731fa4 > ("[media] vsp1: clarify FCP dependency"). > > Fixes: 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") > Signed-off-by: Laurent Pinchart > --- > drivers/media/platform/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig > index 621d63b2001d..81c3ab95c050 100644 > --- a/drivers/media/platform/Kconfig > +++ b/drivers/media/platform/Kconfig > @@ -397,7 +397,7 @@ config VIDEO_RENESAS_FDP1 > tristate "Renesas Fine Display Processor" > depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA > depends on ARCH_RENESAS || COMPILE_TEST > - depends on (!ARCH_RENESAS && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP > + depends on (!ARM64 && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP s/!ARM64/ARCH_RCAR_GEN2/? > select VIDEOBUF2_DMA_CONTIG > select V4L2_MEM2MEM_DEV Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org 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
[PATCH 2/3] v4l: rcar_fdp1: Enable compilation on Gen2 platforms
Commit 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") fixed a compilation breakage when the optional VIDEO_RENESAS_FCP dependency is compiled as a module while the rcar_fdp1 driver is built in. As a side effect it disabled compilation on Gen2 by disallowing the valid combination ARCH_RENESAS && !VIDEO_RENESAS_FCP. Fix it by handling the dependency the same way the vsp1 driver did in commit 199946731fa4 ("[media] vsp1: clarify FCP dependency"). Fixes: 1d3897143815 ("[media] v4l: rcar_fdp1: add FCP dependency") Signed-off-by: Laurent Pinchart--- drivers/media/platform/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig index 621d63b2001d..81c3ab95c050 100644 --- a/drivers/media/platform/Kconfig +++ b/drivers/media/platform/Kconfig @@ -397,7 +397,7 @@ config VIDEO_RENESAS_FDP1 tristate "Renesas Fine Display Processor" depends on VIDEO_DEV && VIDEO_V4L2 && HAS_DMA depends on ARCH_RENESAS || COMPILE_TEST - depends on (!ARCH_RENESAS && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP + depends on (!ARM64 && !VIDEO_RENESAS_FCP) || VIDEO_RENESAS_FCP select VIDEOBUF2_DMA_CONTIG select V4L2_MEM2MEM_DEV ---help--- -- Regards, Laurent Pinchart