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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 68669C27C76 for ; Wed, 25 Jan 2023 15:10:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235964AbjAYPK4 (ORCPT ); Wed, 25 Jan 2023 10:10:56 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38334 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233235AbjAYPKt (ORCPT ); Wed, 25 Jan 2023 10:10:49 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9EE2344BDC; Wed, 25 Jan 2023 07:10:48 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 39E826154D; Wed, 25 Jan 2023 15:10:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A53B0C4339E; Wed, 25 Jan 2023 15:10:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1674659447; bh=TofsbXpZihSKZUVsy9XrANh9lc730J6wSOc2QP9eoTI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=PWvP9JOJGFWee6ASV9x3ADUxkKdcW+FrahCSA7zAki85HcwH0S+Qlftb5AVSx76V4 qBpzvZbPNozQG37LDylg+pITOqfq3rlpjhbTAHNw0Oafns2nuDLECj60B1OrLEWMnV /DBCokptbh1fiSNjleo4CBNt1NKQdn8kPf8iqz68WINnJ4LRJldl0YhHV/lKBLFxj7 iiB6+dKsKTB+B07SQ3EH4R91GxqW6sbp7ZwJOpScy+CeEyBddDco4n5ScTpc1xMwwc EyMfjtYYEe2OMD5XlBJx97JPAnsmnx3im+jMYOD9UXRGxoX7/db0GPAIhfy7NJWA1c KRS15JP4gUAzw== Date: Wed, 25 Jan 2023 15:10:38 +0000 From: Lee Jones To: Rob Herring Cc: Krzysztof Kozlowski , David Airlie , Daniel Vetter , Bartosz Golaszewski , Jean Delvare , Guenter Roeck , Thomas Gleixner , Marc Zyngier , Jassi Brar , Mauro Carvalho Chehab , Ulf Hansson , Richard Weinberger , Vignesh Raghavendra , Sebastian Reichel , Mark Brown , "Rafael J. Wysocki" , Daniel Lezcano , Greg Kroah-Hartman , linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org, linux-gpio@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-media@vger.kernel.org, openbmc@lists.ozlabs.org, linux-mmc@vger.kernel.org, linux-mtd@lists.infradead.org, linux-pm@vger.kernel.org, alsa-devel@alsa-project.org, linux-spi@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH] dt-bindings: Add missing (unevaluated|additional)Properties on child node schemas Message-ID: References: <20230124230048.371144-1-robh@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230124230048.371144-1-robh@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-hwmon@vger.kernel.org On Tue, 24 Jan 2023, Rob Herring wrote: > Just as unevaluatedProperties or additionalProperties are required at > the top level of schemas, they should (and will) also be required for > child node schemas. That ensures only documented properties are > present. > > Add unevaluatedProperties or additionalProperties as appropriate, and > then add any missing properties flagged by the addition. > > Signed-off-by: Rob Herring > --- > To: Krzysztof Kozlowski > To: David Airlie > To: Daniel Vetter > To: Bartosz Golaszewski > To: Jean Delvare > To: Guenter Roeck > To: Thomas Gleixner > To: Marc Zyngier > To: Jassi Brar > To: Mauro Carvalho Chehab > To: Lee Jones > To: Ulf Hansson > To: Richard Weinberger > To: Vignesh Raghavendra > To: Sebastian Reichel > To: Mark Brown > To: "Rafael J. Wysocki" > To: Daniel Lezcano > To: Greg Kroah-Hartman > Cc: linux-arm-kernel@lists.infradead.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-gpio@vger.kernel.org > Cc: linux-hwmon@vger.kernel.org > Cc: linux-media@vger.kernel.org > Cc: openbmc@lists.ozlabs.org > Cc: linux-mmc@vger.kernel.org > Cc: linux-mtd@lists.infradead.org > Cc: linux-pm@vger.kernel.org > Cc: alsa-devel@alsa-project.org > Cc: linux-spi@vger.kernel.org > Cc: linux-usb@vger.kernel.org > --- > .../devicetree/bindings/arm/arm,vexpress-juno.yaml | 1 + > .../bindings/arm/bcm/raspberrypi,bcm2835-firmware.yaml | 5 +++-- > .../bindings/arm/tegra/nvidia,tegra20-pmc.yaml | 4 ++++ > .../bindings/bus/allwinner,sun50i-a64-de2.yaml | 1 + > .../bindings/bus/allwinner,sun8i-a23-rsb.yaml | 1 + > .../bus/intel,ixp4xx-expansion-bus-controller.yaml | 6 ++++++ > Documentation/devicetree/bindings/bus/palmbus.yaml | 1 + > .../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 +++++ > Documentation/devicetree/bindings/example-schema.yaml | 2 ++ > .../devicetree/bindings/gpio/x-powers,axp209-gpio.yaml | 1 + > .../devicetree/bindings/hwmon/adi,ltc2992.yaml | 1 + > .../bindings/interrupt-controller/arm,gic-v3.yaml | 2 ++ > .../bindings/mailbox/xlnx,zynqmp-ipi-mailbox.yaml | 1 + > .../devicetree/bindings/media/i2c/maxim,max9286.yaml | 7 +++++++ > .../bindings/memory-controllers/arm,pl35x-smc.yaml | 1 + > .../bindings/memory-controllers/exynos-srom.yaml | 1 + > .../memory-controllers/nvidia,tegra124-emc.yaml | 1 + > .../bindings/memory-controllers/st,stm32-fmc2-ebi.yaml | 1 + > .../devicetree/bindings/mfd/mediatek,mt6370.yaml | 2 ++ > .../devicetree/bindings/mmc/aspeed,sdhci.yaml | 1 + > Documentation/devicetree/bindings/mtd/mtd.yaml | 1 + > .../devicetree/bindings/power/supply/ti,lp8727.yaml | 1 + > .../devicetree/bindings/soc/imx/fsl,imx93-src.yaml | 3 ++- > .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 1 + > Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml | 1 + > .../devicetree/bindings/sound/marvell,mmp-sspa.yaml | 1 + > .../devicetree/bindings/sound/qcom,wcd934x.yaml | 1 + > .../devicetree/bindings/sound/samsung,odroid.yaml | 2 ++ > .../devicetree/bindings/soundwire/qcom,soundwire.yaml | 1 + > .../bindings/spi/allwinner,sun4i-a10-spi.yaml | 1 + > .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + > .../devicetree/bindings/spi/spi-controller.yaml | 1 + > .../sram/allwinner,sun4i-a10-system-control.yaml | 10 +++++----- > Documentation/devicetree/bindings/sram/qcom,ocmem.yaml | 1 + > .../devicetree/bindings/thermal/thermal-zones.yaml | 1 + > Documentation/devicetree/bindings/usb/qcom,dwc3.yaml | 1 + > 36 files changed, 65 insertions(+), 8 deletions(-) Is this the same as the patch I just reviewed? -- Lee Jones [李琼斯]