* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
The following series adds support for v4l2 display on da850-evm with
a UI board in device tree boot mode.
Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
errors in dts files and bindings, extend the example and the dts for
da850-evm with the output port and address the pinmuxing.
Patch 6/10 enables the relevant modules in the defconfig file.
Patches 7/10 and 8/10 fix two already existing bugs encountered
during development.
Patch 9/10 make it possible to use a different i2c adapter in the
vpif display driver.
The last patch adds the pdata quirks necessary to enable v4l2 display.
Tested with a modified version of yavta[1] as gstreamer support for
v4l2 seems to be broken and results in picture artifacts.
[1] https://github.com/brgl/yavta davinci/vpif-display
Bartosz Golaszewski (10):
media: dt-bindings: vpif: fix whitespace errors
ARM: dts: da850-evm: fix whitespace errors
media: dt-bindings: vpif: extend the example with an output port
ARM: dts: da850-evm: add the output port to the vpif node
ARM: dts: da850: add vpif video display pins
ARM: davinci_all_defconfig: enable VPIF display modules
ARM: davinci: fix a whitespace error
ARM: davinci: fix the DT boot on da850-evm
media: vpif: use a configurable i2c_adapter_id for vpif display
ARM: davinci: add pdata-quirks for da850-evm vpif display
.../devicetree/bindings/media/ti,da850-vpif.txt | 45 ++++++++---
arch/arm/boot/dts/da850-evm.dts | 26 +++---
arch/arm/boot/dts/da850.dtsi | 25 +++++-
arch/arm/configs/davinci_all_defconfig | 2 +
arch/arm/mach-davinci/board-da850-evm.c | 1 +
arch/arm/mach-davinci/pdata-quirks.c | 92 ++++++++++++++++++++--
drivers/media/platform/davinci/vpif_display.c | 2 +-
include/media/davinci/vpif_types.h | 1 +
8 files changed, 164 insertions(+), 30 deletions(-)
--
2.9.3
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
The following series adds support for v4l2 display on da850-evm with
a UI board in device tree boot mode.
Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
errors in dts files and bindings, extend the example and the dts for
da850-evm with the output port and address the pinmuxing.
Patch 6/10 enables the relevant modules in the defconfig file.
Patches 7/10 and 8/10 fix two already existing bugs encountered
during development.
Patch 9/10 make it possible to use a different i2c adapter in the
vpif display driver.
The last patch adds the pdata quirks necessary to enable v4l2 display.
Tested with a modified version of yavta[1] as gstreamer support for
v4l2 seems to be broken and results in picture artifacts.
[1] https://github.com/brgl/yavta davinci/vpif-display
Bartosz Golaszewski (10):
media: dt-bindings: vpif: fix whitespace errors
ARM: dts: da850-evm: fix whitespace errors
media: dt-bindings: vpif: extend the example with an output port
ARM: dts: da850-evm: add the output port to the vpif node
ARM: dts: da850: add vpif video display pins
ARM: davinci_all_defconfig: enable VPIF display modules
ARM: davinci: fix a whitespace error
ARM: davinci: fix the DT boot on da850-evm
media: vpif: use a configurable i2c_adapter_id for vpif display
ARM: davinci: add pdata-quirks for da850-evm vpif display
.../devicetree/bindings/media/ti,da850-vpif.txt | 45 ++++++++---
arch/arm/boot/dts/da850-evm.dts | 26 +++---
arch/arm/boot/dts/da850.dtsi | 25 +++++-
arch/arm/configs/davinci_all_defconfig | 2 +
arch/arm/mach-davinci/board-da850-evm.c | 1 +
arch/arm/mach-davinci/pdata-quirks.c | 92 ++++++++++++++++++++--
drivers/media/platform/davinci/vpif_display.c | 2 +-
include/media/davinci/vpif_types.h | 1 +
8 files changed, 164 insertions(+), 30 deletions(-)
--
2.9.3
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 6d25d7f..9c7510b 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -30,15 +30,15 @@ I2C-connected TVP5147 decoder:
port {
vpif_ch0: endpoint@0 {
- reg = <0>;
- bus-width = <8>;
- remote-endpoint = <&composite>;
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite>;
};
vpif_ch1: endpoint@1 {
- reg = <1>;
- bus-width = <8>;
- data-shift = <8>;
+ reg = <1>;
+ bus-width = <8>;
+ data-shift = <8>;
};
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA, Bartosz Golaszewski
Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 6d25d7f..9c7510b 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -30,15 +30,15 @@ I2C-connected TVP5147 decoder:
port {
vpif_ch0: endpoint@0 {
- reg = <0>;
- bus-width = <8>;
- remote-endpoint = <&composite>;
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite>;
};
vpif_ch1: endpoint@1 {
- reg = <1>;
- bus-width = <8>;
- data-shift = <8>;
+ reg = <1>;
+ bus-width = <8>;
+ data-shift = <8>;
};
};
};
--
2.9.3
--
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 related [flat|nested] 91+ messages in thread
* [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 6d25d7f..9c7510b 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -30,15 +30,15 @@ I2C-connected TVP5147 decoder:
port {
vpif_ch0: endpoint at 0 {
- reg = <0>;
- bus-width = <8>;
- remote-endpoint = <&composite>;
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite>;
};
vpif_ch1: endpoint at 1 {
- reg = <1>;
- bus-width = <8>;
- data-shift = <8>;
+ reg = <1>;
+ bus-width = <8>;
+ data-shift = <8>;
};
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 16:41 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index c970b6e..94938a3 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -301,14 +301,14 @@
/* VPIF capture port */
port {
vpif_ch0: endpoint@0 {
- reg = <0>;
- bus-width = <8>;
+ reg = <0>;
+ bus-width = <8>;
};
vpif_ch1: endpoint@1 {
- reg = <1>;
- bus-width = <8>;
- data-shift = <8>;
+ reg = <1>;
+ bus-width = <8>;
+ data-shift = <8>;
};
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index c970b6e..94938a3 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -301,14 +301,14 @@
/* VPIF capture port */
port {
vpif_ch0: endpoint at 0 {
- reg = <0>;
- bus-width = <8>;
+ reg = <0>;
+ bus-width = <8>;
};
vpif_ch1: endpoint at 1 {
- reg = <1>;
- bus-width = <8>;
- data-shift = <8>;
+ reg = <1>;
+ bus-width = <8>;
+ data-shift = <8>;
};
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
This makes the example more or less correspond with the da850-evm
hardware setup.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
.../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
1 file changed, 29 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 9c7510b..543f6f3 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
reg = <0x217000 0x1000>;
interrupts = <92>;
- port {
- vpif_ch0: endpoint@0 {
+ port@0 {
+ vpif_input_ch0: endpoint@0 {
reg = <0>;
bus-width = <8>;
- remote-endpoint = <&composite>;
+ remote-endpoint = <&composite_in>;
};
- vpif_ch1: endpoint@1 {
+ vpif_input_ch1: endpoint@1 {
reg = <1>;
bus-width = <8>;
data-shift = <8>;
};
};
+
+ port@1 {
+ vpif_output_ch0: endpoint@0 {
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite_out>;
+ };
+ };
};
[ ... ]
@@ -53,13 +61,28 @@ I2C-connected TVP5147 decoder:
status = "okay";
port {
- composite: endpoint {
+ composite_in: endpoint {
hsync-active = <1>;
vsync-active = <1>;
pclk-sample = <0>;
/* VPIF channel 0 (lower 8-bits) */
- remote-endpoint = <&vpif_ch0>;
+ remote-endpoint = <&vpif_input_ch0>;
+ bus-width = <8>;
+ };
+ };
+ };
+
+ adv7343@2a {
+ compatible = "adi,adv7343";
+ reg = <0x2a>;
+
+ port {
+ composite_out: endpoint {
+ adi,dac-enable = <1 1 1>;
+ adi,sd-dac-enable = <1>;
+
+ remote-endpoint = <&vpif_output_ch0>;
bus-width = <8>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA, Bartosz Golaszewski
This makes the example more or less correspond with the da850-evm
hardware setup.
Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
.../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
1 file changed, 29 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 9c7510b..543f6f3 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
reg = <0x217000 0x1000>;
interrupts = <92>;
- port {
- vpif_ch0: endpoint@0 {
+ port@0 {
+ vpif_input_ch0: endpoint@0 {
reg = <0>;
bus-width = <8>;
- remote-endpoint = <&composite>;
+ remote-endpoint = <&composite_in>;
};
- vpif_ch1: endpoint@1 {
+ vpif_input_ch1: endpoint@1 {
reg = <1>;
bus-width = <8>;
data-shift = <8>;
};
};
+
+ port@1 {
+ vpif_output_ch0: endpoint@0 {
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite_out>;
+ };
+ };
};
[ ... ]
@@ -53,13 +61,28 @@ I2C-connected TVP5147 decoder:
status = "okay";
port {
- composite: endpoint {
+ composite_in: endpoint {
hsync-active = <1>;
vsync-active = <1>;
pclk-sample = <0>;
/* VPIF channel 0 (lower 8-bits) */
- remote-endpoint = <&vpif_ch0>;
+ remote-endpoint = <&vpif_input_ch0>;
+ bus-width = <8>;
+ };
+ };
+ };
+
+ adv7343@2a {
+ compatible = "adi,adv7343";
+ reg = <0x2a>;
+
+ port {
+ composite_out: endpoint {
+ adi,dac-enable = <1 1 1>;
+ adi,sd-dac-enable = <1>;
+
+ remote-endpoint = <&vpif_output_ch0>;
bus-width = <8>;
};
};
--
2.9.3
--
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 related [flat|nested] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
This makes the example more or less correspond with the da850-evm
hardware setup.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
.../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
1 file changed, 29 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
index 9c7510b..543f6f3 100644
--- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
+++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
@@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
reg = <0x217000 0x1000>;
interrupts = <92>;
- port {
- vpif_ch0: endpoint at 0 {
+ port at 0 {
+ vpif_input_ch0: endpoint at 0 {
reg = <0>;
bus-width = <8>;
- remote-endpoint = <&composite>;
+ remote-endpoint = <&composite_in>;
};
- vpif_ch1: endpoint at 1 {
+ vpif_input_ch1: endpoint at 1 {
reg = <1>;
bus-width = <8>;
data-shift = <8>;
};
};
+
+ port at 1 {
+ vpif_output_ch0: endpoint at 0 {
+ reg = <0>;
+ bus-width = <8>;
+ remote-endpoint = <&composite_out>;
+ };
+ };
};
[ ... ]
@@ -53,13 +61,28 @@ I2C-connected TVP5147 decoder:
status = "okay";
port {
- composite: endpoint {
+ composite_in: endpoint {
hsync-active = <1>;
vsync-active = <1>;
pclk-sample = <0>;
/* VPIF channel 0 (lower 8-bits) */
- remote-endpoint = <&vpif_ch0>;
+ remote-endpoint = <&vpif_input_ch0>;
+ bus-width = <8>;
+ };
+ };
+ };
+
+ adv7343 at 2a {
+ compatible = "adi,adv7343";
+ reg = <0x2a>;
+
+ port {
+ composite_out: endpoint {
+ adi,dac-enable = <1 1 1>;
+ adi,sd-dac-enable = <1>;
+
+ remote-endpoint = <&vpif_output_ch0>;
bus-width = <8>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 04/10] ARM: dts: da850-evm: add the output port to the vpif node
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 16:41 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Extend the vpif node with an output port with a single channel.
NOTE: this is still just hardware description - the actual driver
is registered using pdata-quirks.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 14 +++++++++++---
arch/arm/boot/dts/da850.dtsi | 8 +++++++-
2 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 94938a3..3d6dd66 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -299,16 +299,24 @@
status = "okay";
/* VPIF capture port */
- port {
- vpif_ch0: endpoint@0 {
+ port@0 {
+ vpif_input_ch0: endpoint@0 {
reg = <0>;
bus-width = <8>;
};
- vpif_ch1: endpoint@1 {
+ vpif_input_ch1: endpoint@1 {
reg = <1>;
bus-width = <8>;
data-shift = <8>;
};
};
+
+ /* VPIF display port */
+ port@1 {
+ vpif_output_ch0: endpoint@0 {
+ reg = <0>;
+ bus-width = <8>;
+ };
+ };
};
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 69ec5e7..768a58c 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -494,7 +494,13 @@
status = "disabled";
/* VPIF capture port */
- port {
+ port@0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ };
+
+ /* VPIF display port */
+ port@1 {
#address-cells = <1>;
#size-cells = <0>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 04/10] ARM: dts: da850-evm: add the output port to the vpif node
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Extend the vpif node with an output port with a single channel.
NOTE: this is still just hardware description - the actual driver
is registered using pdata-quirks.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 14 +++++++++++---
arch/arm/boot/dts/da850.dtsi | 8 +++++++-
2 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 94938a3..3d6dd66 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -299,16 +299,24 @@
status = "okay";
/* VPIF capture port */
- port {
- vpif_ch0: endpoint at 0 {
+ port at 0 {
+ vpif_input_ch0: endpoint at 0 {
reg = <0>;
bus-width = <8>;
};
- vpif_ch1: endpoint at 1 {
+ vpif_input_ch1: endpoint at 1 {
reg = <1>;
bus-width = <8>;
data-shift = <8>;
};
};
+
+ /* VPIF display port */
+ port at 1 {
+ vpif_output_ch0: endpoint at 0 {
+ reg = <0>;
+ bus-width = <8>;
+ };
+ };
};
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 69ec5e7..768a58c 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -494,7 +494,13 @@
status = "disabled";
/* VPIF capture port */
- port {
+ port at 0 {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ };
+
+ /* VPIF display port */
+ port at 1 {
#address-cells = <1>;
#size-cells = <0>;
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 05/10] ARM: dts: da850: add vpif video display pins
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Add a new pinctrl sub-node for vpif display pins. Move VP_CLKIN3 and
VP_CLKIN2 to the display node where they actually belong (vide section
35.2.2 of the da850 datasheet).
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 2 +-
arch/arm/boot/dts/da850.dtsi | 17 +++++++++++++++--
2 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 3d6dd66..6db16ba 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -295,7 +295,7 @@
&vpif {
pinctrl-names = "default";
- pinctrl-0 = <&vpif_capture_pins>;
+ pinctrl-0 = <&vpif_capture_pins>, <&vpif_display_pins>;
status = "okay";
/* VPIF capture port */
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 768a58c..d317bc5 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -218,8 +218,21 @@
0x3c 0x11111111 0xffffffff
/* VP_DIN[8..9] */
0x40 0x00000011 0x000000ff
- /* VP_CLKIN3, VP_CLKIN2 */
- 0x4c 0x00010100 0x000f0f00
+ >;
+ };
+ vpif_display_pins: vpif_display_pins {
+ pinctrl-single,bits = <
+ /* VP_DOUT[2..7] */
+ 0x40 0x11111100 0xffffff00
+ /* VP_DOUT[10..15,0..1] */
+ 0x44 0x11111111 0xffffffff
+ /* VP_DOUT[8..9] */
+ 0x48 0x00000011 0x000000ff
+ /*
+ * VP_CLKOUT3, VP_CLKIN3,
+ * VP_CLKOUT2, VP_CLKIN2
+ */
+ 0x4c 0x00111100 0x00ffff00
>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 05/10] ARM: dts: da850: add vpif video display pins
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA, Bartosz Golaszewski
Add a new pinctrl sub-node for vpif display pins. Move VP_CLKIN3 and
VP_CLKIN2 to the display node where they actually belong (vide section
35.2.2 of the da850 datasheet).
Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
arch/arm/boot/dts/da850-evm.dts | 2 +-
arch/arm/boot/dts/da850.dtsi | 17 +++++++++++++++--
2 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 3d6dd66..6db16ba 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -295,7 +295,7 @@
&vpif {
pinctrl-names = "default";
- pinctrl-0 = <&vpif_capture_pins>;
+ pinctrl-0 = <&vpif_capture_pins>, <&vpif_display_pins>;
status = "okay";
/* VPIF capture port */
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 768a58c..d317bc5 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -218,8 +218,21 @@
0x3c 0x11111111 0xffffffff
/* VP_DIN[8..9] */
0x40 0x00000011 0x000000ff
- /* VP_CLKIN3, VP_CLKIN2 */
- 0x4c 0x00010100 0x000f0f00
+ >;
+ };
+ vpif_display_pins: vpif_display_pins {
+ pinctrl-single,bits = <
+ /* VP_DOUT[2..7] */
+ 0x40 0x11111100 0xffffff00
+ /* VP_DOUT[10..15,0..1] */
+ 0x44 0x11111111 0xffffffff
+ /* VP_DOUT[8..9] */
+ 0x48 0x00000011 0x000000ff
+ /*
+ * VP_CLKOUT3, VP_CLKIN3,
+ * VP_CLKOUT2, VP_CLKIN2
+ */
+ 0x4c 0x00111100 0x00ffff00
>;
};
};
--
2.9.3
--
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 related [flat|nested] 91+ messages in thread
* [PATCH 05/10] ARM: dts: da850: add vpif video display pins
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Add a new pinctrl sub-node for vpif display pins. Move VP_CLKIN3 and
VP_CLKIN2 to the display node where they actually belong (vide section
35.2.2 of the da850 datasheet).
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/boot/dts/da850-evm.dts | 2 +-
arch/arm/boot/dts/da850.dtsi | 17 +++++++++++++++--
2 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index 3d6dd66..6db16ba 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -295,7 +295,7 @@
&vpif {
pinctrl-names = "default";
- pinctrl-0 = <&vpif_capture_pins>;
+ pinctrl-0 = <&vpif_capture_pins>, <&vpif_display_pins>;
status = "okay";
/* VPIF capture port */
diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 768a58c..d317bc5 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -218,8 +218,21 @@
0x3c 0x11111111 0xffffffff
/* VP_DIN[8..9] */
0x40 0x00000011 0x000000ff
- /* VP_CLKIN3, VP_CLKIN2 */
- 0x4c 0x00010100 0x000f0f00
+ >;
+ };
+ vpif_display_pins: vpif_display_pins {
+ pinctrl-single,bits = <
+ /* VP_DOUT[2..7] */
+ 0x40 0x11111100 0xffffff00
+ /* VP_DOUT[10..15,0..1] */
+ 0x44 0x11111111 0xffffffff
+ /* VP_DOUT[8..9] */
+ 0x48 0x00000011 0x000000ff
+ /*
+ * VP_CLKOUT3, VP_CLKIN3,
+ * VP_CLKOUT2, VP_CLKIN2
+ */
+ 0x4c 0x00111100 0x00ffff00
>;
};
};
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 06/10] ARM: davinci_all_defconfig: enable VPIF display modules
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Enable the VPIF display module and the video encoder present on the
da850-evm UI board.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/configs/davinci_all_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/davinci_all_defconfig b/arch/arm/configs/davinci_all_defconfig
index 7815a52..e188b02 100644
--- a/arch/arm/configs/davinci_all_defconfig
+++ b/arch/arm/configs/davinci_all_defconfig
@@ -137,9 +137,11 @@ CONFIG_DRM_DUMB_VGA_DAC=m
CONFIG_MEDIA_SUPPORT=m
CONFIG_MEDIA_CAMERA_SUPPORT=y
CONFIG_V4L_PLATFORM_DRIVERS=y
+CONFIG_VIDEO_DAVINCI_VPIF_DISPLAY=m
CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE=m
# CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
CONFIG_VIDEO_TVP514X=m
+CONFIG_VIDEO_ADV7343=m
CONFIG_FB=y
CONFIG_FIRMWARE_EDID=y
CONFIG_FB_DA8XX=y
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 06/10] ARM: davinci_all_defconfig: enable VPIF display modules
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA, Bartosz Golaszewski
Enable the VPIF display module and the video encoder present on the
da850-evm UI board.
Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
arch/arm/configs/davinci_all_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/davinci_all_defconfig b/arch/arm/configs/davinci_all_defconfig
index 7815a52..e188b02 100644
--- a/arch/arm/configs/davinci_all_defconfig
+++ b/arch/arm/configs/davinci_all_defconfig
@@ -137,9 +137,11 @@ CONFIG_DRM_DUMB_VGA_DAC=m
CONFIG_MEDIA_SUPPORT=m
CONFIG_MEDIA_CAMERA_SUPPORT=y
CONFIG_V4L_PLATFORM_DRIVERS=y
+CONFIG_VIDEO_DAVINCI_VPIF_DISPLAY=m
CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE=m
# CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
CONFIG_VIDEO_TVP514X=m
+CONFIG_VIDEO_ADV7343=m
CONFIG_FB=y
CONFIG_FIRMWARE_EDID=y
CONFIG_FB_DA8XX=y
--
2.9.3
--
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 related [flat|nested] 91+ messages in thread
* [PATCH 06/10] ARM: davinci_all_defconfig: enable VPIF display modules
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Enable the VPIF display module and the video encoder present on the
da850-evm UI board.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/configs/davinci_all_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/configs/davinci_all_defconfig b/arch/arm/configs/davinci_all_defconfig
index 7815a52..e188b02 100644
--- a/arch/arm/configs/davinci_all_defconfig
+++ b/arch/arm/configs/davinci_all_defconfig
@@ -137,9 +137,11 @@ CONFIG_DRM_DUMB_VGA_DAC=m
CONFIG_MEDIA_SUPPORT=m
CONFIG_MEDIA_CAMERA_SUPPORT=y
CONFIG_V4L_PLATFORM_DRIVERS=y
+CONFIG_VIDEO_DAVINCI_VPIF_DISPLAY=m
CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE=m
# CONFIG_MEDIA_SUBDRV_AUTOSELECT is not set
CONFIG_VIDEO_TVP514X=m
+CONFIG_VIDEO_ADV7343=m
CONFIG_FB=y
CONFIG_FIRMWARE_EDID=y
CONFIG_FB_DA8XX=y
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 07/10] ARM: davinci: fix a whitespace error
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 16:41 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
There's a stray tab in da850_vpif_legacy_init(). Remove it.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index a186513..94948c1 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
static void __init da850_vpif_legacy_init(void)
{
int ret;
-
+
/* LCDK doesn't have the 2nd TVP514x on CH1 */
if (of_machine_is_compatible("ti,da850-lcdk"))
da850_vpif_capture_config.subdev_count = 1;
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
There's a stray tab in da850_vpif_legacy_init(). Remove it.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index a186513..94948c1 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
static void __init da850_vpif_legacy_init(void)
{
int ret;
-
+
/* LCDK doesn't have the 2nd TVP514x on CH1 */
if (of_machine_is_compatible("ti,da850-lcdk"))
da850_vpif_capture_config.subdev_count = 1;
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 16:41 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
When we enable vpif capture on the da850-evm we hit a BUG_ON() because
the i2c adapter can't be found. The board file boot uses i2c adapter 1
but in the DT mode it's actually adapter 0. Drop the problematic lines.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index 94948c1..09f62ac 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
if (of_machine_is_compatible("ti,da850-lcdk"))
da850_vpif_capture_config.subdev_count = 1;
- /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
- if (of_machine_is_compatible("ti,da850-evm"))
- da850_vpif_capture_config.i2c_adapter_id = 1;
-
ret = da850_register_vpif_capture(&da850_vpif_capture_config);
if (ret)
pr_warn("%s: VPIF capture setup failed: %d\n",
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
When we enable vpif capture on the da850-evm we hit a BUG_ON() because
the i2c adapter can't be found. The board file boot uses i2c adapter 1
but in the DT mode it's actually adapter 0. Drop the problematic lines.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index 94948c1..09f62ac 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
if (of_machine_is_compatible("ti,da850-lcdk"))
da850_vpif_capture_config.subdev_count = 1;
- /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
- if (of_machine_is_compatible("ti,da850-evm"))
- da850_vpif_capture_config.i2c_adapter_id = 1;
-
ret = da850_register_vpif_capture(&da850_vpif_capture_config);
if (ret)
pr_warn("%s: VPIF capture setup failed: %d\n",
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 16:41 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
The vpif display driver uses a static i2c adapter ID of 1 but on the
da850-evm board in DT boot mode the i2c adapter ID is actually 0.
Make the adapter ID configurable like it already is for vpif capture.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/board-da850-evm.c | 1 +
drivers/media/platform/davinci/vpif_display.c | 2 +-
include/media/davinci/vpif_types.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/board-da850-evm.c b/arch/arm/mach-davinci/board-da850-evm.c
index e5d4ded..fe0bfa7 100644
--- a/arch/arm/mach-davinci/board-da850-evm.c
+++ b/arch/arm/mach-davinci/board-da850-evm.c
@@ -1290,6 +1290,7 @@ static struct vpif_display_config da850_vpif_display_config = {
.output_count = ARRAY_SIZE(da850_ch0_outputs),
},
.card_name = "DA850/OMAP-L138 Video Display",
+ .i2c_adapter_id = 1,
};
static __init void da850_vpif_init(void)
diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c
index 50c3073..7e5cf99 100644
--- a/drivers/media/platform/davinci/vpif_display.c
+++ b/drivers/media/platform/davinci/vpif_display.c
@@ -1287,7 +1287,7 @@ static __init int vpif_probe(struct platform_device *pdev)
}
if (!vpif_obj.config->asd_sizes) {
- i2c_adap = i2c_get_adapter(1);
+ i2c_adap = i2c_get_adapter(vpif_obj.config->i2c_adapter_id);
for (i = 0; i < subdev_count; i++) {
vpif_obj.sd[i] =
v4l2_i2c_new_subdev_board(&vpif_obj.v4l2_dev,
diff --git a/include/media/davinci/vpif_types.h b/include/media/davinci/vpif_types.h
index 4282a7d..0c72b46 100644
--- a/include/media/davinci/vpif_types.h
+++ b/include/media/davinci/vpif_types.h
@@ -57,6 +57,7 @@ struct vpif_display_config {
int (*set_clock)(int, int);
struct vpif_subdev_info *subdevinfo;
int subdev_count;
+ int i2c_adapter_id;
struct vpif_display_chan_config chan_config[VPIF_DISPLAY_MAX_CHANNELS];
const char *card_name;
struct v4l2_async_subdev **asd; /* Flat array, arranged in groups */
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
The vpif display driver uses a static i2c adapter ID of 1 but on the
da850-evm board in DT boot mode the i2c adapter ID is actually 0.
Make the adapter ID configurable like it already is for vpif capture.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/board-da850-evm.c | 1 +
drivers/media/platform/davinci/vpif_display.c | 2 +-
include/media/davinci/vpif_types.h | 1 +
3 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/board-da850-evm.c b/arch/arm/mach-davinci/board-da850-evm.c
index e5d4ded..fe0bfa7 100644
--- a/arch/arm/mach-davinci/board-da850-evm.c
+++ b/arch/arm/mach-davinci/board-da850-evm.c
@@ -1290,6 +1290,7 @@ static struct vpif_display_config da850_vpif_display_config = {
.output_count = ARRAY_SIZE(da850_ch0_outputs),
},
.card_name = "DA850/OMAP-L138 Video Display",
+ .i2c_adapter_id = 1,
};
static __init void da850_vpif_init(void)
diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c
index 50c3073..7e5cf99 100644
--- a/drivers/media/platform/davinci/vpif_display.c
+++ b/drivers/media/platform/davinci/vpif_display.c
@@ -1287,7 +1287,7 @@ static __init int vpif_probe(struct platform_device *pdev)
}
if (!vpif_obj.config->asd_sizes) {
- i2c_adap = i2c_get_adapter(1);
+ i2c_adap = i2c_get_adapter(vpif_obj.config->i2c_adapter_id);
for (i = 0; i < subdev_count; i++) {
vpif_obj.sd[i] =
v4l2_i2c_new_subdev_board(&vpif_obj.v4l2_dev,
diff --git a/include/media/davinci/vpif_types.h b/include/media/davinci/vpif_types.h
index 4282a7d..0c72b46 100644
--- a/include/media/davinci/vpif_types.h
+++ b/include/media/davinci/vpif_types.h
@@ -57,6 +57,7 @@ struct vpif_display_config {
int (*set_clock)(int, int);
struct vpif_subdev_info *subdevinfo;
int subdev_count;
+ int i2c_adapter_id;
struct vpif_display_chan_config chan_config[VPIF_DISPLAY_MAX_CHANNELS];
const char *card_name;
struct v4l2_async_subdev **asd; /* Flat array, arranged in groups */
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media,
Bartosz Golaszewski
Similarly to vpif capture: we need to register the vpif display driver
and the corresponding adv7343 encoder in pdata-quirks as the DT
support is not complete.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
1 file changed, 85 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index 09f62ac..0a55546 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -9,13 +9,17 @@
*/
#include <linux/kernel.h>
#include <linux/of_platform.h>
+#include <linux/gpio.h>
#include <media/i2c/tvp514x.h>
+#include <media/i2c/adv7343.h>
#include <mach/common.h>
#include <mach/da8xx.h>
#include <mach/mux.h>
+#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
+
struct pdata_init {
const char *compatible;
void (*fn)(void);
@@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
},
.card_name = "DA850/OMAP-L138 Video Capture",
};
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
+
+#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
+static void vpif_evm_display_setup(void)
+{
+ int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
+
+ ret = gpio_request(gpio, "sel_c");
+ if (ret) {
+ pr_warn("Cannot open UI expander pin %d\n", gpio);
+ return;
+ }
+
+ gpio_direction_output(gpio, 0);
+}
+
+static struct adv7343_platform_data adv7343_pdata = {
+ .mode_config = {
+ .dac = { 1, 1, 1 },
+ },
+ .sd_config = {
+ .sd_dac_out = { 1 },
+ },
+};
+
+static struct vpif_subdev_info da850_vpif_subdev[] = {
+ {
+ .name = "adv7343",
+ .board_info = {
+ I2C_BOARD_INFO("adv7343", 0x2a),
+ .platform_data = &adv7343_pdata,
+ },
+ },
+};
+static const struct vpif_output da850_ch0_outputs[] = {
+ {
+ .output = {
+ .index = 0,
+ .name = "Composite",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_COMPOSITE_ID,
+ },
+ {
+ .output = {
+ .index = 1,
+ .name = "S-Video",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_SVIDEO_ID,
+ },
+};
+
+static struct vpif_display_config da850_vpif_display_config = {
+ .subdevinfo = da850_vpif_subdev,
+ .subdev_count = ARRAY_SIZE(da850_vpif_subdev),
+ .chan_config[0] = {
+ .outputs = da850_ch0_outputs,
+ .output_count = ARRAY_SIZE(da850_ch0_outputs),
+ },
+ .card_name = "DA850/OMAP-L138 Video Display",
+};
+#endif /* defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
+
+#if IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
static void __init da850_vpif_legacy_init(void)
{
int ret;
@@ -120,8 +195,17 @@ static void __init da850_vpif_legacy_init(void)
if (ret)
pr_warn("%s: VPIF capture setup failed: %d\n",
__func__, ret);
+
+ /* LCDK doesn't support VPIF display */
+ if (of_machine_is_compatible("ti,da850-evm")) {
+ vpif_evm_display_setup();
+ ret = da850_register_vpif_display(&da850_vpif_display_config);
+ if (ret)
+ pr_warn("%s: VPIF display setup failed: %d\n",
+ __func__, ret);
+ }
}
-#endif
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
static void pdata_quirks_check(struct pdata_init *quirks)
{
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA, Bartosz Golaszewski
Similarly to vpif capture: we need to register the vpif display driver
and the corresponding adv7343 encoder in pdata-quirks as the DT
support is not complete.
Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
---
arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
1 file changed, 85 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index 09f62ac..0a55546 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -9,13 +9,17 @@
*/
#include <linux/kernel.h>
#include <linux/of_platform.h>
+#include <linux/gpio.h>
#include <media/i2c/tvp514x.h>
+#include <media/i2c/adv7343.h>
#include <mach/common.h>
#include <mach/da8xx.h>
#include <mach/mux.h>
+#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
+
struct pdata_init {
const char *compatible;
void (*fn)(void);
@@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
},
.card_name = "DA850/OMAP-L138 Video Capture",
};
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
+
+#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
+static void vpif_evm_display_setup(void)
+{
+ int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
+
+ ret = gpio_request(gpio, "sel_c");
+ if (ret) {
+ pr_warn("Cannot open UI expander pin %d\n", gpio);
+ return;
+ }
+
+ gpio_direction_output(gpio, 0);
+}
+
+static struct adv7343_platform_data adv7343_pdata = {
+ .mode_config = {
+ .dac = { 1, 1, 1 },
+ },
+ .sd_config = {
+ .sd_dac_out = { 1 },
+ },
+};
+
+static struct vpif_subdev_info da850_vpif_subdev[] = {
+ {
+ .name = "adv7343",
+ .board_info = {
+ I2C_BOARD_INFO("adv7343", 0x2a),
+ .platform_data = &adv7343_pdata,
+ },
+ },
+};
+static const struct vpif_output da850_ch0_outputs[] = {
+ {
+ .output = {
+ .index = 0,
+ .name = "Composite",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_COMPOSITE_ID,
+ },
+ {
+ .output = {
+ .index = 1,
+ .name = "S-Video",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_SVIDEO_ID,
+ },
+};
+
+static struct vpif_display_config da850_vpif_display_config = {
+ .subdevinfo = da850_vpif_subdev,
+ .subdev_count = ARRAY_SIZE(da850_vpif_subdev),
+ .chan_config[0] = {
+ .outputs = da850_ch0_outputs,
+ .output_count = ARRAY_SIZE(da850_ch0_outputs),
+ },
+ .card_name = "DA850/OMAP-L138 Video Display",
+};
+#endif /* defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
+
+#if IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
static void __init da850_vpif_legacy_init(void)
{
int ret;
@@ -120,8 +195,17 @@ static void __init da850_vpif_legacy_init(void)
if (ret)
pr_warn("%s: VPIF capture setup failed: %d\n",
__func__, ret);
+
+ /* LCDK doesn't support VPIF display */
+ if (of_machine_is_compatible("ti,da850-evm")) {
+ vpif_evm_display_setup();
+ ret = da850_register_vpif_display(&da850_vpif_display_config);
+ if (ret)
+ pr_warn("%s: VPIF display setup failed: %d\n",
+ __func__, ret);
+ }
}
-#endif
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
static void pdata_quirks_check(struct pdata_init *quirks)
{
--
2.9.3
--
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 related [flat|nested] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-07 16:41 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-07 16:41 UTC (permalink / raw)
To: linux-arm-kernel
Similarly to vpif capture: we need to register the vpif display driver
and the corresponding adv7343 encoder in pdata-quirks as the DT
support is not complete.
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
---
arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
1 file changed, 85 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
index 09f62ac..0a55546 100644
--- a/arch/arm/mach-davinci/pdata-quirks.c
+++ b/arch/arm/mach-davinci/pdata-quirks.c
@@ -9,13 +9,17 @@
*/
#include <linux/kernel.h>
#include <linux/of_platform.h>
+#include <linux/gpio.h>
#include <media/i2c/tvp514x.h>
+#include <media/i2c/adv7343.h>
#include <mach/common.h>
#include <mach/da8xx.h>
#include <mach/mux.h>
+#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
+
struct pdata_init {
const char *compatible;
void (*fn)(void);
@@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
},
.card_name = "DA850/OMAP-L138 Video Capture",
};
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
+
+#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
+static void vpif_evm_display_setup(void)
+{
+ int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
+
+ ret = gpio_request(gpio, "sel_c");
+ if (ret) {
+ pr_warn("Cannot open UI expander pin %d\n", gpio);
+ return;
+ }
+
+ gpio_direction_output(gpio, 0);
+}
+
+static struct adv7343_platform_data adv7343_pdata = {
+ .mode_config = {
+ .dac = { 1, 1, 1 },
+ },
+ .sd_config = {
+ .sd_dac_out = { 1 },
+ },
+};
+
+static struct vpif_subdev_info da850_vpif_subdev[] = {
+ {
+ .name = "adv7343",
+ .board_info = {
+ I2C_BOARD_INFO("adv7343", 0x2a),
+ .platform_data = &adv7343_pdata,
+ },
+ },
+};
+static const struct vpif_output da850_ch0_outputs[] = {
+ {
+ .output = {
+ .index = 0,
+ .name = "Composite",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_COMPOSITE_ID,
+ },
+ {
+ .output = {
+ .index = 1,
+ .name = "S-Video",
+ .type = V4L2_OUTPUT_TYPE_ANALOG,
+ .capabilities = V4L2_OUT_CAP_STD,
+ .std = V4L2_STD_ALL,
+ },
+ .subdev_name = "adv7343",
+ .output_route = ADV7343_SVIDEO_ID,
+ },
+};
+
+static struct vpif_display_config da850_vpif_display_config = {
+ .subdevinfo = da850_vpif_subdev,
+ .subdev_count = ARRAY_SIZE(da850_vpif_subdev),
+ .chan_config[0] = {
+ .outputs = da850_ch0_outputs,
+ .output_count = ARRAY_SIZE(da850_ch0_outputs),
+ },
+ .card_name = "DA850/OMAP-L138 Video Display",
+};
+#endif /* defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
+
+#if IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
static void __init da850_vpif_legacy_init(void)
{
int ret;
@@ -120,8 +195,17 @@ static void __init da850_vpif_legacy_init(void)
if (ret)
pr_warn("%s: VPIF capture setup failed: %d\n",
__func__, ret);
+
+ /* LCDK doesn't support VPIF display */
+ if (of_machine_is_compatible("ti,da850-evm")) {
+ vpif_evm_display_setup();
+ ret = da850_register_vpif_display(&da850_vpif_display_config);
+ if (ret)
+ pr_warn("%s: VPIF display setup failed: %d\n",
+ __func__, ret);
+ }
}
-#endif
+#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
static void pdata_quirks_check(struct pdata_init *quirks)
{
--
2.9.3
^ permalink raw reply related [flat|nested] 91+ messages in thread
* Re: [PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 18:16 ` Kevin Hilman
-1 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:16 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> The vpif display driver uses a static i2c adapter ID of 1 but on the
> da850-evm board in DT boot mode the i2c adapter ID is actually 0.
>
> Make the adapter ID configurable like it already is for vpif capture.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/board-da850-evm.c b/arch/arm/mach-davinci/board-da850-evm.c
> index e5d4ded..fe0bfa7 100644
> --- a/arch/arm/mach-davinci/board-da850-evm.c
> +++ b/arch/arm/mach-davinci/board-da850-evm.c
> @@ -1290,6 +1290,7 @@ static struct vpif_display_config da850_vpif_display_config = {
> .output_count = ARRAY_SIZE(da850_ch0_outputs),
> },
> .card_name = "DA850/OMAP-L138 Video Display",
> + .i2c_adapter_id = 1,
> };
>
> static __init void da850_vpif_init(void)
> diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c
> index 50c3073..7e5cf99 100644
> --- a/drivers/media/platform/davinci/vpif_display.c
> +++ b/drivers/media/platform/davinci/vpif_display.c
> @@ -1287,7 +1287,7 @@ static __init int vpif_probe(struct platform_device *pdev)
> }
>
> if (!vpif_obj.config->asd_sizes) {
> - i2c_adap = i2c_get_adapter(1);
> + i2c_adap = i2c_get_adapter(vpif_obj.config->i2c_adapter_id);
> for (i = 0; i < subdev_count; i++) {
> vpif_obj.sd[i] =
> v4l2_i2c_new_subdev_board(&vpif_obj.v4l2_dev,
> diff --git a/include/media/davinci/vpif_types.h b/include/media/davinci/vpif_types.h
> index 4282a7d..0c72b46 100644
> --- a/include/media/davinci/vpif_types.h
> +++ b/include/media/davinci/vpif_types.h
> @@ -57,6 +57,7 @@ struct vpif_display_config {
> int (*set_clock)(int, int);
> struct vpif_subdev_info *subdevinfo;
> int subdev_count;
> + int i2c_adapter_id;
> struct vpif_display_chan_config chan_config[VPIF_DISPLAY_MAX_CHANNELS];
> const char *card_name;
> struct v4l2_async_subdev **asd; /* Flat array, arranged in groups */
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display
@ 2017-02-07 18:16 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:16 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> The vpif display driver uses a static i2c adapter ID of 1 but on the
> da850-evm board in DT boot mode the i2c adapter ID is actually 0.
>
> Make the adapter ID configurable like it already is for vpif capture.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 3 files changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/board-da850-evm.c b/arch/arm/mach-davinci/board-da850-evm.c
> index e5d4ded..fe0bfa7 100644
> --- a/arch/arm/mach-davinci/board-da850-evm.c
> +++ b/arch/arm/mach-davinci/board-da850-evm.c
> @@ -1290,6 +1290,7 @@ static struct vpif_display_config da850_vpif_display_config = {
> .output_count = ARRAY_SIZE(da850_ch0_outputs),
> },
> .card_name = "DA850/OMAP-L138 Video Display",
> + .i2c_adapter_id = 1,
> };
>
> static __init void da850_vpif_init(void)
> diff --git a/drivers/media/platform/davinci/vpif_display.c b/drivers/media/platform/davinci/vpif_display.c
> index 50c3073..7e5cf99 100644
> --- a/drivers/media/platform/davinci/vpif_display.c
> +++ b/drivers/media/platform/davinci/vpif_display.c
> @@ -1287,7 +1287,7 @@ static __init int vpif_probe(struct platform_device *pdev)
> }
>
> if (!vpif_obj.config->asd_sizes) {
> - i2c_adap = i2c_get_adapter(1);
> + i2c_adap = i2c_get_adapter(vpif_obj.config->i2c_adapter_id);
> for (i = 0; i < subdev_count; i++) {
> vpif_obj.sd[i] =
> v4l2_i2c_new_subdev_board(&vpif_obj.v4l2_dev,
> diff --git a/include/media/davinci/vpif_types.h b/include/media/davinci/vpif_types.h
> index 4282a7d..0c72b46 100644
> --- a/include/media/davinci/vpif_types.h
> +++ b/include/media/davinci/vpif_types.h
> @@ -57,6 +57,7 @@ struct vpif_display_config {
> int (*set_clock)(int, int);
> struct vpif_subdev_info *subdevinfo;
> int subdev_count;
> + int i2c_adapter_id;
> struct vpif_display_chan_config chan_config[VPIF_DISPLAY_MAX_CHANNELS];
> const char *card_name;
> struct v4l2_async_subdev **asd; /* Flat array, arranged in groups */
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 18:20 ` Kevin Hilman
-1 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:20 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Similarly to vpif capture: we need to register the vpif display driver
> and the corresponding adv7343 encoder in pdata-quirks as the DT
> support is not complete.
To add a bit more detail to the changelog: DT support is not complete
since there isn't currently a way to define the output_routing in the
V4L2 drivers (c.f. s_routing) via DT.
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
minor nit below, otherwise
Reviewed-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 85 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 09f62ac..0a55546 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -9,13 +9,17 @@
> */
> #include <linux/kernel.h>
> #include <linux/of_platform.h>
> +#include <linux/gpio.h>
>
> #include <media/i2c/tvp514x.h>
> +#include <media/i2c/adv7343.h>
>
> #include <mach/common.h>
> #include <mach/da8xx.h>
> #include <mach/mux.h>
>
> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
> +
> struct pdata_init {
> const char *compatible;
> void (*fn)(void);
> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> },
> .card_name = "DA850/OMAP-L138 Video Capture",
> };
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
> +
> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
Why not IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) also here?
> +static void vpif_evm_display_setup(void)
> +{
> + int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
> +
> + ret = gpio_request(gpio, "sel_c");
> + if (ret) {
> + pr_warn("Cannot open UI expander pin %d\n", gpio);
> + return;
> + }
> +
> + gpio_direction_output(gpio, 0);
> +}
> +
> +static struct adv7343_platform_data adv7343_pdata = {
> + .mode_config = {
> + .dac = { 1, 1, 1 },
> + },
> + .sd_config = {
> + .sd_dac_out = { 1 },
> + },
> +};
> +
> +static struct vpif_subdev_info da850_vpif_subdev[] = {
> + {
> + .name = "adv7343",
> + .board_info = {
> + I2C_BOARD_INFO("adv7343", 0x2a),
> + .platform_data = &adv7343_pdata,
> + },
> + },
> +};
>
> +static const struct vpif_output da850_ch0_outputs[] = {
> + {
> + .output = {
> + .index = 0,
> + .name = "Composite",
> + .type = V4L2_OUTPUT_TYPE_ANALOG,
> + .capabilities = V4L2_OUT_CAP_STD,
> + .std = V4L2_STD_ALL,
> + },
> + .subdev_name = "adv7343",
> + .output_route = ADV7343_COMPOSITE_ID,
> + },
> + {
> + .output = {
> + .index = 1,
> + .name = "S-Video",
> + .type = V4L2_OUTPUT_TYPE_ANALOG,
> + .capabilities = V4L2_OUT_CAP_STD,
> + .std = V4L2_STD_ALL,
> + },
> + .subdev_name = "adv7343",
> + .output_route = ADV7343_SVIDEO_ID,
> + },
> +};
> +
> +static struct vpif_display_config da850_vpif_display_config = {
> + .subdevinfo = da850_vpif_subdev,
> + .subdev_count = ARRAY_SIZE(da850_vpif_subdev),
> + .chan_config[0] = {
> + .outputs = da850_ch0_outputs,
> + .output_count = ARRAY_SIZE(da850_ch0_outputs),
> + },
> + .card_name = "DA850/OMAP-L138 Video Display",
> +};
> +#endif /* defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
> +
> +#if IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> @@ -120,8 +195,17 @@ static void __init da850_vpif_legacy_init(void)
> if (ret)
> pr_warn("%s: VPIF capture setup failed: %d\n",
> __func__, ret);
> +
> + /* LCDK doesn't support VPIF display */
> + if (of_machine_is_compatible("ti,da850-evm")) {
> + vpif_evm_display_setup();
> + ret = da850_register_vpif_display(&da850_vpif_display_config);
> + if (ret)
> + pr_warn("%s: VPIF display setup failed: %d\n",
> + __func__, ret);
> + }
> }
> -#endif
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
>
> static void pdata_quirks_check(struct pdata_init *quirks)
> {
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-07 18:20 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:20 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Similarly to vpif capture: we need to register the vpif display driver
> and the corresponding adv7343 encoder in pdata-quirks as the DT
> support is not complete.
To add a bit more detail to the changelog: DT support is not complete
since there isn't currently a way to define the output_routing in the
V4L2 drivers (c.f. s_routing) via DT.
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
minor nit below, otherwise
Reviewed-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 85 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 09f62ac..0a55546 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -9,13 +9,17 @@
> */
> #include <linux/kernel.h>
> #include <linux/of_platform.h>
> +#include <linux/gpio.h>
>
> #include <media/i2c/tvp514x.h>
> +#include <media/i2c/adv7343.h>
>
> #include <mach/common.h>
> #include <mach/da8xx.h>
> #include <mach/mux.h>
>
> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
> +
> struct pdata_init {
> const char *compatible;
> void (*fn)(void);
> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> },
> .card_name = "DA850/OMAP-L138 Video Capture",
> };
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
> +
> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
Why not IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) also here?
> +static void vpif_evm_display_setup(void)
> +{
> + int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
> +
> + ret = gpio_request(gpio, "sel_c");
> + if (ret) {
> + pr_warn("Cannot open UI expander pin %d\n", gpio);
> + return;
> + }
> +
> + gpio_direction_output(gpio, 0);
> +}
> +
> +static struct adv7343_platform_data adv7343_pdata = {
> + .mode_config = {
> + .dac = { 1, 1, 1 },
> + },
> + .sd_config = {
> + .sd_dac_out = { 1 },
> + },
> +};
> +
> +static struct vpif_subdev_info da850_vpif_subdev[] = {
> + {
> + .name = "adv7343",
> + .board_info = {
> + I2C_BOARD_INFO("adv7343", 0x2a),
> + .platform_data = &adv7343_pdata,
> + },
> + },
> +};
>
> +static const struct vpif_output da850_ch0_outputs[] = {
> + {
> + .output = {
> + .index = 0,
> + .name = "Composite",
> + .type = V4L2_OUTPUT_TYPE_ANALOG,
> + .capabilities = V4L2_OUT_CAP_STD,
> + .std = V4L2_STD_ALL,
> + },
> + .subdev_name = "adv7343",
> + .output_route = ADV7343_COMPOSITE_ID,
> + },
> + {
> + .output = {
> + .index = 1,
> + .name = "S-Video",
> + .type = V4L2_OUTPUT_TYPE_ANALOG,
> + .capabilities = V4L2_OUT_CAP_STD,
> + .std = V4L2_STD_ALL,
> + },
> + .subdev_name = "adv7343",
> + .output_route = ADV7343_SVIDEO_ID,
> + },
> +};
> +
> +static struct vpif_display_config da850_vpif_display_config = {
> + .subdevinfo = da850_vpif_subdev,
> + .subdev_count = ARRAY_SIZE(da850_vpif_subdev),
> + .chan_config[0] = {
> + .outputs = da850_ch0_outputs,
> + .output_count = ARRAY_SIZE(da850_ch0_outputs),
> + },
> + .card_name = "DA850/OMAP-L138 Video Display",
> +};
> +#endif /* defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
> +
> +#if IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> @@ -120,8 +195,17 @@ static void __init da850_vpif_legacy_init(void)
> if (ret)
> pr_warn("%s: VPIF capture setup failed: %d\n",
> __func__, ret);
> +
> + /* LCDK doesn't support VPIF display */
> + if (of_machine_is_compatible("ti,da850-evm")) {
> + vpif_evm_display_setup();
> + ret = da850_register_vpif_display(&da850_vpif_display_config);
> + if (ret)
> + pr_warn("%s: VPIF display setup failed: %d\n",
> + __func__, ret);
> + }
> }
> -#endif
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) || defined(CONFIG_DA850_UI_SD_VIDEO_PORT) */
>
> static void pdata_quirks_check(struct pdata_init *quirks)
> {
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-07 18:21 ` Kevin Hilman
-1 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:21 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
> the i2c adapter can't be found. The board file boot uses i2c adapter 1
> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 94948c1..09f62ac 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
>
> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
> - if (of_machine_is_compatible("ti,da850-evm"))
> - da850_vpif_capture_config.i2c_adapter_id = 1;
> -
oops, my bad.
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ret = da850_register_vpif_capture(&da850_vpif_capture_config);
> if (ret)
> pr_warn("%s: VPIF capture setup failed: %d\n",
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-07 18:21 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-07 18:21 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
> the i2c adapter can't be found. The board file boot uses i2c adapter 1
> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 94948c1..09f62ac 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
>
> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
> - if (of_machine_is_compatible("ti,da850-evm"))
> - da850_vpif_capture_config.i2c_adapter_id = 1;
> -
oops, my bad.
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ret = da850_register_vpif_capture(&da850_vpif_capture_config);
> if (ret)
> pr_warn("%s: VPIF capture setup failed: %d\n",
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
2017-02-07 18:20 ` Kevin Hilman
(?)
@ 2017-02-08 13:26 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-08 13:26 UTC (permalink / raw)
To: Kevin Hilman
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc, linux-media
2017-02-07 19:20 GMT+01:00 Kevin Hilman <khilman@baylibre.com>:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> Similarly to vpif capture: we need to register the vpif display driver
>> and the corresponding adv7343 encoder in pdata-quirks as the DT
>> support is not complete.
>
> To add a bit more detail to the changelog: DT support is not complete
> since there isn't currently a way to define the output_routing in the
> V4L2 drivers (c.f. s_routing) via DT.
>
I'll add this in v2.
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> minor nit below, otherwise
>
> Reviewed-by: Kevin Hilman <khilman@baylibre.com>
>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
>> 1 file changed, 85 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 09f62ac..0a55546 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -9,13 +9,17 @@
>> */
>> #include <linux/kernel.h>
>> #include <linux/of_platform.h>
>> +#include <linux/gpio.h>
>>
>> #include <media/i2c/tvp514x.h>
>> +#include <media/i2c/adv7343.h>
>>
>> #include <mach/common.h>
>> #include <mach/da8xx.h>
>> #include <mach/mux.h>
>>
>> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
>> +
>> struct pdata_init {
>> const char *compatible;
>> void (*fn)(void);
>> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
>> },
>> .card_name = "DA850/OMAP-L138 Video Capture",
>> };
>> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
>> +
>> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
>
> Why not IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) also here?
Just a reflex ;)
I'll fix this too.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-08 13:26 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-08 13:26 UTC (permalink / raw)
To: Kevin Hilman
Cc: Mark Rutland, linux-devicetree, David Lechner, Michael Turquette,
Sekhar Nori, Russell King, LKML, Lad Prabhakar, Rob Herring,
linux-media, Alexandre Bailon, Patrick Titiano,
Mauro Carvalho Chehab, arm-soc, Laurent Pinchart
2017-02-07 19:20 GMT+01:00 Kevin Hilman <khilman@baylibre.com>:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> Similarly to vpif capture: we need to register the vpif display driver
>> and the corresponding adv7343 encoder in pdata-quirks as the DT
>> support is not complete.
>
> To add a bit more detail to the changelog: DT support is not complete
> since there isn't currently a way to define the output_routing in the
> V4L2 drivers (c.f. s_routing) via DT.
>
I'll add this in v2.
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> minor nit below, otherwise
>
> Reviewed-by: Kevin Hilman <khilman@baylibre.com>
>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
>> 1 file changed, 85 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 09f62ac..0a55546 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -9,13 +9,17 @@
>> */
>> #include <linux/kernel.h>
>> #include <linux/of_platform.h>
>> +#include <linux/gpio.h>
>>
>> #include <media/i2c/tvp514x.h>
>> +#include <media/i2c/adv7343.h>
>>
>> #include <mach/common.h>
>> #include <mach/da8xx.h>
>> #include <mach/mux.h>
>>
>> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
>> +
>> struct pdata_init {
>> const char *compatible;
>> void (*fn)(void);
>> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
>> },
>> .card_name = "DA850/OMAP-L138 Video Capture",
>> };
>> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
>> +
>> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
>
> Why not IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) also here?
Just a reflex ;)
I'll fix this too.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-08 13:26 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-08 13:26 UTC (permalink / raw)
To: linux-arm-kernel
2017-02-07 19:20 GMT+01:00 Kevin Hilman <khilman@baylibre.com>:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> Similarly to vpif capture: we need to register the vpif display driver
>> and the corresponding adv7343 encoder in pdata-quirks as the DT
>> support is not complete.
>
> To add a bit more detail to the changelog: DT support is not complete
> since there isn't currently a way to define the output_routing in the
> V4L2 drivers (c.f. s_routing) via DT.
>
I'll add this in v2.
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> minor nit below, otherwise
>
> Reviewed-by: Kevin Hilman <khilman@baylibre.com>
>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
>> 1 file changed, 85 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 09f62ac..0a55546 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -9,13 +9,17 @@
>> */
>> #include <linux/kernel.h>
>> #include <linux/of_platform.h>
>> +#include <linux/gpio.h>
>>
>> #include <media/i2c/tvp514x.h>
>> +#include <media/i2c/adv7343.h>
>>
>> #include <mach/common.h>
>> #include <mach/da8xx.h>
>> #include <mach/mux.h>
>>
>> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
>> +
>> struct pdata_init {
>> const char *compatible;
>> void (*fn)(void);
>> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
>> },
>> .card_name = "DA850/OMAP-L138 Video Capture",
>> };
>> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
>> +
>> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
>
> Why not IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) also here?
Just a reflex ;)
I'll fix this too.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
2017-02-07 18:21 ` Kevin Hilman
(?)
@ 2017-02-09 15:23 ` Sekhar Nori
-1 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-09 15:23 UTC (permalink / raw)
To: Kevin Hilman, Bartosz Golaszewski
Cc: Patrick Titiano, Michael Turquette, Rob Herring, Mark Rutland,
Russell King, Alexandre Bailon, David Lechner, Laurent Pinchart,
Mauro Carvalho Chehab, Lad Prabhakar, devicetree, linux-kernel,
linux-arm-kernel, linux-media
On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 94948c1..09f62ac 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>> if (of_machine_is_compatible("ti,da850-lcdk"))
>> da850_vpif_capture_config.subdev_count = 1;
>>
>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>> - if (of_machine_is_compatible("ti,da850-evm"))
>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>> -
>
> oops, my bad.
>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
The offending code is not in my master branch. Since its almost certain
that VPIF platform support is going to wait for v4.12, can you or Kevin
please update Kevin's original patches with these fixes rolled in?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-09 15:23 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-09 15:23 UTC (permalink / raw)
To: Kevin Hilman, Bartosz Golaszewski
Cc: Mark Rutland, devicetree, David Lechner, Michael Turquette,
Russell King, linux-kernel, Lad Prabhakar, Rob Herring,
linux-media, Alexandre Bailon, Patrick Titiano,
Mauro Carvalho Chehab, linux-arm-kernel, Laurent Pinchart
On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 94948c1..09f62ac 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>> if (of_machine_is_compatible("ti,da850-lcdk"))
>> da850_vpif_capture_config.subdev_count = 1;
>>
>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>> - if (of_machine_is_compatible("ti,da850-evm"))
>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>> -
>
> oops, my bad.
>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
The offending code is not in my master branch. Since its almost certain
that VPIF platform support is going to wait for v4.12, can you or Kevin
please update Kevin's original patches with these fixes rolled in?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-09 15:23 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-09 15:23 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>> 1 file changed, 4 deletions(-)
>>
>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>> index 94948c1..09f62ac 100644
>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>> if (of_machine_is_compatible("ti,da850-lcdk"))
>> da850_vpif_capture_config.subdev_count = 1;
>>
>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>> - if (of_machine_is_compatible("ti,da850-evm"))
>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>> -
>
> oops, my bad.
>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
The offending code is not in my master branch. Since its almost certain
that VPIF platform support is going to wait for v4.12, can you or Kevin
please update Kevin's original patches with these fixes rolled in?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-09 16:44 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-09 16:44 UTC (permalink / raw)
To: Sekhar Nori
Cc: Kevin Hilman, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc, linux-media
2017-02-09 16:23 GMT+01:00 Sekhar Nori <nsekhar@ti.com>:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman@baylibre.com>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
>
> Thanks,
> Sekhar
>
Sure, I based my series on Kevin's integration branch for 4.10.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-09 16:44 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-09 16:44 UTC (permalink / raw)
To: Sekhar Nori
Cc: Kevin Hilman, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc,
linux-media-u79uwXL29TY76Z2rM5mHXA
2017-02-09 16:23 GMT+01:00 Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
>
> Thanks,
> Sekhar
>
Sure, I based my series on Kevin's integration branch for 4.10.
Thanks,
Bartosz
--
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] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-09 16:44 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-09 16:44 UTC (permalink / raw)
To: linux-arm-kernel
2017-02-09 16:23 GMT+01:00 Sekhar Nori <nsekhar@ti.com>:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman@baylibre.com>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
>
> Thanks,
> Sekhar
>
Sure, I based my series on Kevin's integration branch for 4.10.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-10 19:40 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-10 19:40 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Hi Bartosz,
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Similarly to vpif capture: we need to register the vpif display driver
> and the corresponding adv7343 encoder in pdata-quirks as the DT
> support is not complete.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 85 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 09f62ac..0a55546 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -9,13 +9,17 @@
> */
> #include <linux/kernel.h>
> #include <linux/of_platform.h>
> +#include <linux/gpio.h>
>
> #include <media/i2c/tvp514x.h>
> +#include <media/i2c/adv7343.h>
>
> #include <mach/common.h>
> #include <mach/da8xx.h>
> #include <mach/mux.h>
>
> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
> +
> struct pdata_init {
> const char *compatible;
> void (*fn)(void);
> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> },
> .card_name = "DA850/OMAP-L138 Video Capture",
> };
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
> +
> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
> +static void vpif_evm_display_setup(void)
> +{
> + int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
> +
> + ret = gpio_request(gpio, "sel_c");
> + if (ret) {
> + pr_warn("Cannot open UI expander pin %d\n", gpio);
> + return;
> + }
> +
> + gpio_direction_output(gpio, 0);
> +}
I had a closer look at the UI board schematic, and it looks like the
SEL_C line of the GPIO exapander is actualy to select the VPIF *input*
source, not output, so I don't think it should be needed in this patch.
Can you test VPIF display works without calling this function?
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-10 19:40 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-10 19:40 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Hi Bartosz,
Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
> Similarly to vpif capture: we need to register the vpif display driver
> and the corresponding adv7343 encoder in pdata-quirks as the DT
> support is not complete.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 85 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 09f62ac..0a55546 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -9,13 +9,17 @@
> */
> #include <linux/kernel.h>
> #include <linux/of_platform.h>
> +#include <linux/gpio.h>
>
> #include <media/i2c/tvp514x.h>
> +#include <media/i2c/adv7343.h>
>
> #include <mach/common.h>
> #include <mach/da8xx.h>
> #include <mach/mux.h>
>
> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
> +
> struct pdata_init {
> const char *compatible;
> void (*fn)(void);
> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> },
> .card_name = "DA850/OMAP-L138 Video Capture",
> };
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
> +
> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
> +static void vpif_evm_display_setup(void)
> +{
> + int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
> +
> + ret = gpio_request(gpio, "sel_c");
> + if (ret) {
> + pr_warn("Cannot open UI expander pin %d\n", gpio);
> + return;
> + }
> +
> + gpio_direction_output(gpio, 0);
> +}
I had a closer look at the UI board schematic, and it looks like the
SEL_C line of the GPIO exapander is actualy to select the VPIF *input*
source, not output, so I don't think it should be needed in this patch.
Can you test VPIF display works without calling this function?
Kevin
--
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] 91+ messages in thread
* [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm vpif display
@ 2017-02-10 19:40 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-10 19:40 UTC (permalink / raw)
To: linux-arm-kernel
Hi Bartosz,
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Similarly to vpif capture: we need to register the vpif display driver
> and the corresponding adv7343 encoder in pdata-quirks as the DT
> support is not complete.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 86 +++++++++++++++++++++++++++++++++++-
> 1 file changed, 85 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index 09f62ac..0a55546 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -9,13 +9,17 @@
> */
> #include <linux/kernel.h>
> #include <linux/of_platform.h>
> +#include <linux/gpio.h>
>
> #include <media/i2c/tvp514x.h>
> +#include <media/i2c/adv7343.h>
>
> #include <mach/common.h>
> #include <mach/da8xx.h>
> #include <mach/mux.h>
>
> +#define DA850_EVM_UI_EXP_SEL_VPIF_DISP 5
> +
> struct pdata_init {
> const char *compatible;
> void (*fn)(void);
> @@ -107,7 +111,78 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> },
> .card_name = "DA850/OMAP-L138 Video Capture",
> };
> +#endif /* IS_ENABLED(CONFIG_VIDEO_DAVINCI_VPIF_CAPTURE) */
> +
> +#if defined(CONFIG_DA850_UI_SD_VIDEO_PORT)
> +static void vpif_evm_display_setup(void)
> +{
> + int gpio = DAVINCI_N_GPIO + DA850_EVM_UI_EXP_SEL_VPIF_DISP, ret;
> +
> + ret = gpio_request(gpio, "sel_c");
> + if (ret) {
> + pr_warn("Cannot open UI expander pin %d\n", gpio);
> + return;
> + }
> +
> + gpio_direction_output(gpio, 0);
> +}
I had a closer look at the UI board schematic, and it looks like the
SEL_C line of the GPIO exapander is actualy to select the VPIF *input*
source, not output, so I don't think it should be needed in this patch.
Can you test VPIF display works without calling this function?
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
2017-02-07 16:41 ` Bartosz Golaszewski
(?)
@ 2017-02-13 9:22 ` Sekhar Nori
-1 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-13 9:22 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Patrick Titiano,
Michael Turquette, Rob Herring, Mark Rutland, Russell King,
Alexandre Bailon, David Lechner, Laurent Pinchart,
Mauro Carvalho Chehab, Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media
Hi Bartosz,
On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 display.
>
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
>
> [1] https://github.com/brgl/yavta davinci/vpif-display
Can you also share the command line you used ?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-02-13 9:22 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-13 9:22 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Patrick Titiano,
Michael Turquette, Rob Herring, Mark Rutland, Russell King,
Alexandre Bailon, David Lechner, Laurent Pinchart,
Mauro Carvalho Chehab, Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media
Hi Bartosz,
On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 display.
>
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
>
> [1] https://github.com/brgl/yavta davinci/vpif-display
Can you also share the command line you used ?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-02-13 9:22 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-13 9:22 UTC (permalink / raw)
To: linux-arm-kernel
Hi Bartosz,
On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 display.
>
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
>
> [1] https://github.com/brgl/yavta davinci/vpif-display
Can you also share the command line you used ?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-13 18:28 ` Kevin Hilman
-1 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:28 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index c970b6e..94938a3 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -301,14 +301,14 @@
> /* VPIF capture port */
> port {
> vpif_ch0: endpoint@0 {
> - reg = <0>;
> - bus-width = <8>;
> + reg = <0>;
> + bus-width = <8>;
> };
>
> vpif_ch1: endpoint@1 {
> - reg = <1>;
> - bus-width = <8>;
> - data-shift = <8>;
> + reg = <1>;
> + bus-width = <8>;
> + data-shift = <8>;
> };
> };
> };
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Extend the vpif node with an output port with a single channel.
>
> NOTE: this is still just hardware description - the actual driver
> is registered using pdata-quirks.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/boot/dts/da850-evm.dts | 14 +++++++++++---
> arch/arm/boot/dts/da850.dtsi | 8 +++++++-
> 2 files changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index 94938a3..3d6dd66 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -299,16 +299,24 @@
> status = "okay";
>
> /* VPIF capture port */
> - port {
> - vpif_ch0: endpoint@0 {
> + port@0 {
> + vpif_input_ch0: endpoint@0 {
> reg = <0>;
> bus-width = <8>;
> };
>
> - vpif_ch1: endpoint@1 {
> + vpif_input_ch1: endpoint@1 {
> reg = <1>;
> bus-width = <8>;
> data-shift = <8>;
> };
> };
> +
> + /* VPIF display port */
> + port@1 {
> + vpif_output_ch0: endpoint@0 {
> + reg = <0>;
> + bus-width = <8>;
> + };
> + };
> };
> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
> index 69ec5e7..768a58c 100644
> --- a/arch/arm/boot/dts/da850.dtsi
> +++ b/arch/arm/boot/dts/da850.dtsi
> @@ -494,7 +494,13 @@
> status = "disabled";
>
> /* VPIF capture port */
> - port {
> + port@0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + };
> +
> + /* VPIF display port */
> + port@1 {
> #address-cells = <1>;
> #size-cells = <0>;
> };
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index a186513..94948c1 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> -
> +
> /* LCDK doesn't have the 2nd TVP514x on CH1 */
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-13 18:28 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:28 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
> ---
> arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index c970b6e..94938a3 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -301,14 +301,14 @@
> /* VPIF capture port */
> port {
> vpif_ch0: endpoint at 0 {
> - reg = <0>;
> - bus-width = <8>;
> + reg = <0>;
> + bus-width = <8>;
> };
>
> vpif_ch1: endpoint at 1 {
> - reg = <1>;
> - bus-width = <8>;
> - data-shift = <8>;
> + reg = <1>;
> + bus-width = <8>;
> + data-shift = <8>;
> };
> };
> };
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Extend the vpif node with an output port with a single channel.
>
> NOTE: this is still just hardware description - the actual driver
> is registered using pdata-quirks.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/boot/dts/da850-evm.dts | 14 +++++++++++---
> arch/arm/boot/dts/da850.dtsi | 8 +++++++-
> 2 files changed, 18 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index 94938a3..3d6dd66 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -299,16 +299,24 @@
> status = "okay";
>
> /* VPIF capture port */
> - port {
> - vpif_ch0: endpoint at 0 {
> + port at 0 {
> + vpif_input_ch0: endpoint at 0 {
> reg = <0>;
> bus-width = <8>;
> };
>
> - vpif_ch1: endpoint at 1 {
> + vpif_input_ch1: endpoint at 1 {
> reg = <1>;
> bus-width = <8>;
> data-shift = <8>;
> };
> };
> +
> + /* VPIF display port */
> + port at 1 {
> + vpif_output_ch0: endpoint at 0 {
> + reg = <0>;
> + bus-width = <8>;
> + };
> + };
> };
> diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
> index 69ec5e7..768a58c 100644
> --- a/arch/arm/boot/dts/da850.dtsi
> +++ b/arch/arm/boot/dts/da850.dtsi
> @@ -494,7 +494,13 @@
> status = "disabled";
>
> /* VPIF capture port */
> - port {
> + port at 0 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + };
> +
> + /* VPIF display port */
> + port at 1 {
> #address-cells = <1>;
> #size-cells = <0>;
> };
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index a186513..94948c1 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> -
> +
> /* LCDK doesn't have the 2nd TVP514x on CH1 */
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
I'll fold this one into the original since it's not yet merged.
Kevin
> ---
> arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index c970b6e..94938a3 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -301,14 +301,14 @@
> /* VPIF capture port */
> port {
> vpif_ch0: endpoint@0 {
> - reg = <0>;
> - bus-width = <8>;
> + reg = <0>;
> + bus-width = <8>;
> };
>
> vpif_ch1: endpoint@1 {
> - reg = <1>;
> - bus-width = <8>;
> - data-shift = <8>;
> + reg = <1>;
> + bus-width = <8>;
> + data-shift = <8>;
> };
> };
> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
I'll fold this one into the original since it's not yet merged.
Kevin
> ---
> arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index c970b6e..94938a3 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -301,14 +301,14 @@
> /* VPIF capture port */
> port {
> vpif_ch0: endpoint@0 {
> - reg = <0>;
> - bus-width = <8>;
> + reg = <0>;
> + bus-width = <8>;
> };
>
> vpif_ch1: endpoint@1 {
> - reg = <1>;
> - bus-width = <8>;
> - data-shift = <8>;
> + reg = <1>;
> + bus-width = <8>;
> + data-shift = <8>;
> };
> };
> };
--
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] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
I'll fold this one into the original since it's not yet merged.
Kevin
> ---
> arch/arm/boot/dts/da850-evm.dts | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index c970b6e..94938a3 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -301,14 +301,14 @@
> /* VPIF capture port */
> port {
> vpif_ch0: endpoint at 0 {
> - reg = <0>;
> - bus-width = <8>;
> + reg = <0>;
> + bus-width = <8>;
> };
>
> vpif_ch1: endpoint at 1 {
> - reg = <1>;
> - bus-width = <8>;
> - data-shift = <8>;
> + reg = <1>;
> + bus-width = <8>;
> + data-shift = <8>;
> };
> };
> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: Sekhar Nori
Cc: Bartosz Golaszewski, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar, devicetree, linux-kernel, linux-arm-kernel,
linux-media
Sekhar Nori <nsekhar@ti.com> writes:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman@baylibre.com>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
I'm folding this into the original patch.
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: Sekhar Nori
Cc: Bartosz Golaszewski, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar, devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> writes:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
I'm folding this into the original patch.
Kevin
--
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] 91+ messages in thread
* [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm
@ 2017-02-13 18:35 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:35 UTC (permalink / raw)
To: linux-arm-kernel
Sekhar Nori <nsekhar@ti.com> writes:
> On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote:
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because
>>> the i2c adapter can't be found. The board file boot uses i2c adapter 1
>>> but in the DT mode it's actually adapter 0. Drop the problematic lines.
>>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>> ---
>>> arch/arm/mach-davinci/pdata-quirks.c | 4 ----
>>> 1 file changed, 4 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
>>> index 94948c1..09f62ac 100644
>>> --- a/arch/arm/mach-davinci/pdata-quirks.c
>>> +++ b/arch/arm/mach-davinci/pdata-quirks.c
>>> @@ -116,10 +116,6 @@ static void __init da850_vpif_legacy_init(void)
>>> if (of_machine_is_compatible("ti,da850-lcdk"))
>>> da850_vpif_capture_config.subdev_count = 1;
>>>
>>> - /* EVM (UI card) uses i2c adapter 1 (not default: zero) */
>>> - if (of_machine_is_compatible("ti,da850-evm"))
>>> - da850_vpif_capture_config.i2c_adapter_id = 1;
>>> -
>>
>> oops, my bad.
>>
>> Acked-by: Kevin Hilman <khilman@baylibre.com>
>
> The offending code is not in my master branch. Since its almost certain
> that VPIF platform support is going to wait for v4.12, can you or Kevin
> please update Kevin's original patches with these fixes rolled in?
I'm folding this into the original patch.
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 18:36 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:36 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Folding into the original,
Kevin
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index a186513..94948c1 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> -
> +
> /* LCDK doesn't have the 2nd TVP514x on CH1 */
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 18:36 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:36 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
Folding into the original,
Kevin
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index a186513..94948c1 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> -
> +
> /* LCDK doesn't have the 2nd TVP514x on CH1 */
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
--
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] 91+ messages in thread
* [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 18:36 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 18:36 UTC (permalink / raw)
To: linux-arm-kernel
Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Folding into the original,
Kevin
> ---
> arch/arm/mach-davinci/pdata-quirks.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-davinci/pdata-quirks.c b/arch/arm/mach-davinci/pdata-quirks.c
> index a186513..94948c1 100644
> --- a/arch/arm/mach-davinci/pdata-quirks.c
> +++ b/arch/arm/mach-davinci/pdata-quirks.c
> @@ -111,7 +111,7 @@ static struct vpif_capture_config da850_vpif_capture_config = {
> static void __init da850_vpif_legacy_init(void)
> {
> int ret;
> -
> +
> /* LCDK doesn't have the 2nd TVP514x on CH1 */
> if (of_machine_is_compatible("ti,da850-lcdk"))
> da850_vpif_capture_config.subdev_count = 1;
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
2017-02-13 18:35 ` Kevin Hilman
@ 2017-02-13 21:01 ` Kevin Hilman
-1 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 21:01 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Kevin Hilman <khilman@baylibre.com> writes:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> I'll fold this one into the original since it's not yet merged.
Oops, Sekhar has already merged this one to his v4.11/dt branch, so he
can apply it (or fold it in.)
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-13 21:01 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 21:01 UTC (permalink / raw)
To: linux-arm-kernel
Kevin Hilman <khilman@baylibre.com> writes:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> I'll fold this one into the original since it's not yet merged.
Oops, Sekhar has already merged this one to his v4.11/dt branch, so he
can apply it (or fold it in.)
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 21:04 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 21:04 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
Kevin Hilman <khilman@baylibre.com> writes:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Folding into the original,
Looks like the version in Sekhar's v4.11/soc branch already has this
fixed.
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 21:04 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 21:04 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Sekhar Nori, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Kevin Hilman <khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
> Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org> writes:
>
>> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>
> Folding into the original,
Looks like the version in Sekhar's v4.11/soc branch already has this
fixed.
Kevin
--
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] 91+ messages in thread
* [PATCH 07/10] ARM: davinci: fix a whitespace error
@ 2017-02-13 21:04 ` Kevin Hilman
0 siblings, 0 replies; 91+ messages in thread
From: Kevin Hilman @ 2017-02-13 21:04 UTC (permalink / raw)
To: linux-arm-kernel
Kevin Hilman <khilman@baylibre.com> writes:
> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>
>> There's a stray tab in da850_vpif_legacy_init(). Remove it.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>
> Folding into the original,
Looks like the version in Sekhar's v4.11/soc branch already has this
fixed.
Kevin
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
2017-02-13 21:01 ` Kevin Hilman
(?)
@ 2017-02-14 6:30 ` Sekhar Nori
-1 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-14 6:30 UTC (permalink / raw)
To: Kevin Hilman, Bartosz Golaszewski
Cc: Patrick Titiano, Michael Turquette, Rob Herring, Mark Rutland,
Russell King, Alexandre Bailon, David Lechner, Laurent Pinchart,
Mauro Carvalho Chehab, Lad Prabhakar, devicetree, linux-kernel,
linux-arm-kernel, linux-media
On Tuesday 14 February 2017 02:31 AM, Kevin Hilman wrote:
> Kevin Hilman <khilman@baylibre.com> writes:
>
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> I'll fold this one into the original since it's not yet merged.
>
> Oops, Sekhar has already merged this one to his v4.11/dt branch, so he
> can apply it (or fold it in.)
Alright, I will apply this.
Bartosz, for future, please add a commit description for trivial patches
too. Even if its more or less just a rewording of subject line.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-14 6:30 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-14 6:30 UTC (permalink / raw)
To: Kevin Hilman, Bartosz Golaszewski
Cc: Mark Rutland, devicetree, David Lechner, Michael Turquette,
Russell King, linux-kernel, Lad Prabhakar, Rob Herring,
linux-media, Alexandre Bailon, Patrick Titiano,
Mauro Carvalho Chehab, linux-arm-kernel, Laurent Pinchart
On Tuesday 14 February 2017 02:31 AM, Kevin Hilman wrote:
> Kevin Hilman <khilman@baylibre.com> writes:
>
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> I'll fold this one into the original since it's not yet merged.
>
> Oops, Sekhar has already merged this one to his v4.11/dt branch, so he
> can apply it (or fold it in.)
Alright, I will apply this.
Bartosz, for future, please add a commit description for trivial patches
too. Even if its more or less just a rewording of subject line.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 02/10] ARM: dts: da850-evm: fix whitespace errors
@ 2017-02-14 6:30 ` Sekhar Nori
0 siblings, 0 replies; 91+ messages in thread
From: Sekhar Nori @ 2017-02-14 6:30 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 14 February 2017 02:31 AM, Kevin Hilman wrote:
> Kevin Hilman <khilman@baylibre.com> writes:
>
>> Bartosz Golaszewski <bgolaszewski@baylibre.com> writes:
>>
>>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> I'll fold this one into the original since it's not yet merged.
>
> Oops, Sekhar has already merged this one to his v4.11/dt branch, so he
> can apply it (or fold it in.)
Alright, I will apply this.
Bartosz, for future, please add a commit description for trivial patches
too. Even if its more or less just a rewording of subject line.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-02-15 22:01 ` Rob Herring
-1 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:01 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
On Tue, Feb 07, 2017 at 05:41:14PM +0100, Bartosz Golaszewski wrote:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors
@ 2017-02-15 22:01 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:01 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Feb 07, 2017 at 05:41:14PM +0100, Bartosz Golaszewski wrote:
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> Documentation/devicetree/bindings/media/ti,da850-vpif.txt | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:03 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:03 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:03 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:03 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
--
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] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:03 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:03 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:08 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:08 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree, linux-kernel, linux-arm-kernel, linux-media
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Spoke too soon...
>
> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> index 9c7510b..543f6f3 100644
> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> reg = <0x217000 0x1000>;
> interrupts = <92>;
>
> - port {
> - vpif_ch0: endpoint@0 {
> + port@0 {
> + vpif_input_ch0: endpoint@0 {
> reg = <0>;
> bus-width = <8>;
> - remote-endpoint = <&composite>;
> + remote-endpoint = <&composite_in>;
> };
>
> - vpif_ch1: endpoint@1 {
> + vpif_input_ch1: endpoint@1 {
> reg = <1>;
> bus-width = <8>;
> data-shift = <8>;
> };
> };
> +
> + port@1 {
The binding doc says nothing about supporting a 2nd port.
> + vpif_output_ch0: endpoint@0 {
> + reg = <0>;
Don't need reg here.
> + bus-width = <8>;
> + remote-endpoint = <&composite_out>;
> + };
> + };
> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:08 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:08 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Spoke too soon...
>
> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> index 9c7510b..543f6f3 100644
> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> reg = <0x217000 0x1000>;
> interrupts = <92>;
>
> - port {
> - vpif_ch0: endpoint@0 {
> + port@0 {
> + vpif_input_ch0: endpoint@0 {
> reg = <0>;
> bus-width = <8>;
> - remote-endpoint = <&composite>;
> + remote-endpoint = <&composite_in>;
> };
>
> - vpif_ch1: endpoint@1 {
> + vpif_input_ch1: endpoint@1 {
> reg = <1>;
> bus-width = <8>;
> data-shift = <8>;
> };
> };
> +
> + port@1 {
The binding doc says nothing about supporting a 2nd port.
> + vpif_output_ch0: endpoint@0 {
> + reg = <0>;
Don't need reg here.
> + bus-width = <8>;
> + remote-endpoint = <&composite_out>;
> + };
> + };
> };
--
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] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-15 22:08 ` Rob Herring
0 siblings, 0 replies; 91+ messages in thread
From: Rob Herring @ 2017-02-15 22:08 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> This makes the example more or less correspond with the da850-evm
> hardware setup.
>
> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> ---
> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
> 1 file changed, 29 insertions(+), 6 deletions(-)
Spoke too soon...
>
> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> index 9c7510b..543f6f3 100644
> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> reg = <0x217000 0x1000>;
> interrupts = <92>;
>
> - port {
> - vpif_ch0: endpoint at 0 {
> + port at 0 {
> + vpif_input_ch0: endpoint at 0 {
> reg = <0>;
> bus-width = <8>;
> - remote-endpoint = <&composite>;
> + remote-endpoint = <&composite_in>;
> };
>
> - vpif_ch1: endpoint at 1 {
> + vpif_input_ch1: endpoint at 1 {
> reg = <1>;
> bus-width = <8>;
> data-shift = <8>;
> };
> };
> +
> + port at 1 {
The binding doc says nothing about supporting a 2nd port.
> + vpif_output_ch0: endpoint at 0 {
> + reg = <0>;
Don't need reg here.
> + bus-width = <8>;
> + remote-endpoint = <&composite_out>;
> + };
> + };
> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
2017-02-15 22:08 ` Rob Herring
(?)
@ 2017-02-16 7:32 ` Laurent Pinchart
-1 siblings, 0 replies; 91+ messages in thread
From: Laurent Pinchart @ 2017-02-16 7:32 UTC (permalink / raw)
To: Rob Herring
Cc: Bartosz Golaszewski, Kevin Hilman, Sekhar Nori, Patrick Titiano,
Michael Turquette, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Mauro Carvalho Chehab, Lad Prabhakar, devicetree,
linux-kernel, linux-arm-kernel, linux-media
Hi Rob,
On Wednesday 15 Feb 2017 16:08:22 Rob Herring wrote:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> > This makes the example more or less correspond with the da850-evm
> > hardware setup.
> >
> > Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > ---
> >
> > .../devicetree/bindings/media/ti,da850-vpif.txt | 35 +++++++++++++----
> > 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
> > diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt index
> > 9c7510b..543f6f3 100644
> > --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> > reg = <0x217000 0x1000>;
> > interrupts = <92>;
> >
> > - port {
> > - vpif_ch0: endpoint@0 {
> > + port@0 {
> > + vpif_input_ch0: endpoint@0 {
> > reg = <0>;
> > bus-width = <8>;
> > - remote-endpoint = <&composite>;
> > + remote-endpoint = <&composite_in>;
> > };
> >
> > - vpif_ch1: endpoint@1 {
> > + vpif_input_ch1: endpoint@1 {
> > reg = <1>;
> > bus-width = <8>;
> > data-shift = <8>;
> > };
> > };
> > +
> > + port@1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
> > + vpif_output_ch0: endpoint@0 {
> > + reg = <0>;
>
> Don't need reg here.
And the node should then be named endpoint, not endpoint@0.
> > + bus-width = <8>;
> > + remote-endpoint = <&composite_out>;
> > + };
> > + };
> > };
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-16 7:32 ` Laurent Pinchart
0 siblings, 0 replies; 91+ messages in thread
From: Laurent Pinchart @ 2017-02-16 7:32 UTC (permalink / raw)
To: Rob Herring
Cc: Bartosz Golaszewski, Kevin Hilman, Sekhar Nori, Patrick Titiano,
Michael Turquette, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Mauro Carvalho Chehab, Lad Prabhakar,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-media-u79uwXL29TY76Z2rM5mHXA
Hi Rob,
On Wednesday 15 Feb 2017 16:08:22 Rob Herring wrote:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> > This makes the example more or less correspond with the da850-evm
> > hardware setup.
> >
> > Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> > ---
> >
> > .../devicetree/bindings/media/ti,da850-vpif.txt | 35 +++++++++++++----
> > 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
> > diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt index
> > 9c7510b..543f6f3 100644
> > --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> > reg = <0x217000 0x1000>;
> > interrupts = <92>;
> >
> > - port {
> > - vpif_ch0: endpoint@0 {
> > + port@0 {
> > + vpif_input_ch0: endpoint@0 {
> > reg = <0>;
> > bus-width = <8>;
> > - remote-endpoint = <&composite>;
> > + remote-endpoint = <&composite_in>;
> > };
> >
> > - vpif_ch1: endpoint@1 {
> > + vpif_input_ch1: endpoint@1 {
> > reg = <1>;
> > bus-width = <8>;
> > data-shift = <8>;
> > };
> > };
> > +
> > + port@1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
> > + vpif_output_ch0: endpoint@0 {
> > + reg = <0>;
>
> Don't need reg here.
And the node should then be named endpoint, not endpoint@0.
> > + bus-width = <8>;
> > + remote-endpoint = <&composite_out>;
> > + };
> > + };
> > };
--
Regards,
Laurent Pinchart
--
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] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-16 7:32 ` Laurent Pinchart
0 siblings, 0 replies; 91+ messages in thread
From: Laurent Pinchart @ 2017-02-16 7:32 UTC (permalink / raw)
To: linux-arm-kernel
Hi Rob,
On Wednesday 15 Feb 2017 16:08:22 Rob Herring wrote:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
> > This makes the example more or less correspond with the da850-evm
> > hardware setup.
> >
> > Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> > ---
> >
> > .../devicetree/bindings/media/ti,da850-vpif.txt | 35 +++++++++++++----
> > 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
> > diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt index
> > 9c7510b..543f6f3 100644
> > --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
> > @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
> > reg = <0x217000 0x1000>;
> > interrupts = <92>;
> >
> > - port {
> > - vpif_ch0: endpoint at 0 {
> > + port at 0 {
> > + vpif_input_ch0: endpoint at 0 {
> > reg = <0>;
> > bus-width = <8>;
> > - remote-endpoint = <&composite>;
> > + remote-endpoint = <&composite_in>;
> > };
> >
> > - vpif_ch1: endpoint at 1 {
> > + vpif_input_ch1: endpoint at 1 {
> > reg = <1>;
> > bus-width = <8>;
> > data-shift = <8>;
> > };
> > };
> > +
> > + port at 1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
> > + vpif_output_ch0: endpoint at 0 {
> > + reg = <0>;
>
> Don't need reg here.
And the node should then be named endpoint, not endpoint at 0.
> > + bus-width = <8>;
> > + remote-endpoint = <&composite_out>;
> > + };
> > + };
> > };
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
2017-02-15 22:08 ` Rob Herring
(?)
@ 2017-02-16 16:06 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-16 16:06 UTC (permalink / raw)
To: Rob Herring
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc, linux-media
2017-02-15 23:08 GMT+01:00 Rob Herring <robh@kernel.org>:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
>> This makes the example more or less correspond with the da850-evm
>> hardware setup.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
>> 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
>>
>> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> index 9c7510b..543f6f3 100644
>> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
>> reg = <0x217000 0x1000>;
>> interrupts = <92>;
>>
>> - port {
>> - vpif_ch0: endpoint@0 {
>> + port@0 {
>> + vpif_input_ch0: endpoint@0 {
>> reg = <0>;
>> bus-width = <8>;
>> - remote-endpoint = <&composite>;
>> + remote-endpoint = <&composite_in>;
>> };
>>
>> - vpif_ch1: endpoint@1 {
>> + vpif_input_ch1: endpoint@1 {
>> reg = <1>;
>> bus-width = <8>;
>> data-shift = <8>;
>> };
>> };
>> +
>> + port@1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
I assumed that "It should contain at least one port child node" means
there can be more than one.
Thanks,
Bartosz
>
>> + vpif_output_ch0: endpoint@0 {
>> + reg = <0>;
>
> Don't need reg here.
>
>> + bus-width = <8>;
>> + remote-endpoint = <&composite_out>;
>> + };
>> + };
>> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-16 16:06 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-16 16:06 UTC (permalink / raw)
To: Rob Herring
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc,
linux-media-u79uwXL29TY76Z2rM5mHXA
2017-02-15 23:08 GMT+01:00 Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
>> This makes the example more or less correspond with the da850-evm
>> hardware setup.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
>> ---
>> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
>> 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
>>
>> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> index 9c7510b..543f6f3 100644
>> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
>> reg = <0x217000 0x1000>;
>> interrupts = <92>;
>>
>> - port {
>> - vpif_ch0: endpoint@0 {
>> + port@0 {
>> + vpif_input_ch0: endpoint@0 {
>> reg = <0>;
>> bus-width = <8>;
>> - remote-endpoint = <&composite>;
>> + remote-endpoint = <&composite_in>;
>> };
>>
>> - vpif_ch1: endpoint@1 {
>> + vpif_input_ch1: endpoint@1 {
>> reg = <1>;
>> bus-width = <8>;
>> data-shift = <8>;
>> };
>> };
>> +
>> + port@1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
I assumed that "It should contain at least one port child node" means
there can be more than one.
Thanks,
Bartosz
>
>> + vpif_output_ch0: endpoint@0 {
>> + reg = <0>;
>
> Don't need reg here.
>
>> + bus-width = <8>;
>> + remote-endpoint = <&composite_out>;
>> + };
>> + };
>> };
--
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] 91+ messages in thread
* [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port
@ 2017-02-16 16:06 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-16 16:06 UTC (permalink / raw)
To: linux-arm-kernel
2017-02-15 23:08 GMT+01:00 Rob Herring <robh@kernel.org>:
> On Tue, Feb 07, 2017 at 05:41:16PM +0100, Bartosz Golaszewski wrote:
>> This makes the example more or less correspond with the da850-evm
>> hardware setup.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>> ---
>> .../devicetree/bindings/media/ti,da850-vpif.txt | 35 ++++++++++++++++++----
>> 1 file changed, 29 insertions(+), 6 deletions(-)
>
> Spoke too soon...
>
>>
>> diff --git a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> index 9c7510b..543f6f3 100644
>> --- a/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> +++ b/Documentation/devicetree/bindings/media/ti,da850-vpif.txt
>> @@ -28,19 +28,27 @@ I2C-connected TVP5147 decoder:
>> reg = <0x217000 0x1000>;
>> interrupts = <92>;
>>
>> - port {
>> - vpif_ch0: endpoint at 0 {
>> + port at 0 {
>> + vpif_input_ch0: endpoint at 0 {
>> reg = <0>;
>> bus-width = <8>;
>> - remote-endpoint = <&composite>;
>> + remote-endpoint = <&composite_in>;
>> };
>>
>> - vpif_ch1: endpoint at 1 {
>> + vpif_input_ch1: endpoint at 1 {
>> reg = <1>;
>> bus-width = <8>;
>> data-shift = <8>;
>> };
>> };
>> +
>> + port at 1 {
>
> The binding doc says nothing about supporting a 2nd port.
>
I assumed that "It should contain at least one port child node" means
there can be more than one.
Thanks,
Bartosz
>
>> + vpif_output_ch0: endpoint at 0 {
>> + reg = <0>;
>
> Don't need reg here.
>
>> + bus-width = <8>;
>> + remote-endpoint = <&composite_out>;
>> + };
>> + };
>> };
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
2017-02-13 9:22 ` Sekhar Nori
@ 2017-02-16 17:48 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-16 17:48 UTC (permalink / raw)
To: Sekhar Nori
Cc: Kevin Hilman, Patrick Titiano, Michael Turquette, Rob Herring,
Mark Rutland, Russell King, Alexandre Bailon, David Lechner,
Laurent Pinchart, Mauro Carvalho Chehab, Lad Prabhakar,
linux-devicetree, LKML, arm-soc, linux-media
2017-02-13 10:22 GMT+01:00 Sekhar Nori <nsekhar@ti.com>:
> Hi Bartosz,
>
> On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>>
>> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
>> errors in dts files and bindings, extend the example and the dts for
>> da850-evm with the output port and address the pinmuxing.
>>
>> Patch 6/10 enables the relevant modules in the defconfig file.
>>
>> Patches 7/10 and 8/10 fix two already existing bugs encountered
>> during development.
>>
>> Patch 9/10 make it possible to use a different i2c adapter in the
>> vpif display driver.
>>
>> The last patch adds the pdata quirks necessary to enable v4l2 display.
>>
>> Tested with a modified version of yavta[1] as gstreamer support for
>> v4l2 seems to be broken and results in picture artifacts.
>>
>> [1] https://github.com/brgl/yavta davinci/vpif-display
>
> Can you also share the command line you used ?
>
> Thanks,
> Sekhar
Will do. I'll also send separate sets of patches for your different
branches as advised by Kevin.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-02-16 17:48 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-02-16 17:48 UTC (permalink / raw)
To: linux-arm-kernel
2017-02-13 10:22 GMT+01:00 Sekhar Nori <nsekhar@ti.com>:
> Hi Bartosz,
>
> On Tuesday 07 February 2017 10:11 PM, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>>
>> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
>> errors in dts files and bindings, extend the example and the dts for
>> da850-evm with the output port and address the pinmuxing.
>>
>> Patch 6/10 enables the relevant modules in the defconfig file.
>>
>> Patches 7/10 and 8/10 fix two already existing bugs encountered
>> during development.
>>
>> Patch 9/10 make it possible to use a different i2c adapter in the
>> vpif display driver.
>>
>> The last patch adds the pdata quirks necessary to enable v4l2 display.
>>
>> Tested with a modified version of yavta[1] as gstreamer support for
>> v4l2 seems to be broken and results in picture artifacts.
>>
>> [1] https://github.com/brgl/yavta davinci/vpif-display
>
> Can you also share the command line you used ?
>
> Thanks,
> Sekhar
Will do. I'll also send separate sets of patches for your different
branches as advised by Kevin.
Thanks,
Bartosz
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
2017-02-07 16:41 ` Bartosz Golaszewski
@ 2017-03-09 10:53 ` Hans Verkuil
-1 siblings, 0 replies; 91+ messages in thread
From: Hans Verkuil @ 2017-03-09 10:53 UTC (permalink / raw)
To: Bartosz Golaszewski, Kevin Hilman, Sekhar Nori, Patrick Titiano,
Michael Turquette, Rob Herring, Mark Rutland, Russell King,
Alexandre Bailon, David Lechner, Laurent Pinchart,
Mauro Carvalho Chehab, Lad Prabhakar
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-media
On 07/02/17 17:41, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
As far as I could tell from the comments this patch series will see a
second version, so I am marking it as 'Changes Requested' in patchwork.
If I'm wrong, then please let me know.
Regards,
Hans
>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 display.
>
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
>
> [1] https://github.com/brgl/yavta davinci/vpif-display
>
> Bartosz Golaszewski (10):
> media: dt-bindings: vpif: fix whitespace errors
> ARM: dts: da850-evm: fix whitespace errors
> media: dt-bindings: vpif: extend the example with an output port
> ARM: dts: da850-evm: add the output port to the vpif node
> ARM: dts: da850: add vpif video display pins
> ARM: davinci_all_defconfig: enable VPIF display modules
> ARM: davinci: fix a whitespace error
> ARM: davinci: fix the DT boot on da850-evm
> media: vpif: use a configurable i2c_adapter_id for vpif display
> ARM: davinci: add pdata-quirks for da850-evm vpif display
>
> .../devicetree/bindings/media/ti,da850-vpif.txt | 45 ++++++++---
> arch/arm/boot/dts/da850-evm.dts | 26 +++---
> arch/arm/boot/dts/da850.dtsi | 25 +++++-
> arch/arm/configs/davinci_all_defconfig | 2 +
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> arch/arm/mach-davinci/pdata-quirks.c | 92 ++++++++++++++++++++--
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 8 files changed, 164 insertions(+), 30 deletions(-)
>
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-03-09 10:53 ` Hans Verkuil
0 siblings, 0 replies; 91+ messages in thread
From: Hans Verkuil @ 2017-03-09 10:53 UTC (permalink / raw)
To: linux-arm-kernel
On 07/02/17 17:41, Bartosz Golaszewski wrote:
> The following series adds support for v4l2 display on da850-evm with
> a UI board in device tree boot mode.
As far as I could tell from the comments this patch series will see a
second version, so I am marking it as 'Changes Requested' in patchwork.
If I'm wrong, then please let me know.
Regards,
Hans
>
> Patches 1/10 - 5/10 deal with the device tree: we fix whitespace
> errors in dts files and bindings, extend the example and the dts for
> da850-evm with the output port and address the pinmuxing.
>
> Patch 6/10 enables the relevant modules in the defconfig file.
>
> Patches 7/10 and 8/10 fix two already existing bugs encountered
> during development.
>
> Patch 9/10 make it possible to use a different i2c adapter in the
> vpif display driver.
>
> The last patch adds the pdata quirks necessary to enable v4l2 display.
>
> Tested with a modified version of yavta[1] as gstreamer support for
> v4l2 seems to be broken and results in picture artifacts.
>
> [1] https://github.com/brgl/yavta davinci/vpif-display
>
> Bartosz Golaszewski (10):
> media: dt-bindings: vpif: fix whitespace errors
> ARM: dts: da850-evm: fix whitespace errors
> media: dt-bindings: vpif: extend the example with an output port
> ARM: dts: da850-evm: add the output port to the vpif node
> ARM: dts: da850: add vpif video display pins
> ARM: davinci_all_defconfig: enable VPIF display modules
> ARM: davinci: fix a whitespace error
> ARM: davinci: fix the DT boot on da850-evm
> media: vpif: use a configurable i2c_adapter_id for vpif display
> ARM: davinci: add pdata-quirks for da850-evm vpif display
>
> .../devicetree/bindings/media/ti,da850-vpif.txt | 45 ++++++++---
> arch/arm/boot/dts/da850-evm.dts | 26 +++---
> arch/arm/boot/dts/da850.dtsi | 25 +++++-
> arch/arm/configs/davinci_all_defconfig | 2 +
> arch/arm/mach-davinci/board-da850-evm.c | 1 +
> arch/arm/mach-davinci/pdata-quirks.c | 92 ++++++++++++++++++++--
> drivers/media/platform/davinci/vpif_display.c | 2 +-
> include/media/davinci/vpif_types.h | 1 +
> 8 files changed, 164 insertions(+), 30 deletions(-)
>
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
2017-03-09 10:53 ` Hans Verkuil
(?)
@ 2017-03-09 11:00 ` Bartosz Golaszewski
-1 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-03-09 11:00 UTC (permalink / raw)
To: Hans Verkuil
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar, linux-devicetree, LKML, arm-soc, linux-media
2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>
> As far as I could tell from the comments this patch series will see a
> second version, so I am marking it as 'Changes Requested' in patchwork.
>
> If I'm wrong, then please let me know.
>
> Regards,
>
> Hans
>
Hi Hans,
this series has since been split into several smaller ones and most of
the code is already merged into Sekhar Nori's DaVinci tree.
There are two series that still need merging with the media tree[1][2].
Thanks,
Bartosz Golaszewski
[1] https://lkml.org/lkml/2017/2/28/426
[2] https://patchwork.kernel.org/patch/9577965/
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-03-09 11:00 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-03-09 11:00 UTC (permalink / raw)
To: Hans Verkuil
Cc: Mark Rutland, linux-devicetree, David Lechner, Kevin Hilman,
Michael Turquette, Sekhar Nori, Russell King, LKML,
Lad Prabhakar, Rob Herring, linux-media, Alexandre Bailon,
Patrick Titiano, Mauro Carvalho Chehab, arm-soc,
Laurent Pinchart
2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>
> As far as I could tell from the comments this patch series will see a
> second version, so I am marking it as 'Changes Requested' in patchwork.
>
> If I'm wrong, then please let me know.
>
> Regards,
>
> Hans
>
Hi Hans,
this series has since been split into several smaller ones and most of
the code is already merged into Sekhar Nori's DaVinci tree.
There are two series that still need merging with the media tree[1][2].
Thanks,
Bartosz Golaszewski
[1] https://lkml.org/lkml/2017/2/28/426
[2] https://patchwork.kernel.org/patch/9577965/
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-03-09 11:00 ` Bartosz Golaszewski
0 siblings, 0 replies; 91+ messages in thread
From: Bartosz Golaszewski @ 2017-03-09 11:00 UTC (permalink / raw)
To: linux-arm-kernel
2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>> The following series adds support for v4l2 display on da850-evm with
>> a UI board in device tree boot mode.
>
> As far as I could tell from the comments this patch series will see a
> second version, so I am marking it as 'Changes Requested' in patchwork.
>
> If I'm wrong, then please let me know.
>
> Regards,
>
> Hans
>
Hi Hans,
this series has since been split into several smaller ones and most of
the code is already merged into Sekhar Nori's DaVinci tree.
There are two series that still need merging with the media tree[1][2].
Thanks,
Bartosz Golaszewski
[1] https://lkml.org/lkml/2017/2/28/426
[2] https://patchwork.kernel.org/patch/9577965/
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
2017-03-09 11:00 ` Bartosz Golaszewski
(?)
@ 2017-03-09 12:05 ` Hans Verkuil
-1 siblings, 0 replies; 91+ messages in thread
From: Hans Verkuil @ 2017-03-09 12:05 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Kevin Hilman, Sekhar Nori, Patrick Titiano, Michael Turquette,
Rob Herring, Mark Rutland, Russell King, Alexandre Bailon,
David Lechner, Laurent Pinchart, Mauro Carvalho Chehab,
Lad Prabhakar, linux-devicetree, LKML, arm-soc, linux-media
On 09/03/17 12:00, Bartosz Golaszewski wrote:
> 2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
>> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>>> The following series adds support for v4l2 display on da850-evm with
>>> a UI board in device tree boot mode.
>>
>> As far as I could tell from the comments this patch series will see a
>> second version, so I am marking it as 'Changes Requested' in patchwork.
>>
>> If I'm wrong, then please let me know.
>>
>> Regards,
>>
>> Hans
>>
>
> Hi Hans,
>
> this series has since been split into several smaller ones and most of
> the code is already merged into Sekhar Nori's DaVinci tree.
>
> There are two series that still need merging with the media tree[1][2].
Merged, thanks!
Regards,
Hans
>
> Thanks,
> Bartosz Golaszewski
>
> [1] https://lkml.org/lkml/2017/2/28/426
> [2] https://patchwork.kernel.org/patch/9577965/
>
^ permalink raw reply [flat|nested] 91+ messages in thread
* Re: [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-03-09 12:05 ` Hans Verkuil
0 siblings, 0 replies; 91+ messages in thread
From: Hans Verkuil @ 2017-03-09 12:05 UTC (permalink / raw)
To: Bartosz Golaszewski
Cc: Mark Rutland, linux-devicetree, David Lechner, Kevin Hilman,
Michael Turquette, Sekhar Nori, Russell King, LKML,
Lad Prabhakar, Rob Herring, linux-media, Alexandre Bailon,
Patrick Titiano, Mauro Carvalho Chehab, arm-soc,
Laurent Pinchart
On 09/03/17 12:00, Bartosz Golaszewski wrote:
> 2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
>> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>>> The following series adds support for v4l2 display on da850-evm with
>>> a UI board in device tree boot mode.
>>
>> As far as I could tell from the comments this patch series will see a
>> second version, so I am marking it as 'Changes Requested' in patchwork.
>>
>> If I'm wrong, then please let me know.
>>
>> Regards,
>>
>> Hans
>>
>
> Hi Hans,
>
> this series has since been split into several smaller ones and most of
> the code is already merged into Sekhar Nori's DaVinci tree.
>
> There are two series that still need merging with the media tree[1][2].
Merged, thanks!
Regards,
Hans
>
> Thanks,
> Bartosz Golaszewski
>
> [1] https://lkml.org/lkml/2017/2/28/426
> [2] https://patchwork.kernel.org/patch/9577965/
>
^ permalink raw reply [flat|nested] 91+ messages in thread
* [PATCH 00/10] ARM: davinci: add vpif display support
@ 2017-03-09 12:05 ` Hans Verkuil
0 siblings, 0 replies; 91+ messages in thread
From: Hans Verkuil @ 2017-03-09 12:05 UTC (permalink / raw)
To: linux-arm-kernel
On 09/03/17 12:00, Bartosz Golaszewski wrote:
> 2017-03-09 11:53 GMT+01:00 Hans Verkuil <hverkuil@xs4all.nl>:
>> On 07/02/17 17:41, Bartosz Golaszewski wrote:
>>> The following series adds support for v4l2 display on da850-evm with
>>> a UI board in device tree boot mode.
>>
>> As far as I could tell from the comments this patch series will see a
>> second version, so I am marking it as 'Changes Requested' in patchwork.
>>
>> If I'm wrong, then please let me know.
>>
>> Regards,
>>
>> Hans
>>
>
> Hi Hans,
>
> this series has since been split into several smaller ones and most of
> the code is already merged into Sekhar Nori's DaVinci tree.
>
> There are two series that still need merging with the media tree[1][2].
Merged, thanks!
Regards,
Hans
>
> Thanks,
> Bartosz Golaszewski
>
> [1] https://lkml.org/lkml/2017/2/28/426
> [2] https://patchwork.kernel.org/patch/9577965/
>
^ permalink raw reply [flat|nested] 91+ messages in thread
end of thread, other threads:[~2017-03-09 12:08 UTC | newest]
Thread overview: 91+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-07 16:41 [PATCH 00/10] ARM: davinci: add vpif display support Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` [PATCH 01/10] media: dt-bindings: vpif: fix whitespace errors Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-15 22:01 ` Rob Herring
2017-02-15 22:01 ` Rob Herring
2017-02-07 16:41 ` [PATCH 02/10] ARM: dts: da850-evm: " Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-13 18:28 ` Kevin Hilman
2017-02-13 18:28 ` Kevin Hilman
2017-02-13 18:35 ` Kevin Hilman
2017-02-13 18:35 ` Kevin Hilman
2017-02-13 18:35 ` Kevin Hilman
2017-02-13 21:01 ` Kevin Hilman
2017-02-13 21:01 ` Kevin Hilman
2017-02-14 6:30 ` Sekhar Nori
2017-02-14 6:30 ` Sekhar Nori
2017-02-14 6:30 ` Sekhar Nori
2017-02-07 16:41 ` [PATCH 03/10] media: dt-bindings: vpif: extend the example with an output port Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-15 22:03 ` Rob Herring
2017-02-15 22:03 ` Rob Herring
2017-02-15 22:03 ` Rob Herring
2017-02-15 22:08 ` Rob Herring
2017-02-15 22:08 ` Rob Herring
2017-02-15 22:08 ` Rob Herring
2017-02-16 7:32 ` Laurent Pinchart
2017-02-16 7:32 ` Laurent Pinchart
2017-02-16 7:32 ` Laurent Pinchart
2017-02-16 16:06 ` Bartosz Golaszewski
2017-02-16 16:06 ` Bartosz Golaszewski
2017-02-16 16:06 ` Bartosz Golaszewski
2017-02-07 16:41 ` [PATCH 04/10] ARM: dts: da850-evm: add the output port to the vpif node Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` [PATCH 05/10] ARM: dts: da850: add vpif video display pins Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` [PATCH 06/10] ARM: davinci_all_defconfig: enable VPIF display modules Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` [PATCH 07/10] ARM: davinci: fix a whitespace error Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-13 18:36 ` Kevin Hilman
2017-02-13 18:36 ` Kevin Hilman
2017-02-13 18:36 ` Kevin Hilman
2017-02-13 21:04 ` Kevin Hilman
2017-02-13 21:04 ` Kevin Hilman
2017-02-13 21:04 ` Kevin Hilman
2017-02-07 16:41 ` [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 18:21 ` Kevin Hilman
2017-02-07 18:21 ` Kevin Hilman
2017-02-09 15:23 ` Sekhar Nori
2017-02-09 15:23 ` Sekhar Nori
2017-02-09 15:23 ` Sekhar Nori
2017-02-09 16:44 ` Bartosz Golaszewski
2017-02-09 16:44 ` Bartosz Golaszewski
2017-02-09 16:44 ` Bartosz Golaszewski
2017-02-13 18:35 ` Kevin Hilman
2017-02-13 18:35 ` Kevin Hilman
2017-02-13 18:35 ` Kevin Hilman
2017-02-07 16:41 ` [PATCH 09/10] media: vpif: use a configurable i2c_adapter_id for vpif display Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 18:16 ` Kevin Hilman
2017-02-07 18:16 ` Kevin Hilman
2017-02-07 16:41 ` [PATCH 10/10] ARM: davinci: add pdata-quirks for da850-evm " Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 16:41 ` Bartosz Golaszewski
2017-02-07 18:20 ` Kevin Hilman
2017-02-07 18:20 ` Kevin Hilman
2017-02-08 13:26 ` Bartosz Golaszewski
2017-02-08 13:26 ` Bartosz Golaszewski
2017-02-08 13:26 ` Bartosz Golaszewski
2017-02-10 19:40 ` Kevin Hilman
2017-02-10 19:40 ` Kevin Hilman
2017-02-10 19:40 ` Kevin Hilman
2017-02-13 9:22 ` [PATCH 00/10] ARM: davinci: add vpif display support Sekhar Nori
2017-02-13 9:22 ` Sekhar Nori
2017-02-13 9:22 ` Sekhar Nori
2017-02-16 17:48 ` Bartosz Golaszewski
2017-02-16 17:48 ` Bartosz Golaszewski
2017-03-09 10:53 ` Hans Verkuil
2017-03-09 10:53 ` Hans Verkuil
2017-03-09 11:00 ` Bartosz Golaszewski
2017-03-09 11:00 ` Bartosz Golaszewski
2017-03-09 11:00 ` Bartosz Golaszewski
2017-03-09 12:05 ` Hans Verkuil
2017-03-09 12:05 ` Hans Verkuil
2017-03-09 12:05 ` Hans Verkuil
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.