diff mbox

[v5,4/5] arm64: dts: exynos: Remove the OF graph from DSI node

Message ID 1488948852-5380-5-git-send-email-hoegeun.kwon@samsung.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hoegeun Kwon March 8, 2017, 4:54 a.m. UTC
The OF graph is not needed because the panel is a child of dsi. Remove
the ports node in DSI node.

Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
---
 arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 --------------
 1 file changed, 14 deletions(-)

Comments

Krzysztof Kozlowski May 15, 2017, 4:48 p.m. UTC | #1
On Wed, Mar 08, 2017 at 01:54:11PM +0900, Hoegeun Kwon wrote:
> The OF graph is not needed because the panel is a child of dsi. Remove
> the ports node in DSI node.
> 
> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
> ---
>  arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 --------------
>  1 file changed, 14 deletions(-)
>

Hi,

I wanted to apply this (and 5/5) but it seems that DRM change is not
merged. Was it included in v4.12-rc1?

Best regards,
Krzysztof
Hoegeun Kwon May 19, 2017, 6:17 a.m. UTC | #2
On 05/16/2017 01:48 AM, Krzysztof Kozlowski wrote:
> On Wed, Mar 08, 2017 at 01:54:11PM +0900, Hoegeun Kwon wrote:
>> The OF graph is not needed because the panel is a child of dsi. Remove
>> the ports node in DSI node.
>>
>> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
>> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
>> Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
>> ---
>>   arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 --------------
>>   1 file changed, 14 deletions(-)
>>
> Hi,
>
> I wanted to apply this (and 5/5) but it seems that DRM change is not
> merged. Was it included in v4.12-rc1?
>
> Best regards,
> Krzysztof

Hi,

Sorry for my belated reply.

The DRM patch(3/5) is not merged in v4.12-rc1 (mainline)...
I have checked the David's branch, but not merged.
However it was merged into Inki's exynos drm branch.

Inki, Could you check the patch(3/5).

Best regards,
Hoegeun
Hoegeun Kwon June 7, 2017, 6:14 a.m. UTC | #3
On 05/19/2017 03:17 PM, Hoegeun Kwon wrote:
> On 05/16/2017 01:48 AM, Krzysztof Kozlowski wrote:
>> On Wed, Mar 08, 2017 at 01:54:11PM +0900, Hoegeun Kwon wrote:
>>> The OF graph is not needed because the panel is a child of dsi. Remove
>>> the ports node in DSI node.
>>>
>>> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
>>> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
>>> Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
>>> ---
>>>   arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 
>>> --------------
>>>   1 file changed, 14 deletions(-)
>>>
>> Hi,
>>
>> I wanted to apply this (and 5/5) but it seems that DRM change is not
>> merged. Was it included in v4.12-rc1?
>>
>> Best regards,
>> Krzysztof
>
> Hi,
>
> Sorry for my belated reply.
>
> The DRM patch(3/5) is not merged in v4.12-rc1 (mainline)...
> I have checked the David's branch, but not merged.
> However it was merged into Inki's exynos drm branch.
>
> Inki, Could you check the patch(3/5).
>
> Best regards,
> Hoegeun

Hi Krzysztof,

The drm patch(3/5) has been merged into v4.12-rc4.

Best regards,
Hoegeun
Krzysztof Kozlowski June 9, 2017, 7:37 a.m. UTC | #4
On Wed, Jun 07, 2017 at 03:14:20PM +0900, Hoegeun Kwon wrote:
> On 05/19/2017 03:17 PM, Hoegeun Kwon wrote:
> >On 05/16/2017 01:48 AM, Krzysztof Kozlowski wrote:
> >>On Wed, Mar 08, 2017 at 01:54:11PM +0900, Hoegeun Kwon wrote:
> >>>The OF graph is not needed because the panel is a child of dsi. Remove
> >>>the ports node in DSI node.
> >>>
> >>>Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
> >>>Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> >>>Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
> >>>---
> >>>  arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14
> >>>--------------
> >>>  1 file changed, 14 deletions(-)
> >>>
> >>Hi,
> >>
> >>I wanted to apply this (and 5/5) but it seems that DRM change is not
> >>merged. Was it included in v4.12-rc1?
> >>
> >>Best regards,
> >>Krzysztof
> >
> >Hi,
> >
> >Sorry for my belated reply.
> >
> >The DRM patch(3/5) is not merged in v4.12-rc1 (mainline)...
> >I have checked the David's branch, but not merged.
> >However it was merged into Inki's exynos drm branch.
> >
> >Inki, Could you check the patch(3/5).
> >
> >Best regards,
> >Hoegeun
> 
> Hi Krzysztof,
> 
> The drm patch(3/5) has been merged into v4.12-rc4.

Thanks for letting me know. My branch is rebased on v4.12-rc1 so to
maintain bisectabiltiy I would have to merge v4.12-rc4... thus making a
messy history.

It is just a cleanup so I will take it after releasing v4.12, so this
will end up in v4.14. No need to resend, I'll keep it in Patchwork.

Best regards,
Krzysztof
Krzysztof Kozlowski July 18, 2017, 5:03 a.m. UTC | #5
On Wed, Mar 08, 2017 at 01:54:11PM +0900, Hoegeun Kwon wrote:
> The OF graph is not needed because the panel is a child of dsi. Remove
> the ports node in DSI node.
> 
> Signed-off-by: Hoegeun Kwon <hoegeun.kwon@samsung.com>
> Reviewed-by: Andrzej Hajda <a.hajda@samsung.com>
> Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
> ---
>  arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi | 14 --------------
>  1 file changed, 14 deletions(-)
> 

Thanks, applied this and 5/5.

Best regards,
Krzysztof
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
index e31e20c..77ba238 100644
--- a/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
+++ b/arch/arm64/boot/dts/exynos/exynos5433-tm2-common.dtsi
@@ -303,20 +303,6 @@ 
 	samsung,pll-clock-frequency = <24000000>;
 	pinctrl-names = "default";
 	pinctrl-0 = <&te_irq>;
-
-	ports {
-		#address-cells = <1>;
-		#size-cells = <0>;
-
-		port@1 {
-			reg = <1>;
-
-			dsi_out: endpoint {
-				samsung,burst-clock-frequency = <512000000>;
-				samsung,esc-clock-frequency = <16000000>;
-			};
-		};
-	};
 };
 
 &hdmi {