Hi Laurent, On Wed, Jan 18, 2023 at 7:16 PM Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > > Hi Peter, > > Thank you for the patch. > > On Sun, Jan 08, 2023 at 06:04:01AM +0000, Peter Robinson wrote: > > There's a few components in the rcar-du drm directory that > > don't make sense to be selectable if DRM_RCAR_DU isn't because > > they are part of the IP block so add a dependency and add > > compile check to ensure they're still tested. > > > > Signed-off-by: Peter Robinson <pbrobinson@xxxxxxxxx> > > --- > > > > v2: > > - typo fix in commit message > > - s/ARCH_RENESAS/DRM_RCAR_DU > > > > drivers/gpu/drm/rcar-du/Kconfig | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/drivers/gpu/drm/rcar-du/Kconfig b/drivers/gpu/drm/rcar-du/Kconfig > > index b2bddbeca878..362fb6099e9f 100644 > > --- a/drivers/gpu/drm/rcar-du/Kconfig > > +++ b/drivers/gpu/drm/rcar-du/Kconfig > > @@ -25,6 +25,7 @@ config DRM_RCAR_CMM > > config DRM_RCAR_DW_HDMI > > tristate "R-Car Gen3 and RZ/G2 DU HDMI Encoder Support" > > depends on DRM && OF > > + depends on DRM_RCAR_DU || COMPILE_TEST > > How about dropping COMPILE_TEST ? DRM_RCAR_DU itself has > > depends on ARCH_RENESAS || COMPILE_TEST > > so if COMPILE_TEST is set we'll be able to test these drivers when > DRM_RCAR_DU is enabled. Same below. > > I can update this when taking the patch in my tree, there's no need to > send a v3. Works for me! Thanks > > select DRM_DW_HDMI > > help > > Enable support for R-Car Gen3 or RZ/G2 internal HDMI encoder. > > @@ -32,6 +33,7 @@ config DRM_RCAR_DW_HDMI > > config DRM_RCAR_USE_LVDS > > bool "R-Car DU LVDS Encoder Support" > > depends on DRM_BRIDGE && OF > > + depends on DRM_RCAR_DU || COMPILE_TEST > > default DRM_RCAR_DU > > help > > Enable support for the R-Car Display Unit embedded LVDS encoders. > > @@ -45,6 +47,7 @@ config DRM_RCAR_LVDS > > config DRM_RCAR_USE_MIPI_DSI > > bool "R-Car DU MIPI DSI Encoder Support" > > depends on DRM_BRIDGE && OF > > + depends on DRM_RCAR_DU || COMPILE_TEST > > default DRM_RCAR_DU > > help > > Enable support for the R-Car Display Unit embedded MIPI DSI encoders. > > -- > Regards, > > Laurent Pinchart
