[PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
Another round of wack-a-mole. The json-schema default is additional unknown properties are allowed, but for DT all properties should be defined. Cc: Thierry Reding Cc: Linus Walleij Cc: Stephen Boyd Cc: Shawn Guo Cc: Bjorn Andersson Cc: Baolin Wang Cc: Guenter Roeck Cc: Jonathan Cameron Cc: Mauro Carvalho Chehab Cc: Laurent Pinchart Cc: Lee Jones Cc: Ulf Hansson Cc: "David S. Miller" Cc: Bjorn Helgaas Cc: Vinod Koul Cc: Liam Girdwood Cc: Mark Brown Cc: Greg Kroah-Hartman Cc: Daniel Lezcano Cc: linux-...@vger.kernel.org Cc: dri-de...@lists.freedesktop.org Cc: linux-...@vger.kernel.org Cc: linux-g...@vger.kernel.org Cc: linux-hw...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: openipmi-develo...@lists.sourceforge.net Cc: linux-l...@vger.kernel.org Cc: linux-me...@vger.kernel.org Cc: linux-rockc...@lists.infradead.org Cc: linux-st...@st-md-mailman.stormreply.com Cc: linux-m...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: net...@vger.kernel.org Cc: linux-...@vger.kernel.org Cc: linux...@vger.kernel.org Cc: linux-remotep...@vger.kernel.org Cc: linux-ser...@vger.kernel.org Cc: alsa-de...@alsa-project.org Cc: linux-...@vger.kernel.org Signed-off-by: Rob Herring --- I'll take this thru the DT tree. .../arm/bcm/raspberrypi,bcm2835-firmware.yaml | 2 ++ .../arm/mediatek/mediatek,pericfg.yaml| 2 ++ .../devicetree/bindings/arm/pmu.yaml | 2 ++ .../devicetree/bindings/arm/primecell.yaml| 3 +++ .../bindings/arm/samsung/sysreg.yaml | 2 ++ .../arm/tegra/nvidia,tegra20-pmc.yaml | 2 ++ .../bindings/bus/mti,mips-cdmm.yaml | 2 ++ .../bus/socionext,uniphier-system-bus.yaml| 7 +++ .../bindings/clock/arm,syscon-icst.yaml | 2 ++ .../bindings/clock/idt,versaclock5.yaml | 20 ++- .../bindings/clock/imx6q-clock.yaml | 2 ++ .../bindings/clock/imx6sl-clock.yaml | 2 ++ .../bindings/clock/imx6sll-clock.yaml | 2 ++ .../bindings/clock/imx6sx-clock.yaml | 2 ++ .../bindings/clock/imx6ul-clock.yaml | 2 ++ .../bindings/clock/intel,cgu-lgm.yaml | 2 ++ .../bindings/clock/qcom,gcc-sm8250.yaml | 2 ++ .../bindings/clock/sprd,sc9863a-clk.yaml | 2 ++ .../bindings/clock/ti,am654-ehrpwm-tbclk.yaml | 2 ++ .../bindings/display/bridge/ite,it6505.yaml | 5 + .../bindings/display/bridge/lvds-codec.yaml | 3 +++ .../devicetree/bindings/display/msm/gmu.yaml | 2 ++ .../devicetree/bindings/edac/dmc-520.yaml | 2 ++ .../devicetree/bindings/fsi/ibm,fsi2spi.yaml | 2 ++ .../gpio/socionext,uniphier-gpio.yaml | 2 ++ .../bindings/hwmon/adi,axi-fan-control.yaml | 2 ++ .../devicetree/bindings/hwmon/adt7475.yaml| 2 ++ .../bindings/iio/accel/kionix,kxsd9.yaml | 4 .../bindings/iio/adc/maxim,max1238.yaml | 2 ++ .../bindings/iio/adc/maxim,max1363.yaml | 2 ++ .../bindings/iio/adc/qcom,spmi-vadc.yaml | 4 .../bindings/iio/adc/samsung,exynos-adc.yaml | 2 ++ .../bindings/iio/adc/ti,ads8688.yaml | 4 .../bindings/iio/amplifiers/adi,hmc425a.yaml | 2 ++ .../bindings/iio/imu/invensense,icm42600.yaml | 6 ++ .../bindings/iio/light/amstaos,tsl2563.yaml | 2 ++ .../bindings/iio/light/dynaimage,al3010.yaml | 2 ++ .../bindings/iio/light/dynaimage,al3320a.yaml | 2 ++ .../bindings/iio/light/sharp,gp2ap002.yaml| 2 ++ .../iio/magnetometer/asahi-kasei,ak8975.yaml | 2 ++ .../iio/proximity/vishay,vcnl3020.yaml| 2 ++ .../interrupt-controller/ingenic,intc.yaml| 2 ++ .../loongson,pch-msi.yaml | 2 ++ .../loongson,pch-pic.yaml | 2 ++ .../devicetree/bindings/ipmi/ipmi-smic.yaml | 2 ++ .../devicetree/bindings/leds/leds-lp55xx.yaml | 8 .../bindings/media/i2c/chrontel,ch7322.yaml | 2 ++ .../bindings/media/i2c/imi,rdacm2x-gmsl.yaml | 2 ++ .../bindings/media/nxp,imx8mq-vpu.yaml| 2 ++ .../bindings/media/qcom,msm8916-venus.yaml| 2 ++ .../bindings/media/qcom,msm8996-venus.yaml| 2 ++ .../bindings/media/qcom,sc7180-venus.yaml | 2 ++ .../bindings/media/qcom,sdm845-venus-v2.yaml | 2 ++ .../bindings/media/qcom,sdm845-venus.yaml | 2 ++ .../bindings/memory-controllers/fsl/mmdc.yaml | 2 ++ .../memory-controllers/st,stm32-fmc2-ebi.yaml | 2 ++ .../bindings/mfd/gateworks-gsc.yaml | 2 ++ .../bindings/mfd/xylon,logicvc.yaml | 14 +++-- .../bindings/mips/ingenic/ingenic,cpu.yaml| 6 -- .../bindings/mips/loongson/rs780e-acpi.yaml | 2 ++ .../bindings/mmc/mmc-pwrseq-emmc.yaml | 2 ++ .../bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 ++ .../bindings/mmc/mmc-pwrseq-simple.yaml | 2 ++ .../devicetree/bindings/net/qcom,ipa.yaml | 2 ++ .../bindings/net/realtek-bluetooth.yaml | 4 +++- .../net/wireless/microchip,wilc1000.yaml | 4 .../devicetree/bindings/pci/rcar-pci-ep.yaml | 2 ++ .../phy/a
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, 02 Oct 2020, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. [...] > .../bindings/mfd/gateworks-gsc.yaml | 2 ++ > .../bindings/mfd/xylon,logicvc.yaml | 14 +++-- Acked-by: Lee Jones -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Sat, 3 Oct 2020 at 01:41, Rob Herring wrote: > > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. > [...] > .../bindings/mmc/mmc-pwrseq-emmc.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-simple.yaml | 2 ++ Acked-by: Ulf Hansson Kind regards Uffe
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, 2 Oct 2020 18:41:43 -0500 Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- Hi Rob, Acked-by: Jonathan Cameron # for iio However, one of these made me wonder if the binding was simply wrong... (definitely highlights why we should have additionalProperties: false where ever possible). ... > diff --git > a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml > b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml > index abd8d25e1136..4c1c083d0e92 100644 > --- a/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml > +++ b/Documentation/devicetree/bindings/iio/imu/invensense,icm42600.yaml > @@ -47,11 +47,17 @@ properties: >vddio-supply: > description: Regulator that provides power to the bus > > + spi-max-frequency: true > + spi-cpha: true > + spi-cpol: true It isn't completely unheard of for a device to operate in multiple SPI modes, but it does seem to be fairly unusual. I took a look at the datasheet and at least from the provided timing diagrams, these are both required in SPI mode. http://invensense.tdk.com/wp-content/uploads/2020/09/DS-000292-ICM-42605-v1.5.pdf That doesn't make the binding wrong as such, but we could be tighter in checking this! I'll add this to my list to take a closer look at sometime soonish. Thanks. Jonathan > + > required: >- compatible >- reg >- interrupts > > +additionalProperties: false > + > examples: >- |
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, Oct 02, 2020 at 06:41:43PM -0500, Rob Herring wrote: [...] > .../arm/tegra/nvidia,tegra20-pmc.yaml | 2 ++ [...] > .../bindings/sound/nvidia,tegra186-dspk.yaml | 2 ++ > .../sound/nvidia,tegra210-admaif.yaml | 2 ++ > .../bindings/sound/nvidia,tegra210-dmic.yaml | 2 ++ > .../bindings/sound/nvidia,tegra210-i2s.yaml | 2 ++ [...] > .../bindings/usb/nvidia,tegra-xudc.yaml | 2 ++ [...] Acked-by: Thierry Reding signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, Oct 02, 2020 at 06:41:43PM -0500, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. Acked-by: Mark Brown signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
Hi! > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. for leds: Acked-by: Pavel Machek I assume you apply it..? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, Oct 02, 2020 at 06:41:43PM -0500, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. > > [...] > .../bindings/power/supply/cw2015_battery.yaml | 2 ++ > .../bindings/power/supply/rohm,bd99954.yaml | 8 > [...] Acked-by: Sebastian Reichel -- Sebastian signature.asc Description: PGP signature
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, 02 Oct 2020, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. Acked-by: Lee Jones -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On 10/2/20 4:41 PM, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org For hwmon: Reviewed-by: Guenter Roeck > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. > > .../arm/bcm/raspberrypi,bcm2835-firmware.yaml | 2 ++ > .../arm/mediatek/mediatek,pericfg.yaml| 2 ++ > .../devicetree/bindings/arm/pmu.yaml | 2 ++ > .../devicetree/bindings/arm/primecell.yaml| 3 +++ > .../bindings/arm/samsung/sysreg.yaml | 2 ++ > .../arm/tegra/nvidia,tegra20-pmc.yaml | 2 ++ > .../bindings/bus/mti,mips-cdmm.yaml | 2 ++ > .../bus/socionext,uniphier-system-bus.yaml| 7 +++ > .../bindings/clock/arm,syscon-icst.yaml | 2 ++ > .../bindings/clock/idt,versaclock5.yaml | 20 ++- > .../bindings/clock/imx6q-clock.yaml | 2 ++ > .../bindings/clock/imx6sl-clock.yaml | 2 ++ > .../bindings/clock/imx6sll-clock.yaml | 2 ++ > .../bindings/clock/imx6sx-clock.yaml | 2 ++ > .../bindings/clock/imx6ul-clock.yaml | 2 ++ > .../bindings/clock/intel,cgu-lgm.yaml | 2 ++ > .../bindings/clock/qcom,gcc-sm8250.yaml | 2 ++ > .../bindings/clock/sprd,sc9863a-clk.yaml | 2 ++ > .../bindings/clock/ti,am654-ehrpwm-tbclk.yaml | 2 ++ > .../bindings/display/bridge/ite,it6505.yaml | 5 + > .../bindings/display/bridge/lvds-codec.yaml | 3 +++ > .../devicetree/bindings/display/msm/gmu.yaml | 2 ++ > .../devicetree/bindings/edac/dmc-520.yaml | 2 ++ > .../devicetree/bindings/fsi/ibm,fsi2spi.yaml | 2 ++ > .../gpio/socionext,uniphier-gpio.yaml | 2 ++ > .../bindings/hwmon/adi,axi-fan-control.yaml | 2 ++ > .../devicetree/bindings/hwmon/adt7475.yaml| 2 ++ > .../bindings/iio/accel/kionix,kxsd9.yaml | 4 > .../bindings/iio/adc/maxim,max1238.yaml | 2 ++ > .../bindings/iio/adc/maxim,max1363.yaml | 2 ++ > .../bindings/iio/adc/qcom,spmi-vadc.yaml | 4 > .../bindings/iio/adc/samsung,exynos-adc.yaml | 2 ++ > .../bindings/iio/adc/ti,ads8688.yaml | 4 > .../bindings/iio/amplifiers/adi,hmc425a.yaml | 2 ++ > .../bindings/iio/imu/invensense,icm42600.yaml | 6 ++ > .../bindings/iio/light/amstaos,tsl2563.yaml | 2 ++ > .../bindings/iio/light/dynaimage,al3010.yaml | 2 ++ > .../bindings/iio/light/dynaimage,al3320a.yaml | 2 ++ > .../bindings/iio/light/sharp,gp2ap002.yaml| 2 ++ > .../iio/magnetometer/asahi-kasei,ak8975.yaml | 2 ++ > .../iio/proximity/vishay,vcnl3020.yaml| 2 ++ > .../interrupt-controller/ingenic,intc.yaml| 2 ++ > .../loongson,pch-msi.yaml | 2 ++ > .../loongson,pch-pic.yaml | 2 ++ > .../devicetree/bindings/ipmi/ipmi-smic.yaml | 2 ++ > .../devicetree/bindings/leds/leds-lp55xx.yaml | 8 > .../bindings/media/i2c/chrontel,ch7322.yaml | 2 ++ > .../bindings/media/i2c/imi,rdacm2x-gmsl.yaml | 2 ++ > .../bindings/media/nxp,imx8mq-vpu.yaml| 2 ++ > .../bindings/media/qcom,msm8916-venus.yaml| 2 ++ > .../bindings/media/qcom,msm8996-venus.yaml| 2 ++ > .../bindings/media/qcom,sc7180-venus.yaml | 2 ++ > .../bindings/media/qcom,sdm845-venus-v2.yaml | 2 ++ > .../bindings/media/qcom,sdm845-venus.yaml | 2 ++ > .../bindings/memory-controllers/fsl/mmdc.yaml | 2 ++ > .../memory-controllers/st,stm32-fmc2-ebi.yaml | 2 ++ > .../bindings/mfd/gateworks-gsc.yaml | 2 ++ > .../bindings/mfd/xylon,logicvc.yaml | 14 +++-- > .../bindings/mips/ingenic/ingenic,cpu.yaml| 6 -- > .../bindings/mips/loongson/rs780e-acpi.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-emmc.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-sd8787.yaml
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, Oct 02, 2020 at 06:41:43PM -0500, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. For USB: Reviewed-by: Greg Kroah-Hartman
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On Fri, Oct 02, 2020 at 06:41:43PM -0500, Rob Herring wrote: > Another round of wack-a-mole. The json-schema default is additional > unknown properties are allowed, but for DT all properties should be > defined. > > Cc: Thierry Reding > Cc: Linus Walleij > Cc: Stephen Boyd > Cc: Shawn Guo > Cc: Bjorn Andersson > Cc: Baolin Wang > Cc: Guenter Roeck > Cc: Jonathan Cameron > Cc: Mauro Carvalho Chehab > Cc: Laurent Pinchart > Cc: Lee Jones > Cc: Ulf Hansson > Cc: "David S. Miller" > Cc: Bjorn Helgaas > Cc: Vinod Koul > Cc: Liam Girdwood > Cc: Mark Brown > Cc: Greg Kroah-Hartman > Cc: Daniel Lezcano > Cc: linux-...@vger.kernel.org > Cc: dri-de...@lists.freedesktop.org > Cc: linux-...@vger.kernel.org > Cc: linux-g...@vger.kernel.org > Cc: linux-hw...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: openipmi-develo...@lists.sourceforge.net > Cc: linux-l...@vger.kernel.org > Cc: linux-me...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-m...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: net...@vger.kernel.org > Cc: linux-...@vger.kernel.org > Cc: linux...@vger.kernel.org > Cc: linux-remotep...@vger.kernel.org > Cc: linux-ser...@vger.kernel.org > Cc: alsa-de...@alsa-project.org > Cc: linux-...@vger.kernel.org > Signed-off-by: Rob Herring > --- > > I'll take this thru the DT tree. > > .../arm/bcm/raspberrypi,bcm2835-firmware.yaml | 2 ++ > .../arm/mediatek/mediatek,pericfg.yaml| 2 ++ > .../devicetree/bindings/arm/pmu.yaml | 2 ++ > .../devicetree/bindings/arm/primecell.yaml| 3 +++ > .../bindings/arm/samsung/sysreg.yaml | 2 ++ > .../arm/tegra/nvidia,tegra20-pmc.yaml | 2 ++ > .../bindings/bus/mti,mips-cdmm.yaml | 2 ++ > .../bus/socionext,uniphier-system-bus.yaml| 7 +++ > .../bindings/clock/arm,syscon-icst.yaml | 2 ++ > .../bindings/clock/idt,versaclock5.yaml | 20 ++- > .../bindings/clock/imx6q-clock.yaml | 2 ++ > .../bindings/clock/imx6sl-clock.yaml | 2 ++ > .../bindings/clock/imx6sll-clock.yaml | 2 ++ > .../bindings/clock/imx6sx-clock.yaml | 2 ++ > .../bindings/clock/imx6ul-clock.yaml | 2 ++ > .../bindings/clock/intel,cgu-lgm.yaml | 2 ++ > .../bindings/clock/qcom,gcc-sm8250.yaml | 2 ++ > .../bindings/clock/sprd,sc9863a-clk.yaml | 2 ++ > .../bindings/clock/ti,am654-ehrpwm-tbclk.yaml | 2 ++ > .../bindings/display/bridge/ite,it6505.yaml | 5 + > .../bindings/display/bridge/lvds-codec.yaml | 3 +++ > .../devicetree/bindings/display/msm/gmu.yaml | 2 ++ > .../devicetree/bindings/edac/dmc-520.yaml | 2 ++ > .../devicetree/bindings/fsi/ibm,fsi2spi.yaml | 2 ++ > .../gpio/socionext,uniphier-gpio.yaml | 2 ++ > .../bindings/hwmon/adi,axi-fan-control.yaml | 2 ++ > .../devicetree/bindings/hwmon/adt7475.yaml| 2 ++ > .../bindings/iio/accel/kionix,kxsd9.yaml | 4 > .../bindings/iio/adc/maxim,max1238.yaml | 2 ++ > .../bindings/iio/adc/maxim,max1363.yaml | 2 ++ > .../bindings/iio/adc/qcom,spmi-vadc.yaml | 4 > .../bindings/iio/adc/samsung,exynos-adc.yaml | 2 ++ > .../bindings/iio/adc/ti,ads8688.yaml | 4 > .../bindings/iio/amplifiers/adi,hmc425a.yaml | 2 ++ > .../bindings/iio/imu/invensense,icm42600.yaml | 6 ++ > .../bindings/iio/light/amstaos,tsl2563.yaml | 2 ++ > .../bindings/iio/light/dynaimage,al3010.yaml | 2 ++ > .../bindings/iio/light/dynaimage,al3320a.yaml | 2 ++ > .../bindings/iio/light/sharp,gp2ap002.yaml| 2 ++ > .../iio/magnetometer/asahi-kasei,ak8975.yaml | 2 ++ > .../iio/proximity/vishay,vcnl3020.yaml| 2 ++ > .../interrupt-controller/ingenic,intc.yaml| 2 ++ > .../loongson,pch-msi.yaml | 2 ++ > .../loongson,pch-pic.yaml | 2 ++ > .../devicetree/bindings/ipmi/ipmi-smic.yaml | 2 ++ > .../devicetree/bindings/leds/leds-lp55xx.yaml | 8 > .../bindings/media/i2c/chrontel,ch7322.yaml | 2 ++ > .../bindings/media/i2c/imi,rdacm2x-gmsl.yaml | 2 ++ > .../bindings/media/nxp,imx8mq-vpu.yaml| 2 ++ > .../bindings/media/qcom,msm8916-venus.yaml| 2 ++ > .../bindings/media/qcom,msm8996-venus.yaml| 2 ++ > .../bindings/media/qcom,sc7180-venus.yaml | 2 ++ > .../bindings/media/qcom,sdm845-venus-v2.yaml | 2 ++ > .../bindings/media/qcom,sdm845-venus.yaml | 2 ++ > .../bindings/memory-controllers/fsl/mmdc.yaml | 2 ++ > .../memory-controllers/st,stm32-fmc2-ebi.yaml | 2 ++ > .../bindings/mfd/gateworks-gsc.yaml | 2 ++ > .../bindings/mfd/xylon,logicvc.yaml | 14 +++-- > .../bindings/mips/ingenic/ingenic,cpu.yaml| 6 -- > .../bindings/mips/loongson/rs780e-acpi.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-emmc.yaml | 2 ++ > .../bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 ++ > .../b
Re: [PATCH] dt-bindings: Another round of adding missing 'additionalProperties'
On 02-10-20, 18:41, Rob Herring wrote: > .../phy/amlogic,meson-g12a-usb2-phy.yaml | 2 ++ > .../bindings/phy/qcom,ipq806x-usb-phy-hs.yaml | 2 ++ > .../bindings/phy/qcom,ipq806x-usb-phy-ss.yaml | 2 ++ > .../bindings/phy/qcom,qusb2-phy.yaml | 1 + > .../bindings/phy/qcom-usb-ipq4019-phy.yaml| 2 ++ For phy changes: Acked-By: Vinod Koul -- ~Vinod