Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

Date Prev

Date Next

Thread Prev

Thread Next

Date Index

Thread Index

Subject: Re: [PATCH 4/4] dt-bindings: Explicitly allow additional properties in common schemas

From: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>

Date: Tue, 6 Oct 2020 09:22:48 +0100

Cc: Albert Ou <aou@xxxxxxxxxxxxxxxxx>, Alessandro Zummo <a.zummo@xxxxxxxxxxxx>, Andrew Lunn <andrew@xxxxxxx>, Bjorn Helgaas <bhelgaas@xxxxxxxxxx>, Chanwoo Choi <cw00.choi@xxxxxxxxxxx>, Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, dmaengine@xxxxxxxxxxxxxxx, Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Heiner Kallweit <hkallweit1@xxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Jason Cooper <jason@xxxxxxxxxxxxxx>, Jens Axboe <axboe@xxxxxxxxx>, Jonathan Cameron <jic23@xxxxxxxxxx>, Krzysztof Kozlowski <krzk@xxxxxxxxxx>, Lars-Peter Clausen <lars@xxxxxxxxxx>, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>, Lee Jones <lee.jones@xxxxxxxxxx>, Liam Girdwood <lgirdwood@xxxxxxxxx>, Linus Walleij <linus.walleij@xxxxxxxxxx>, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>, Marc Zyngier <maz@xxxxxxxxxx>, Mark Brown <broonie@xxxxxxxxxx>, Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Miquel Raynal <miquel.raynal@xxxxxxxxxxx>, MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>, Palmer Dabbelt <palmer@xxxxxxxxxxx>, Paul Walmsley <paul.walmsley@xxxxxxxxxx>, Pavel Machek <pavel@xxxxxx>, Peter Meerwald-Stadler <pmeerw@xxxxxxxxxx>, Richard Weinberger <richard@xxxxxx>, Sam Ravnborg <sam@xxxxxxxxxxxx>, Sebastian Reichel <sre@xxxxxxxxxx>, Stephen Boyd <sboyd@xxxxxxxxxx>, Thierry Reding <thierry.reding@xxxxxxxxx>, Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Ulf Hansson <ulf.hansson@xxxxxxxxxx>, Vignesh Raghavendra <vigneshr@xxxxxx>, Vinod Koul <vkoul@xxxxxxxxxx>, Viresh Kumar <vireshk@xxxxxxxxxx>, Vivien Didelot <vivien.didelot@xxxxxxxxx>, Wim Van Sebroeck <wim@xxxxxxxxxxxxxxxxxx>, Wolfgang Grandegger <wg@xxxxxxxxxxxxxx>, Zhang Rui <rui.zhang@xxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-can@xxxxxxxxxxxxxxx, linux-clk@xxxxxxxxxxxxxxx, linux-gpio@xxxxxxxxxxxxxxx, linux-hwmon@xxxxxxxxxxxxxxx, linux-i2c@xxxxxxxxxxxxxxx, linux-ide@xxxxxxxxxxxxxxx, linux-iio@xxxxxxxxxxxxxxx, linux-input@xxxxxxxxxxxxxxx, linux-leds@xxxxxxxxxxxxxxx, linux-media@xxxxxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxxx, linux-mmc@xxxxxxxxxxxxxxx, linux-mtd@xxxxxxxxxxxxxxxxxxx, linux-pci@xxxxxxxxxxxxxxx, linux-pm@xxxxxxxxxxxxxxx, linux-pwm@xxxxxxxxxxxxxxx, linux-riscv@xxxxxxxxxxxxxxxxxxx, linux-rtc@xxxxxxxxxxxxxxx, linux-serial@xxxxxxxxxxxxxxx, linux-spi@xxxxxxxxxxxxxxx, linux-usb@xxxxxxxxxxxxxxx, linux-watchdog@xxxxxxxxxxxxxxx

In-reply-to: <20201005183830.486085-5-robh@kernel.org>

References: <20201005183830.486085-1-robh@kernel.org> <20201005183830.486085-5-robh@kernel.org>

User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0



On 05/10/2020 19:38, Rob Herring wrote:

In order to add meta-schema checks for additional/unevaluatedProperties
being present, all schema need to make this explicit. As common/shared
schema are included by other schemas, they should always allow for
additionalProperties.

Signed-off-by: Rob Herring<robh@xxxxxxxxxx>

  Documentation/devicetree/bindings/nvmem/nvmem-consumer.yaml  | 2 ++
  Documentation/devicetree/bindings/nvmem/nvmem.yaml           | 2 ++


for nvmem parts,

Acked-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>

thanks,
srini