* [PATCH 0/5] i.MX6Q IPP DI support
@ 2022-10-13 13:18 Alexander Stein
2022-10-13 13:18 ` [PATCH 1/5] ARM: dts: imx6qdl: add clocks to clock controller node Alexander Stein
` (4 more replies)
0 siblings, 5 replies; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
Hi,
this series add support for IPP_DI[01] clock input.
1st patch is just preparation to handle anaclk1/anaclk2 similar to osc &
friends.
2nd & 3rd patch adds the clock to bindings and device tree.
4th patch adds the clock to the clk tree and also adds support for IPU
clock source.
5th patch adds the pinctrl defines for imx6q (only). I was only able to test
this on pad DA14 on imx6q, but not on imx6dl.
Putting all together it is possible to use those IPP DI0/1 clocks as input
for IPU pixel clock.
Best regards,
Alexander
Alexander Stein (5):
ARM: dts: imx6qdl: add clocks to clock controller node
dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings
ARM: dts: imx6qdl: add missing ipp_di clocks to clock controller node
clk: imx6q: Add ipp-di0 and ipp-di1 clocks
ARM: dts: imx6q: add ipp_di0 and ipp_di1 pad configuration
.../bindings/clock/imx6q-clock.yaml | 4 ++
arch/arm/boot/dts/imx6dl-pinfunc.h | 3 ++
arch/arm/boot/dts/imx6q-pinfunc.h | 3 ++
arch/arm/boot/dts/imx6qdl.dtsi | 38 +++++++++++++++++--
drivers/clk/imx/clk-imx6q.c | 20 ++++++----
include/dt-bindings/clock/imx6qdl-clock.h | 4 +-
6 files changed, 60 insertions(+), 12 deletions(-)
--
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 [flat|nested] 8+ messages in thread
* [PATCH 1/5] ARM: dts: imx6qdl: add clocks to clock controller node
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
@ 2022-10-13 13:18 ` Alexander Stein
2022-10-13 13:18 ` [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings Alexander Stein
` (3 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
Add missing anaclk clocks and needed labels for the existing clocks.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm/boot/dts/imx6qdl.dtsi | 22 +++++++++++++++++++---
1 file changed, 19 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index 1a4b299815ca..e357ee2df779 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -54,23 +54,35 @@ aliases {
};
clocks {
- ckil {
+ ckil: ckil {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32768>;
};
- ckih1 {
+ ckih1: ckih1 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <0>;
};
- osc {
+ osc: osc {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <24000000>;
};
+
+ anaclk1: anaclk1 {
+ compatible = "fixed-clock";
+ #clock-cells = <0>;
+ clock-frequency = <0>;
+ };
+
+ anaclk2: anaclk2 {
+ compatible = "fixed-clock";
+ #clock-cells = <0>;
+ clock-frequency = <0>;
+ };
};
ldb: ldb {
@@ -686,6 +698,10 @@ clks: clock-controller@20c4000 {
interrupts = <0 87 IRQ_TYPE_LEVEL_HIGH>,
<0 88 IRQ_TYPE_LEVEL_HIGH>;
#clock-cells = <1>;
+ clocks = <&osc>, <&ckil>, <&ckih1>,
+ <&anaclk1>, <&anaclk2>;
+ clock-names = "osc", "ckil", "ckih1",
+ "anaclk1", "anaclk2";
};
anatop: anatop@20c8000 {
--
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] 8+ messages in thread
* [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
2022-10-13 13:18 ` [PATCH 1/5] ARM: dts: imx6qdl: add clocks to clock controller node Alexander Stein
@ 2022-10-13 13:18 ` Alexander Stein
2022-10-13 13:48 ` Krzysztof Kozlowski
2022-10-13 13:18 ` [PATCH 3/5] ARM: dts: imx6qdl: add missing ipp_di clocks to clock controller node Alexander Stein
` (2 subsequent siblings)
4 siblings, 1 reply; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
These are alternative input clocks for IPU available as external inputs.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
Documentation/devicetree/bindings/clock/imx6q-clock.yaml | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/clock/imx6q-clock.yaml b/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
index 4f4637eddb8b..329e4cb95dc0 100644
--- a/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
@@ -34,6 +34,8 @@ properties:
- description: ckih1 clock input
- description: anaclk1 clock input
- description: anaclk2 clock input
+ - description: ipp_di0 clock input
+ - description: ipp_di1 clock input
clock-names:
items:
@@ -42,6 +44,8 @@ properties:
- const: ckih1
- const: anaclk1
- const: anaclk2
+ - const: ipp_di0
+ - const: ipp_di1
fsl,pmic-stby-poweroff:
$ref: /schemas/types.yaml#/definitions/flag
--
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] 8+ messages in thread
* [PATCH 3/5] ARM: dts: imx6qdl: add missing ipp_di clocks to clock controller node
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
2022-10-13 13:18 ` [PATCH 1/5] ARM: dts: imx6qdl: add clocks to clock controller node Alexander Stein
2022-10-13 13:18 ` [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings Alexander Stein
@ 2022-10-13 13:18 ` Alexander Stein
2022-10-13 13:18 ` [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks Alexander Stein
2022-10-13 13:18 ` [PATCH 5/5] ARM: dts: imx6q: add ipp_di0 and ipp_di1 pad configuration Alexander Stein
4 siblings, 0 replies; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
These are external input clocks and a rate must be provided explicitly.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm/boot/dts/imx6qdl.dtsi | 20 ++++++++++++++++++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
index e357ee2df779..def4e4e4b29e 100644
--- a/arch/arm/boot/dts/imx6qdl.dtsi
+++ b/arch/arm/boot/dts/imx6qdl.dtsi
@@ -83,6 +83,20 @@ anaclk2: anaclk2 {
#clock-cells = <0>;
clock-frequency = <0>;
};
+
+ ipp_di0: ipp-di0 {
+ compatible = "fixed-clock";
+ #clock-cells = <0>;
+ clock-frequency = <0>;
+ clock-output-names = "ipp_di0";
+ };
+
+ ipp_di1: ipp-di1 {
+ compatible = "fixed-clock";
+ #clock-cells = <0>;
+ clock-frequency = <0>;
+ clock-output-names = "ipp_di1";
+ };
};
ldb: ldb {
@@ -699,9 +713,11 @@ clks: clock-controller@20c4000 {
<0 88 IRQ_TYPE_LEVEL_HIGH>;
#clock-cells = <1>;
clocks = <&osc>, <&ckil>, <&ckih1>,
- <&anaclk1>, <&anaclk2>;
+ <&anaclk1>, <&anaclk2>,
+ <&ipp_di0>, <&ipp_di1>;
clock-names = "osc", "ckil", "ckih1",
- "anaclk1", "anaclk2";
+ "anaclk1", "anaclk2",
+ "ipp_di0", "ipp_di1";
};
anatop: anatop@20c8000 {
--
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] 8+ messages in thread
* [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
` (2 preceding siblings ...)
2022-10-13 13:18 ` [PATCH 3/5] ARM: dts: imx6qdl: add missing ipp_di clocks to clock controller node Alexander Stein
@ 2022-10-13 13:18 ` Alexander Stein
2022-10-13 13:49 ` Krzysztof Kozlowski
2022-10-13 13:18 ` [PATCH 5/5] ARM: dts: imx6q: add ipp_di0 and ipp_di1 pad configuration Alexander Stein
4 siblings, 1 reply; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
Described in CCM_CHSCCDR and CCM_CSCDR2 in the fields ipuX_diX_pre_clk_sel.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
drivers/clk/imx/clk-imx6q.c | 20 ++++++++++++--------
include/dt-bindings/clock/imx6qdl-clock.h | 4 +++-
2 files changed, 15 insertions(+), 9 deletions(-)
diff --git a/drivers/clk/imx/clk-imx6q.c b/drivers/clk/imx/clk-imx6q.c
index de36f58d551c..6f25361302a6 100644
--- a/drivers/clk/imx/clk-imx6q.c
+++ b/drivers/clk/imx/clk-imx6q.c
@@ -38,14 +38,14 @@ static const char *gpu3d_shader_sels[] = { "mmdc_ch0_axi", "pll3_usb_otg", "pll2
static const char *ipu_sels[] = { "mmdc_ch0_axi", "pll2_pfd2_396m", "pll3_120m", "pll3_pfd1_540m", };
static const char *ldb_di_sels[] = { "pll5_video_div", "pll2_pfd0_352m", "pll2_pfd2_396m", "mmdc_ch1_axi", "pll3_usb_otg", };
static const char *ipu_di_pre_sels[] = { "mmdc_ch0_axi", "pll3_usb_otg", "pll5_video_div", "pll2_pfd0_352m", "pll2_pfd2_396m", "pll3_pfd1_540m", };
-static const char *ipu1_di0_sels[] = { "ipu1_di0_pre", "dummy", "dummy", "ldb_di0", "ldb_di1", };
-static const char *ipu1_di1_sels[] = { "ipu1_di1_pre", "dummy", "dummy", "ldb_di0", "ldb_di1", };
-static const char *ipu2_di0_sels[] = { "ipu2_di0_pre", "dummy", "dummy", "ldb_di0", "ldb_di1", };
-static const char *ipu2_di1_sels[] = { "ipu2_di1_pre", "dummy", "dummy", "ldb_di0", "ldb_di1", };
-static const char *ipu1_di0_sels_2[] = { "ipu1_di0_pre", "dummy", "dummy", "ldb_di0_podf", "ldb_di1_podf", };
-static const char *ipu1_di1_sels_2[] = { "ipu1_di1_pre", "dummy", "dummy", "ldb_di0_podf", "ldb_di1_podf", };
-static const char *ipu2_di0_sels_2[] = { "ipu2_di0_pre", "dummy", "dummy", "ldb_di0_podf", "ldb_di1_podf", };
-static const char *ipu2_di1_sels_2[] = { "ipu2_di1_pre", "dummy", "dummy", "ldb_di0_podf", "ldb_di1_podf", };
+static const char *ipu1_di0_sels[] = { "ipu1_di0_pre", "ipp_di0", "ipp_di1", "ldb_di0", "ldb_di1", };
+static const char *ipu1_di1_sels[] = { "ipu1_di1_pre", "ipp_di0", "ipp_di1", "ldb_di0", "ldb_di1", };
+static const char *ipu2_di0_sels[] = { "ipu2_di0_pre", "ipp_di0", "ipp_di1", "ldb_di0", "ldb_di1", };
+static const char *ipu2_di1_sels[] = { "ipu2_di1_pre", "ipp_di0", "ipp_di1", "ldb_di0", "ldb_di1", };
+static const char *ipu1_di0_sels_2[] = { "ipu1_di0_pre", "ipp_di0", "ipp_di1", "ldb_di0_podf", "ldb_di1_podf", };
+static const char *ipu1_di1_sels_2[] = { "ipu1_di1_pre", "ipp_di0", "ipp_di1", "ldb_di0_podf", "ldb_di1_podf", };
+static const char *ipu2_di0_sels_2[] = { "ipu2_di0_pre", "ipp_di0", "ipp_di1", "ldb_di0_podf", "ldb_di1_podf", };
+static const char *ipu2_di1_sels_2[] = { "ipu2_di1_pre", "ipp_di0", "ipp_di1", "ldb_di0_podf", "ldb_di1_podf", };
static const char *hsi_tx_sels[] = { "pll3_120m", "pll2_pfd2_396m", };
static const char *pcie_axi_sels[] = { "axi", "ahb", };
static const char *ssi_sels[] = { "pll3_pfd2_508m", "pll3_pfd3_454m", "pll4_audio_div", };
@@ -452,6 +452,10 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
hws[IMX6QDL_CLK_ANACLK1] = imx6q_obtain_fixed_clk_hw(ccm_node, "anaclk1", 0);
hws[IMX6QDL_CLK_ANACLK2] = imx6q_obtain_fixed_clk_hw(ccm_node, "anaclk2", 0);
+ /* ipp_di clocks are external input */
+ hws[IMX6QDL_CLK_IPP_DI0] = imx6q_obtain_fixed_clk_hw(ccm_node, "ipp_di0", 0);
+ hws[IMX6QDL_CLK_IPP_DI1] = imx6q_obtain_fixed_clk_hw(ccm_node, "ipp_di1", 0);
+
np = of_find_compatible_node(NULL, NULL, "fsl,imx6q-anatop");
anatop_base = base = of_iomap(np, 0);
WARN_ON(!base);
diff --git a/include/dt-bindings/clock/imx6qdl-clock.h b/include/dt-bindings/clock/imx6qdl-clock.h
index e20c43cc36f6..2beb8738ae3d 100644
--- a/include/dt-bindings/clock/imx6qdl-clock.h
+++ b/include/dt-bindings/clock/imx6qdl-clock.h
@@ -273,6 +273,8 @@
#define IMX6QDL_CLK_MMDC_P0_IPG 263
#define IMX6QDL_CLK_DCIC1 264
#define IMX6QDL_CLK_DCIC2 265
-#define IMX6QDL_CLK_END 266
+#define IMX6QDL_CLK_IPP_DI0 266
+#define IMX6QDL_CLK_IPP_DI1 267
+#define IMX6QDL_CLK_END 268
#endif /* __DT_BINDINGS_CLOCK_IMX6QDL_H */
--
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] 8+ messages in thread
* [PATCH 5/5] ARM: dts: imx6q: add ipp_di0 and ipp_di1 pad configuration
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
` (3 preceding siblings ...)
2022-10-13 13:18 ` [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks Alexander Stein
@ 2022-10-13 13:18 ` Alexander Stein
4 siblings, 0 replies; 8+ messages in thread
From: Alexander Stein @ 2022-10-13 13:18 UTC (permalink / raw)
To: Abel Vesa, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Shawn Guo, Sascha Hauer, Fabio Estevam
Cc: Alexander Stein, Pengutronix Kernel Team, NXP Linux Team,
Anson Huang, linux-clk, devicetree, linux-arm-kernel
The reference manual does not mention this setting, but it works.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm/boot/dts/imx6dl-pinfunc.h | 3 +++
arch/arm/boot/dts/imx6q-pinfunc.h | 3 +++
2 files changed, 6 insertions(+)
diff --git a/arch/arm/boot/dts/imx6dl-pinfunc.h b/arch/arm/boot/dts/imx6dl-pinfunc.h
index 9d88d09f9bf6..876ff4545c30 100644
--- a/arch/arm/boot/dts/imx6dl-pinfunc.h
+++ b/arch/arm/boot/dts/imx6dl-pinfunc.h
@@ -525,11 +525,13 @@
#define MX6QDL_PAD_EIM_DA12__EPDC_DATA02 0x194 0x564 0x000 0x8 0x0
#define MX6QDL_PAD_EIM_DA13__EIM_AD13 0x198 0x568 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_DA13__IPU1_DI1_D0_CS 0x198 0x568 0x000 0x1 0x0
+#define MX6QDL_PAD_EIM_DA13__IPP_DI1_CLK 0x198 0x568 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_DA13__GPIO3_IO13 0x198 0x568 0x000 0x5 0x0
#define MX6QDL_PAD_EIM_DA13__SRC_BOOT_CFG13 0x198 0x568 0x000 0x7 0x0
#define MX6QDL_PAD_EIM_DA13__EPDC_DATA13 0x198 0x568 0x000 0x8 0x0
#define MX6QDL_PAD_EIM_DA14__EIM_AD14 0x19c 0x56c 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_DA14__IPU1_DI1_D1_CS 0x19c 0x56c 0x000 0x1 0x0
+#define MX6QDL_PAD_EIM_DA14__IPP_DI0_CLK 0x19c 0x56c 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_DA14__GPIO3_IO14 0x19c 0x56c 0x000 0x5 0x0
#define MX6QDL_PAD_EIM_DA14__SRC_BOOT_CFG14 0x19c 0x56c 0x000 0x7 0x0
#define MX6QDL_PAD_EIM_DA14__EPDC_DATA14 0x19c 0x56c 0x000 0x8 0x0
@@ -602,6 +604,7 @@
#define MX6QDL_PAD_EIM_EB1__EPDC_SDSHR 0x1c8 0x598 0x000 0x8 0x0
#define MX6QDL_PAD_EIM_EB2__EIM_EB2_B 0x1cc 0x59c 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_EB2__ECSPI1_SS0 0x1cc 0x59c 0x7e4 0x1 0x2
+#define MX6QDL_PAD_EIM_EB2__IPP_DI1_CLK 0x1cc 0x59c 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_EB2__IPU1_CSI1_DATA19 0x1cc 0x59c 0x8ac 0x3 0x1
#define MX6QDL_PAD_EIM_EB2__HDMI_TX_DDC_SCL 0x1cc 0x59c 0x860 0x4 0x0
#define MX6QDL_PAD_EIM_EB2__GPIO2_IO30 0x1cc 0x59c 0x000 0x5 0x0
diff --git a/arch/arm/boot/dts/imx6q-pinfunc.h b/arch/arm/boot/dts/imx6q-pinfunc.h
index e40409d04b97..40e53273c2e2 100644
--- a/arch/arm/boot/dts/imx6q-pinfunc.h
+++ b/arch/arm/boot/dts/imx6q-pinfunc.h
@@ -76,6 +76,7 @@
#define MX6QDL_PAD_EIM_A25__HDMI_TX_CEC_LINE 0x088 0x39c 0x88c 0x6 0x0
#define MX6QDL_PAD_EIM_EB2__EIM_EB2_B 0x08c 0x3a0 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_EB2__ECSPI1_SS0 0x08c 0x3a0 0x800 0x1 0x0
+#define MX6QDL_PAD_EIM_EB2__IPP_DI1_CLK 0x08c 0x3a0 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_EB2__IPU2_CSI1_DATA19 0x08c 0x3a0 0x8d4 0x3 0x0
#define MX6QDL_PAD_EIM_EB2__HDMI_TX_DDC_SCL 0x08c 0x3a0 0x890 0x4 0x0
#define MX6QDL_PAD_EIM_EB2__GPIO2_IO30 0x08c 0x3a0 0x000 0x5 0x0
@@ -373,10 +374,12 @@
#define MX6QDL_PAD_EIM_DA12__SRC_BOOT_CFG12 0x144 0x458 0x000 0x7 0x0
#define MX6QDL_PAD_EIM_DA13__EIM_AD13 0x148 0x45c 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_DA13__IPU1_DI1_D0_CS 0x148 0x45c 0x000 0x1 0x0
+#define MX6QDL_PAD_EIM_DA13__IPP_DI1_CLK 0x148 0x45c 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_DA13__GPIO3_IO13 0x148 0x45c 0x000 0x5 0x0
#define MX6QDL_PAD_EIM_DA13__SRC_BOOT_CFG13 0x148 0x45c 0x000 0x7 0x0
#define MX6QDL_PAD_EIM_DA14__EIM_AD14 0x14c 0x460 0x000 0x0 0x0
#define MX6QDL_PAD_EIM_DA14__IPU1_DI1_D1_CS 0x14c 0x460 0x000 0x1 0x0
+#define MX6QDL_PAD_EIM_DA14__IPP_DI0_CLK 0x14c 0x460 0x000 0x2 0x0
#define MX6QDL_PAD_EIM_DA14__GPIO3_IO14 0x14c 0x460 0x000 0x5 0x0
#define MX6QDL_PAD_EIM_DA14__SRC_BOOT_CFG14 0x14c 0x460 0x000 0x7 0x0
#define MX6QDL_PAD_EIM_DA15__EIM_AD15 0x150 0x464 0x000 0x0 0x0
--
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] 8+ messages in thread
* Re: [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings
2022-10-13 13:18 ` [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings Alexander Stein
@ 2022-10-13 13:48 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-10-13 13:48 UTC (permalink / raw)
To: Alexander Stein, Abel Vesa, Michael Turquette, Stephen Boyd,
Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Fabio Estevam
Cc: Pengutronix Kernel Team, NXP Linux Team, Anson Huang, linux-clk,
devicetree, linux-arm-kernel
On 13/10/2022 09:18, Alexander Stein wrote:
> These are alternative input clocks for IPU available as external inputs.
You made them required, not alternative. Please describe in commit why
they are required (what was broken).
>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
> Documentation/devicetree/bindings/clock/imx6q-clock.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
Best regards,
Krzysztof
_______________________________________________
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] 8+ messages in thread
* Re: [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks
2022-10-13 13:18 ` [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks Alexander Stein
@ 2022-10-13 13:49 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2022-10-13 13:49 UTC (permalink / raw)
To: Alexander Stein, Abel Vesa, Michael Turquette, Stephen Boyd,
Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Fabio Estevam
Cc: Pengutronix Kernel Team, NXP Linux Team, Anson Huang, linux-clk,
devicetree, linux-arm-kernel
On 13/10/2022 09:18, Alexander Stein wrote:
> Described in CCM_CHSCCDR and CCM_CSCDR2 in the fields ipuX_diX_pre_clk_sel.
>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> ---
> drivers/clk/imx/clk-imx6q.c | 20 ++++++++++++--------
> include/dt-bindings/clock/imx6qdl-clock.h | 4 +++-
Don't mix bindings with drivers.
Best regards,
Krzysztof
_______________________________________________
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] 8+ messages in thread
end of thread, other threads:[~2022-10-13 13:58 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-13 13:18 [PATCH 0/5] i.MX6Q IPP DI support Alexander Stein
2022-10-13 13:18 ` [PATCH 1/5] ARM: dts: imx6qdl: add clocks to clock controller node Alexander Stein
2022-10-13 13:18 ` [PATCH 2/5] dt-bindings: clock: Add ipp_di0 and ipp_di1 clocks to i.MX6Q bindings Alexander Stein
2022-10-13 13:48 ` Krzysztof Kozlowski
2022-10-13 13:18 ` [PATCH 3/5] ARM: dts: imx6qdl: add missing ipp_di clocks to clock controller node Alexander Stein
2022-10-13 13:18 ` [PATCH 4/5] clk: imx6q: Add ipp-di0 and ipp-di1 clocks Alexander Stein
2022-10-13 13:49 ` Krzysztof Kozlowski
2022-10-13 13:18 ` [PATCH 5/5] ARM: dts: imx6q: add ipp_di0 and ipp_di1 pad configuration Alexander Stein
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).