diff mbox series

media: nxp: Restrict VIDEO_IMX_MIPI_CSIS to ARCH_MXC or COMPILE_TEST

Message ID 20220318203735.5923-1-laurent.pinchart+renesas@ideasonboard.com (mailing list archive)
State New, archived
Headers show
Series media: nxp: Restrict VIDEO_IMX_MIPI_CSIS to ARCH_MXC or COMPILE_TEST | expand

Commit Message

Laurent Pinchart March 18, 2022, 8:37 p.m. UTC 
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

The imx-mipi-csis driver is specific to NXP platforms. Restrict it to
those by default, and enable compilation with COMPILE_TEST to keep a
wide test coverage.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 drivers/media/platform/nxp/Kconfig | 1 +
 1 file changed, 1 insertion(+)


base-commit: 71e6d0608e4d1b79069990c7dacb3600ced28a3b

Comments

Rui Miguel Silva March 18, 2022, 9:44 p.m. UTC | #1 
Hi Laurent,
On Fri, Mar 18, 2022 at 10:37:35PM +0200, Laurent Pinchart wrote:
> From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> The imx-mipi-csis driver is specific to NXP platforms. Restrict it to
> those by default, and enable compilation with COMPILE_TEST to keep a
> wide test coverage.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thanks, LGTM.

Reviewed-by: Rui Miguel Silva <rmfrfs@gmail.com>

Cheers,
     Rui
> ---
>  drivers/media/platform/nxp/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/platform/nxp/Kconfig b/drivers/media/platform/nxp/Kconfig
> index 7473096f5885..28f2bafc14d2 100644
> --- a/drivers/media/platform/nxp/Kconfig
> +++ b/drivers/media/platform/nxp/Kconfig
> @@ -6,6 +6,7 @@ comment "NXP media platform drivers"
>  
>  config VIDEO_IMX_MIPI_CSIS
>  	tristate "NXP MIPI CSI-2 CSIS receiver found on i.MX7 and i.MX8 models"
> +	depends on ARCH_MXC || COMPILE_TEST
>  	select MEDIA_CONTROLLER
>  	select V4L2_FWNODE
>  	select VIDEO_V4L2_SUBDEV_API
> 
> base-commit: 71e6d0608e4d1b79069990c7dacb3600ced28a3b
> -- 
> Regards,
> 
> Laurent Pinchart
>
Kieran Bingham March 21, 2022, 2:03 p.m. UTC | #2 
Quoting Laurent Pinchart (2022-03-18 20:37:35)
> From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> The imx-mipi-csis driver is specific to NXP platforms. Restrict it to
> those by default, and enable compilation with COMPILE_TEST to keep a
> wide test coverage.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>

> ---
>  drivers/media/platform/nxp/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/platform/nxp/Kconfig b/drivers/media/platform/nxp/Kconfig
> index 7473096f5885..28f2bafc14d2 100644
> --- a/drivers/media/platform/nxp/Kconfig
> +++ b/drivers/media/platform/nxp/Kconfig
> @@ -6,6 +6,7 @@ comment "NXP media platform drivers"
>  
>  config VIDEO_IMX_MIPI_CSIS
>         tristate "NXP MIPI CSI-2 CSIS receiver found on i.MX7 and i.MX8 models"
> +       depends on ARCH_MXC || COMPILE_TEST
>         select MEDIA_CONTROLLER
>         select V4L2_FWNODE
>         select VIDEO_V4L2_SUBDEV_API
> 
> base-commit: 71e6d0608e4d1b79069990c7dacb3600ced28a3b
> -- 
> Regards,
> 
> Laurent Pinchart
>
Paul Elder March 23, 2022, 9:29 a.m. UTC | #3 
Hi Laurent,

On Fri, Mar 18, 2022 at 10:37:35PM +0200, Laurent Pinchart wrote:
> From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
> The imx-mipi-csis driver is specific to NXP platforms. Restrict it to
> those by default, and enable compilation with COMPILE_TEST to keep a
> wide test coverage.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>

> ---
>  drivers/media/platform/nxp/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/platform/nxp/Kconfig b/drivers/media/platform/nxp/Kconfig
> index 7473096f5885..28f2bafc14d2 100644
> --- a/drivers/media/platform/nxp/Kconfig
> +++ b/drivers/media/platform/nxp/Kconfig
> @@ -6,6 +6,7 @@ comment "NXP media platform drivers"
>  
>  config VIDEO_IMX_MIPI_CSIS
>  	tristate "NXP MIPI CSI-2 CSIS receiver found on i.MX7 and i.MX8 models"
> +	depends on ARCH_MXC || COMPILE_TEST
>  	select MEDIA_CONTROLLER
>  	select V4L2_FWNODE
>  	select VIDEO_V4L2_SUBDEV_API
> 
> base-commit: 71e6d0608e4d1b79069990c7dacb3600ced28a3b
> -- 
> Regards,
> 
> Laurent Pinchart
>
diff mbox series

Patch

diff --git a/drivers/media/platform/nxp/Kconfig b/drivers/media/platform/nxp/Kconfig
index 7473096f5885..28f2bafc14d2 100644
--- a/drivers/media/platform/nxp/Kconfig
+++ b/drivers/media/platform/nxp/Kconfig
@@ -6,6 +6,7 @@  comment "NXP media platform drivers"
 
 config VIDEO_IMX_MIPI_CSIS
 	tristate "NXP MIPI CSI-2 CSIS receiver found on i.MX7 and i.MX8 models"
+	depends on ARCH_MXC || COMPILE_TEST
 	select MEDIA_CONTROLLER
 	select V4L2_FWNODE
 	select VIDEO_V4L2_SUBDEV_API