Re: [PATCH RFC] ARM: dts: samsung: add rga-lvds panel in itop elite

[Andrzej Hajda]

Hi,


On 08/30/2016 12:50 AM, Randy Li wrote:
> It is actually a lvds panel connected through a rga-lvds bridge.
> But I really have no idea about what does a port mean in fimd node.
>
> Also how should I configure this panel size? I think the i2c found
> on the panel schematic, but it more likely to be used a touch screen
> touth. Also the touch screen is not supported in currently driver.
>
> Signed-off-by: Randy Li <ayaka@xxxxxxxxxxx>
> ---
>  arch/arm/boot/dts/exynos4412-itop-elite.dts | 35 ++++++++++++++++++++++++++---
>  1 file changed, 32 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos4412-itop-elite.dts b/arch/arm/boot/dts/exynos4412-itop-elite.dts
> index e1cda54..2d67385 100644
> --- a/arch/arm/boot/dts/exynos4412-itop-elite.dts
> +++ b/arch/arm/boot/dts/exynos4412-itop-elite.dts
> @@ -139,6 +139,20 @@
>  		assigned-clocks = <&clock CLK_MOUT_CAM0>;
>  		assigned-clock-parents = <&clock CLK_XUSBXTI>;
>  	};
> +	
> +	vcc_sys_lcd: sys-lcd {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vcc_5v";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		gpio = <&gpl0 4 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	panel: panel {
> +		ddc-i2c-bus = <&i2c_3>;
> +		power-supply = <&vcc_sys_lcd>;
> +		enable-gpios = <&gpl0 2 GPIO_ACTIVE_HIGH>;
> +	};

Panel has no compatible string, which driver will handle this node?

Regards
Andrzej

>  };
>  
>  &pinctrl_1 {
> @@ -221,9 +235,6 @@
>  };
>  
>  &pwm {
> -	pinctrl-0 = <&pwm1_out>;
> -	pinctrl-names = "default";
> -	samsung,pwm-outputs = <1>;
>  	status = "okay";
>  };
>  
> @@ -239,3 +250,21 @@
>  	assigned-clock-parents = <&clock CLK_MOUT_MPLL_USER_T>;
>  	assigned-clock-rates = <0>, <176000000>;
>  };
> +
> +&i2c_3 {
> +	status = "okay";
> +};
> +
> +&fimd {
> +	pinctrl-0 = <&lcd_clk &lcd_data24 &pwm1_out>;
> +	pinctrl-names = "default";
> +	status = "okay";
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +	port@0 {
> +		reg = <0>;
> +		rga_lvds: endpoint {
> +			remote-endpoint = <&panel>;
> +		};
> +	};
> +};