Message ID | 20210901091852.479202-1-maxime@cerno.tech |
---|---|
Headers | show |
Series | ARM: dts: Last round of DT schema fixes | expand |
On Wed, Sep 01, 2021 at 11:18:05AM +0200, Maxime Ripard wrote: > The Reserved Memory mechanism is supported by Linux thanks to its device > tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Mailing List <devicetree-spec@vger.kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Minor request. As device tree bindings are used and valid and applicable outside of Linux, it shouldn't be mentioned in the commit message either. This binding is just as valid for U-Boot, FreeBSD, etc as it is for Linux :) Thanks!
On Wed, 01 Sep 2021 11:18:02 +0200, Maxime Ripard wrote: > Bluetooth SCO Link are supported by Linux with a matching device tree > binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: alsa-devel@alsa-project.org > Cc: devicetree@vger.kernel.org > Cc: Liam Girdwood <lgirdwood@gmail.com> > Cc: Mark Brown <broonie@kernel.org> > Cc: Samuel Holland <samuel@sholland.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Accept either a 0 or 1 #sound-dai-cells > --- > .../devicetree/bindings/sound/bt-sco.txt | 13 ------- > .../bindings/sound/linux,bt-sco.yaml | 38 +++++++++++++++++++ > 2 files changed, 38 insertions(+), 13 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/sound/bt-sco.txt > create mode 100644 Documentation/devicetree/bindings/sound/linux,bt-sco.yaml > Reviewed-by: Rob Herring <robh@kernel.org>
On Wed, 01 Sep 2021 11:18:05 +0200, Maxime Ripard wrote: > The Reserved Memory mechanism is supported by Linux thanks to its device > tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Mailing List <devicetree-spec@vger.kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Fixed the license > - Added a mention that the file has moved > - Added a documentation for memory-region > - Split the reserved-memory binding in two: one for the generic part > and one for shared-dma-pool > --- > .../reserved-memory/memory-region.yaml | 40 ++++ > .../reserved-memory/reserved-memory.txt | 172 +----------------- > .../reserved-memory/reserved-memory.yaml | 96 ++++++++++ > .../reserved-memory/shared-dma-pool.yaml | 87 +++++++++ > 4 files changed, 224 insertions(+), 171 deletions(-) > create mode 100644 Documentation/devicetree/bindings/reserved-memory/memory-region.yaml > create mode 100644 Documentation/devicetree/bindings/reserved-memory/reserved-memory.yaml > create mode 100644 Documentation/devicetree/bindings/reserved-memory/shared-dma-pool.yaml > I reworked the commit msg and a couple of nits, and applied, thanks!
On Wed, 01 Sep 2021 11:18:06 +0200, Maxime Ripard wrote: > The ARM CCI-400 Interconnect is supported by Linux thanks to its device > tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Reduced the max number of interrupts > - Comented the pl330 compatible to avoid a warning > - Added cci-control-port to the cpus schemas > --- > .../devicetree/bindings/arm/arm,cci-400.yaml | 216 +++++++++++++++++ > .../bindings/arm/cci-control-port.yaml | 38 +++ > Documentation/devicetree/bindings/arm/cci.txt | 224 ------------------ > .../devicetree/bindings/arm/cpus.yaml | 2 + > 4 files changed, 256 insertions(+), 224 deletions(-) > create mode 100644 Documentation/devicetree/bindings/arm/arm,cci-400.yaml > create mode 100644 Documentation/devicetree/bindings/arm/cci-control-port.yaml > delete mode 100644 Documentation/devicetree/bindings/arm/cci.txt > Applied, thanks!
On Wed, 01 Sep 2021 11:18:13 +0200, Maxime Ripard wrote: > The Mitsubishi AA140XD12 Device Tree Binding was requiring a vcc-supply > property. However, neither the existing device trees using that binding, > nor the driver were actually using that property which is also redundant > with power-supply. Let's just drop it. > > Cc: dri-devel@lists.freedesktop.org > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Cc: Sam Ravnborg <sam@ravnborg.org> > Cc: Thierry Reding <thierry.reding@gmail.com> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > --- > .../bindings/display/panel/mitsubishi,aa104xd12.yaml | 5 ----- > 1 file changed, 5 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Wed, 01 Sep 2021 11:18:17 +0200, Maxime Ripard wrote: > The Mitsubishi AA121TD01 Device Tree Binding was requiring a > data-mapping property value which was set to another value in the > existing Device Trees. Fix this. > > Cc: dri-devel@lists.freedesktop.org > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Cc: Sam Ravnborg <sam@ravnborg.org> > Cc: Thierry Reding <thierry.reding@gmail.com> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > --- > .../bindings/display/panel/mitsubishi,aa121td01.yaml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Rob Herring <robh@kernel.org>
On Wed, 01 Sep 2021 11:18:21 +0200, Maxime Ripard wrote: > The UBlox Neo 6M is supported by Linux thanks to its device tree > binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Johan Hovold <johan@kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Fixed a typo > - Fixed the example node name > --- > .../bindings/gnss/u-blox,neo-6m.yaml | 62 +++++++++++++++++++ > .../devicetree/bindings/gnss/u-blox.txt | 45 -------------- > 2 files changed, 62 insertions(+), 45 deletions(-) > create mode 100644 Documentation/devicetree/bindings/gnss/u-blox,neo-6m.yaml > delete mode 100644 Documentation/devicetree/bindings/gnss/u-blox.txt > Applied, thanks!
On Wed, 01 Sep 2021 11:18:23 +0200, Maxime Ripard wrote: > Even though we had the iio-hwmon driver for some time and a number of > boards using it already, we never had a binding for it. Let's add it > based on what the driver expects and the boards are providing. > > Cc: Guenter Roeck <linux@roeck-us.net> > Cc: Jean Delvare <jdelvare@suse.com> > Cc: Jonathan Cameron <jic23@kernel.org> > Cc: linux-hwmon@vger.kernel.org > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Reduced the maximum number of io-channels, and added a comment for > the limitation > --- > .../devicetree/bindings/hwmon/iio-hwmon.yaml | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml > Reviewed-by: Rob Herring <robh@kernel.org>
On Wed, Sep 01, 2021 at 11:18:27AM +0200, Maxime Ripard wrote: > The Omnivision OV5640 is supported by Linux thanks to its device tree > binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: linux-media@vger.kernel.org > Cc: Mauro Carvalho Chehab <mchehab@kernel.org> > Cc: Steve Longerbeam <slongerbeam@gmail.com> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > --- > .../devicetree/bindings/media/i2c/ov5640.txt | 92 ----------- > .../bindings/media/i2c/ovti,ov5640.yaml | 154 ++++++++++++++++++ > 2 files changed, 154 insertions(+), 92 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5640.txt > create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov5640.yaml > > diff --git a/Documentation/devicetree/bindings/media/i2c/ov5640.txt b/Documentation/devicetree/bindings/media/i2c/ov5640.txt > deleted file mode 100644 > index c97c2f2da12d..000000000000 > --- a/Documentation/devicetree/bindings/media/i2c/ov5640.txt > +++ /dev/null > @@ -1,92 +0,0 @@ > -* Omnivision OV5640 MIPI CSI-2 / parallel sensor > - > -Required Properties: > -- compatible: should be "ovti,ov5640" > -- clocks: reference to the xclk input clock. > -- clock-names: should be "xclk". > -- DOVDD-supply: Digital I/O voltage supply, 1.8 volts > -- AVDD-supply: Analog voltage supply, 2.8 volts > -- DVDD-supply: Digital core voltage supply, 1.5 volts > - > -Optional Properties: > -- reset-gpios: reference to the GPIO connected to the reset pin, if any. > - This is an active low signal to the OV5640. > -- powerdown-gpios: reference to the GPIO connected to the powerdown pin, > - if any. This is an active high signal to the OV5640. > -- rotation: as defined in > - Documentation/devicetree/bindings/media/video-interfaces.txt, > - valid values are 0 (sensor mounted upright) and 180 (sensor > - mounted upside down). > - > -The device node must contain one 'port' child node for its digital output > -video port, in accordance with the video interface bindings defined in > -Documentation/devicetree/bindings/media/video-interfaces.txt. > - > -OV5640 can be connected to a MIPI CSI-2 bus or a parallel bus endpoint. > - > -Endpoint node required properties for CSI-2 connection are: > -- remote-endpoint: a phandle to the bus receiver's endpoint node. > -- clock-lanes: should be set to <0> (clock lane on hardware lane 0) > -- data-lanes: should be set to <1> or <1 2> (one or two CSI-2 lanes supported) > - > -Endpoint node required properties for parallel connection are: > -- remote-endpoint: a phandle to the bus receiver's endpoint node. > -- bus-width: shall be set to <8> for 8 bits parallel bus > - or <10> for 10 bits parallel bus > -- data-shift: shall be set to <2> for 8 bits parallel bus > - (lines 9:2 are used) or <0> for 10 bits parallel bus > -- hsync-active: active state of the HSYNC signal, 0/1 for LOW/HIGH respectively. > -- vsync-active: active state of the VSYNC signal, 0/1 for LOW/HIGH respectively. > -- pclk-sample: sample data on rising (1) or falling (0) edge of the pixel clock > - signal. > - > -Examples: > - > -&i2c1 { > - ov5640: camera@3c { > - compatible = "ovti,ov5640"; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_ov5640>; > - reg = <0x3c>; > - clocks = <&clks IMX6QDL_CLK_CKO>; > - clock-names = "xclk"; > - DOVDD-supply = <&vgen4_reg>; /* 1.8v */ > - AVDD-supply = <&vgen3_reg>; /* 2.8v */ > - DVDD-supply = <&vgen2_reg>; /* 1.5v */ > - powerdown-gpios = <&gpio1 19 GPIO_ACTIVE_HIGH>; > - reset-gpios = <&gpio1 20 GPIO_ACTIVE_LOW>; > - rotation = <180>; > - > - port { > - /* MIPI CSI-2 bus endpoint */ > - ov5640_to_mipi_csi2: endpoint { > - remote-endpoint = <&mipi_csi2_from_ov5640>; > - clock-lanes = <0>; > - data-lanes = <1 2>; > - }; > - }; > - }; > -}; > - > -&i2c1 { > - ov5640: camera@3c { > - compatible = "ovti,ov5640"; > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_ov5640>; > - reg = <0x3c>; > - clocks = <&clk_ext_camera>; > - clock-names = "xclk"; > - > - port { > - /* Parallel bus endpoint */ > - ov5640_to_parallel: endpoint { > - remote-endpoint = <¶llel_from_ov5640>; > - bus-width = <8>; > - data-shift = <2>; /* lines 9:2 are used */ > - hsync-active = <0>; > - vsync-active = <0>; > - pclk-sample = <1>; > - }; > - }; > - }; > -}; > diff --git a/Documentation/devicetree/bindings/media/i2c/ovti,ov5640.yaml b/Documentation/devicetree/bindings/media/i2c/ovti,ov5640.yaml > new file mode 100644 > index 000000000000..0283b1895a75 > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/i2c/ovti,ov5640.yaml > @@ -0,0 +1,154 @@ > +# SPDX-License-Identifier: GPL-2.0 > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/i2c/ovti,ov5640.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: OmniVision OV5640 Image Sensor Device Tree Bindings > + > +maintainers: > + - Steve Longerbeam <slongerbeam@gmail.com> > + > +allOf: > + - $ref: /schemas/media/video-interface-devices.yaml# > + > +properties: > + compatible: > + const: ovti,ov5640 > + > + reg: > + maxItems: 1 > + > + clocks: > + description: XCLK Input Clock > + > + clock-names: > + const: xclk > + > + AVDD-supply: > + description: Analog voltage supply, 2.8 volts > + > + DVDD-supply: > + description: Digital core voltage supply, 1.5 volts > + > + DOVDD-supply: > + description: Digital I/O voltage supply, 1.8 volts > + > + powerdown-gpios: > + maxItems: 1 > + description: > > + Reference to the GPIO connected to the powerdown pin, if any. > + > + reset-gpios: > + maxItems: 1 > + description: > > + Reference to the GPIO connected to the reset pin, if any. > + > + rotation: > + enum: > + - 0 > + - 180 > + > + port: > + description: Digital Output Port > + $ref: /schemas/graph.yaml#/properties/port This needs to be /schemas/graph.yaml#/$defs/port-base Otherwise, it will fail when we start checking unevaluatedProperties. > + additionalProperties: false > + > + properties: > + endpoint: > + $ref: /schemas/media/video-interfaces.yaml# > + unevaluatedProperties: false > + > + properties: > + clock-lanes: > + const: 0 > + > + data-lanes: > + minItems: 1 > + maxItems: 2 > + items: > + enum: [1, 2] > + > + bus-width: > + enum: [8, 10] > + > + data-shift: > + enum: [0, 2] > + > +required: > + - compatible > + - reg > + - clocks > + - clock-names > + - AVDD-supply > + - DVDD-supply > + - DOVDD-supply > + - port > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/imx6qdl-clock.h> > + #include <dt-bindings/gpio/gpio.h> > + > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + camera@3c { > + compatible = "ovti,ov5640"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ov5640>; > + reg = <0x3c>; > + clocks = <&clks IMX6QDL_CLK_CKO>; > + clock-names = "xclk"; > + DOVDD-supply = <&vgen4_reg>; /* 1.8v */ > + AVDD-supply = <&vgen3_reg>; /* 2.8v */ > + DVDD-supply = <&vgen2_reg>; /* 1.5v */ > + powerdown-gpios = <&gpio1 19 GPIO_ACTIVE_HIGH>; > + reset-gpios = <&gpio1 20 GPIO_ACTIVE_LOW>; > + rotation = <180>; > + > + port { > + /* MIPI CSI-2 bus endpoint */ > + ov5640_to_mipi_csi2: endpoint { > + remote-endpoint = <&mipi_csi2_from_ov5640>; > + clock-lanes = <0>; > + data-lanes = <1 2>; > + }; > + }; > + }; > + }; > + > + - | > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + camera@3c { > + compatible = "ovti,ov5640"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_ov5640>; > + reg = <0x3c>; > + clocks = <&clk_ext_camera>; > + clock-names = "xclk"; > + DOVDD-supply = <&vgen4_reg>; /* 1.8v */ > + AVDD-supply = <&vgen3_reg>; /* 2.8v */ > + DVDD-supply = <&vgen2_reg>; /* 1.5v */ > + > + port { > + /* Parallel bus endpoint */ > + ov5640_to_parallel: endpoint { > + remote-endpoint = <¶llel_from_ov5640>; > + bus-width = <8>; > + data-shift = <2>; /* lines 9:2 are used */ > + hsync-active = <0>; > + vsync-active = <0>; > + pclk-sample = <1>; > + }; > + }; > + }; > + }; > + > +... > -- > 2.31.1 > >
On Wed, 01 Sep 2021 11:18:29 +0200, Maxime Ripard wrote: > The X-Powers AXP PMICs are supported by Linux thanks to its device tree > binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Chen-Yu Tsai <wens@csie.org> > Cc: Lee Jones <lee.jones@linaro.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Add GPIO example > - Remove the limitation on regulator-ramp-delay > --- > .../i2c/allwinner,sun6i-a31-p2wi.yaml | 2 +- > .../devicetree/bindings/mfd/axp20x.txt | 273 ------------ > .../bindings/mfd/x-powers,axp152.yaml | 400 ++++++++++++++++++ > 3 files changed, 401 insertions(+), 274 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt > create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > Reviewed-by: Rob Herring <robh@kernel.org> Note that the gpio child schema needs to be applied with this one.
On Wed, 01 Sep 2021 11:18:36 +0200, Maxime Ripard wrote: > The SMSC USB3503 USB Hub Controller is supported by Linux thanks to > its device tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Dongjin Kim <tobetter@gmail.com> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > Cc: linux-usb@vger.kernel.org > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Added maximum number of items for clocks and gpios > - Fixed the example node name > --- > .../devicetree/bindings/usb/smsc,usb3503.yaml | 108 ++++++++++++++++++ > .../devicetree/bindings/usb/usb3503.txt | 39 ------- > 2 files changed, 108 insertions(+), 39 deletions(-) > create mode 100644 Documentation/devicetree/bindings/usb/smsc,usb3503.yaml > delete mode 100644 Documentation/devicetree/bindings/usb/usb3503.txt > Reviewed-by: Rob Herring <robh@kernel.org>
On Wed, 01 Sep 2021 11:18:38 +0200, Maxime Ripard wrote: > Bitbanged 1-Wire buses are supported by Linux thanks to their device > tree binding. > > Now that we have the DT validation in place, let's convert the device > tree bindings for that driver over to a YAML schema. > > Cc: Daniel Mack <zonque@gmail.com> > Cc: Evgeniy Polyakov <zbr@ioremap.net> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > --- > > Changes from v1: > - Changed additionalproperties to make sure all the other items in the > schema are nodes > --- > .../devicetree/bindings/w1/w1-gpio.txt | 27 ------------ > .../devicetree/bindings/w1/w1-gpio.yaml | 44 +++++++++++++++++++ > 2 files changed, 44 insertions(+), 27 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/w1/w1-gpio.txt > create mode 100644 Documentation/devicetree/bindings/w1/w1-gpio.yaml > Applied, thanks!
On Wed, Sep 01, 2021 at 11:18:23AM +0200, Maxime Ripard wrote: > Even though we had the iio-hwmon driver for some time and a number of > boards using it already, we never had a binding for it. Let's add it > based on what the driver expects and the boards are providing. > > Cc: Guenter Roeck <linux@roeck-us.net> > Cc: Jean Delvare <jdelvare@suse.com> > Cc: Jonathan Cameron <jic23@kernel.org> > Cc: linux-hwmon@vger.kernel.org > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > Reviewed-by: Rob Herring <robh@kernel.org> Applied (assuming this is supposed to be pushed through hwmon). Thanks, Guenter
Dne sreda, 01. september 2021 ob 11:18:09 CEST je Maxime Ripard napisal(a): > Even though both the driver and the device trees all use the > allwinner,sun50i-h6-de3-clk, we documented the compatible as > allwinner,sun50i-h6-de2-clk in the binding. Let's fix this. > > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:25 CEST je Maxime Ripard napisal(a): > The R40 MBUS compatible was introduced recently but it was never > documented. > > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:31 CEST je Maxime Ripard napisal(a): > Even though both the driver and the device trees all use the > allwinner,sun8i-r40-gmac compatible, we documented the compatible as > allwinner,sun8i-r40-emac in the binding. Let's fix this. > > Cc: Alexandre Torgue <alexandre.torgue@st.com> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Giuseppe Cavallaro <peppe.cavallaro@st.com> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: Jose Abreu <joabreu@synopsys.com> > Cc: netdev@vger.kernel.org > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:35 CEST je Maxime Ripard napisal(a): > Even though we've used the A80 PRCM driver for some time and a number of > boards using it already, we never had a binding for it. Let's add it > based on what the driver expects and the boards are providing. > > Reviewed-by: Rob Herring <robh@kernel.org> > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:41 CEST je Maxime Ripard napisal(a): > Even though it translates to the same thing down to the binary level, we > should have an array of 2 number cells to describe each OPP, which in > turns create a validation warning. > > Let's fix this. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:42 CEST je Maxime Ripard napisal(a): > The operating-points-v2 nodes are named inconsistently, but mostly > either opp_table0 or gpu-opp-table. However, the underscore is an > invalid character for a node name and the thermal zone binding > explicitly requires that zones are called opp-table-*. Let's fix it. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:43 CEST je Maxime Ripard napisal(a): > According to the SPI NOR bindings, the flash node names are supposed to > be flash@<address>. Let's fix our users to use that new scheme. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:45 CEST je Maxime Ripard napisal(a): > The edt,edt-ft5x06 compatible has never been a valid compatible > according to the binding. Let's change for one that is. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:47 CEST je Maxime Ripard napisal(a): > The GPU thermal zone is named gpu_thermal. However, the underscore is > an invalid character for a node name and the thermal zone binding > explicitly requires that zones are called *-thermal. Let's fix it. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:49 CEST je Maxime Ripard napisal(a): > The thermal zones one the A100 are called $device-thermal-zone. > > However, the thermal zone binding explicitly requires that zones are > called *-thermal. Let's fix it. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
Dne sreda, 01. september 2021 ob 11:18:51 CEST je Maxime Ripard napisal(a): > The anx6345 bridge mandates that the input port is named port@0. Since > we have a unit-address, this implies that we need a reg property with > the same value, but it was found to be missing in the Teres-I device > tree. Make sure it's there. > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej
On Fri, 03 Sep 2021, Rob Herring wrote: > On Wed, 01 Sep 2021 11:18:29 +0200, Maxime Ripard wrote: > > The X-Powers AXP PMICs are supported by Linux thanks to its device tree > > binding. > > > > Now that we have the DT validation in place, let's convert the device > > tree bindings for that driver over to a YAML schema. > > > > Cc: Chen-Yu Tsai <wens@csie.org> > > Cc: Lee Jones <lee.jones@linaro.org> > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > > > --- > > > > Changes from v1: > > - Add GPIO example > > - Remove the limitation on regulator-ramp-delay > > --- > > .../i2c/allwinner,sun6i-a31-p2wi.yaml | 2 +- > > .../devicetree/bindings/mfd/axp20x.txt | 273 ------------ > > .../bindings/mfd/x-powers,axp152.yaml | 400 ++++++++++++++++++ > > 3 files changed, 401 insertions(+), 274 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt > > create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > Note that the gpio child schema needs to be applied with this one. Not sure I'm in possession of that. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
On Sun, Sep 05, 2021 at 10:50:32PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:09 CEST je Maxime Ripard napisal(a): > > Even though both the driver and the device trees all use the > > allwinner,sun50i-h6-de3-clk, we documented the compatible as > > allwinner,sun50i-h6-de2-clk in the binding. Let's fix this. > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:01:17PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:35 CEST je Maxime Ripard napisal(a): > > Even though we've used the A80 PRCM driver for some time and a number of > > boards using it already, we never had a binding for it. Let's add it > > based on what the driver expects and the boards are providing. > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Reviewed-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:04:59PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:41 CEST je Maxime Ripard napisal(a): > > Even though it translates to the same thing down to the binary level, we > > should have an array of 2 number cells to describe each OPP, which in > > turns create a validation warning. > > > > Let's fix this. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks
On Sun, Sep 05, 2021 at 11:05:58PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:42 CEST je Maxime Ripard napisal(a): > > The operating-points-v2 nodes are named inconsistently, but mostly > > either opp_table0 or gpu-opp-table. However, the underscore is an > > invalid character for a node name and the thermal zone binding > > explicitly requires that zones are called opp-table-*. Let's fix it. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:06:45PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:43 CEST je Maxime Ripard napisal(a): > > According to the SPI NOR bindings, the flash node names are supposed to > > be flash@<address>. Let's fix our users to use that new scheme. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:11:58PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:45 CEST je Maxime Ripard napisal(a): > > The edt,edt-ft5x06 compatible has never been a valid compatible > > according to the binding. Let's change for one that is. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:12:51PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:47 CEST je Maxime Ripard napisal(a): > > The GPU thermal zone is named gpu_thermal. However, the underscore is > > an invalid character for a node name and the thermal zone binding > > explicitly requires that zones are called *-thermal. Let's fix it. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:13:46PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:49 CEST je Maxime Ripard napisal(a): > > The thermal zones one the A100 are called $device-thermal-zone. > > > > However, the thermal zone binding explicitly requires that zones are > > called *-thermal. Let's fix it. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Sun, Sep 05, 2021 at 11:14:34PM +0200, Jernej Škrabec wrote: > Dne sreda, 01. september 2021 ob 11:18:51 CEST je Maxime Ripard napisal(a): > > The anx6345 bridge mandates that the input port is named port@0. Since > > we have a unit-address, this implies that we need a reg property with > > the same value, but it was found to be missing in the Teres-I device > > tree. Make sure it's there. > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Applied, thanks Maxime
On Wed, 1 Sep 2021 11:18:00 +0200, Maxime Ripard wrote: > Here's another round of schema warnings fixes for the Allwinner platform. > > There's a fair share of new schemas in there since the schema tools now warn > when a compatible is not documented in a schema. > > We don't have any warning anymore if we use the OPP binding Rob submitted, and > since that means we have all our devices properly validated I don't expect more > fixes now, aside from the usual bunch of regressions. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [01/52] ASoC: dt-bindings: Add WM8978 Binding commit: 5bd5699c494f42a3c23e377bb35be5d716b85012 [02/52] ASoC: dt-bindings: Convert Bluetooth SCO Link binding to a schema commit: 6ef239699102580b104bfca11d9b94abfbc8caec [03/52] ASoC: dt-bindings: Convert SPDIF Transmitter binding to a schema commit: 1c02b74ba20838ba194850beb0adfc6b5e11bb2a [04/52] ASoC: dt-bindings: Convert Simple Amplifier binding to a schema commit: 50a41ce8c56d07f42a97112077acbaeda8b86ed9 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
On Mon, Sep 06, 2021 at 10:02:26AM +0100, Lee Jones wrote: > On Fri, 03 Sep 2021, Rob Herring wrote: > > > On Wed, 01 Sep 2021 11:18:29 +0200, Maxime Ripard wrote: > > > The X-Powers AXP PMICs are supported by Linux thanks to its device tree > > > binding. > > > > > > Now that we have the DT validation in place, let's convert the device > > > tree bindings for that driver over to a YAML schema. > > > > > > Cc: Chen-Yu Tsai <wens@csie.org> > > > Cc: Lee Jones <lee.jones@linaro.org> > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > > > > > --- > > > > > > Changes from v1: > > > - Add GPIO example > > > - Remove the limitation on regulator-ramp-delay > > > --- > > > .../i2c/allwinner,sun6i-a31-p2wi.yaml | 2 +- > > > .../devicetree/bindings/mfd/axp20x.txt | 273 ------------ > > > .../bindings/mfd/x-powers,axp152.yaml | 400 ++++++++++++++++++ > > > 3 files changed, 401 insertions(+), 274 deletions(-) > > > delete mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt > > > create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > > > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > > > Note that the gpio child schema needs to be applied with this one. > > Not sure I'm in possession of that. So, how do you want to merge that? Through the DT tree? Maxime
On Tue, Sep 14, 2021 at 2:12 AM Maxime Ripard <maxime@cerno.tech> wrote: > > On Mon, Sep 06, 2021 at 10:02:26AM +0100, Lee Jones wrote: > > On Fri, 03 Sep 2021, Rob Herring wrote: > > > > > On Wed, 01 Sep 2021 11:18:29 +0200, Maxime Ripard wrote: > > > > The X-Powers AXP PMICs are supported by Linux thanks to its device tree > > > > binding. > > > > > > > > Now that we have the DT validation in place, let's convert the device > > > > tree bindings for that driver over to a YAML schema. > > > > > > > > Cc: Chen-Yu Tsai <wens@csie.org> > > > > Cc: Lee Jones <lee.jones@linaro.org> > > > > Signed-off-by: Maxime Ripard <maxime@cerno.tech> > > > > > > > > --- > > > > > > > > Changes from v1: > > > > - Add GPIO example > > > > - Remove the limitation on regulator-ramp-delay > > > > --- > > > > .../i2c/allwinner,sun6i-a31-p2wi.yaml | 2 +- > > > > .../devicetree/bindings/mfd/axp20x.txt | 273 ------------ > > > > .../bindings/mfd/x-powers,axp152.yaml | 400 ++++++++++++++++++ > > > > 3 files changed, 401 insertions(+), 274 deletions(-) > > > > delete mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt > > > > create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml > > > > > > > > > > Reviewed-by: Rob Herring <robh@kernel.org> > > > > > > Note that the gpio child schema needs to be applied with this one. > > > > Not sure I'm in possession of that. > > So, how do you want to merge that? Through the DT tree? Resend the MFD and child patches in a series to Lee. Rob