On 02/09/2025 08:38, Krzysztof Kozlowski wrote: > On Mon, Sep 01, 2025 at 05:57:30PM +0800, Yongxing Mou wrote: >> + - if: >> + properties: >> + compatible: >> + contains: >> + enum: >> + # QCS8300 only has one DP controller that supports 4 >> + # streams MST. >> + - qcom,qcs8300-dp >> + then: >> + properties: >> + reg: >> + minItems: 9 >> + maxItems: 9 >> + clocks: >> + minItems: 8 >> + maxItems: 8 > > Clocks have only five items, reg has 5. At least in my next from few > days ago. > > Nothing explains any patchset dependencies, so this makes reviewing more > difficult than it should be. OK, I found the dependency in cover letter (I was looking at wrong cover letter), but the dependency does not have relevant clock changes, so this is still wrong.
I suggest to slow down with this patchset. It's v10 already with simultaneous other changes and two (!!!) dependencies. I don't know how I am supposed to review this. Best regards, Krzysztof
