Hi Lad,
On Tue, Mar 10, 2020 at 9:42 PM Lad Prabhakar prabhakar.mahadev-lad.rj@bp.renesas.com wrote:
From: Fabrizio Castro fabrizio.castro@bp.renesas.com
The HiHope RZ/G2M is advertised as compatible with panel idk-1110wr from Advantech, however the panel isn't sold alongside the board. A new dts, adding everything that's required to get the panel to work the HiHope RZ/G2M, is the most convenient way to support the HiHope RZ/G2M when it's connected to the idk-1110wr.
Signed-off-by: Fabrizio Castro fabrizio.castro@bp.renesas.com Acked-by: Laurent Pinchart laurent.pinchart@ideasonboard.com Signed-off-by: Lad Prabhakar prabhakar.mahadev-lad.rj@bp.renesas.com
Thanks for picking up this patch!
--- /dev/null +++ b/arch/arm64/boot/dts/renesas/r8a774a1-hihope-rzg2m-ex-idk-1110wr.dts @@ -0,0 +1,86 @@ +// SPDX-License-Identifier: GPL-2.0 +/*
- Device Tree Source for the HiHope RZ/G2M sub board connected to an
- Advantech IDK-1110WR 10.1" LVDS panel
- Copyright (C) 2020 Renesas Electronics Corp.
- */
+#include "r8a774a1-hihope-rzg2m-ex.dts"
+/ {
backlight {compatible = "pwm-backlight";pwms = <&pwm0 0 50000>;brightness-levels = <0 2 8 16 32 64 128 255>;default-brightness-level = <6>;};panel-lvds {compatible = "advantech,idk-1110wr", "panel-lvds";width-mm = <223>;height-mm = <125>;data-mapping = "jeida-24";panel-timing {/* 1024x600 @60Hz */clock-frequency = <51200000>;hactive = <1024>;vactive = <600>;hsync-len = <240>;hfront-porch = <40>;hback-porch = <40>;vfront-porch = <15>;vback-porch = <10>;vsync-len = <10>;};port {panel_in: endpoint {remote-endpoint = <&lvds0_out>;};};};
I believe the plan was to include the existing rzg2-advantech-idk-1110wr-panel.dtsi instead, to provide the panel-lvds node...
+};
+&gpio1 {
/** When GP1_20 is LOW LVDS0 is connected to the LVDS connector* When GP1_20 is HIGH LVDS0 is connected to the LT8918L*/lvds-connector-en-gpio {gpio-hog;gpios = <20 GPIO_ACTIVE_HIGH>;output-low;line-name = "lvds-connector-en-gpio";};+};
+&lvds0 {
status = "okay";ports {port@1 {lvds0_out: endpoint {remote-endpoint = <&panel_in>;};};};+};
... and the lvds_connector endpoint configuration.
The rest looks good to me.
Gr{oetje,eeting}s,
Geert