On Thu, Dec 19, 2024 at 04:05:51PM +0800, Damon Ding wrote: > Compared with RK3288/RK3399, the HBR2 link rate support is the main > improvement of RK3588 eDP TX controller, and there are also two > independent eDP display interfaces on RK3588 Soc. > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Drop, patch was very different. > Signed-off-by: Damon Ding <damon.ding@xxxxxxxxxxxxxx> > > --- > > Changes in v2: > - Add the main defferences of the RK3588 eDP and the previous versions > in commit message > > Changes in v3: > - expand the property clock-names, resets and reset-names Adding new properties invalidates reviews. The patch is entirely, really significantly different so keeping the review is not acceptable. > --- > .../display/rockchip/rockchip,analogix-dp.yaml | 14 +++++++++++--- > 1 file changed, 11 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml b/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml > index 60dedf9b2be7..4f84e8849876 100644 > --- a/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml > +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml > @@ -15,6 +15,7 @@ properties: > enum: > - rockchip,rk3288-dp > - rockchip,rk3399-edp > + - rockchip,rk3588-edp > > clocks: > minItems: 2 > @@ -25,16 +26,23 @@ properties: > items: > - const: dp > - const: pclk > - - const: grf > + - enum: > + - grf > + - spdif No, this cannot be flexible for existing variants. > > power-domains: > maxItems: 1 > > resets: > - maxItems: 1 > + minItems: 1 > + maxItems: 2 No, you must list the items. > > reset-names: > - const: dp > + minItems: 1 > + items: > + - const: dp > + - enum: > + - apb I don't understand what happened here but all this is NAK. Why suddenly existing variants became different/changed? Nothing in commit msg explains this. Best regards, Krzysztof
