* [PATCH] ARM: dts: am335x-evm: Use drm simple-panel instead of tilcdc-panel
@ 2019-12-02 18:57 Jyri Sarha
2019-12-03 7:29 ` Laurent Pinchart
0 siblings, 1 reply; 2+ messages in thread
From: Jyri Sarha @ 2019-12-02 18:57 UTC (permalink / raw)
To: dri-devel, tony, bcousson, devicetree, linux-omap
Cc: tomi.valkeinen, laurent.pinchart, peter.ujfalusi, bparrot
Move to use the new drm panel support in tilcdc together with added
"tfc,s9700rtwv43tr-01b"-panel support in drm panel-simple.
Signed-off-by: Jyri Sarha <jsarha@ti.com>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
---
"tfc,s9700rtwv43tr-01b" in panel-simple has been in for some time now
so it is about time to get this in too.
arch/arm/boot/dts/am335x-evm.dts | 44 +++++++++++++-------------------
1 file changed, 18 insertions(+), 26 deletions(-)
diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
index a00145705c9b..0e286a1c9cc5 100644
--- a/arch/arm/boot/dts/am335x-evm.dts
+++ b/arch/arm/boot/dts/am335x-evm.dts
@@ -113,7 +113,7 @@
};
};
- backlight {
+ backlight: backlight {
compatible = "pwm-backlight";
pwms = <&ecap0 0 50000 0>;
brightness-levels = <0 51 53 56 62 75 101 152 255>;
@@ -121,35 +121,19 @@
};
panel {
- compatible = "ti,tilcdc,panel";
+ compatible = "tfc,s9700rtwv43tr-01b";
+
status = "okay";
+
pinctrl-names = "default";
pinctrl-0 = <&lcd_pins_s0>;
- panel-info {
- ac-bias = <255>;
- ac-bias-intrpt = <0>;
- dma-burst-sz = <16>;
- bpp = <32>;
- fdd = <0x80>;
- sync-edge = <0>;
- sync-ctrl = <1>;
- raster-order = <0>;
- fifo-th = <0>;
- };
- display-timings {
- 800x480p62 {
- clock-frequency = <30000000>;
- hactive = <800>;
- vactive = <480>;
- hfront-porch = <39>;
- hback-porch = <39>;
- hsync-len = <47>;
- vback-porch = <29>;
- vfront-porch = <13>;
- vsync-len = <2>;
- hsync-active = <1>;
- vsync-active = <1>;
+ backlight = <&backlight>;
+ ports {
+ port {
+ panel_0: endpoint@0 {
+ remote-endpoint = <&lcdc_0>;
+ };
};
};
};
@@ -525,6 +509,14 @@
status = "okay";
blue-and-red-wiring = "crossed";
+
+ ports {
+ port {
+ lcdc_0: endpoint@0 {
+ remote-endpoint = <&panel_0>;
+ };
+ };
+ };
};
&elm {
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM: dts: am335x-evm: Use drm simple-panel instead of tilcdc-panel
2019-12-02 18:57 [PATCH] ARM: dts: am335x-evm: Use drm simple-panel instead of tilcdc-panel Jyri Sarha
@ 2019-12-03 7:29 ` Laurent Pinchart
0 siblings, 0 replies; 2+ messages in thread
From: Laurent Pinchart @ 2019-12-03 7:29 UTC (permalink / raw)
To: Jyri Sarha
Cc: dri-devel, tony, bcousson, devicetree, linux-omap,
tomi.valkeinen, peter.ujfalusi, bparrot
Hi Jyri,
Thank you for the patch.
On Mon, Dec 02, 2019 at 08:57:29PM +0200, Jyri Sarha wrote:
> Move to use the new drm panel support in tilcdc together with added
> "tfc,s9700rtwv43tr-01b"-panel support in drm panel-simple.
>
> Signed-off-by: Jyri Sarha <jsarha@ti.com>
> Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
> "tfc,s9700rtwv43tr-01b" in panel-simple has been in for some time now
> so it is about time to get this in too.
>
> arch/arm/boot/dts/am335x-evm.dts | 44 +++++++++++++-------------------
> 1 file changed, 18 insertions(+), 26 deletions(-)
>
> diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
> index a00145705c9b..0e286a1c9cc5 100644
> --- a/arch/arm/boot/dts/am335x-evm.dts
> +++ b/arch/arm/boot/dts/am335x-evm.dts
> @@ -113,7 +113,7 @@
> };
> };
>
> - backlight {
> + backlight: backlight {
> compatible = "pwm-backlight";
> pwms = <&ecap0 0 50000 0>;
> brightness-levels = <0 51 53 56 62 75 101 152 255>;
> @@ -121,35 +121,19 @@
> };
>
> panel {
> - compatible = "ti,tilcdc,panel";
> + compatible = "tfc,s9700rtwv43tr-01b";
> +
> status = "okay";
I think this can be dropped as it's implicit.
> +
> pinctrl-names = "default";
> pinctrl-0 = <&lcd_pins_s0>;
> - panel-info {
> - ac-bias = <255>;
> - ac-bias-intrpt = <0>;
> - dma-burst-sz = <16>;
> - bpp = <32>;
> - fdd = <0x80>;
> - sync-edge = <0>;
> - sync-ctrl = <1>;
> - raster-order = <0>;
> - fifo-th = <0>;
> - };
>
> - display-timings {
> - 800x480p62 {
> - clock-frequency = <30000000>;
> - hactive = <800>;
> - vactive = <480>;
> - hfront-porch = <39>;
> - hback-porch = <39>;
> - hsync-len = <47>;
> - vback-porch = <29>;
> - vfront-porch = <13>;
> - vsync-len = <2>;
> - hsync-active = <1>;
> - vsync-active = <1>;
> + backlight = <&backlight>;
I would add a blank line here.
> + ports {
> + port {
> + panel_0: endpoint@0 {
> + remote-endpoint = <&lcdc_0>;
> + };
> };
> };
The outter ports node isn't required if there's a single port, up to
you.
> };
> @@ -525,6 +509,14 @@
> status = "okay";
>
> blue-and-red-wiring = "crossed";
> +
> + ports {
> + port {
> + lcdc_0: endpoint@0 {
> + remote-endpoint = <&panel_0>;
> + };
> + };
> + };
Same here.
With the above issues addressed as applicable,
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> };
>
> &elm {
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-12-03 7:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-02 18:57 [PATCH] ARM: dts: am335x-evm: Use drm simple-panel instead of tilcdc-panel Jyri Sarha
2019-12-03 7:29 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).