diff mbox

[v4,3/3] ARM: dts: exynos: Remove the display-timing and delay from rinato dts

Message ID 1499912443-3671-4-git-send-email-hoegeun.kwon@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hoegeun Kwon July 13, 2017, 2:20 a.m. UTC
The display-timing and delay are included in the panel driver. So it
should be removed in dts.

Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
---
 arch/arm/boot/dts/exynos3250-rinato.dts | 22 ----------------------
 1 file changed, 22 deletions(-)

Comments

Hoegeun Kwon Aug. 29, 2017, 2:52 a.m. UTC | #1
Hi Krzysztof,

The driver has been merged into exynos-drm-misc.
Could you please check this patch(3/3).

Best regards,
Hoegeun

On 07/13/2017 11:20 AM, Hoegeun Kwon wrote:
> The display-timing and delay are included in the panel driver. So it
> should be removed in dts.
>
> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
> ---
>   arch/arm/boot/dts/exynos3250-rinato.dts | 22 ----------------------
>   1 file changed, 22 deletions(-)
>
> diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts
> index 443e0c9..6b70c8d 100644
> --- a/arch/arm/boot/dts/exynos3250-rinato.dts
> +++ b/arch/arm/boot/dts/exynos3250-rinato.dts
> @@ -242,28 +242,6 @@
>   		vci-supply = <&ldo20_reg>;
>   		reset-gpios = <&gpe0 1 GPIO_ACTIVE_LOW>;
>   		te-gpios = <&gpx0 6 GPIO_ACTIVE_HIGH>;
> -		power-on-delay= <30>;
> -		power-off-delay= <120>;
> -		reset-delay = <5>;
> -		init-delay = <100>;
> -		flip-horizontal;
> -		flip-vertical;
> -		panel-width-mm = <29>;
> -		panel-height-mm = <29>;
> -
> -		display-timings {
> -			timing-0 {
> -				clock-frequency = <4600000>;
> -				hactive = <320>;
> -				vactive = <320>;
> -				hfront-porch = <1>;
> -				hback-porch = <1>;
> -				hsync-len = <1>;
> -				vfront-porch = <150>;
> -				vback-porch = <1>;
> -				vsync-len = <2>;
> -			};
> -		};
>   
>   		port {
>   			dsi_in: endpoint {
Krzysztof Kozlowski Aug. 29, 2017, 5:58 a.m. UTC | #2
On Tue, Aug 29, 2017 at 4:52 AM, Hoegeun Kwon <hoegeun.kwon@samsung.com> wrote:
> Hi Krzysztof,
>
> The driver has been merged into exynos-drm-misc.
> Could you please check this patch(3/3).

Hi, OK, no problems for me but it is too late for current cycle so it
will go in for v4.15.

Best regards,
Krzysztof

>
> Best regards,
> Hoegeun
>
>
> On 07/13/2017 11:20 AM, Hoegeun Kwon wrote:
>>
>> The display-timing and delay are included in the panel driver. So it
>> should be removed in dts.
>>
>> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
>> ---
>>   arch/arm/boot/dts/exynos3250-rinato.dts | 22 ----------------------
>>   1 file changed, 22 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts
>> b/arch/arm/boot/dts/exynos3250-rinato.dts
>> index 443e0c9..6b70c8d 100644
>> --- a/arch/arm/boot/dts/exynos3250-rinato.dts
>> +++ b/arch/arm/boot/dts/exynos3250-rinato.dts
>> @@ -242,28 +242,6 @@
>>                 vci-supply = <&ldo20_reg>;
>>                 reset-gpios = <&gpe0 1 GPIO_ACTIVE_LOW>;
>>                 te-gpios = <&gpx0 6 GPIO_ACTIVE_HIGH>;
>> -               power-on-delay= <30>;
>> -               power-off-delay= <120>;
>> -               reset-delay = <5>;
>> -               init-delay = <100>;
>> -               flip-horizontal;
>> -               flip-vertical;
>> -               panel-width-mm = <29>;
>> -               panel-height-mm = <29>;
>> -
>> -               display-timings {
>> -                       timing-0 {
>> -                               clock-frequency = <4600000>;
>> -                               hactive = <320>;
>> -                               vactive = <320>;
>> -                               hfront-porch = <1>;
>> -                               hback-porch = <1>;
>> -                               hsync-len = <1>;
>> -                               vfront-porch = <150>;
>> -                               vback-porch = <1>;
>> -                               vsync-len = <2>;
>> -                       };
>> -               };
>>                 port {
>>                         dsi_in: endpoint {
>
>
Krzysztof Kozlowski Sept. 17, 2017, 7:32 a.m. UTC | #3
On Tue, Aug 29, 2017 at 07:58:52AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Aug 29, 2017 at 4:52 AM, Hoegeun Kwon <hoegeun.kwon@samsung.com> wrote:
> > Hi Krzysztof,
> >
> > The driver has been merged into exynos-drm-misc.
> > Could you please check this patch(3/3).
> 
> Hi, OK, no problems for me but it is too late for current cycle so it
> will go in for v4.15.
>

Patch does not apply. Could you rebase on current mainline (v4.14-rc1) and resend?

Best regards,
Krzysztof
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts
index 443e0c9..6b70c8d 100644
--- a/arch/arm/boot/dts/exynos3250-rinato.dts
+++ b/arch/arm/boot/dts/exynos3250-rinato.dts
@@ -242,28 +242,6 @@ 
 		vci-supply = <&ldo20_reg>;
 		reset-gpios = <&gpe0 1 GPIO_ACTIVE_LOW>;
 		te-gpios = <&gpx0 6 GPIO_ACTIVE_HIGH>;
-		power-on-delay= <30>;
-		power-off-delay= <120>;
-		reset-delay = <5>;
-		init-delay = <100>;
-		flip-horizontal;
-		flip-vertical;
-		panel-width-mm = <29>;
-		panel-height-mm = <29>;
-
-		display-timings {
-			timing-0 {
-				clock-frequency = <4600000>;
-				hactive = <320>;
-				vactive = <320>;
-				hfront-porch = <1>;
-				hback-porch = <1>;
-				hsync-len = <1>;
-				vfront-porch = <150>;
-				vback-porch = <1>;
-				vsync-len = <2>;
-			};
-		};
 
 		port {
 			dsi_in: endpoint {