Hello.
On 01/29/2015 07:19 PM, William Towle wrote:
No signed off? Although, looking at the patch, I'm not very surprised...
---
arch/arm/boot/dts/r8a7790-lager.dts | 51 +++
1 file changed, 34 insertions(+), 17 deletions(-)
diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
b/arch/arm/boot/dts/r8a7790-lager.dts
index be44493..c20b6cb 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -249,9 +249,9 @@
renesas,function = "usb2";
};
- vin1_pins: vin {
- renesas,groups = "vin1_data8", "vin1_clk";
- renesas,function = "vin1";
I'm not sure why are you removing the VIN1 pins while adding VIN0 pins.
+ vin0_pins: vin0 {
+ renesas,groups = "vin0_data24", "vin0_sync", "vin0_field", "vin0_clkenb",
"vin0_clk";
+ renesas,function = "vin0";
};
};
@@ -391,15 +391,15 @@
pinctrl-0 = <&iic2_pins>;
pinctrl-names = "default";
- composite-in@20 {
- compatible = "adi,adv7180";
- reg = <0x20>;
- remote = <&vin1>;
Why remove it?
+ adv7612: adv7612@0x4c {
We don't call the nodes with chip names. According to the ePAPR standard
section 2.2.2, node names should reflect the general function of the device.
And drop "0x" prefix from prt of the name please.
+ compatible = "adi,adv7612";
+ reg = <0x4c>;
+ remote = <&vin0>;
port {
- adv7180: endpoint {
- bus-width = <8>;
- remote-endpoint = <&vin1ep0>;
+ adv7612_1: endpoint {
+ remote-endpoint = <&vin0ep0>;
+ default-input = <0>;
};
};
};
@@ -419,6 +419,19 @@
i2c-gpio,delay-us = <1>; /* ~100 kHz */
#address-cells = <1>;
#size-cells = <0>;
+
+ adv7612: adv7612@0x4c {
Two identical labels? Do we really have 2 ADV7162 chips on the board?
And the same comments on the node name...
+ compatible = "adi,adv7612";
+ reg = <0x4c>;
+ remote = <&vin0>;
+
+ port {
+ adv7612_1: endpoint {
Two identical labels?
+ remote-endpoint = <&vin0ep0>;
+ default-input = <0>;
+ };
+ };
+ };
};
};
#endif
@@ -457,9 +470,9 @@
pinctrl-names = "default";
};
-/* composite video input */
-&vin1 {
- pinctrl-0 = <&vin1_pins>;
Why remove it?
+/* HDMI video input */
+&vin0 {
+ pinctrl-0 = <&vin0_pins>;
pinctrl-names = "default";
status = "ok";
@@ -468,9 +481,13 @@
#address-cells = <1>;
#size-cells = <0>;
- vin1ep0: endpoint {
- remote-endpoint = <&adv7180>;
- bus-width = <8>;
+ vin0ep0: endpoint {
+ remote-endpoint = <&adv7612_1>;
+ bus-width = <24>;
+ hsync-active = <0>;
+ vsync-active = <0>;
+ pclk-sample = <1>;
+ data-active = <1>;
};
};
[...]
WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html