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 <unit-address> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
