From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4FDBDCD4851 for ; Fri, 15 May 2026 09:05:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=d+OGVwo7tV7QO9EfJdCj6RNBtoSTwlo4CIAJDKBdNME=; b=fDd4EWlnbXeFSLts/wSpBrmnmV AmXR8Pd84r9bsK+JjJa73vhf1s4iN/5ftU+Z0tbgUiGUoLnkwKn+60aqPgVnX/+5i/+zeof5eNTDL sVWCjot8Wp9TkyADbnaRs+JVdM9doL0OGl3KD3a9rBPbCbK5seDmg1cP5z3665dcZikxklHYWMk4Q mn+hdgWHDc9W6WDND0p65HeZq4NPk3LEaaTwm4h9yEyCVb68Kc1w69yIAQRucAe2iSVBsfKJHWb8Z ewOohWhpuDM0EE1LLD+k3aW18jLn+T+TapkJGB31x69Jb0L8fn0D4Qqbp8aXVDhe9/nAehkdwdjHx yJ8twJAQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wNoTq-00000007q53-2UYT; Fri, 15 May 2026 09:05:14 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wNoTm-00000007q4E-3Cy0; Fri, 15 May 2026 09:05:12 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1778835910; x=1810371910; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=JF3uZsy6llHlDV7C7vLRRmWCmal9y3CobKnloKpst78=; b=ZJLs4KcruAf40DLEE7vJTivHTBR4tb4kI8omSBSouzZZ7Z2rlrR4PXim Ona2PF9OpHVsWaM4U4wBHBNM89MdcQqy2KmihXIaNaqDNH1XyChd1YP1g ZaFfopK9Ijy88vTB2vaK1dIp3iCPz3OVLlFdU+oyZkIxLntc+f+jY3/77 /sqqL4t74/YBwCvnnZCmk4WiC9wuFJZ8E3yKw7RjOedBBJpS2+jhQw8ht 9LwRsiINbM6q9LL9o6D5/gmBBw/AvILR9x2a9vvR3cbYP85ToGsKqBUFq IOUbt1Mr5i1vsrUF3asxbox+ca2J5tBWMKyrw7rd6Ly2EpGSgzJrlN7xM g==; X-CSE-ConnectionGUID: aBy8wa7JRn+FUchmSVlp4w== X-CSE-MsgGUID: fbVAwTCTQfC/IGhAcLgkpA== X-IronPort-AV: E=Sophos;i="6.23,236,1770620400"; d="asc'?scan'208";a="57964607" X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN Received: from unknown (HELO email.microchip.com) ([170.129.1.10]) by esa2.microchip.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 May 2026 02:05:08 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.87.71) by chn-vm-ex2.mchp-main.com (10.10.87.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.2.2562.37; Fri, 15 May 2026 02:05:08 -0700 Received: from wendy (10.10.85.11) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.58 via Frontend Transport; Fri, 15 May 2026 02:05:03 -0700 Date: Fri, 15 May 2026 10:04:25 +0100 From: Conor Dooley To: Damon Ding Subject: Re: [PATCH v1 1/3] dt-bindings: display: rockchip: analogix-dp: Expose inherited properties Message-ID: <20260515-confess-hungrily-4f64ce0d8de2@wendy> References: <20260514070133.2275069-1-damon.ding@rock-chips.com> <20260514070133.2275069-2-damon.ding@rock-chips.com> <20260514-upstate-sneer-0b6e78682798@spud> <2f653664-27e9-4632-97e9-8b59cf7e585e@rock-chips.com> MIME-Version: 1.0 In-Reply-To: <2f653664-27e9-4632-97e9-8b59cf7e585e@rock-chips.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260515_020511_064951_CC0182ED X-CRM114-Status: GOOD ( 25.88 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: heiko@sntech.de, dri-devel@lists.freedesktop.org, dianders@chromium.org, Laurent.pinchart@ideasonboard.com, andrzej.hajda@intel.com, airlied@gmail.com, m.szyprowski@samsung.com, simona@ffwll.ch, robh@kernel.org, rfoss@kernel.org, sebastian.reichel@collabora.com, jernej.skrabec@gmail.com, linux-rockchip@lists.infradead.org, luca.ceresoli@bootlin.com, devicetree@vger.kernel.org, conor+dt@kernel.org, jonas@kwiboo.se, maarten.lankhorst@linux.intel.com, mripard@kernel.org, linux-arm-kernel@lists.infradead.org, dmitry.baryshkov@oss.qualcomm.com, neil.armstrong@linaro.org, hjc@rock-chips.com, Conor Dooley , linux-kernel@vger.kernel.org, tzimmermann@suse.de, andy.yan@rock-chips.com, krzk+dt@kernel.org, nicolas.frattaroli@collabora.com Content-Type: multipart/mixed; boundary="===============5258423530061441651==" Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org --===============5258423530061441651== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="JTEOF7u1knQLxomm" Content-Disposition: inline --JTEOF7u1knQLxomm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, May 15, 2026 at 11:57:58AM +0800, Damon Ding wrote: > Hi Conor, >=20 > 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. > > >=20 > > > Signed-off-by: Damon Ding > >=20 > > Given it's unevaluatedProperties, not addtionalProperties, this patch > > shouldn't be needed? > >=20 >=20 > When I remove both the top-level data-lanes property and those explicit > "xxx: true" property entries and run the dtbs check with: >=20 > make CHECK_DTBS=3Dy CROSS_COMPILE=3Daarch64-linux-gnu- LT0=3Dnone LLVM=3D1 > LLVM_IAS=3D1 ARCH=3Darm64 rockchip/rk3588-evb1-v10.dtb > rockchip/rk3588s-evb1-v10.dtb rockchip/rk3399-sapphire-excavator.dtb > rockchip/rk3576-evb1-v10.dtb -j4 >=20 > It results in validation errors like these: >=20 > /home/ding/drm-misc/arch/arm64/boot/dts/rockchip/rk3576-evb1-v10.dtb: > edp@27dc0000: ports:port@1:endpoint: Unevaluated properties are not allow= ed > ('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# >=20 > I suspect that the properties defined in the child binding are > overriding/masking all the inherited properties from the parent Analogix = DP > schema. >=20 > Is there a better way to fix this issue without explicitly listing all > inherited properties as true? The example in this file uses most of the properties that you mention above: dp@ff970000 { compatible =3D "rockchip,rk3288-dp"; reg =3D <0xff970000 0x4000>; interrupts =3D ; clocks =3D <&cru SCLK_EDP>, <&cru PCLK_EDP_CTRL>; clock-names =3D "dp", "pclk"; phys =3D <&dp_phy>; phy-names =3D "dp"; resets =3D <&cru 111>; reset-names =3D "dp"; rockchip,grf =3D <&grf>; pinctrl-0 =3D <&edp_hpd>; pinctrl-names =3D "default"; dt_binding_check reports no problems with this node, so I think the problem might lie elsewhere? There's no edp node in the dts you mention above, so this looks like an interaction with something that's not yet upstream. If this is required for the rk3576 edp, then you should include this patch in the rk3576 edp support series rather than this one anyway where it can actually be evaluated alongside the node it apparently causes problems with. pw-bot: changes-requested Thanks, Conor. >=20 > Best regards, > Damon >=20 > > > --- > > > .../bindings/display/rockchip/rockchip,analogix-dp.yaml | 7 ++++= +++ > > > 1 file changed, 7 insertions(+) > > >=20 > > > diff --git a/Documentation/devicetree/bindings/display/rockchip/rockc= hip,analogix-dp.yaml b/Documentation/devicetree/bindings/display/rockchip/r= ockchip,analogix-dp.yaml > > > index bb75d898a5c5..896ded87880f 100644 > > > --- a/Documentation/devicetree/bindings/display/rockchip/rockchip,ana= logix-dp.yaml > > > +++ b/Documentation/devicetree/bindings/display/rockchip/rockchip,ana= logix-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 > > > --=20 > > > 2.34.1 > > >=20 > > >=20 >=20 --JTEOF7u1knQLxomm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCagbhmQAKCRB4tDGHoIJi 0hL9AQDMzOGCBAaj30x0MldbIEzDvxHY9JZ/Hr59JunC6U0AhQEAhrUYE74I+05k njcJ1OaOANoefame5d09LLbMsepD7Q4= =jYCg -----END PGP SIGNATURE----- --JTEOF7u1knQLxomm-- --===============5258423530061441651== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip --===============5258423530061441651==--