linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support
@ 2018-11-06 12:03 Fabio Estevam
  2018-11-06 12:03 ` [PATCH 2/2] ARM: imx_v6_v7_defconfig: Select the PXP driver Fabio Estevam
  2018-11-16  5:49 ` [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Shawn Guo
  0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2018-11-06 12:03 UTC (permalink / raw)
  To: linux-arm-kernel

According to Documentation/devicetree/bindings/media/fsl-pxp.txt,
only one PXP clock needs to be described and it should be named
"axi".

Also pass the compatible string as suggested in the bindings doc.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/imx6sx.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
index 84b7687..b79b6c4 100644
--- a/arch/arm/boot/dts/imx6sx.dtsi
+++ b/arch/arm/boot/dts/imx6sx.dtsi
@@ -1194,11 +1194,11 @@
 				};
 
 				pxp: pxp at 2218000 {
+					compatible = "fsl,imx6sx-pxp", "fsl,imx6ull-pxp";
 					reg = <0x02218000 0x4000>;
 					interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
-					clocks = <&clks IMX6SX_CLK_PXP_AXI>,
-						 <&clks IMX6SX_CLK_DISPLAY_AXI>;
-					clock-names = "pxp-axi", "disp-axi";
+					clocks = <&clks IMX6SX_CLK_PXP_AXI>;
+					clock-names = "axi";
 					status = "disabled";
 				};
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 2/2] ARM: imx_v6_v7_defconfig: Select the PXP driver
  2018-11-06 12:03 [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Fabio Estevam
@ 2018-11-06 12:03 ` Fabio Estevam
  2018-11-16  5:49 ` [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Fabio Estevam @ 2018-11-06 12:03 UTC (permalink / raw)
  To: linux-arm-kernel

The Pixel Pipeline (PXP) block is present on several i.MX SoCs
such as imx6dl, imx6sl, imx6ul, imx6sx, imx6ull and imx7d.

Select the PXP driver by default.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/configs/imx_v6_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/imx_v6_v7_defconfig b/arch/arm/configs/imx_v6_v7_defconfig
index 1ad5736..3e2ab0a 100644
--- a/arch/arm/configs/imx_v6_v7_defconfig
+++ b/arch/arm/configs/imx_v6_v7_defconfig
@@ -257,6 +257,7 @@ CONFIG_VIDEO_MUX=y
 CONFIG_SOC_CAMERA=y
 CONFIG_V4L_MEM2MEM_DRIVERS=y
 CONFIG_VIDEO_CODA=m
+CONFIG_VIDEO_IMX_PXP=y
 # CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
 CONFIG_VIDEO_ADV7180=m
 CONFIG_VIDEO_OV5640=m
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support
  2018-11-06 12:03 [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Fabio Estevam
  2018-11-06 12:03 ` [PATCH 2/2] ARM: imx_v6_v7_defconfig: Select the PXP driver Fabio Estevam
@ 2018-11-16  5:49 ` Shawn Guo
  1 sibling, 0 replies; 3+ messages in thread
From: Shawn Guo @ 2018-11-16  5:49 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Nov 06, 2018 at 10:03:42AM -0200, Fabio Estevam wrote:
> According to Documentation/devicetree/bindings/media/fsl-pxp.txt,
> only one PXP clock needs to be described and it should be named
> "axi".
> 
> Also pass the compatible string as suggested in the bindings doc.
> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Applied both, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-11-16  5:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-06 12:03 [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Fabio Estevam
2018-11-06 12:03 ` [PATCH 2/2] ARM: imx_v6_v7_defconfig: Select the PXP driver Fabio Estevam
2018-11-16  5:49 ` [PATCH 1/2] ARM: dts: imx6sx: Complete the PXP support Shawn Guo

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).