* [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC
@ 2022-06-24 12:10 ` Geert Uytterhoeven
0 siblings, 0 replies; 18+ messages in thread
From: Geert Uytterhoeven @ 2022-06-24 12:10 UTC (permalink / raw)
To: Liu Ying, Andrzej Hajda, Neil Armstrong, Robert Foss,
David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer,
Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Sakari Ailus, Marek Vasut
Cc: dri-devel, linux-arm-kernel, linux-kernel, Geert Uytterhoeven
The various Freescale i.MX8 display bridges are only present on
Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent
asking the user about these drivers when configuring a kernel without
i.MX SoC support.
Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm")
Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp")
Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support")
Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support")
Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Liu Ying <victor.liu@nxp.com>
---
v2:
- s/i.MX8MP/i.MX8/,
- Add Reviewed-by.
---
drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig
index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644
--- a/drivers/gpu/drm/bridge/imx/Kconfig
+++ b/drivers/gpu/drm/bridge/imx/Kconfig
@@ -1,3 +1,5 @@
+if ARCH_MXC || COMPILE_TEST
+
config DRM_IMX8QM_LDB
tristate "Freescale i.MX8QM LVDS display bridge"
depends on OF
@@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI
help
Choose this to enable pixel link to display pixel interface(PXL2DPI)
found in Freescale i.MX8qxp processor.
+
+endif # ARCH_MXC || COMPILE_TEST
--
2.25.1
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-24 12:10 ` Geert Uytterhoeven 0 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2022-06-24 12:10 UTC (permalink / raw) To: Liu Ying, Andrzej Hajda, Neil Armstrong, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel, Geert Uytterhoeven The various Freescale i.MX8 display bridges are only present on Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent asking the user about these drivers when configuring a kernel without i.MX SoC support. Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm") Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp") Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support") Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support") Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support") Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Liu Ying <victor.liu@nxp.com> --- v2: - s/i.MX8MP/i.MX8/, - Add Reviewed-by. --- drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 --- a/drivers/gpu/drm/bridge/imx/Kconfig +++ b/drivers/gpu/drm/bridge/imx/Kconfig @@ -1,3 +1,5 @@ +if ARCH_MXC || COMPILE_TEST + config DRM_IMX8QM_LDB tristate "Freescale i.MX8QM LVDS display bridge" depends on OF @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI help Choose this to enable pixel link to display pixel interface(PXL2DPI) found in Freescale i.MX8qxp processor. + +endif # ARCH_MXC || COMPILE_TEST -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-24 12:10 ` Geert Uytterhoeven 0 siblings, 0 replies; 18+ messages in thread From: Geert Uytterhoeven @ 2022-06-24 12:10 UTC (permalink / raw) To: Liu Ying, Andrzej Hajda, Neil Armstrong, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: Geert Uytterhoeven, linux-arm-kernel, dri-devel, linux-kernel The various Freescale i.MX8 display bridges are only present on Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent asking the user about these drivers when configuring a kernel without i.MX SoC support. Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm") Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp") Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support") Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support") Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support") Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Liu Ying <victor.liu@nxp.com> --- v2: - s/i.MX8MP/i.MX8/, - Add Reviewed-by. --- drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 --- a/drivers/gpu/drm/bridge/imx/Kconfig +++ b/drivers/gpu/drm/bridge/imx/Kconfig @@ -1,3 +1,5 @@ +if ARCH_MXC || COMPILE_TEST + config DRM_IMX8QM_LDB tristate "Freescale i.MX8QM LVDS display bridge" depends on OF @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI help Choose this to enable pixel link to display pixel interface(PXL2DPI) found in Freescale i.MX8qxp processor. + +endif # ARCH_MXC || COMPILE_TEST -- 2.25.1 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC 2022-06-24 12:10 ` Geert Uytterhoeven (?) @ 2022-06-27 12:22 ` Neil Armstrong -1 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-27 12:22 UTC (permalink / raw) To: Geert Uytterhoeven, Liu Ying, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel Hi, On 24/06/2022 14:10, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm") > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp") > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support") > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support") > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > Reviewed-by: Liu Ying <victor.liu@nxp.com> > --- > v2: > - s/i.MX8MP/i.MX8/, > - Add Reviewed-by. > --- > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > --- a/drivers/gpu/drm/bridge/imx/Kconfig > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > @@ -1,3 +1,5 @@ > +if ARCH_MXC || COMPILE_TEST > + > config DRM_IMX8QM_LDB > tristate "Freescale i.MX8QM LVDS display bridge" > depends on OF > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > help > Choose this to enable pixel link to display pixel interface(PXL2DPI) > found in Freescale i.MX8qxp processor. > + > +endif # ARCH_MXC || COMPILE_TEST I was wondering why those were added in drivers/gpu/drm/bridge/imx since they are specific to NXP SoCs, I think they should be moved in the right drm imx subsystem instead of this change. Neil ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-27 12:22 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-27 12:22 UTC (permalink / raw) To: Geert Uytterhoeven, Liu Ying, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel Hi, On 24/06/2022 14:10, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm") > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp") > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support") > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support") > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > Reviewed-by: Liu Ying <victor.liu@nxp.com> > --- > v2: > - s/i.MX8MP/i.MX8/, > - Add Reviewed-by. > --- > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > --- a/drivers/gpu/drm/bridge/imx/Kconfig > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > @@ -1,3 +1,5 @@ > +if ARCH_MXC || COMPILE_TEST > + > config DRM_IMX8QM_LDB > tristate "Freescale i.MX8QM LVDS display bridge" > depends on OF > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > help > Choose this to enable pixel link to display pixel interface(PXL2DPI) > found in Freescale i.MX8qxp processor. > + > +endif # ARCH_MXC || COMPILE_TEST I was wondering why those were added in drivers/gpu/drm/bridge/imx since they are specific to NXP SoCs, I think they should be moved in the right drm imx subsystem instead of this change. Neil _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-27 12:22 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-27 12:22 UTC (permalink / raw) To: Geert Uytterhoeven, Liu Ying, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: linux-arm-kernel, dri-devel, linux-kernel Hi, On 24/06/2022 14:10, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for i.MX8qm") > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for i.MX8qxp") > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link to DPI support") > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display pixel link support") > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel combiner support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > Reviewed-by: Liu Ying <victor.liu@nxp.com> > --- > v2: > - s/i.MX8MP/i.MX8/, > - Add Reviewed-by. > --- > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > --- a/drivers/gpu/drm/bridge/imx/Kconfig > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > @@ -1,3 +1,5 @@ > +if ARCH_MXC || COMPILE_TEST > + > config DRM_IMX8QM_LDB > tristate "Freescale i.MX8QM LVDS display bridge" > depends on OF > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > help > Choose this to enable pixel link to display pixel interface(PXL2DPI) > found in Freescale i.MX8qxp processor. > + > +endif # ARCH_MXC || COMPILE_TEST I was wondering why those were added in drivers/gpu/drm/bridge/imx since they are specific to NXP SoCs, I think they should be moved in the right drm imx subsystem instead of this change. Neil ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC 2022-06-27 12:22 ` Neil Armstrong (?) @ 2022-06-28 2:31 ` Liu Ying -1 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-06-28 2:31 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: linux-arm-kernel, dri-devel, linux-kernel On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > Hi, Hi, > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > The various Freescale i.MX8 display bridges are only present on > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > prevent > > asking the user about these drivers when configuring a kernel > > without > > i.MX SoC support. > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > i.MX8qm") > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > i.MX8qxp") > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link > > to DPI support") > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display > > pixel link support") > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel > > combiner support") > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > --- > > v2: > > - s/i.MX8MP/i.MX8/, > > - Add Reviewed-by. > > --- > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > b/drivers/gpu/drm/bridge/imx/Kconfig > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > @@ -1,3 +1,5 @@ > > +if ARCH_MXC || COMPILE_TEST > > + > > config DRM_IMX8QM_LDB > > tristate "Freescale i.MX8QM LVDS display bridge" > > depends on OF > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > help > > Choose this to enable pixel link to display pixel > > interface(PXL2DPI) > > found in Freescale i.MX8qxp processor. > > + > > +endif # ARCH_MXC || COMPILE_TEST > > I was wondering why those were added in drivers/gpu/drm/bridge/imx > since they are specific to NXP SoCs, > I think they should be moved in the right drm imx subsystem instead > of this change. There are 2 directories which contain display controller drivers for i.MX SoCs: a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and i.MX8qm/qxp DPU([1], not landed yet) b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp LCDIFv3([2], not landed yet) Bridges added in drivers/gpu/drm/bridge/imx make it possible to share bridge drivers across display controllers. I see chance to use the LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. [1] https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ [2] https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ Regards, Liu Ying ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-28 2:31 ` Liu Ying 0 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-06-28 2:31 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > Hi, Hi, > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > The various Freescale i.MX8 display bridges are only present on > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > prevent > > asking the user about these drivers when configuring a kernel > > without > > i.MX SoC support. > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > i.MX8qm") > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > i.MX8qxp") > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link > > to DPI support") > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display > > pixel link support") > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel > > combiner support") > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > --- > > v2: > > - s/i.MX8MP/i.MX8/, > > - Add Reviewed-by. > > --- > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > b/drivers/gpu/drm/bridge/imx/Kconfig > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > @@ -1,3 +1,5 @@ > > +if ARCH_MXC || COMPILE_TEST > > + > > config DRM_IMX8QM_LDB > > tristate "Freescale i.MX8QM LVDS display bridge" > > depends on OF > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > help > > Choose this to enable pixel link to display pixel > > interface(PXL2DPI) > > found in Freescale i.MX8qxp processor. > > + > > +endif # ARCH_MXC || COMPILE_TEST > > I was wondering why those were added in drivers/gpu/drm/bridge/imx > since they are specific to NXP SoCs, > I think they should be moved in the right drm imx subsystem instead > of this change. There are 2 directories which contain display controller drivers for i.MX SoCs: a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and i.MX8qm/qxp DPU([1], not landed yet) b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp LCDIFv3([2], not landed yet) Bridges added in drivers/gpu/drm/bridge/imx make it possible to share bridge drivers across display controllers. I see chance to use the LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. [1] https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ [2] https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ Regards, Liu Ying _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-28 2:31 ` Liu Ying 0 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-06-28 2:31 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > Hi, Hi, > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > The various Freescale i.MX8 display bridges are only present on > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > prevent > > asking the user about these drivers when configuring a kernel > > without > > i.MX SoC support. > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > i.MX8qm") > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > i.MX8qxp") > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link > > to DPI support") > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display > > pixel link support") > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel > > combiner support") > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > --- > > v2: > > - s/i.MX8MP/i.MX8/, > > - Add Reviewed-by. > > --- > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > b/drivers/gpu/drm/bridge/imx/Kconfig > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > @@ -1,3 +1,5 @@ > > +if ARCH_MXC || COMPILE_TEST > > + > > config DRM_IMX8QM_LDB > > tristate "Freescale i.MX8QM LVDS display bridge" > > depends on OF > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > help > > Choose this to enable pixel link to display pixel > > interface(PXL2DPI) > > found in Freescale i.MX8qxp processor. > > + > > +endif # ARCH_MXC || COMPILE_TEST > > I was wondering why those were added in drivers/gpu/drm/bridge/imx > since they are specific to NXP SoCs, > I think they should be moved in the right drm imx subsystem instead > of this change. There are 2 directories which contain display controller drivers for i.MX SoCs: a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and i.MX8qm/qxp DPU([1], not landed yet) b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp LCDIFv3([2], not landed yet) Bridges added in drivers/gpu/drm/bridge/imx make it possible to share bridge drivers across display controllers. I see chance to use the LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. [1] https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ [2] https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ Regards, Liu Ying ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC 2022-06-28 2:31 ` Liu Ying (?) @ 2022-06-28 7:10 ` Neil Armstrong -1 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-28 7:10 UTC (permalink / raw) To: Liu Ying, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel On 28/06/2022 04:31, Liu Ying wrote: > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: >> Hi, > > Hi, > >> >> On 24/06/2022 14:10, Geert Uytterhoeven wrote: >>> The various Freescale i.MX8 display bridges are only present on >>> Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to >>> prevent >>> asking the user about these drivers when configuring a kernel >>> without >>> i.MX SoC support. >>> >>> Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for >>> i.MX8qm") >>> Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for >>> i.MX8qxp") >>> Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link >>> to DPI support") >>> Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display >>> pixel link support") >>> Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel >>> combiner support") >>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> >>> Reviewed-by: Liu Ying <victor.liu@nxp.com> >>> --- >>> v2: >>> - s/i.MX8MP/i.MX8/, >>> - Add Reviewed-by. >>> --- >>> drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/bridge/imx/Kconfig >>> b/drivers/gpu/drm/bridge/imx/Kconfig >>> index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 >>> --- a/drivers/gpu/drm/bridge/imx/Kconfig >>> +++ b/drivers/gpu/drm/bridge/imx/Kconfig >>> @@ -1,3 +1,5 @@ >>> +if ARCH_MXC || COMPILE_TEST >>> + >>> config DRM_IMX8QM_LDB >>> tristate "Freescale i.MX8QM LVDS display bridge" >>> depends on OF >>> @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI >>> help >>> Choose this to enable pixel link to display pixel >>> interface(PXL2DPI) >>> found in Freescale i.MX8qxp processor. >>> + >>> +endif # ARCH_MXC || COMPILE_TEST >> >> I was wondering why those were added in drivers/gpu/drm/bridge/imx >> since they are specific to NXP SoCs, >> I think they should be moved in the right drm imx subsystem instead >> of this change. > > There are 2 directories which contain display controller drivers for > i.MX SoCs: > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > i.MX8qm/qxp DPU([1], not landed yet) > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > LCDIFv3([2], not landed yet) > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to share > bridge drivers across display controllers. I see chance to use the > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. Thanks for the explanation, LGTM Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> > > [1] > https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ > [2] > https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ > > Regards, > Liu Ying > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-28 7:10 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-28 7:10 UTC (permalink / raw) To: Liu Ying, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel On 28/06/2022 04:31, Liu Ying wrote: > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: >> Hi, > > Hi, > >> >> On 24/06/2022 14:10, Geert Uytterhoeven wrote: >>> The various Freescale i.MX8 display bridges are only present on >>> Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to >>> prevent >>> asking the user about these drivers when configuring a kernel >>> without >>> i.MX SoC support. >>> >>> Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for >>> i.MX8qm") >>> Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for >>> i.MX8qxp") >>> Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link >>> to DPI support") >>> Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display >>> pixel link support") >>> Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel >>> combiner support") >>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> >>> Reviewed-by: Liu Ying <victor.liu@nxp.com> >>> --- >>> v2: >>> - s/i.MX8MP/i.MX8/, >>> - Add Reviewed-by. >>> --- >>> drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/bridge/imx/Kconfig >>> b/drivers/gpu/drm/bridge/imx/Kconfig >>> index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 >>> --- a/drivers/gpu/drm/bridge/imx/Kconfig >>> +++ b/drivers/gpu/drm/bridge/imx/Kconfig >>> @@ -1,3 +1,5 @@ >>> +if ARCH_MXC || COMPILE_TEST >>> + >>> config DRM_IMX8QM_LDB >>> tristate "Freescale i.MX8QM LVDS display bridge" >>> depends on OF >>> @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI >>> help >>> Choose this to enable pixel link to display pixel >>> interface(PXL2DPI) >>> found in Freescale i.MX8qxp processor. >>> + >>> +endif # ARCH_MXC || COMPILE_TEST >> >> I was wondering why those were added in drivers/gpu/drm/bridge/imx >> since they are specific to NXP SoCs, >> I think they should be moved in the right drm imx subsystem instead >> of this change. > > There are 2 directories which contain display controller drivers for > i.MX SoCs: > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > i.MX8qm/qxp DPU([1], not landed yet) > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > LCDIFv3([2], not landed yet) > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to share > bridge drivers across display controllers. I see chance to use the > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. Thanks for the explanation, LGTM Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> > > [1] > https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ > [2] > https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ > > Regards, > Liu Ying > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-06-28 7:10 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-06-28 7:10 UTC (permalink / raw) To: Liu Ying, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: linux-arm-kernel, dri-devel, linux-kernel On 28/06/2022 04:31, Liu Ying wrote: > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: >> Hi, > > Hi, > >> >> On 24/06/2022 14:10, Geert Uytterhoeven wrote: >>> The various Freescale i.MX8 display bridges are only present on >>> Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to >>> prevent >>> asking the user about these drivers when configuring a kernel >>> without >>> i.MX SoC support. >>> >>> Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for >>> i.MX8qm") >>> Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for >>> i.MX8qxp") >>> Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel link >>> to DPI support") >>> Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp display >>> pixel link support") >>> Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp pixel >>> combiner support") >>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> >>> Reviewed-by: Liu Ying <victor.liu@nxp.com> >>> --- >>> v2: >>> - s/i.MX8MP/i.MX8/, >>> - Add Reviewed-by. >>> --- >>> drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/bridge/imx/Kconfig >>> b/drivers/gpu/drm/bridge/imx/Kconfig >>> index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 >>> --- a/drivers/gpu/drm/bridge/imx/Kconfig >>> +++ b/drivers/gpu/drm/bridge/imx/Kconfig >>> @@ -1,3 +1,5 @@ >>> +if ARCH_MXC || COMPILE_TEST >>> + >>> config DRM_IMX8QM_LDB >>> tristate "Freescale i.MX8QM LVDS display bridge" >>> depends on OF >>> @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI >>> help >>> Choose this to enable pixel link to display pixel >>> interface(PXL2DPI) >>> found in Freescale i.MX8qxp processor. >>> + >>> +endif # ARCH_MXC || COMPILE_TEST >> >> I was wondering why those were added in drivers/gpu/drm/bridge/imx >> since they are specific to NXP SoCs, >> I think they should be moved in the right drm imx subsystem instead >> of this change. > > There are 2 directories which contain display controller drivers for > i.MX SoCs: > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > i.MX8qm/qxp DPU([1], not landed yet) > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > LCDIFv3([2], not landed yet) > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to share > bridge drivers across display controllers. I see chance to use the > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. Thanks for the explanation, LGTM Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> > > [1] > https://patchwork.kernel.org/project/dri-devel/patch/20220407091156.1211923-6-victor.liu@nxp.com/ > [2] > https://patchwork.kernel.org/project/dri-devel/patch/20220624180201.150417-2-marex@denx.de/ > > Regards, > Liu Ying > ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC 2022-06-28 7:10 ` Neil Armstrong (?) @ 2022-07-01 7:09 ` Liu Ying -1 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-07-01 7:09 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel Hi Neil, Robert, On Tue, 2022-06-28 at 09:10 +0200, Neil Armstrong wrote: > On 28/06/2022 04:31, Liu Ying wrote: > > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > > > Hi, > > > > Hi, > > > > > > > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > > > The various Freescale i.MX8 display bridges are only present on > > > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > > > prevent > > > > asking the user about these drivers when configuring a kernel > > > > without > > > > i.MX SoC support. > > > > > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > > > i.MX8qm") > > > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > > > i.MX8qxp") > > > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel > > > > link > > > > to DPI support") > > > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp > > > > display > > > > pixel link support") > > > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp > > > > pixel > > > > combiner support") > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > > > --- > > > > v2: > > > > - s/i.MX8MP/i.MX8/, > > > > - Add Reviewed-by. > > > > --- > > > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > > > 1 file changed, 4 insertions(+) > > > > > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > > > b/drivers/gpu/drm/bridge/imx/Kconfig > > > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > > > @@ -1,3 +1,5 @@ > > > > +if ARCH_MXC || COMPILE_TEST > > > > + > > > > config DRM_IMX8QM_LDB > > > > tristate "Freescale i.MX8QM LVDS display bridge" > > > > depends on OF > > > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > > > help > > > > Choose this to enable pixel link to display pixel > > > > interface(PXL2DPI) > > > > found in Freescale i.MX8qxp processor. > > > > + > > > > +endif # ARCH_MXC || COMPILE_TEST > > > > > > I was wondering why those were added in > > > drivers/gpu/drm/bridge/imx > > > since they are specific to NXP SoCs, > > > I think they should be moved in the right drm imx subsystem > > > instead > > > of this change. > > > > There are 2 directories which contain display controller drivers > > for > > i.MX SoCs: > > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > > i.MX8qm/qxp DPU([1], not landed yet) > > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > > LCDIFv3([2], not landed yet) > > > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to > > share > > bridge drivers across display controllers. I see chance to use the > > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. > > Thanks for the explanation, LGTM > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> Care to pick this up? I don't have permission to push this to drm-misc. Regards, Liu Ying ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-07-01 7:09 ` Liu Ying 0 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-07-01 7:09 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: dri-devel, linux-arm-kernel, linux-kernel Hi Neil, Robert, On Tue, 2022-06-28 at 09:10 +0200, Neil Armstrong wrote: > On 28/06/2022 04:31, Liu Ying wrote: > > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > > > Hi, > > > > Hi, > > > > > > > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > > > The various Freescale i.MX8 display bridges are only present on > > > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > > > prevent > > > > asking the user about these drivers when configuring a kernel > > > > without > > > > i.MX SoC support. > > > > > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > > > i.MX8qm") > > > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > > > i.MX8qxp") > > > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel > > > > link > > > > to DPI support") > > > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp > > > > display > > > > pixel link support") > > > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp > > > > pixel > > > > combiner support") > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > > > --- > > > > v2: > > > > - s/i.MX8MP/i.MX8/, > > > > - Add Reviewed-by. > > > > --- > > > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > > > 1 file changed, 4 insertions(+) > > > > > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > > > b/drivers/gpu/drm/bridge/imx/Kconfig > > > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > > > @@ -1,3 +1,5 @@ > > > > +if ARCH_MXC || COMPILE_TEST > > > > + > > > > config DRM_IMX8QM_LDB > > > > tristate "Freescale i.MX8QM LVDS display bridge" > > > > depends on OF > > > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > > > help > > > > Choose this to enable pixel link to display pixel > > > > interface(PXL2DPI) > > > > found in Freescale i.MX8qxp processor. > > > > + > > > > +endif # ARCH_MXC || COMPILE_TEST > > > > > > I was wondering why those were added in > > > drivers/gpu/drm/bridge/imx > > > since they are specific to NXP SoCs, > > > I think they should be moved in the right drm imx subsystem > > > instead > > > of this change. > > > > There are 2 directories which contain display controller drivers > > for > > i.MX SoCs: > > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > > i.MX8qm/qxp DPU([1], not landed yet) > > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > > LCDIFv3([2], not landed yet) > > > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to > > share > > bridge drivers across display controllers. I see chance to use the > > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. > > Thanks for the explanation, LGTM > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> Care to pick this up? I don't have permission to push this to drm-misc. Regards, Liu Ying _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-07-01 7:09 ` Liu Ying 0 siblings, 0 replies; 18+ messages in thread From: Liu Ying @ 2022-07-01 7:09 UTC (permalink / raw) To: Neil Armstrong, Geert Uytterhoeven, Andrzej Hajda, Robert Foss, David Airlie, Daniel Vetter, Shawn Guo, Sascha Hauer, Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Sakari Ailus, Marek Vasut Cc: linux-arm-kernel, dri-devel, linux-kernel Hi Neil, Robert, On Tue, 2022-06-28 at 09:10 +0200, Neil Armstrong wrote: > On 28/06/2022 04:31, Liu Ying wrote: > > On Mon, 2022-06-27 at 14:22 +0200, Neil Armstrong wrote: > > > Hi, > > > > Hi, > > > > > > > > On 24/06/2022 14:10, Geert Uytterhoeven wrote: > > > > The various Freescale i.MX8 display bridges are only present on > > > > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to > > > > prevent > > > > asking the user about these drivers when configuring a kernel > > > > without > > > > i.MX SoC support. > > > > > > > > Fixes: e60c4354840b2fe8 ("drm/bridge: imx: Add LDB support for > > > > i.MX8qm") > > > > Fixes: 3818715f62b42b5c ("drm/bridge: imx: Add LDB support for > > > > i.MX8qxp") > > > > Fixes: 96988a526c97cfbe ("drm/bridge: imx: Add i.MX8qxp pixel > > > > link > > > > to DPI support") > > > > Fixes: 1ec17c26bc06289d ("drm/bridge: imx: Add i.MX8qm/qxp > > > > display > > > > pixel link support") > > > > Fixes: 93e163a9e0392aca ("drm/bridge: imx: Add i.MX8qm/qxp > > > > pixel > > > > combiner support") > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > Reviewed-by: Liu Ying <victor.liu@nxp.com> > > > > --- > > > > v2: > > > > - s/i.MX8MP/i.MX8/, > > > > - Add Reviewed-by. > > > > --- > > > > drivers/gpu/drm/bridge/imx/Kconfig | 4 ++++ > > > > 1 file changed, 4 insertions(+) > > > > > > > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig > > > > b/drivers/gpu/drm/bridge/imx/Kconfig > > > > index 212a7b0e64fd8b5a..608f47f41bcd1c81 100644 > > > > --- a/drivers/gpu/drm/bridge/imx/Kconfig > > > > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > > > > @@ -1,3 +1,5 @@ > > > > +if ARCH_MXC || COMPILE_TEST > > > > + > > > > config DRM_IMX8QM_LDB > > > > tristate "Freescale i.MX8QM LVDS display bridge" > > > > depends on OF > > > > @@ -41,3 +43,5 @@ config DRM_IMX8QXP_PIXEL_LINK_TO_DPI > > > > help > > > > Choose this to enable pixel link to display pixel > > > > interface(PXL2DPI) > > > > found in Freescale i.MX8qxp processor. > > > > + > > > > +endif # ARCH_MXC || COMPILE_TEST > > > > > > I was wondering why those were added in > > > drivers/gpu/drm/bridge/imx > > > since they are specific to NXP SoCs, > > > I think they should be moved in the right drm imx subsystem > > > instead > > > of this change. > > > > There are 2 directories which contain display controller drivers > > for > > i.MX SoCs: > > a. drivers/gpu/drm/imx - i.MX51/53/6qdl IPUv3, i.MX8mq DCSS and > > i.MX8qm/qxp DPU([1], not landed yet) > > b. drivers/gpu/drm/mxsfb - i.MX23/28/6sx/8mq LCDIF and i.MX8mp > > LCDIFv3([2], not landed yet) > > > > Bridges added in drivers/gpu/drm/bridge/imx make it possible to > > share > > bridge drivers across display controllers. I see chance to use the > > LVDS Display Bridge(LDB) helper(imx-ldb-helper.c) for i.MX6sx LDB. > > Thanks for the explanation, LGTM > Reviewed-by: Neil Armstrong <narmstrong@baylibre.com> Care to pick this up? I don't have permission to push this to drm-misc. Regards, Liu Ying ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC 2022-06-24 12:10 ` Geert Uytterhoeven (?) @ 2022-07-01 8:29 ` Neil Armstrong -1 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-07-01 8:29 UTC (permalink / raw) To: NXP Linux Team, Sascha Hauer, Pengutronix Kernel Team, Jernej Skrabec, Fabio Estevam, Jonas Karlman, Daniel Vetter, Shawn Guo, Laurent Pinchart, Robert Foss, David Airlie, Marek Vasut, Andrzej Hajda, Geert Uytterhoeven, Sakari Ailus, Liu Ying Cc: linux-kernel, dri-devel, linux-arm-kernel, Neil Armstrong Hi, On Fri, 24 Jun 2022 14:10:51 +0200, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Thanks, Applied to https://anongit.freedesktop.org/git/drm/drm-misc.git (drm-misc-next) [1/1] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC https://cgit.freedesktop.org/drm/drm-misc/commit/?id=5cd561688b3e8a248fdbb37a1b10801b06736865 -- Neil ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-07-01 8:29 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-07-01 8:29 UTC (permalink / raw) To: NXP Linux Team, Sascha Hauer, Pengutronix Kernel Team, Jernej Skrabec, Fabio Estevam, Jonas Karlman, Daniel Vetter, Shawn Guo, Laurent Pinchart, Robert Foss, David Airlie, Marek Vasut, Andrzej Hajda, Geert Uytterhoeven, Sakari Ailus, Liu Ying Cc: Neil Armstrong, linux-arm-kernel, dri-devel, linux-kernel Hi, On Fri, 24 Jun 2022 14:10:51 +0200, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Thanks, Applied to https://anongit.freedesktop.org/git/drm/drm-misc.git (drm-misc-next) [1/1] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC https://cgit.freedesktop.org/drm/drm-misc/commit/?id=5cd561688b3e8a248fdbb37a1b10801b06736865 -- Neil _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC @ 2022-07-01 8:29 ` Neil Armstrong 0 siblings, 0 replies; 18+ messages in thread From: Neil Armstrong @ 2022-07-01 8:29 UTC (permalink / raw) To: NXP Linux Team, Sascha Hauer, Pengutronix Kernel Team, Jernej Skrabec, Fabio Estevam, Jonas Karlman, Daniel Vetter, Shawn Guo, Laurent Pinchart, Robert Foss, David Airlie, Marek Vasut, Andrzej Hajda, Geert Uytterhoeven, Sakari Ailus, Liu Ying Cc: Neil Armstrong, linux-arm-kernel, dri-devel, linux-kernel Hi, On Fri, 24 Jun 2022 14:10:51 +0200, Geert Uytterhoeven wrote: > The various Freescale i.MX8 display bridges are only present on > Freescale i.MX8 SoCs. Hence add a dependency on ARCH_MXC, to prevent > asking the user about these drivers when configuring a kernel without > i.MX SoC support. > > Thanks, Applied to https://anongit.freedesktop.org/git/drm/drm-misc.git (drm-misc-next) [1/1] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC https://cgit.freedesktop.org/drm/drm-misc/commit/?id=5cd561688b3e8a248fdbb37a1b10801b06736865 -- Neil ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2022-07-01 8:31 UTC | newest] Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-24 12:10 [PATCH v2] drm/bridge: imx: i.MX8 bridge drivers should depend on ARCH_MXC Geert Uytterhoeven 2022-06-24 12:10 ` Geert Uytterhoeven 2022-06-24 12:10 ` Geert Uytterhoeven 2022-06-27 12:22 ` Neil Armstrong 2022-06-27 12:22 ` Neil Armstrong 2022-06-27 12:22 ` Neil Armstrong 2022-06-28 2:31 ` Liu Ying 2022-06-28 2:31 ` Liu Ying 2022-06-28 2:31 ` Liu Ying 2022-06-28 7:10 ` Neil Armstrong 2022-06-28 7:10 ` Neil Armstrong 2022-06-28 7:10 ` Neil Armstrong 2022-07-01 7:09 ` Liu Ying 2022-07-01 7:09 ` Liu Ying 2022-07-01 7:09 ` Liu Ying 2022-07-01 8:29 ` Neil Armstrong 2022-07-01 8:29 ` Neil Armstrong 2022-07-01 8:29 ` Neil Armstrong
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.