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: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

Date: Tue, 6 Oct 2020 10:23:28 +0900

Cc: Albert Ou <aou@xxxxxxxxxxxxxxxxx>, Alessandro Zummo <a.zummo@xxxxxxxxxxxx>, Andrew Lunn <andrew@xxxxxxx>, Bjorn Helgaas <bhelgaas@xxxxxxxxxx>, 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>, Srinivas Kandagatla <srinivas.kandagatla@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

Cms-type: 101P

Dlp-filter: Pass

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

Organization: Samsung Electronics

References: <20201005183830.486085-1-robh@kernel.org> <CGME20201005183933epcas1p36d2c9cc1f3ad9180421269ca74330d65@epcas1p3.samsung.com> <20201005183830.486085-5-robh@kernel.org>

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

On 10/6/20 3:38 AM, 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>
> ---
(snip)

>  Documentation/devicetree/bindings/extcon/wlf,arizona.yaml    | 2 ++
(snip)

For the extcon part,
Acked-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics