* [PATCH v3 0/4] r8a7793 Gose video input support
@ 2017-05-19 13:07 ` Ulrich Hecht
0 siblings, 0 replies; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA, geert-Td1EMuHUCqxL1ZNQvxDV9g,
magnus.damm-Re5JQEeQqe8AvxtiuMwx3w,
hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w,
niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw,
sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
horms-/R6kz+dDXgpPR4JQBCEnsQ, devicetree-u79uwXL29TY76Z2rM5mHXA,
Ulrich Hecht
Hi!
This is a by-the-datasheet implementation of analog and digital video input
on the Gose board.
This revision adds new bindings that distinguish between ADV7180 variants
with three and six input ports. There are numerous variants of this chip,
but since all that have "CP" in their names have three inputs, and all that
have "ST" have six, I have limited myself to two new compatible strings,
"adv7180cp" and "adv7180st".
The digital input patch has received minor tweaks of the port names for
consistency, and the Gose analog input patch has been modified to use the
new bindings, and a composite video connector has been added.
CU
Uli
Changes since v2:
- hdmi: port hdmi_con renamed to hdmi_con_out
- adv7180: added new compatibility strings and bindings
- composite: added connector, use new bindings
Changes since v1:
- r8a7793.dtsi: added VIN2
- modeled HDMI decoder input/output and connector
- added "renesas,rcar-gen2-vin" compat strings
- removed unnecessary "remote" node and aliases
- set ADV7612 interrupt to GP4_2
Ulrich Hecht (4):
ARM: dts: gose: add HDMI input
media: adv7180: add adv7180cp, adv7180st compatible strings
media: adv7180: Add adv7180cp, adv7180st bindings
ARM: dts: gose: add composite video input
.../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
drivers/media/i2c/adv7180.c | 2 +
3 files changed, 142 insertions(+), 2 deletions(-)
--
2.7.4
--
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] 16+ messages in thread
* [PATCH v3 0/4] r8a7793 Gose video input support
@ 2017-05-19 13:07 ` Ulrich Hecht
0 siblings, 0 replies; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree, Ulrich Hecht
Hi!
This is a by-the-datasheet implementation of analog and digital video input
on the Gose board.
This revision adds new bindings that distinguish between ADV7180 variants
with three and six input ports. There are numerous variants of this chip,
but since all that have "CP" in their names have three inputs, and all that
have "ST" have six, I have limited myself to two new compatible strings,
"adv7180cp" and "adv7180st".
The digital input patch has received minor tweaks of the port names for
consistency, and the Gose analog input patch has been modified to use the
new bindings, and a composite video connector has been added.
CU
Uli
Changes since v2:
- hdmi: port hdmi_con renamed to hdmi_con_out
- adv7180: added new compatibility strings and bindings
- composite: added connector, use new bindings
Changes since v1:
- r8a7793.dtsi: added VIN2
- modeled HDMI decoder input/output and connector
- added "renesas,rcar-gen2-vin" compat strings
- removed unnecessary "remote" node and aliases
- set ADV7612 interrupt to GP4_2
Ulrich Hecht (4):
ARM: dts: gose: add HDMI input
media: adv7180: add adv7180cp, adv7180st compatible strings
media: adv7180: Add adv7180cp, adv7180st bindings
ARM: dts: gose: add composite video input
.../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
drivers/media/i2c/adv7180.c | 2 +
3 files changed, 142 insertions(+), 2 deletions(-)
--
2.7.4
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v3 1/4] ARM: dts: gose: add HDMI input
2017-05-19 13:07 ` Ulrich Hecht
(?)
@ 2017-05-19 13:07 ` Ulrich Hecht
-1 siblings, 0 replies; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree, Ulrich Hecht
Identical to the setup on Koelsch.
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
arch/arm/boot/dts/r8a7793-gose.dts | 68 ++++++++++++++++++++++++++++++++++++--
1 file changed, 66 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/r8a7793-gose.dts b/arch/arm/boot/dts/r8a7793-gose.dts
index 95e51b7..30f0835 100644
--- a/arch/arm/boot/dts/r8a7793-gose.dts
+++ b/arch/arm/boot/dts/r8a7793-gose.dts
@@ -253,12 +253,23 @@
};
};
+ hdmi-in {
+ compatible = "hdmi-connector";
+ type = "a";
+
+ port {
+ hdmi_con_in: endpoint {
+ remote-endpoint = <&adv7612_in>;
+ };
+ };
+ };
+
hdmi-out {
compatible = "hdmi-connector";
type = "a";
port {
- hdmi_con: endpoint {
+ hdmi_con_out: endpoint {
remote-endpoint = <&adv7511_out>;
};
};
@@ -395,6 +406,11 @@
groups = "audio_clk_a";
function = "audio_clk";
};
+
+ vin0_pins: vin0 {
+ groups = "vin0_data24", "vin0_sync", "vin0_clkenb", "vin0_clk";
+ function = "vin0";
+ };
};
ðer {
@@ -552,7 +568,34 @@
port@1 {
reg = <1>;
adv7511_out: endpoint {
- remote-endpoint = <&hdmi_con>;
+ remote-endpoint = <&hdmi_con_out>;
+ };
+ };
+ };
+ };
+
+ hdmi-in@4c {
+ compatible = "adi,adv7612";
+ reg = <0x4c>;
+ interrupt-parent = <&gpio4>;
+ interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
+ default-input = <0>;
+
+ port {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ adv7612_in: endpoint {
+ remote-endpoint = <&hdmi_con_in>;
+ };
+ };
+
+ port@2 {
+ reg = <2>;
+ adv7612_out: endpoint {
+ remote-endpoint = <&vin0ep2>;
};
};
};
@@ -606,3 +649,24 @@
&ssi1 {
shared-pin;
};
+
+/* HDMI video input */
+&vin0 {
+ status = "okay";
+ pinctrl-0 = <&vin0_pins>;
+ pinctrl-names = "default";
+
+ port {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ vin0ep2: endpoint {
+ remote-endpoint = <&adv7612_out>;
+ bus-width = <24>;
+ hsync-active = <0>;
+ vsync-active = <0>;
+ pclk-sample = <1>;
+ data-active = <1>;
+ };
+ };
+};
--
2.7.4
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings
2017-05-19 13:07 ` Ulrich Hecht
(?)
(?)
@ 2017-05-19 13:07 ` Ulrich Hecht
[not found] ` <1495199224-16337-3-git-send-email-ulrich.hecht+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
-1 siblings, 1 reply; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree, Ulrich Hecht
Used to differentiate between models with 3 and 6 inputs.
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
---
drivers/media/i2c/adv7180.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
index bdbbf8c..78de7dd 100644
--- a/drivers/media/i2c/adv7180.c
+++ b/drivers/media/i2c/adv7180.c
@@ -1452,6 +1452,8 @@ static SIMPLE_DEV_PM_OPS(adv7180_pm_ops, adv7180_suspend, adv7180_resume);
#ifdef CONFIG_OF
static const struct of_device_id adv7180_of_id[] = {
{ .compatible = "adi,adv7180", },
+ { .compatible = "adi,adv7180cp", },
+ { .compatible = "adi,adv7180st", },
{ .compatible = "adi,adv7182", },
{ .compatible = "adi,adv7280", },
{ .compatible = "adi,adv7280-m", },
--
2.7.4
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 3/4] media: adv7180: Add adv7180cp, adv7180st bindings
2017-05-19 13:07 ` Ulrich Hecht
` (2 preceding siblings ...)
(?)
@ 2017-05-19 13:07 ` Ulrich Hecht
2017-05-23 15:08 ` Rob Herring
-1 siblings, 1 reply; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree, Ulrich Hecht
To differentiate between two classes of chip packages that have
different numbers of input ports.
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
---
Documentation/devicetree/bindings/media/i2c/adv7180.txt | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/Documentation/devicetree/bindings/media/i2c/adv7180.txt b/Documentation/devicetree/bindings/media/i2c/adv7180.txt
index 4da486f..552b6a8 100644
--- a/Documentation/devicetree/bindings/media/i2c/adv7180.txt
+++ b/Documentation/devicetree/bindings/media/i2c/adv7180.txt
@@ -6,6 +6,8 @@ digital interfaces like MIPI CSI-2 or parallel video.
Required Properties :
- compatible : value must be one of
"adi,adv7180"
+ "adi,adv7180cp"
+ "adi,adv7180st"
"adi,adv7182"
"adi,adv7280"
"adi,adv7280-m"
@@ -15,6 +17,19 @@ Required Properties :
"adi,adv7282"
"adi,adv7282-m"
+Device nodes of "adi,adv7180cp" and "adi,adv7180st" must contain one
+'port' child node per device input and output port, in accordance with the
+video interface bindings defined in
+Documentation/devicetree/bindings/media/video-interfaces.txt. The port
+nodes are numbered as follows.
+
+ Port adv7180cp adv7180st
+-------------------------------------------------------------------
+ Input 0-2 0-5
+ Output 3 6
+
+The digital output port node must contain at least one endpoint.
+
Optional Properties :
- powerdown-gpios: reference to the GPIO connected to the powerdown pin,
if any.
--
2.7.4
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 4/4] ARM: dts: gose: add composite video input
2017-05-19 13:07 ` Ulrich Hecht
` (3 preceding siblings ...)
(?)
@ 2017-05-19 13:07 ` Ulrich Hecht
-1 siblings, 0 replies; 16+ messages in thread
From: Ulrich Hecht @ 2017-05-19 13:07 UTC (permalink / raw)
To: linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree, Ulrich Hecht
Adds VIN, decoder and connector.
Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
---
arch/arm/boot/dts/r8a7793-gose.dts | 59 ++++++++++++++++++++++++++++++++++++++
1 file changed, 59 insertions(+)
diff --git a/arch/arm/boot/dts/r8a7793-gose.dts b/arch/arm/boot/dts/r8a7793-gose.dts
index 30f0835..76e3aca 100644
--- a/arch/arm/boot/dts/r8a7793-gose.dts
+++ b/arch/arm/boot/dts/r8a7793-gose.dts
@@ -275,6 +275,16 @@
};
};
+ composite-in {
+ compatible = "composite-video-connector";
+
+ port {
+ composite_con_in: endpoint {
+ remote-endpoint = <&adv7180_in>;
+ };
+ };
+ };
+
x2_clk: x2-clock {
compatible = "fixed-clock";
#clock-cells = <0>;
@@ -411,6 +421,11 @@
groups = "vin0_data24", "vin0_sync", "vin0_clkenb", "vin0_clk";
function = "vin0";
};
+
+ vin1_pins: vin1 {
+ groups = "vin1_data8", "vin1_clk";
+ function = "vin1";
+ };
};
ðer {
@@ -542,6 +557,32 @@
reg = <0x12>;
};
+ composite-in@20 {
+ compatible = "adi,adv7180cp";
+ reg = <0x20>;
+ remote = <&vin1>;
+
+ port {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ adv7180_in: endpoint {
+ remote-endpoint = <&composite_con_in>;
+ };
+ };
+
+ port@3 {
+ reg = <3>;
+ adv7180_out: endpoint {
+ bus-width = <8>;
+ remote-endpoint = <&vin1ep>;
+ };
+ };
+ };
+ };
+
hdmi@39 {
compatible = "adi,adv7511w";
reg = <0x39>;
@@ -670,3 +711,21 @@
};
};
};
+
+/* composite video input */
+&vin1 {
+ pinctrl-0 = <&vin1_pins>;
+ pinctrl-names = "default";
+
+ status = "okay";
+
+ port {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ vin1ep: endpoint {
+ remote-endpoint = <&adv7180_out>;
+ bus-width = <8>;
+ };
+ };
+};
--
2.7.4
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/4] media: adv7180: Add adv7180cp, adv7180st bindings
2017-05-19 13:07 ` [PATCH v3 3/4] media: adv7180: Add adv7180cp, adv7180st bindings Ulrich Hecht
@ 2017-05-23 15:08 ` Rob Herring
0 siblings, 0 replies; 16+ messages in thread
From: Rob Herring @ 2017-05-23 15:08 UTC (permalink / raw)
To: Ulrich Hecht
Cc: linux-renesas-soc, laurent.pinchart, linux-media, geert,
magnus.damm, hans.verkuil, niklas.soderlund, sergei.shtylyov,
horms, devicetree
On Fri, May 19, 2017 at 03:07:03PM +0200, Ulrich Hecht wrote:
> To differentiate between two classes of chip packages that have
> different numbers of input ports.
>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
> ---
> Documentation/devicetree/bindings/media/i2c/adv7180.txt | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
2017-05-19 13:07 ` Ulrich Hecht
@ 2017-05-26 6:49 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2017-05-26 6:49 UTC (permalink / raw)
To: Ulrich Hecht
Cc: linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw,
linux-media-u79uwXL29TY76Z2rM5mHXA, geert-Td1EMuHUCqxL1ZNQvxDV9g,
magnus.damm-Re5JQEeQqe8AvxtiuMwx3w,
hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w,
niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw,
sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Fri, May 19, 2017 at 03:07:00PM +0200, Ulrich Hecht wrote:
> Hi!
>
> This is a by-the-datasheet implementation of analog and digital video input
> on the Gose board.
>
> This revision adds new bindings that distinguish between ADV7180 variants
> with three and six input ports. There are numerous variants of this chip,
> but since all that have "CP" in their names have three inputs, and all that
> have "ST" have six, I have limited myself to two new compatible strings,
> "adv7180cp" and "adv7180st".
>
> The digital input patch has received minor tweaks of the port names for
> consistency, and the Gose analog input patch has been modified to use the
> new bindings, and a composite video connector has been added.
>
> CU
> Uli
>
>
> Changes since v2:
> - hdmi: port hdmi_con renamed to hdmi_con_out
> - adv7180: added new compatibility strings and bindings
> - composite: added connector, use new bindings
>
> Changes since v1:
> - r8a7793.dtsi: added VIN2
> - modeled HDMI decoder input/output and connector
> - added "renesas,rcar-gen2-vin" compat strings
> - removed unnecessary "remote" node and aliases
> - set ADV7612 interrupt to GP4_2
>
>
> Ulrich Hecht (4):
> ARM: dts: gose: add HDMI input
I have queued-up the above patch for v4.13.
> media: adv7180: add adv7180cp, adv7180st compatible strings
> media: adv7180: Add adv7180cp, adv7180st bindings
> ARM: dts: gose: add composite video input
I have marked the above dts patch as "deferred" pending acceptance
of the binding. Please repost or otherwise ping me once that has happened.
> .../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
> arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
> drivers/media/i2c/adv7180.c | 2 +
> 3 files changed, 142 insertions(+), 2 deletions(-)
>
> --
> 2.7.4
>
--
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] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
@ 2017-05-26 6:49 ` Simon Horman
0 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2017-05-26 6:49 UTC (permalink / raw)
To: Ulrich Hecht
Cc: linux-renesas-soc, laurent.pinchart, linux-media, geert,
magnus.damm, hans.verkuil, niklas.soderlund, sergei.shtylyov,
devicetree
On Fri, May 19, 2017 at 03:07:00PM +0200, Ulrich Hecht wrote:
> Hi!
>
> This is a by-the-datasheet implementation of analog and digital video input
> on the Gose board.
>
> This revision adds new bindings that distinguish between ADV7180 variants
> with three and six input ports. There are numerous variants of this chip,
> but since all that have "CP" in their names have three inputs, and all that
> have "ST" have six, I have limited myself to two new compatible strings,
> "adv7180cp" and "adv7180st".
>
> The digital input patch has received minor tweaks of the port names for
> consistency, and the Gose analog input patch has been modified to use the
> new bindings, and a composite video connector has been added.
>
> CU
> Uli
>
>
> Changes since v2:
> - hdmi: port hdmi_con renamed to hdmi_con_out
> - adv7180: added new compatibility strings and bindings
> - composite: added connector, use new bindings
>
> Changes since v1:
> - r8a7793.dtsi: added VIN2
> - modeled HDMI decoder input/output and connector
> - added "renesas,rcar-gen2-vin" compat strings
> - removed unnecessary "remote" node and aliases
> - set ADV7612 interrupt to GP4_2
>
>
> Ulrich Hecht (4):
> ARM: dts: gose: add HDMI input
I have queued-up the above patch for v4.13.
> media: adv7180: add adv7180cp, adv7180st compatible strings
> media: adv7180: Add adv7180cp, adv7180st bindings
> ARM: dts: gose: add composite video input
I have marked the above dts patch as "deferred" pending acceptance
of the binding. Please repost or otherwise ping me once that has happened.
> .../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
> arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
> drivers/media/i2c/adv7180.c | 2 +
> 3 files changed, 142 insertions(+), 2 deletions(-)
>
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
2017-05-19 13:07 ` Ulrich Hecht
@ 2017-05-29 9:08 ` Hans Verkuil
-1 siblings, 0 replies; 16+ messages in thread
From: Hans Verkuil @ 2017-05-29 9:08 UTC (permalink / raw)
To: Ulrich Hecht, linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA, geert-Td1EMuHUCqxL1ZNQvxDV9g,
magnus.damm-Re5JQEeQqe8AvxtiuMwx3w,
hans.verkuil-FYB4Gu1CFyUAvxtiuMwx3w,
niklas.soderlund-1zkq55x86MTxsAP9Fp7wbw,
sergei.shtylyov-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8,
horms-/R6kz+dDXgpPR4JQBCEnsQ, devicetree-u79uwXL29TY76Z2rM5mHXA
On 05/19/2017 03:07 PM, Ulrich Hecht wrote:
> Hi!
>
> This is a by-the-datasheet implementation of analog and digital video input
> on the Gose board.
>
> This revision adds new bindings that distinguish between ADV7180 variants
> with three and six input ports. There are numerous variants of this chip,
> but since all that have "CP" in their names have three inputs, and all that
> have "ST" have six, I have limited myself to two new compatible strings,
> "adv7180cp" and "adv7180st".
>
> The digital input patch has received minor tweaks of the port names for
> consistency, and the Gose analog input patch has been modified to use the
> new bindings, and a composite video connector has been added.
Looks good. I assume the dts changes go through linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org?
I'll pick up the adv7180 changes.
Regards,
Hans
>
> CU
> Uli
>
>
> Changes since v2:
> - hdmi: port hdmi_con renamed to hdmi_con_out
> - adv7180: added new compatibility strings and bindings
> - composite: added connector, use new bindings
>
> Changes since v1:
> - r8a7793.dtsi: added VIN2
> - modeled HDMI decoder input/output and connector
> - added "renesas,rcar-gen2-vin" compat strings
> - removed unnecessary "remote" node and aliases
> - set ADV7612 interrupt to GP4_2
>
>
> Ulrich Hecht (4):
> ARM: dts: gose: add HDMI input
> media: adv7180: add adv7180cp, adv7180st compatible strings
> media: adv7180: Add adv7180cp, adv7180st bindings
> ARM: dts: gose: add composite video input
>
> .../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
> arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
> drivers/media/i2c/adv7180.c | 2 +
> 3 files changed, 142 insertions(+), 2 deletions(-)
>
--
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] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
@ 2017-05-29 9:08 ` Hans Verkuil
0 siblings, 0 replies; 16+ messages in thread
From: Hans Verkuil @ 2017-05-29 9:08 UTC (permalink / raw)
To: Ulrich Hecht, linux-renesas-soc, laurent.pinchart
Cc: linux-media, geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, horms, devicetree
On 05/19/2017 03:07 PM, Ulrich Hecht wrote:
> Hi!
>
> This is a by-the-datasheet implementation of analog and digital video input
> on the Gose board.
>
> This revision adds new bindings that distinguish between ADV7180 variants
> with three and six input ports. There are numerous variants of this chip,
> but since all that have "CP" in their names have three inputs, and all that
> have "ST" have six, I have limited myself to two new compatible strings,
> "adv7180cp" and "adv7180st".
>
> The digital input patch has received minor tweaks of the port names for
> consistency, and the Gose analog input patch has been modified to use the
> new bindings, and a composite video connector has been added.
Looks good. I assume the dts changes go through linux-renesas-soc@vger.kernel.org?
I'll pick up the adv7180 changes.
Regards,
Hans
>
> CU
> Uli
>
>
> Changes since v2:
> - hdmi: port hdmi_con renamed to hdmi_con_out
> - adv7180: added new compatibility strings and bindings
> - composite: added connector, use new bindings
>
> Changes since v1:
> - r8a7793.dtsi: added VIN2
> - modeled HDMI decoder input/output and connector
> - added "renesas,rcar-gen2-vin" compat strings
> - removed unnecessary "remote" node and aliases
> - set ADV7612 interrupt to GP4_2
>
>
> Ulrich Hecht (4):
> ARM: dts: gose: add HDMI input
> media: adv7180: add adv7180cp, adv7180st compatible strings
> media: adv7180: Add adv7180cp, adv7180st bindings
> ARM: dts: gose: add composite video input
>
> .../devicetree/bindings/media/i2c/adv7180.txt | 15 +++
> arch/arm/boot/dts/r8a7793-gose.dts | 127 ++++++++++++++++++++-
> drivers/media/i2c/adv7180.c | 2 +
> 3 files changed, 142 insertions(+), 2 deletions(-)
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
2017-05-29 9:08 ` Hans Verkuil
(?)
@ 2017-05-30 8:44 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2017-05-30 8:44 UTC (permalink / raw)
To: Hans Verkuil
Cc: Ulrich Hecht, linux-renesas-soc, laurent.pinchart, linux-media,
geert, magnus.damm, hans.verkuil, niklas.soderlund,
sergei.shtylyov, devicetree
On Mon, May 29, 2017 at 11:08:12AM +0200, Hans Verkuil wrote:
> On 05/19/2017 03:07 PM, Ulrich Hecht wrote:
> >Hi!
> >
> >This is a by-the-datasheet implementation of analog and digital video input
> >on the Gose board.
> >
> >This revision adds new bindings that distinguish between ADV7180 variants
> >with three and six input ports. There are numerous variants of this chip,
> >but since all that have "CP" in their names have three inputs, and all that
> >have "ST" have six, I have limited myself to two new compatible strings,
> >"adv7180cp" and "adv7180st".
> >
> >The digital input patch has received minor tweaks of the port names for
> >consistency, and the Gose analog input patch has been modified to use the
> >new bindings, and a composite video connector has been added.
>
> Looks good. I assume the dts changes go through linux-renesas-soc@vger.kernel.org?
Yes, I will pick up the dts changes.
> I'll pick up the adv7180 changes.
Thanks!
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 0/4] r8a7793 Gose video input support
2017-05-26 6:49 ` Simon Horman
(?)
@ 2017-05-30 8:45 ` Simon Horman
-1 siblings, 0 replies; 16+ messages in thread
From: Simon Horman @ 2017-05-30 8:45 UTC (permalink / raw)
To: Ulrich Hecht
Cc: linux-renesas-soc, laurent.pinchart, linux-media, geert,
magnus.damm, hans.verkuil, niklas.soderlund, sergei.shtylyov,
devicetree
On Fri, May 26, 2017 at 08:49:07AM +0200, Simon Horman wrote:
> On Fri, May 19, 2017 at 03:07:00PM +0200, Ulrich Hecht wrote:
> > Hi!
> >
> > This is a by-the-datasheet implementation of analog and digital video input
> > on the Gose board.
> >
> > This revision adds new bindings that distinguish between ADV7180 variants
> > with three and six input ports. There are numerous variants of this chip,
> > but since all that have "CP" in their names have three inputs, and all that
> > have "ST" have six, I have limited myself to two new compatible strings,
> > "adv7180cp" and "adv7180st".
> >
> > The digital input patch has received minor tweaks of the port names for
> > consistency, and the Gose analog input patch has been modified to use the
> > new bindings, and a composite video connector has been added.
> >
> > CU
> > Uli
> >
> >
> > Changes since v2:
> > - hdmi: port hdmi_con renamed to hdmi_con_out
> > - adv7180: added new compatibility strings and bindings
> > - composite: added connector, use new bindings
> >
> > Changes since v1:
> > - r8a7793.dtsi: added VIN2
> > - modeled HDMI decoder input/output and connector
> > - added "renesas,rcar-gen2-vin" compat strings
> > - removed unnecessary "remote" node and aliases
> > - set ADV7612 interrupt to GP4_2
> >
> >
> > Ulrich Hecht (4):
> > ARM: dts: gose: add HDMI input
>
> I have queued-up the above patch for v4.13.
>
> > media: adv7180: add adv7180cp, adv7180st compatible strings
> > media: adv7180: Add adv7180cp, adv7180st bindings
> > ARM: dts: gose: add composite video input
>
> I have marked the above dts patch as "deferred" pending acceptance
> of the binding. Please repost or otherwise ping me once that has happened.
It looks like Hans will pick up the driver patches.
Accordingly I have queued-up the last dts patch above.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings
2017-05-19 13:07 ` [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings Ulrich Hecht
@ 2017-06-30 9:19 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2017-06-30 9:19 UTC (permalink / raw)
To: Ulrich Hecht
Cc: Linux-Renesas, Laurent Pinchart, Linux Media Mailing List,
Magnus Damm, Hans Verkuil, Niklas Söderlund,
Sergei Shtylyov, Simon Horman, devicetree-u79uwXL29TY76Z2rM5mHXA
Hi Ulrich,
On Fri, May 19, 2017 at 3:07 PM, Ulrich Hecht
<ulrich.hecht+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> Used to differentiate between models with 3 and 6 inputs.
>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> drivers/media/i2c/adv7180.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
> index bdbbf8c..78de7dd 100644
> --- a/drivers/media/i2c/adv7180.c
> +++ b/drivers/media/i2c/adv7180.c
> @@ -1452,6 +1452,8 @@ static SIMPLE_DEV_PM_OPS(adv7180_pm_ops, adv7180_suspend, adv7180_resume);
> #ifdef CONFIG_OF
> static const struct of_device_id adv7180_of_id[] = {
> { .compatible = "adi,adv7180", },
> + { .compatible = "adi,adv7180cp", },
> + { .compatible = "adi,adv7180st", },
> { .compatible = "adi,adv7182", },
> { .compatible = "adi,adv7280", },
> { .compatible = "adi,adv7280-m", },
Adding compatible entries here is not sufficient, and causes a crash on
r8a7793/gose with renesas-drivers-2017-06-27-v4.12-rc7:
adv7180 2-0020: chip found @ 0x20 (e6530000.i2c)
Unable to handle kernel NULL pointer dereference at virtual address 00000014
pgd = c0003000
[00000014] *pgd=80000040004003, *pmd=00000000
Internal error: Oops: 207 [#1] SMP ARM
Modules linked in:
CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.12.0-rc7-rcar2-initrd #37
Hardware name: Generic R-Car Gen2 (Flattened Device Tree)
task: df427040 task.stack: df436000
PC is at adv7180_probe+0x84/0x3cc
In the absence of an entry in adv7180_id[], the passed i2c_device_id
pointer is NULL, and thus dereferencing it crashes the system.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
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] 16+ messages in thread
* Re: [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings
@ 2017-06-30 9:19 ` Geert Uytterhoeven
0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2017-06-30 9:19 UTC (permalink / raw)
To: Ulrich Hecht
Cc: Linux-Renesas, Laurent Pinchart, Linux Media Mailing List,
Magnus Damm, Hans Verkuil, Niklas Söderlund,
Sergei Shtylyov, Simon Horman, devicetree
Hi Ulrich,
On Fri, May 19, 2017 at 3:07 PM, Ulrich Hecht
<ulrich.hecht+renesas@gmail.com> wrote:
> Used to differentiate between models with 3 and 6 inputs.
>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
> ---
> drivers/media/i2c/adv7180.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
> index bdbbf8c..78de7dd 100644
> --- a/drivers/media/i2c/adv7180.c
> +++ b/drivers/media/i2c/adv7180.c
> @@ -1452,6 +1452,8 @@ static SIMPLE_DEV_PM_OPS(adv7180_pm_ops, adv7180_suspend, adv7180_resume);
> #ifdef CONFIG_OF
> static const struct of_device_id adv7180_of_id[] = {
> { .compatible = "adi,adv7180", },
> + { .compatible = "adi,adv7180cp", },
> + { .compatible = "adi,adv7180st", },
> { .compatible = "adi,adv7182", },
> { .compatible = "adi,adv7280", },
> { .compatible = "adi,adv7280-m", },
Adding compatible entries here is not sufficient, and causes a crash on
r8a7793/gose with renesas-drivers-2017-06-27-v4.12-rc7:
adv7180 2-0020: chip found @ 0x20 (e6530000.i2c)
Unable to handle kernel NULL pointer dereference at virtual address 00000014
pgd = c0003000
[00000014] *pgd=80000040004003, *pmd=00000000
Internal error: Oops: 207 [#1] SMP ARM
Modules linked in:
CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.12.0-rc7-rcar2-initrd #37
Hardware name: Generic R-Car Gen2 (Flattened Device Tree)
task: df427040 task.stack: df436000
PC is at adv7180_probe+0x84/0x3cc
In the absence of an entry in adv7180_id[], the passed i2c_device_id
pointer is NULL, and thus dereferencing it crashes the system.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings
2017-06-30 9:19 ` Geert Uytterhoeven
(?)
@ 2017-07-03 8:44 ` Ulrich Hecht
-1 siblings, 0 replies; 16+ messages in thread
From: Ulrich Hecht @ 2017-07-03 8:44 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Linux-Renesas, Laurent Pinchart, Linux Media Mailing List,
Magnus Damm, Hans Verkuil, Niklas Söderlund,
Sergei Shtylyov, Simon Horman, devicetree
On Fri, Jun 30, 2017 at 11:19 AM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Ulrich,
>
> On Fri, May 19, 2017 at 3:07 PM, Ulrich Hecht
> <ulrich.hecht+renesas@gmail.com> wrote:
>> Used to differentiate between models with 3 and 6 inputs.
>>
>> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@gmail.com>
>> ---
>> drivers/media/i2c/adv7180.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
>> index bdbbf8c..78de7dd 100644
>> --- a/drivers/media/i2c/adv7180.c
>> +++ b/drivers/media/i2c/adv7180.c
>> @@ -1452,6 +1452,8 @@ static SIMPLE_DEV_PM_OPS(adv7180_pm_ops, adv7180_suspend, adv7180_resume);
>> #ifdef CONFIG_OF
>> static const struct of_device_id adv7180_of_id[] = {
>> { .compatible = "adi,adv7180", },
>> + { .compatible = "adi,adv7180cp", },
>> + { .compatible = "adi,adv7180st", },
>> { .compatible = "adi,adv7182", },
>> { .compatible = "adi,adv7280", },
>> { .compatible = "adi,adv7280-m", },
>
> Adding compatible entries here is not sufficient, and causes a crash on
> r8a7793/gose with renesas-drivers-2017-06-27-v4.12-rc7:
>
> adv7180 2-0020: chip found @ 0x20 (e6530000.i2c)
> Unable to handle kernel NULL pointer dereference at virtual address 00000014
> pgd = c0003000
> [00000014] *pgd=80000040004003, *pmd=00000000
> Internal error: Oops: 207 [#1] SMP ARM
> Modules linked in:
> CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.12.0-rc7-rcar2-initrd #37
> Hardware name: Generic R-Car Gen2 (Flattened Device Tree)
> task: df427040 task.stack: df436000
> PC is at adv7180_probe+0x84/0x3cc
>
> In the absence of an entry in adv7180_id[], the passed i2c_device_id
> pointer is NULL, and thus dereferencing it crashes the system.
Thank you for testing this. I have sent a fix (" media: adv7180: add
missing adv7180cp, adv7180st i2c device IDs"), could you please check
if it unbreaks things for you?
CU
Uli
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2017-07-03 8:44 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-19 13:07 [PATCH v3 0/4] r8a7793 Gose video input support Ulrich Hecht
2017-05-19 13:07 ` Ulrich Hecht
2017-05-19 13:07 ` [PATCH v3 1/4] ARM: dts: gose: add HDMI input Ulrich Hecht
2017-05-19 13:07 ` [PATCH v3 2/4] media: adv7180: add adv7180cp, adv7180st compatible strings Ulrich Hecht
[not found] ` <1495199224-16337-3-git-send-email-ulrich.hecht+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-06-30 9:19 ` Geert Uytterhoeven
2017-06-30 9:19 ` Geert Uytterhoeven
2017-07-03 8:44 ` Ulrich Hecht
2017-05-19 13:07 ` [PATCH v3 3/4] media: adv7180: Add adv7180cp, adv7180st bindings Ulrich Hecht
2017-05-23 15:08 ` Rob Herring
2017-05-19 13:07 ` [PATCH v3 4/4] ARM: dts: gose: add composite video input Ulrich Hecht
[not found] ` <1495199224-16337-1-git-send-email-ulrich.hecht+renesas-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-05-26 6:49 ` [PATCH v3 0/4] r8a7793 Gose video input support Simon Horman
2017-05-26 6:49 ` Simon Horman
2017-05-30 8:45 ` Simon Horman
2017-05-29 9:08 ` Hans Verkuil
2017-05-29 9:08 ` Hans Verkuil
2017-05-30 8:44 ` Simon Horman
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.