Skip to contentSkip to conversation

ARM: dts: imx6sx: Complete the PXP support

PATCH New Fabio Estevam 2 participants 3 messages Nov 6, 2018 – Nov 16, 2018

Subsystems (3)
lore
Share & tools
lore

Fabio Estevam

author reviewer
#0 lore raw

PATCH 1/2 ARM: dts: imx6sx: Complete the PXP support

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.

---
 arch/arm/boot/dts/imx6sx.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
arch/arm/boot/dts/imx6sx.dtsi +3 -3
index 84b7687..b79b6c4 100644
@@ -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";
 				};
 
Signature · 1 line
2.7.4
author reviewer
#1 lore raw

PATCH 2/2 ARM: imx_v6_v7_defconfig: Select the PXP driver

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.

---
 arch/arm/configs/imx_v6_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)
arch/arm/configs/imx_v6_v7_defconfig +1 -0
index 1ad5736..3e2ab0a 100644
@@ -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
Signature · 1 line
2.7.4
Applied
#2 lore raw

PATCH 1/2 ARM: dts: imx6sx: Complete the PXP support

In reply to Fabio Estevam
Quoted 7 lines · Fabio Estevam

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.

Applied both, thanks.