On 10/8/25 2:49 PM, Dmitry Baryshkov wrote:
> On Wed, Oct 08, 2025 at 02:40:35PM +0200, Konrad Dybcio wrote:
>> On 9/26/25 10:59 AM, Mani Chandana Ballary Kuntumalla wrote:
>>> Add device tree nodes for the mdss1 DPTX0 and DPTX1 controllers
>>> with their corresponding PHYs.
>>>
>>> Signed-off-by: Mani Chandana Ballary Kuntumalla <[email protected]>
>>> ---
>>
>> [...]
>>
>>> + mdss1_dp0: displayport-controller@22154000 {
>>> + compatible = "qcom,sa8775p-dp";
>>> +
>>> + reg = <0x0 0x22154000 0x0 0x104>,
>>
>> sz = 0x200
>
> Hmm, why? I think the memory map specifies these sizes.
Yeah and I went in line with the allocated region size (not last register)
[...]
>>> dispcc1: clock-controller@22100000 {
>>> @@ -6872,6 +7115,8 @@ dispcc1: clock-controller@22100000 {
>>> <&rpmhcc RPMH_CXO_CLK>,
>>> <&rpmhcc RPMH_CXO_CLK_A>,
>>> <&sleep_clk>,
>>> + <&mdss1_dp0_phy 0>, <&mdss1_dp0_phy 1>,
>>> + <&mdss1_dp1_phy 0>, <&mdss1_dp1_phy 1>,
>>> <0>, <0>, <0>, <0>,
>>
>> You need to remove the same amount of zeroes that you added
>
> Nice catch. Which means that somebody was sending untested patches.
It would work.. until someone tried to add DSI to the second row of zeroes
instead of the first one
Konrad> Nice.
>
