* [PATCH] staging: media: imx: drop dependency on ipuv3
@ 2020-11-09 9:13 Martin Kepplinger
2020-11-09 9:22 ` Philipp Zabel
2020-12-02 8:24 ` [PATCH] staging: media: imx: drop dependency on ipuv3 Hans Verkuil
0 siblings, 2 replies; 5+ messages in thread
From: Martin Kepplinger @ 2020-11-09 9:13 UTC (permalink / raw)
To: rogerio.silva, slongerbeam, p.zabel, mchehab, shawnguo, festevam
Cc: iain.galloway, kernel, kernel, linux-imx, linux-media,
linux-arm-kernel, linux-kernel, Martin Kepplinger
As described in NXPs' linux tree, the imx8m SoC includes the same
CSI bridge hardware that is part of imx7d. We should be able to
use the "fsl,imx7-csi" driver for imx8m directly.
Since ipuv3 is not relevant for imx8m, drop the build dependency
for it.
Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
---
drivers/staging/media/imx/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig
index f555aac8a9d5..98272fd92fe4 100644
--- a/drivers/staging/media/imx/Kconfig
+++ b/drivers/staging/media/imx/Kconfig
@@ -2,7 +2,7 @@
config VIDEO_IMX_MEDIA
tristate "i.MX5/6 V4L2 media core driver"
depends on ARCH_MXC || COMPILE_TEST
- depends on VIDEO_V4L2 && IMX_IPUV3_CORE
+ depends on VIDEO_V4L2
select MEDIA_CONTROLLER
select VIDEO_V4L2_SUBDEV_API
depends on HAS_DMA
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: media: imx: drop dependency on ipuv3
2020-11-09 9:13 [PATCH] staging: media: imx: drop dependency on ipuv3 Martin Kepplinger
@ 2020-11-09 9:22 ` Philipp Zabel
2020-11-09 9:46 ` [PATCH] staging: media: imx: Split config option in 2 Martin Kepplinger
2020-12-02 8:24 ` [PATCH] staging: media: imx: drop dependency on ipuv3 Hans Verkuil
1 sibling, 1 reply; 5+ messages in thread
From: Philipp Zabel @ 2020-11-09 9:22 UTC (permalink / raw)
To: Martin Kepplinger, rogerio.silva, slongerbeam, mchehab, shawnguo,
festevam
Cc: iain.galloway, kernel, kernel, linux-imx, linux-media,
linux-arm-kernel, linux-kernel
Hi Martin,
On Mon, 2020-11-09 at 10:13 +0100, Martin Kepplinger wrote:
> As described in NXPs' linux tree, the imx8m SoC includes the same
> CSI bridge hardware that is part of imx7d. We should be able to
> use the "fsl,imx7-csi" driver for imx8m directly.
>
> Since ipuv3 is not relevant for imx8m, drop the build dependency
> for it.
>
> Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
> ---
> drivers/staging/media/imx/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig
> index f555aac8a9d5..98272fd92fe4 100644
> --- a/drivers/staging/media/imx/Kconfig
> +++ b/drivers/staging/media/imx/Kconfig
> @@ -2,7 +2,7 @@
> config VIDEO_IMX_MEDIA
> tristate "i.MX5/6 V4L2 media core driver"
VIDEO_IMX_MEDIA builds imx6-media, which does depend on IMX_IPUV3_CORE.
You only want imx-media-common. I think we have to split the
configuration option in two.
> depends on ARCH_MXC || COMPILE_TEST
> - depends on VIDEO_V4L2 && IMX_IPUV3_CORE
> + depends on VIDEO_V4L2
> select MEDIA_CONTROLLER
> select VIDEO_V4L2_SUBDEV_API
> depends on HAS_DMA
regards
Philipp
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] staging: media: imx: Split config option in 2
2020-11-09 9:22 ` Philipp Zabel
@ 2020-11-09 9:46 ` Martin Kepplinger
2020-11-09 11:27 ` Philipp Zabel
0 siblings, 1 reply; 5+ messages in thread
From: Martin Kepplinger @ 2020-11-09 9:46 UTC (permalink / raw)
To: rogerio.silva, slongerbeam, p.zabel, mchehab, shawnguo, festevam
Cc: iain.galloway, kernel, kernel, linux-imx, linux-media,
linux-arm-kernel, linux-kernel, Martin Kepplinger
As described in NXPs' linux tree, the imx8m SoC includes the same
CSI bridge hardware that is part of imx7d. We should be able to
use the "fsl,imx7-csi" driver for imx8m directly.
Since ipuv3 is not relevant for imx8m we create VIDEO_IMX7_MEDIA and
split up the configuration option in 2 menus (on 1 entry each
for now but that can be changed later).
Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
---
thanks, you're right. did you have something like this in mind?
martin
drivers/staging/media/imx/Kconfig | 27 ++++++++++++++++++++++++---
1 file changed, 24 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig
index f555aac8a9d5..a888d9b918b5 100644
--- a/drivers/staging/media/imx/Kconfig
+++ b/drivers/staging/media/imx/Kconfig
@@ -13,6 +13,20 @@ config VIDEO_IMX_MEDIA
Say yes here to enable support for video4linux media controller
driver for the i.MX5/6 SOC.
+config VIDEO_IMX7_MEDIA
+ tristate "i.MX7/8 V4L2 media core driver"
+ depends on ARCH_MXC || COMPILE_TEST
+ depends on VIDEO_V4L2
+ select MEDIA_CONTROLLER
+ select VIDEO_V4L2_SUBDEV_API
+ depends on HAS_DMA
+ select VIDEOBUF2_DMA_CONTIG
+ select V4L2_FWNODE
+ select V4L2_MEM2MEM_DEV
+ help
+ Say yes here to enable support for video4linux media controller
+ driver for the i.MX7/8M SOC.
+
if VIDEO_IMX_MEDIA
menu "i.MX5/6/7 Media Sub devices"
@@ -23,12 +37,19 @@ config VIDEO_IMX_CSI
help
A video4linux camera sensor interface driver for i.MX5/6.
+endmenu
+endif
+
+if VIDEO_IMX7_MEDIA
+menu "i.MX7/8 Media Sub devices"
+
config VIDEO_IMX7_CSI
- tristate "i.MX6UL/L / i.MX7 Camera Sensor Interface driver"
- depends on VIDEO_IMX_MEDIA && VIDEO_DEV && I2C
+ tristate "i.MX7 / i.MX8M Camera Sensor Interface driver"
+ depends on VIDEO_IMX7_MEDIA && VIDEO_DEV && I2C
default y
help
Enable support for video4linux camera sensor interface driver for
- i.MX6UL/L or i.MX7.
+ i.MX6UL/L, i.MX7 or i.MX8M.
+
endmenu
endif
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: media: imx: Split config option in 2
2020-11-09 9:46 ` [PATCH] staging: media: imx: Split config option in 2 Martin Kepplinger
@ 2020-11-09 11:27 ` Philipp Zabel
0 siblings, 0 replies; 5+ messages in thread
From: Philipp Zabel @ 2020-11-09 11:27 UTC (permalink / raw)
To: Martin Kepplinger, rogerio.silva, slongerbeam, mchehab, shawnguo,
festevam
Cc: iain.galloway, kernel, kernel, linux-imx, linux-media,
linux-arm-kernel, linux-kernel
On Mon, 2020-11-09 at 10:46 +0100, Martin Kepplinger wrote:
> As described in NXPs' linux tree, the imx8m SoC includes the same
> CSI bridge hardware that is part of imx7d. We should be able to
> use the "fsl,imx7-csi" driver for imx8m directly.
>
> Since ipuv3 is not relevant for imx8m we create VIDEO_IMX7_MEDIA and
> split up the configuration option in 2 menus (on 1 entry each
> for now but that can be changed later).
>
> Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
> ---
>
> thanks, you're right. did you have something like this in mind?
Not quite, we need a separate option for the imx-media-common module, so
the Makefile has to be changed as well. That option should be selected
by VIDEO_IMX_MEDIA.
I'm not sure if introducing VIDEO_IMX7_MEDIA is necessary, the new
option could also be hidden if selected VIDEO_IMX7_CSI directly.
regards
Philipp
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: media: imx: drop dependency on ipuv3
2020-11-09 9:13 [PATCH] staging: media: imx: drop dependency on ipuv3 Martin Kepplinger
2020-11-09 9:22 ` Philipp Zabel
@ 2020-12-02 8:24 ` Hans Verkuil
1 sibling, 0 replies; 5+ messages in thread
From: Hans Verkuil @ 2020-12-02 8:24 UTC (permalink / raw)
To: Martin Kepplinger, rogerio.silva, slongerbeam, p.zabel, mchehab,
shawnguo, festevam
Cc: iain.galloway, kernel, kernel, linux-imx, linux-media,
linux-arm-kernel, linux-kernel
On 09/11/2020 10:13, Martin Kepplinger wrote:
> As described in NXPs' linux tree, the imx8m SoC includes the same
> CSI bridge hardware that is part of imx7d. We should be able to
> use the "fsl,imx7-csi" driver for imx8m directly.
>
> Since ipuv3 is not relevant for imx8m, drop the build dependency
> for it.
>
> Signed-off-by: Martin Kepplinger <martin.kepplinger@puri.sm>
> ---
> drivers/staging/media/imx/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/imx/Kconfig b/drivers/staging/media/imx/Kconfig
> index f555aac8a9d5..98272fd92fe4 100644
> --- a/drivers/staging/media/imx/Kconfig
> +++ b/drivers/staging/media/imx/Kconfig
> @@ -2,7 +2,7 @@
> config VIDEO_IMX_MEDIA
> tristate "i.MX5/6 V4L2 media core driver"
Isn't this text rather out of date if imx7 & 8 are also supported?
Something to fix for the next version...
Regards,
Hans
> depends on ARCH_MXC || COMPILE_TEST
> - depends on VIDEO_V4L2 && IMX_IPUV3_CORE
> + depends on VIDEO_V4L2
> select MEDIA_CONTROLLER
> select VIDEO_V4L2_SUBDEV_API
> depends on HAS_DMA
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-12-02 8:25 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-09 9:13 [PATCH] staging: media: imx: drop dependency on ipuv3 Martin Kepplinger
2020-11-09 9:22 ` Philipp Zabel
2020-11-09 9:46 ` [PATCH] staging: media: imx: Split config option in 2 Martin Kepplinger
2020-11-09 11:27 ` Philipp Zabel
2020-12-02 8:24 ` [PATCH] staging: media: imx: drop dependency on ipuv3 Hans Verkuil
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).