Hi Conor,

On 5/15/2026 2:16 AM, Conor Dooley wrote:
On Thu, May 14, 2026 at 03:01:31PM +0800, Damon Ding wrote:
Expose the inherited properties from the base analogix-dp schema
to satisfy unevaluatedProperties constraints.

Signed-off-by: Damon Ding <[email protected]>

Given it's unevaluatedProperties, not addtionalProperties, this patch
shouldn't be needed?


When I remove both the top-level data-lanes property and those explicit "xxx: true" property entries and run the dtbs check with:

make CHECK_DTBS=y CROSS_COMPILE=aarch64-linux-gnu- LT0=none LLVM=1 LLVM_IAS=1 ARCH=arm64 rockchip/rk3588-evb1-v10.dtb rockchip/rk3588s-evb1-v10.dtb rockchip/rk3399-sapphire-excavator.dtb rockchip/rk3576-evb1-v10.dtb -j4

It results in validation errors like these:

/home/ding/drm-misc/arch/arm64/boot/dts/rockchip/rk3576-evb1-v10.dtb: edp@27dc0000: ports:port@1:endpoint: Unevaluated properties are not allowed ('data-lanes' was unexpected) from schema $id: http://devicetree.org/schemas/display/rockchip/rockchip,analogix-dp.yaml# /home/ding/drm-misc/arch/arm64/boot/dts/rockchip/rk3576-evb1-v10.dtb: edp@27dc0000: Unevaluated properties are not allowed ('force-hpd', 'interrupts', 'phy-names', 'phys', 'ports', 'reg' were unexpected) from schema $id: http://devicetree.org/schemas/display/rockchip/rockchip,analogix-dp.yaml#

I suspect that the properties defined in the child binding are overriding/masking all the inherited properties from the parent Analogix DP schema.

Is there a better way to fix this issue without explicitly listing all inherited properties as true?

Best regards,
Damon

---
  .../bindings/display/rockchip/rockchip,analogix-dp.yaml    | 7 +++++++
  1 file changed, 7 insertions(+)

diff --git 
a/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml 
b/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml
index bb75d898a5c5..896ded87880f 100644
--- 
a/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml
+++ 
b/Documentation/devicetree/bindings/display/rockchip/rockchip,analogix-dp.yaml
@@ -50,6 +50,13 @@ properties:
    aux-bus:
      $ref: /schemas/display/dp-aux-bus.yaml#
+ reg: true
+  interrupts: true
+  phys: true
+  phy-names: true
+  force-hpd: true
+  ports: true
+
  required:
    - compatible
    - clocks
--
2.34.1



Reply via email to