Re: [PATCH 6/6] dt-bindings: display: samsung,exynos7-decon: add exynos7870 compatible

[Kaustabh Chakraborty <kauschluss@disroot.org>]

On 2024-09-26 05:34, Kwanghoon Son wrote:
> On Wed, 2024-09-25 at 20:05 +0000, Kaustabh Chakraborty wrote:
>> On 2024-09-25 19:56, Krzysztof Kozlowski wrote:
>> > On 25/09/2024 21:36, Kaustabh Chakraborty wrote:
>> > > On 2024-09-25 19:25, Krzysztof Kozlowski wrote:
>> > > > On 25/09/2024 20:42, Kaustabh Chakraborty wrote:
>> > > > > On 2024-09-20 12:39, Krzysztof Kozlowski wrote:
>> > > > > > On 19/09/2024 17:20, Kaustabh Chakraborty wrote:
>> > > > > > > Add the compatible string of Exynos7870 to the existing list.
>> > > > > > > 
>> > > > > > > Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org>
>> > > > > > 
>> > > > > > ... and the DTS is <please provide lore ink in changelog>?
>> > > > > 
>> > > > > Didn't quite understand. The patch adds the compatible string
>> > > > > for Exynos7870 DECON in documentation. There's no DTS involved
>> > > > > in here, right?
>> > > > 
>> > > > Provide lore link to the DTS submission.
>> > > 
>> > > There aren't any DTS submissions *yet* which use the compatible.
>> > > Is that an issue?
>> > > 
>> > 
>> > Yeah, users are supposed to be upstream. Not downstream.
>> 
>> I understand that. I had plans to submit it in the future.
>> If that's how it's meant to be done, I'll have to revisit this
>> submission at a later date then.
> 
> Hi, may I ask for reason that you don't submit dts?

It's not that I don't want, I have nowhere to submit it to yet. As
pointed out by Krzysztof, support for exynos7870 doesn't exist in
upstream. I'll get back to this after I get that finished.

> I am asking because I wonder if there is an issue related to DPP.

Exynos7870 doesn't have DPP blocks as far as I could tell. Let me
know if I misunderstood anything.

> 
> https://lore.kernel.org/linux-samsung-soc/2e4d3d180f535e57d9cb98e7bac1d14b51ffc5d4.camel@gmail.com/#t
> 
> Best regards,
> kwang.
> 
>> 
>> > 
>> > Best regards,
>> > Krzysztof