devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Yakir Yang <ykk@rock-chips.com>
To: Jingoo Han <jingoohan1@gmail.com>,
	'Inki Dae' <inki.dae@samsung.com>,
	'Mark Yao' <mark.yao@rock-chips.com>,
	'Heiko Stuebner' <heiko@sntech.de>
Cc: devicetree@vger.kernel.org,
	'Krzysztof Kozlowski' <k.kozlowski@samsung.com>,
	linux-samsung-soc@vger.kernel.org,
	'Russell King' <linux@arm.linux.org.uk>,
	linux-rockchip@lists.infradead.org, emil.l.velikov@gmail.com,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	'Kishon Vijay Abraham I' <kishon@ti.com>,
	javier@osg.samsung.com, 'Rob Herring' <robh+dt@kernel.org>,
	'Andy Yan' <andy.yan@rock-chips.com>,
	'Thierry Reding' <treding@nvidia.com>,
	'Gustavo Padovan' <gustavo.padovan@collabora.co.uk>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v11 06/19] ARM: dts: exynos/dp: remove some properties that deprecated by analogix_dp driver
Date: Wed, 23 Dec 2015 08:47:12 +0800	[thread overview]
Message-ID: <5679EF10.7070807@rock-chips.com> (raw)
In-Reply-To: <000801d13cb2$43079270$c916b750$@com>


On 12/22/2015 08:13 PM, Jingoo Han wrote:
> On Wednesday, December 16, 2015 12:35 PM, Yakir Yang wrote:
>> After exynos_dp have been split the common IP code into analogix_dp driver,
>> the analogix_dp driver have deprecated some Samsung platform properties which
>> could be dynamically parsed from EDID/MODE/DPCD message, so this is an update
>> for Exynos DTS file for dp-controller.
>>
>> Beside the backward compatibility is fully preserved, so there are no
>> bisectability break that make this change in a separate patch.
>>
>> Signed-off-by: Yakir Yang <ykk@rock-chips.com>
>> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>> Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>
> Reviewed-by: Jingoo Han < jingoohan1@gmail.com >

Thanks,
- Yakir

> Best regards,
> Jingoo Han
>
>> ---
>> Changes in v11: None
>> Changes in v10: None
>> Changes in v9: None
>> Changes in v8: None
>> Changes in v7: None
>> Changes in v6:
>> - Fix Peach Pit hpd property name error:
>> -       hpd-gpio = <&gpx2 6 0>;
>> +       hpd-gpios = <&gpx2 6 0>;
>>
>> Changes in v5:
>> - Correct the misspell in commit message. (Krzysztof)
>>
>> Changes in v4:
>> - Separate all DTS changes to a separate patch. (Krzysztof)
>>
>> Changes in v3: None
>> Changes in v2: None
>>
>>   arch/arm/boot/dts/exynos5250-arndale.dts      | 2 --
>>   arch/arm/boot/dts/exynos5250-smdk5250.dts     | 2 --
>>   arch/arm/boot/dts/exynos5250-snow-common.dtsi | 4 +---
>>   arch/arm/boot/dts/exynos5250-spring.dts       | 4 +---
>>   arch/arm/boot/dts/exynos5420-peach-pit.dts    | 4 +---
>>   arch/arm/boot/dts/exynos5420-smdk5420.dts     | 2 --
>>   arch/arm/boot/dts/exynos5800-peach-pi.dts     | 4 +---
>>   7 files changed, 4 insertions(+), 18 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts
>> index c000532..b1790cf 100644
>> --- a/arch/arm/boot/dts/exynos5250-arndale.dts
>> +++ b/arch/arm/boot/dts/exynos5250-arndale.dts
>> @@ -124,8 +124,6 @@
>>   &dp {
>>   	status = "okay";
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <4>;
>> diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts b/arch/arm/boot/dts/exynos5250-smdk5250.dts
>> index 0f5dcd4..f30c2db 100644
>> --- a/arch/arm/boot/dts/exynos5250-smdk5250.dts
>> +++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts
>> @@ -80,8 +80,6 @@
>>
>>   &dp {
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <4>;
>> diff --git a/arch/arm/boot/dts/exynos5250-snow-common.dtsi b/arch/arm/boot/dts/exynos5250-snow-
>> common.dtsi
>> index 5cb33ba..b96624b 100644
>> --- a/arch/arm/boot/dts/exynos5250-snow-common.dtsi
>> +++ b/arch/arm/boot/dts/exynos5250-snow-common.dtsi
>> @@ -236,12 +236,10 @@
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&dp_hpd>;
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <2>;
>> -	samsung,hpd-gpio = <&gpx0 7 GPIO_ACTIVE_HIGH>;
>> +	hpd-gpios = <&gpx0 7 GPIO_ACTIVE_HIGH>;
>>
>>   	ports {
>>   		port@0 {
>> diff --git a/arch/arm/boot/dts/exynos5250-spring.dts b/arch/arm/boot/dts/exynos5250-spring.dts
>> index c1edd6d..91881d7 100644
>> --- a/arch/arm/boot/dts/exynos5250-spring.dts
>> +++ b/arch/arm/boot/dts/exynos5250-spring.dts
>> @@ -74,12 +74,10 @@
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&dp_hpd_gpio>;
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <1>;
>> -	samsung,hpd-gpio = <&gpc3 0 GPIO_ACTIVE_HIGH>;
>> +	hpd-gpios = <&gpc3 0 GPIO_ACTIVE_HIGH>;
>>   };
>>
>>   &ehci {
>> diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts
>> index 35cfb07..2f37c87 100644
>> --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts
>> +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts
>> @@ -148,12 +148,10 @@
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&dp_hpd_gpio>;
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x06>;
>>   	samsung,lane-count = <2>;
>> -	samsung,hpd-gpio = <&gpx2 6 GPIO_ACTIVE_HIGH>;
>> +	hpd-gpios = <&gpx2 6 GPIO_ACTIVE_HIGH>;
>>
>>   	ports {
>>   		port@0 {
>> diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
>> index ac35aef..f67344f 100644
>> --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
>> +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
>> @@ -93,8 +93,6 @@
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&dp_hpd>;
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <4>;
>> diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts
>> index 7b018e4..363c95f 100644
>> --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts
>> +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts
>> @@ -142,12 +142,10 @@
>>   	pinctrl-names = "default";
>>   	pinctrl-0 = <&dp_hpd_gpio>;
>>   	samsung,color-space = <0>;
>> -	samsung,dynamic-range = <0>;
>> -	samsung,ycbcr-coeff = <0>;
>>   	samsung,color-depth = <1>;
>>   	samsung,link-rate = <0x0a>;
>>   	samsung,lane-count = <2>;
>> -	samsung,hpd-gpio = <&gpx2 6 GPIO_ACTIVE_HIGH>;
>> +	hpd-gpios = <&gpx2 6 GPIO_ACTIVE_HIGH>;
>>   	panel = <&panel>;
>>   };
>>
>> --
>> 1.9.1
>
>
>
>


_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-12-23  0:47 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-16  3:20 [PATCH v11 0/19] Add Analogix Core Display Port Driver Yakir Yang
2015-12-16  3:26 ` [PATCH v11 02/19] drm: bridge: analogix/dp: fix some obvious code style Yakir Yang
2015-12-22 12:05   ` Jingoo Han
2015-12-23  0:50     ` Yakir Yang
2015-12-16  3:30 ` [PATCH v11 04/19] drm: bridge: analogix/dp: dynamic parse sync_pol & interlace & dynamic_range Yakir Yang
2015-12-16  3:34 ` [PATCH v11 06/19] ARM: dts: exynos/dp: remove some properties that deprecated by analogix_dp driver Yakir Yang
     [not found]   ` <1450236876-1951-1-git-send-email-ykk-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2015-12-22 12:13     ` Jingoo Han
2015-12-23  0:47       ` Yakir Yang [this message]
     [not found] ` <1450236018-1118-1-git-send-email-ykk-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2015-12-16  3:22   ` [PATCH v11 01/19] drm: bridge: analogix/dp: split exynos dp driver to bridge directory Yakir Yang
2015-12-16  3:28   ` [PATCH v11 03/19] drm: bridge: analogix/dp: remove duplicate configuration of link rate and link count Yakir Yang
2015-12-22 12:09     ` Jingoo Han
2015-12-23  0:49       ` Yakir Yang
2015-12-16  3:32   ` [PATCH v11 05/19] dt-bindings: add document for analogix display port driver Yakir Yang
2015-12-16  3:36   ` [PATCH v11 07/19] drm: rockchip: dp: add rockchip platform dp driver Yakir Yang
2015-12-16  3:38 ` [PATCH v11 08/19] dt-bindings: add document for rockchip variant of analogix_dp Yakir Yang
2015-12-16  3:40 ` [PATCH v11 09/19] phy: Add driver for rockchip Display Port PHY Yakir Yang
2015-12-22 12:20   ` Jingoo Han
2015-12-23  0:46     ` Yakir Yang
2015-12-16  3:43 ` [PATCH v11 10/19] dt-bindings: add document for rockchip dp phy Yakir Yang
2015-12-16  3:45 ` [PATCH v11 11/19] drm: rockchip: vop: add bpc and color mode setting Yakir Yang
2015-12-16  3:47 ` [PATCH v11 12/19] drm: bridge: analogix/dp: add some rk3288 special registers setting Yakir Yang
2015-12-16  3:49 ` [PATCH v11 13/19] drm: bridge: analogix/dp: add max link rate and lane count limit for RK3288 Yakir Yang
2015-12-16  3:51 ` [PATCH v11 14/19] drm: bridge: analogix/dp: try force hpd after plug in lookup failed Yakir Yang
2015-12-16  3:53 ` [PATCH v11 15/19] drm: bridge: analogix/dp: move hpd detect to connector detect function Yakir Yang
2015-12-16  3:55 ` [PATCH v11 16/19] drm: bridge: analogix/dp: add edid modes parse in get_modes method Yakir Yang
2015-12-16  3:57 ` [PATCH v11 17/19] drm: bridge: analogix/dp: expand the look time for waiting AUX CH reply Yakir Yang
2015-12-22 12:26   ` Jingoo Han
2015-12-23  4:24     ` Yakir Yang
2015-12-23  6:00       ` Yakir Yang
2015-12-23 15:15         ` Jingoo Han
2015-12-16  3:59 ` [PATCH v11 18/19] drm: bridge: analogix/dp: add panel prepare/unprepare in suspend/resume time Yakir Yang
2015-12-16  4:01 ` [PATCH v11 19/19] drm: bridge: analogix/dp: Fix the possible dead lock in bridge disable time Yakir Yang
2015-12-17 23:51 ` [PATCH v11 0/19] Add Analogix Core Display Port Driver Heiko Stübner
2015-12-18  0:38   ` Yakir Yang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5679EF10.7070807@rock-chips.com \
    --to=ykk@rock-chips.com \
    --cc=andy.yan@rock-chips.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=emil.l.velikov@gmail.com \
    --cc=gustavo.padovan@collabora.co.uk \
    --cc=heiko@sntech.de \
    --cc=inki.dae@samsung.com \
    --cc=javier@osg.samsung.com \
    --cc=jingoohan1@gmail.com \
    --cc=k.kozlowski@samsung.com \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.yao@rock-chips.com \
    --cc=robh+dt@kernel.org \
    --cc=treding@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).