* [PATCH] ARM: DT: imx53: fix lvds channel 1 port
@ 2014-09-04 12:05 Markus Niebel
0 siblings, 0 replies; 5+ messages in thread
From: Markus Niebel @ 2014-09-04 12:05 UTC (permalink / raw)
To: linux-arm-kernel; +Cc: Markus Niebel, devicetree, festevam, shawn.guo, p.zabel
using LVDS channel 1 on an i.MX53 leads to following error:
imx-ldb 53fa8008.ldb: unable to set di0 parent clock to ldb_di1
This comes from imx_ldb_set_clock with mux = 0. Mux parameter must be "1" for
reparenting di1 clock to ldb_di1. The value of the mux param comes from device
tree port settings.
On i.MX5, the internal two-input-multiplexer is used. Due to hardware limitations,
only one port (port@[0,1]) can be used for each channel (lvds-channel@[0,1],
respectively)
Documentation update suggested by Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Markus Niebel <Markus.Niebel@tq-group.com>
---
Documentation/devicetree/bindings/staging/imx-drm/ldb.txt | 15 +++++++++++++--
arch/arm/boot/dts/imx53.dtsi | 12 ++++++++++--
2 files changed, 23 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
index 578a1fc..443bcb6 100644
--- a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
+++ b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
@@ -56,6 +56,9 @@ Required properties:
- fsl,data-width : should be <18> or <24>
- port: A port node with endpoint definitions as defined in
Documentation/devicetree/bindings/media/video-interfaces.txt.
+ On i.MX5, the internal two-input-multiplexer is used.
+ Due to hardware limitations, only one port (port@[0,1])
+ can be used for each channel (lvds-channel@[0,1], respectively)
On i.MX6, there should be four ports (port@[0-3]) that correspond
to the four LVDS multiplexer inputs.
@@ -78,6 +81,8 @@ ldb: ldb@53fa8008 {
"di0", "di1";
lvds-channel@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <0>;
fsl,data-mapping = "spwg";
fsl,data-width = <24>;
@@ -86,7 +91,9 @@ ldb: ldb@53fa8008 {
/* ... */
};
- port {
+ port@0 {
+ reg = <0>;
+
lvds0_in: endpoint {
remote-endpoint = <&ipu_di0_lvds0>;
};
@@ -94,6 +101,8 @@ ldb: ldb@53fa8008 {
};
lvds-channel@1 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <1>;
fsl,data-mapping = "spwg";
fsl,data-width = <24>;
@@ -102,7 +111,9 @@ ldb: ldb@53fa8008 {
/* ... */
};
- port {
+ port@1 {
+ reg = <1>;
+
lvds1_in: endpoint {
remote-endpoint = <&ipu_di1_lvds1>;
};
diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
index 6456a00..1a2202d 100644
--- a/arch/arm/boot/dts/imx53.dtsi
+++ b/arch/arm/boot/dts/imx53.dtsi
@@ -419,10 +419,14 @@
status = "disabled";
lvds-channel@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <0>;
status = "disabled";
- port {
+ port@0 {
+ reg = <0>;
+
lvds0_in: endpoint {
remote-endpoint = <&ipu_di0_lvds0>;
};
@@ -430,10 +434,14 @@
};
lvds-channel@1 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <1>;
status = "disabled";
- port {
+ port@1 {
+ reg = <1>;
+
lvds1_in: endpoint {
remote-endpoint = <&ipu_di1_lvds1>;
};
--
2.1.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] ARM: DT: imx53: fix lvds channel 1 port
@ 2014-09-04 16:43 Markus Niebel
2014-09-10 1:28 ` Shawn Guo
[not found] ` <1409848985-16140-1-git-send-email-list-09_linux_arm-7U5DVgMjAv4@public.gmane.org>
0 siblings, 2 replies; 5+ messages in thread
From: Markus Niebel @ 2014-09-04 16:43 UTC (permalink / raw)
To: linux-arm-kernel; +Cc: Markus Niebel, devicetree, festevam, shawn.guo, p.zabel
From: Markus Niebel <Markus.Niebel@tq-group.com>
using LVDS channel 1 on an i.MX53 leads to following error:
imx-ldb 53fa8008.ldb: unable to set di0 parent clock to ldb_di1
This comes from imx_ldb_set_clock with mux = 0. Mux parameter must be "1" for
reparenting di1 clock to ldb_di1. The value of the mux param comes from device
tree port settings.
On i.MX5, the internal two-input-multiplexer is used. Due to hardware limitations,
only one port (port@[0,1]) can be used for each channel (lvds-channel@[0,1],
respectively)
Documentation update suggested by Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Markus Niebel <Markus.Niebel@tq-group.com>
---
Documentation/devicetree/bindings/staging/imx-drm/ldb.txt | 15 +++++++++++++--
arch/arm/boot/dts/imx53.dtsi | 12 ++++++++++--
2 files changed, 23 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
index 578a1fc..443bcb6 100644
--- a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
+++ b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
@@ -56,6 +56,9 @@ Required properties:
- fsl,data-width : should be <18> or <24>
- port: A port node with endpoint definitions as defined in
Documentation/devicetree/bindings/media/video-interfaces.txt.
+ On i.MX5, the internal two-input-multiplexer is used.
+ Due to hardware limitations, only one port (port@[0,1])
+ can be used for each channel (lvds-channel@[0,1], respectively)
On i.MX6, there should be four ports (port@[0-3]) that correspond
to the four LVDS multiplexer inputs.
@@ -78,6 +81,8 @@ ldb: ldb@53fa8008 {
"di0", "di1";
lvds-channel@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <0>;
fsl,data-mapping = "spwg";
fsl,data-width = <24>;
@@ -86,7 +91,9 @@ ldb: ldb@53fa8008 {
/* ... */
};
- port {
+ port@0 {
+ reg = <0>;
+
lvds0_in: endpoint {
remote-endpoint = <&ipu_di0_lvds0>;
};
@@ -94,6 +101,8 @@ ldb: ldb@53fa8008 {
};
lvds-channel@1 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <1>;
fsl,data-mapping = "spwg";
fsl,data-width = <24>;
@@ -102,7 +111,9 @@ ldb: ldb@53fa8008 {
/* ... */
};
- port {
+ port@1 {
+ reg = <1>;
+
lvds1_in: endpoint {
remote-endpoint = <&ipu_di1_lvds1>;
};
diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
index 6456a00..1a2202d 100644
--- a/arch/arm/boot/dts/imx53.dtsi
+++ b/arch/arm/boot/dts/imx53.dtsi
@@ -419,10 +419,14 @@
status = "disabled";
lvds-channel@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <0>;
status = "disabled";
- port {
+ port@0 {
+ reg = <0>;
+
lvds0_in: endpoint {
remote-endpoint = <&ipu_di0_lvds0>;
};
@@ -430,10 +434,14 @@
};
lvds-channel@1 {
+ #address-cells = <1>;
+ #size-cells = <0>;
reg = <1>;
status = "disabled";
- port {
+ port@1 {
+ reg = <1>;
+
lvds1_in: endpoint {
remote-endpoint = <&ipu_di1_lvds1>;
};
--
2.1.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: DT: imx53: fix lvds channel 1 port
2014-09-04 16:43 [PATCH] ARM: DT: imx53: fix lvds channel 1 port Markus Niebel
@ 2014-09-10 1:28 ` Shawn Guo
2014-09-10 13:15 ` Philipp Zabel
[not found] ` <1409848985-16140-1-git-send-email-list-09_linux_arm-7U5DVgMjAv4@public.gmane.org>
1 sibling, 1 reply; 5+ messages in thread
From: Shawn Guo @ 2014-09-10 1:28 UTC (permalink / raw)
To: Markus Niebel, p.zabel
Cc: devicetree, festevam, Markus Niebel, linux-arm-kernel
On Thu, Sep 04, 2014 at 06:43:05PM +0200, Markus Niebel wrote:
> From: Markus Niebel <Markus.Niebel@tq-group.com>
>
> using LVDS channel 1 on an i.MX53 leads to following error:
>
> imx-ldb 53fa8008.ldb: unable to set di0 parent clock to ldb_di1
>
> This comes from imx_ldb_set_clock with mux = 0. Mux parameter must be "1" for
> reparenting di1 clock to ldb_di1. The value of the mux param comes from device
> tree port settings.
>
> On i.MX5, the internal two-input-multiplexer is used. Due to hardware limitations,
> only one port (port@[0,1]) can be used for each channel (lvds-channel@[0,1],
> respectively)
>
> Documentation update suggested by Philipp Zabel <p.zabel@pengutronix.de>
>
> Signed-off-by: Markus Niebel <Markus.Niebel@tq-group.com>
Philipp,
Does the patch look good to you? I feel more comfortable to send it as
a fix and stable material only with your ACK at this stage.
Shawn
> ---
> Documentation/devicetree/bindings/staging/imx-drm/ldb.txt | 15 +++++++++++++--
> arch/arm/boot/dts/imx53.dtsi | 12 ++++++++++--
> 2 files changed, 23 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
> index 578a1fc..443bcb6 100644
> --- a/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
> +++ b/Documentation/devicetree/bindings/staging/imx-drm/ldb.txt
> @@ -56,6 +56,9 @@ Required properties:
> - fsl,data-width : should be <18> or <24>
> - port: A port node with endpoint definitions as defined in
> Documentation/devicetree/bindings/media/video-interfaces.txt.
> + On i.MX5, the internal two-input-multiplexer is used.
> + Due to hardware limitations, only one port (port@[0,1])
> + can be used for each channel (lvds-channel@[0,1], respectively)
> On i.MX6, there should be four ports (port@[0-3]) that correspond
> to the four LVDS multiplexer inputs.
>
> @@ -78,6 +81,8 @@ ldb: ldb@53fa8008 {
> "di0", "di1";
>
> lvds-channel@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> reg = <0>;
> fsl,data-mapping = "spwg";
> fsl,data-width = <24>;
> @@ -86,7 +91,9 @@ ldb: ldb@53fa8008 {
> /* ... */
> };
>
> - port {
> + port@0 {
> + reg = <0>;
> +
> lvds0_in: endpoint {
> remote-endpoint = <&ipu_di0_lvds0>;
> };
> @@ -94,6 +101,8 @@ ldb: ldb@53fa8008 {
> };
>
> lvds-channel@1 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> reg = <1>;
> fsl,data-mapping = "spwg";
> fsl,data-width = <24>;
> @@ -102,7 +111,9 @@ ldb: ldb@53fa8008 {
> /* ... */
> };
>
> - port {
> + port@1 {
> + reg = <1>;
> +
> lvds1_in: endpoint {
> remote-endpoint = <&ipu_di1_lvds1>;
> };
> diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
> index 6456a00..1a2202d 100644
> --- a/arch/arm/boot/dts/imx53.dtsi
> +++ b/arch/arm/boot/dts/imx53.dtsi
> @@ -419,10 +419,14 @@
> status = "disabled";
>
> lvds-channel@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> reg = <0>;
> status = "disabled";
>
> - port {
> + port@0 {
> + reg = <0>;
> +
> lvds0_in: endpoint {
> remote-endpoint = <&ipu_di0_lvds0>;
> };
> @@ -430,10 +434,14 @@
> };
>
> lvds-channel@1 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> reg = <1>;
> status = "disabled";
>
> - port {
> + port@1 {
> + reg = <1>;
> +
> lvds1_in: endpoint {
> remote-endpoint = <&ipu_di1_lvds1>;
> };
> --
> 2.1.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: DT: imx53: fix lvds channel 1 port
2014-09-10 1:28 ` Shawn Guo
@ 2014-09-10 13:15 ` Philipp Zabel
0 siblings, 0 replies; 5+ messages in thread
From: Philipp Zabel @ 2014-09-10 13:15 UTC (permalink / raw)
To: Shawn Guo
Cc: Markus Niebel, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA,
festevam-Re5JQEeQqe8AvxtiuMwx3w, Markus Niebel
Hi Shawn,
Am Mittwoch, den 10.09.2014, 09:28 +0800 schrieb Shawn Guo:
> On Thu, Sep 04, 2014 at 06:43:05PM +0200, Markus Niebel wrote:
> > From: Markus Niebel <Markus.Niebel-HVi9AzVXQk5Wk0Htik3J/w@public.gmane.org>
> >
> > using LVDS channel 1 on an i.MX53 leads to following error:
> >
> > imx-ldb 53fa8008.ldb: unable to set di0 parent clock to ldb_di1
> >
> > This comes from imx_ldb_set_clock with mux = 0. Mux parameter must be "1" for
> > reparenting di1 clock to ldb_di1. The value of the mux param comes from device
> > tree port settings.
> >
> > On i.MX5, the internal two-input-multiplexer is used. Due to hardware limitations,
> > only one port (port@[0,1]) can be used for each channel (lvds-channel@[0,1],
> > respectively)
> >
> > Documentation update suggested by Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
> >
> > Signed-off-by: Markus Niebel <Markus.Niebel-HVi9AzVXQk5Wk0Htik3J/w@public.gmane.org>
>
> Philipp,
>
> Does the patch look good to you? I feel more comfortable to send it as
> a fix and stable material only with your ACK at this stage.
Acked-by: Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
regards
Philipp
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: DT: imx53: fix lvds channel 1 port
[not found] ` <1409848985-16140-1-git-send-email-list-09_linux_arm-7U5DVgMjAv4@public.gmane.org>
@ 2014-09-11 7:44 ` Shawn Guo
0 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2014-09-11 7:44 UTC (permalink / raw)
To: Markus Niebel
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA, p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ,
festevam-Re5JQEeQqe8AvxtiuMwx3w, Markus Niebel
On Thu, Sep 04, 2014 at 06:43:05PM +0200, Markus Niebel wrote:
> From: Markus Niebel <Markus.Niebel-HVi9AzVXQk5Wk0Htik3J/w@public.gmane.org>
>
> using LVDS channel 1 on an i.MX53 leads to following error:
>
> imx-ldb 53fa8008.ldb: unable to set di0 parent clock to ldb_di1
>
> This comes from imx_ldb_set_clock with mux = 0. Mux parameter must be "1" for
> reparenting di1 clock to ldb_di1. The value of the mux param comes from device
> tree port settings.
>
> On i.MX5, the internal two-input-multiplexer is used. Due to hardware limitations,
> only one port (port@[0,1]) can be used for each channel (lvds-channel@[0,1],
> respectively)
>
> Documentation update suggested by Philipp Zabel <p.zabel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
>
> Signed-off-by: Markus Niebel <Markus.Niebel-HVi9AzVXQk5Wk0Htik3J/w@public.gmane.org>
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-09-11 7:44 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-04 16:43 [PATCH] ARM: DT: imx53: fix lvds channel 1 port Markus Niebel
2014-09-10 1:28 ` Shawn Guo
2014-09-10 13:15 ` Philipp Zabel
[not found] ` <1409848985-16140-1-git-send-email-list-09_linux_arm-7U5DVgMjAv4@public.gmane.org>
2014-09-11 7:44 ` Shawn Guo
-- strict thread matches above, loose matches on Subject: below --
2014-09-04 12:05 Markus Niebel
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).