RE: [PATCH v6 1/2] dt-bindings: it6505: add properties to restrict output bandwidth

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



hi

-----Original Message-----
From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> 
Sent: Thursday, November 3, 2022 12:34 AM
To: Allen Chen (陳柏宇) <allen.chen@xxxxxxxxxx>
Cc: treapking@xxxxxxxxxxxx; Jau-Chih Tseng (曾昭智) <Jau-Chih.Tseng@xxxxxxxxxx>; Hermes Wu (吳佳宏) <Hermes.Wu@xxxxxxxxxx>; Kenneth Hung (洪家倫) <Kenneth.Hung@xxxxxxxxxx>; andrzej.hajda@xxxxxxxxx; narmstrong@xxxxxxxxxxxx; robert.foss@xxxxxxxxxx; Laurent.pinchart@xxxxxxxxxxxxxxxx; jonas@xxxxxxxxx; jernej.skrabec@xxxxxxxxx; airlied@xxxxxxxx; daniel@xxxxxxxx; robh+dt@xxxxxxxxxx; krzysztof.kozlowski+dt@xxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH v6 1/2] dt-bindings: it6505: add properties to restrict output bandwidth

On 02/11/2022 02:54, allen.chen@xxxxxxxxxx wrote:
> On 26/10/2022 23:01, allen wrote:
>> From: allen chen <allen.chen@xxxxxxxxxx>
>>
>> Add properties to restrict dp output data-lanes and clock.
> 
> This is a friendly reminder during the review process.
> 
> It seems my previous comments were not fully addressed. Maybe my feedback got lost between the quotes, maybe you just forgot to apply it.
> Please go back to the previous discussion and either implement all requested changes or keep discussing them.
> 
> Thank you.
> 
> ==> I am sorry I didn't find the missing advice you said. Could you explain it again?
> 
> Rob asked - Commit msg should explain reason for breaking users.

And where is the reason? I saw only one sentence "Add properties to restrict dp output data-lanes and clock."
==> I will add the commit msg in v7. Thanks for the suggestion.

Best regards,
Krzysztof





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux