mbox series

[v2,00/52] ARM: dts: Last round of DT schema fixes

Message ID 20210901091852.479202-1-maxime@cerno.tech
Headers show
Series ARM: dts: Last round of DT schema fixes | expand

Message

Maxime Ripard Sept. 1, 2021, 9:18 a.m. UTC
Hi,

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.

Let me know what you think,
Maxime

Maxime Ripard (52):
  ASoC: dt-bindings: Add WM8978 Binding
  ASoC: dt-bindings: Convert Bluetooth SCO Link binding to a schema
  ASoC: dt-bindings: Convert SPDIF Transmitter binding to a schema
  ASoC: dt-bindings: Convert Simple Amplifier binding to a schema
  dt-bindings: Convert Reserved Memory binding to a schema
  dt-bindings: arm: Convert ARM CCI-400 binding to a schema
  dt-bindings: bluetooth: broadcom: Fix clocks check
  dt-bindings: bluetooth: realtek: Add missing max-speed
  dt-bindings: clocks: Fix typo in the H6 compatible
  dt-bindings: display: Move idk-1110wr to panel-lvds
  dt-bindings: display: Move idk-2121wr to panel-lvds
  dt-bindings: display: Move ee101ia-01d to panel-lvds
  dt-bindings: display: aa104xd12: Remove unused vcc-supply
  dt-bindings: display: aa104xd12: Fix data-mapping
  dt-bindings: display: Move aa104xd12 to panel-lvds
  dt-bindings: display: aa121td01: Remove unused vcc-supply
  dt-bindings: display: aa121td01: Fix data-mapping
  dt-bindings: display: Move aa121td01 to panel-lvds
  dt-bindings: display: Move gktw70sdae4se to panel-lvds
  dt-bindings: display: panel-lvds: Document missing panel compatibles
  dt-bindings: gnss: Convert UBlox Neo-6M binding to a schema
  dt-bindings: gpio: Convert X-Powers AXP209 GPIO binding to a schema
  dt-bindings: hwmon: Add IIO HWMON binding
  dt-bindings: input: Convert Silead GSL1680 binding to a schema
  dt-bindings: interconnect: sunxi: Add R40 MBUS compatible
  dt-bindings: media: ti,cal: Fix example
  dt-bindings: media: Convert OV5640 binding to a schema
  dt-bindings: mfd: Convert X-Powers AC100 binding to a schema
  dt-bindings: mfd: Convert X-Powers AXP binding to a schema
  dt-bindings: mmc: Convert MMC Card binding to a schema
  dt-bindings: net: dwmac: Fix typo in the R40 compatible
  dt-bindings: net: wireless: Convert ESP ESP8089 binding to a schema
  dt-bindings: regulator: Convert SY8106A binding to a schema
  dt-bindings: sunxi: Add CPU Configuration Controller Binding
  dt-bindings: sunxi: Add Allwinner A80 PRCM Binding
  dt-bindings: usb: Convert SMSC USB3503 binding to a schema
  dt-bindings: usb: dwc3: Fix usb-phy check
  dt-bindings: w1: Convert 1-Wire GPIO binding to a schema
  ARM: dts: sunxi: Rename power-supply names
  ARM: dts: sunxi: Rename gpio pinctrl names
  ARM: dts: sunxi: Fix OPP arrays
  ARM: dts: sunxi: Fix OPPs node name
  ARM: dts: sunxi: Fix the SPI NOR node names
  ARM: dts: v3s: Remove useless DMA properties
  ARM: dts: tbs711: Fix touchscreen compatible
  ARM: dts: cubieboard4: Remove the dumb-vga-dac compatible
  arm64: dts: allwinner: h5: Fix GPU thermal zone node name
  arm64: dts: allwinner: h6: Fix de3 parent clocks ordering
  arm64: dts: allwinner: a100: Fix thermal zone node name
  arm64: dts: allwinner: pinetab: Change regulator node name to avoid
    warning
  arm64: dts: allwinner: teres-i: Add missing reg
  arm64: dts: allwinner: teres-i: Remove wakekup-source from the PMIC

 .../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 +
 .../arm/sunxi/allwinner,sun4i-a10-mbus.yaml   |   1 +
 .../sunxi/allwinner,sun6i-a31-cpuconfig.yaml  |  38 ++
 .../arm/sunxi/allwinner,sun9i-a80-prcm.yaml   |  33 ++
 .../clock/allwinner,sun8i-a83t-de2-clk.yaml   |   2 +-
 .../display/panel/advantech,idk-1110wr.yaml   |  69 ---
 .../display/panel/advantech,idk-2121wr.yaml   | 121 ------
 .../display/panel/innolux,ee101ia-01d.yaml    |  31 --
 .../bindings/display/panel/lvds.yaml          | 130 +++++-
 .../display/panel/mitsubishi,aa104xd12.yaml   |  75 ----
 .../display/panel/mitsubishi,aa121td01.yaml   |  74 ----
 .../display/panel/sgd,gktw70sdae4se.yaml      |  68 ---
 .../bindings/gnss/u-blox,neo-6m.yaml          |  62 +++
 .../devicetree/bindings/gnss/u-blox.txt       |  45 --
 .../devicetree/bindings/gpio/gpio-axp209.txt  |  75 ----
 .../bindings/gpio/x-powers,axp209-gpio.yaml   |  55 +++
 .../devicetree/bindings/hwmon/iio-hwmon.yaml  |  37 ++
 .../i2c/allwinner,sun6i-a31-p2wi.yaml         |   2 +-
 .../input/touchscreen/silead,gsl1680.yaml     |  91 ++++
 .../input/touchscreen/silead_gsl1680.txt      |  44 --
 .../devicetree/bindings/media/i2c/ov5640.txt  |  92 ----
 .../bindings/media/i2c/ovti,ov5640.yaml       | 154 +++++++
 .../devicetree/bindings/media/ti,cal.yaml     |   4 +-
 .../devicetree/bindings/mfd/ac100.txt         |  50 ---
 .../devicetree/bindings/mfd/axp20x.txt        | 273 ------------
 .../bindings/mfd/x-powers,ac100.yaml          | 116 +++++
 .../bindings/mfd/x-powers,axp152.yaml         | 400 ++++++++++++++++++
 .../devicetree/bindings/mmc/mmc-card.txt      |  30 --
 .../devicetree/bindings/mmc/mmc-card.yaml     |  48 +++
 .../bindings/mmc/mmc-controller.yaml          |   6 -
 .../net/allwinner,sun8i-a83t-emac.yaml        |   4 +-
 .../bindings/net/broadcom-bluetooth.yaml      |  17 +-
 .../bindings/net/realtek-bluetooth.yaml       |   2 +
 .../devicetree/bindings/net/snps,dwmac.yaml   |   6 +-
 .../bindings/net/wireless/esp,esp8089.txt     |  30 --
 .../bindings/net/wireless/esp,esp8089.yaml    |  43 ++
 .../bindings/regulator/silergy,sy8106a.yaml   |  52 +++
 .../bindings/regulator/sy8106a-regulator.txt  |  23 -
 .../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 ++++
 .../devicetree/bindings/sound/bt-sco.txt      |  13 -
 .../bindings/sound/linux,bt-sco.yaml          |  38 ++
 .../bindings/sound/linux,spdif-dit.yaml       |  32 ++
 .../bindings/sound/simple-amplifier.txt       |  17 -
 .../sound/simple-audio-amplifier.yaml         |  45 ++
 .../bindings/sound/spdif-transmitter.txt      |  10 -
 .../devicetree/bindings/sound/wlf,wm8978.yaml |  58 +++
 .../devicetree/bindings/usb/smsc,usb3503.yaml | 108 +++++
 .../devicetree/bindings/usb/snps,dwc3.yaml    |  10 +-
 .../devicetree/bindings/usb/usb3503.txt       |  39 --
 .../devicetree/bindings/w1/w1-gpio.txt        |  27 --
 .../devicetree/bindings/w1/w1-gpio.yaml       |  44 ++
 arch/arm/boot/dts/axp209.dtsi                 |   6 +-
 arch/arm/boot/dts/axp22x.dtsi                 |   6 +-
 arch/arm/boot/dts/axp81x.dtsi                 |  10 +-
 .../arm/boot/dts/sun4i-a10-olinuxino-lime.dts |  11 +-
 arch/arm/boot/dts/sun4i-a10.dtsi              |  11 +-
 arch/arm/boot/dts/sun5i-a13.dtsi              |  15 +-
 arch/arm/boot/dts/sun6i-a31.dtsi              |  44 +-
 arch/arm/boot/dts/sun7i-a20-bananapi.dts      |  17 +-
 arch/arm/boot/dts/sun7i-a20.dtsi              |  34 +-
 arch/arm/boot/dts/sun8i-a33.dtsi              |   4 +-
 arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts     |   2 +-
 arch/arm/boot/dts/sun8i-a83t.dtsi             |   4 +-
 arch/arm/boot/dts/sun8i-h3.dtsi               |   4 +-
 arch/arm/boot/dts/sun8i-v3-sl631.dtsi         |   2 +-
 arch/arm/boot/dts/sun8i-v3s.dtsi              |   2 -
 arch/arm/boot/dts/sun9i-a80-cubieboard4.dts   |   2 +-
 .../boot/dts/sunxi-libretech-all-h3-it.dtsi   |   2 +-
 arch/arm64/boot/dts/allwinner/axp803.dtsi     |  10 +-
 .../arm64/boot/dts/allwinner/sun50i-a100.dtsi |   6 +-
 .../dts/allwinner/sun50i-a64-cpu-opp.dtsi     |   2 +-
 .../dts/allwinner/sun50i-a64-orangepi-win.dts |   2 +-
 .../boot/dts/allwinner/sun50i-a64-pinetab.dts |   2 +-
 .../boot/dts/allwinner/sun50i-a64-teres-i.dts |   3 +-
 .../boot/dts/allwinner/sun50i-h5-cpu-opp.dtsi |   2 +-
 arch/arm64/boot/dts/allwinner/sun50i-h5.dtsi  |   2 +-
 .../boot/dts/allwinner/sun50i-h6-cpu-opp.dtsi |   2 +-
 arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi  |   8 +-
 84 files changed, 2191 insertions(+), 1743 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
 create mode 100644 Documentation/devicetree/bindings/arm/sunxi/allwinner,sun6i-a31-cpuconfig.yaml
 create mode 100644 Documentation/devicetree/bindings/arm/sunxi/allwinner,sun9i-a80-prcm.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/advantech,idk-1110wr.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/advantech,idk-2121wr.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/innolux,ee101ia-01d.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/mitsubishi,aa104xd12.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/mitsubishi,aa121td01.yaml
 delete mode 100644 Documentation/devicetree/bindings/display/panel/sgd,gktw70sdae4se.yaml
 create mode 100644 Documentation/devicetree/bindings/gnss/u-blox,neo-6m.yaml
 delete mode 100644 Documentation/devicetree/bindings/gnss/u-blox.txt
 delete mode 100644 Documentation/devicetree/bindings/gpio/gpio-axp209.txt
 create mode 100644 Documentation/devicetree/bindings/gpio/x-powers,axp209-gpio.yaml
 create mode 100644 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
 create mode 100644 Documentation/devicetree/bindings/input/touchscreen/silead,gsl1680.yaml
 delete mode 100644 Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/ov5640.txt
 create mode 100644 Documentation/devicetree/bindings/media/i2c/ovti,ov5640.yaml
 delete mode 100644 Documentation/devicetree/bindings/mfd/ac100.txt
 delete mode 100644 Documentation/devicetree/bindings/mfd/axp20x.txt
 create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,ac100.yaml
 create mode 100644 Documentation/devicetree/bindings/mfd/x-powers,axp152.yaml
 delete mode 100644 Documentation/devicetree/bindings/mmc/mmc-card.txt
 create mode 100644 Documentation/devicetree/bindings/mmc/mmc-card.yaml
 delete mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.txt
 create mode 100644 Documentation/devicetree/bindings/net/wireless/esp,esp8089.yaml
 create mode 100644 Documentation/devicetree/bindings/regulator/silergy,sy8106a.yaml
 delete mode 100644 Documentation/devicetree/bindings/regulator/sy8106a-regulator.txt
 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
 delete mode 100644 Documentation/devicetree/bindings/sound/bt-sco.txt
 create mode 100644 Documentation/devicetree/bindings/sound/linux,bt-sco.yaml
 create mode 100644 Documentation/devicetree/bindings/sound/linux,spdif-dit.yaml
 delete mode 100644 Documentation/devicetree/bindings/sound/simple-amplifier.txt
 create mode 100644 Documentation/devicetree/bindings/sound/simple-audio-amplifier.yaml
 delete mode 100644 Documentation/devicetree/bindings/sound/spdif-transmitter.txt
 create mode 100644 Documentation/devicetree/bindings/sound/wlf,wm8978.yaml
 create mode 100644 Documentation/devicetree/bindings/usb/smsc,usb3503.yaml
 delete mode 100644 Documentation/devicetree/bindings/usb/usb3503.txt
 delete mode 100644 Documentation/devicetree/bindings/w1/w1-gpio.txt
 create mode 100644 Documentation/devicetree/bindings/w1/w1-gpio.yaml

Comments

Jernej Škrabec Sept. 5, 2021, 9:02 p.m. UTC | #1
Dne sreda, 01. september 2021 ob 11:18:39 CEST je Maxime Ripard napisal(a):
> The name of our PMIC power supply names conflict with the generic
> regulator supply check that matches anything called *-supply, including
> the nodes, and then makes sure it's a phandle.
> 
> A node is obviously not a phandle, so let's change our power supplies
> names to avoid any conflict.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Jernej Škrabec Sept. 5, 2021, 9:03 p.m. UTC | #2
Dne sreda, 01. september 2021 ob 11:18:40 CEST je Maxime Ripard napisal(a):
> We've had a pinctrl node name convention for a while now, let's follow
> it for the AXP pinctrl nodes as well.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Jernej Škrabec Sept. 5, 2021, 9:04 p.m. UTC | #3
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
Jernej Škrabec Sept. 5, 2021, 9:05 p.m. UTC | #4
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
Jernej Škrabec Sept. 5, 2021, 9:06 p.m. UTC | #5
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
Jernej Škrabec Sept. 5, 2021, 9:08 p.m. UTC | #6
Hi Maxime!

Dne sreda, 01. september 2021 ob 11:18:44 CEST je Maxime Ripard napisal(a):
> The crypto engine binding doesn't allow for any DMA property, yet the
> v3s define some. Remove them.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> ---
>  arch/arm/boot/dts/sun8i-v3s.dtsi | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/sun8i-v3s.dtsi b/arch/arm/boot/dts/sun8i-
v3s.dtsi
> index b30bc1a25ebb..44382e36e825 100644
> --- a/arch/arm/boot/dts/sun8i-v3s.dtsi
> +++ b/arch/arm/boot/dts/sun8i-v3s.dtsi
> @@ -285,8 +285,6 @@ crypto@1c15000 {
>  			interrupts = <GIC_SPI 80 
IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&ccu CLK_BUS_CE>, <&ccu CLK_CE>;
>  			clock-names = "ahb", "mod";
> -			dmas = <&dma 16>, <&dma 16>;
> -			dma-names = "rx", "tx";

Since V3s documentation mentions those DMA channels, wouldn't be better to 
update bindings instead?

Best regards,
Jernej

>  			resets = <&ccu RST_BUS_CE>;
>  			reset-names = "ahb";
>  		};
> -- 
> 2.31.1
> 
>
Jernej Škrabec Sept. 5, 2021, 9:11 p.m. UTC | #7
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
Jernej Škrabec Sept. 5, 2021, 9:12 p.m. UTC | #8
Dne sreda, 01. september 2021 ob 11:18:46 CEST je Maxime Ripard napisal(a):
> The dumb-vga-dac and adi,adv7123 compatibles are not supposed to be used
> together according to the binding.
> 
> Since the corpro gm7123 is a drop-in replacement for the adv7123, let's
> remove dumb-vga-dac from our compatible list.
> 
> Reviewed-by: Chen-Yu Tsai <wens@csie.org>
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Jernej Škrabec Sept. 5, 2021, 9:12 p.m. UTC | #9
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
Jernej Škrabec Sept. 5, 2021, 9:13 p.m. UTC | #10
Dne sreda, 01. september 2021 ob 11:18:48 CEST je Maxime Ripard napisal(a):
> While it doesn't really matter from a functional point of view in this
> driver's case, it's usually a good practice to list the clocks in a
> driver in the same driver across all its users.
> 
> The H6 is using the inverse order than all the other users, so let's
> make it consistent.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Jernej Škrabec Sept. 5, 2021, 9:13 p.m. UTC | #11
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
Jernej Škrabec Sept. 5, 2021, 9:14 p.m. UTC | #12
Dne sreda, 01. september 2021 ob 11:18:50 CEST je Maxime Ripard napisal(a):
> The fixed regulator clock name has a unit address, but no reg property,
> which generates a warning in DTC. Change its name to remove its useless
> unit address.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Jernej Škrabec Sept. 5, 2021, 9:14 p.m. UTC | #13
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
Jernej Škrabec Sept. 5, 2021, 9:24 p.m. UTC | #14
Dne sreda, 01. september 2021 ob 11:18:52 CEST je Maxime Ripard napisal(a):
> Neither the binding nor the driver make any use of the wakeup-source
> property for the AXP803. Remove it.
> 
> Signed-off-by: Maxime Ripard <maxime@cerno.tech>

Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Best regards,
Jernej
Maxime Ripard Sept. 6, 2021, 1:36 p.m. UTC | #15
On Sun, Sep 05, 2021 at 11:02:44PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:39 CEST je Maxime Ripard napisal(a):
> > The name of our PMIC power supply names conflict with the generic
> > regulator supply check that matches anything called *-supply, including
> > the nodes, and then makes sure it's a phandle.
> > 
> > A node is obviously not a phandle, so let's change our power supplies
> > names to avoid any conflict.
> > 
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Maxime Ripard Sept. 6, 2021, 1:37 p.m. UTC | #16
On Sun, Sep 05, 2021 at 11:03:11PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:40 CEST je Maxime Ripard napisal(a):
> > We've had a pinctrl node name convention for a while now, let's follow
> > it for the AXP pinctrl nodes as well.
> > 
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Maxime Ripard Sept. 6, 2021, 1:37 p.m. UTC | #17
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
Maxime Ripard Sept. 6, 2021, 1:38 p.m. UTC | #18
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
Maxime Ripard Sept. 6, 2021, 1:38 p.m. UTC | #19
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
Maxime Ripard Sept. 6, 2021, 1:38 p.m. UTC | #20
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
Maxime Ripard Sept. 6, 2021, 1:39 p.m. UTC | #21
On Sun, Sep 05, 2021 at 11:12:27PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:46 CEST je Maxime Ripard napisal(a):
> > The dumb-vga-dac and adi,adv7123 compatibles are not supposed to be used
> > together according to the binding.
> > 
> > Since the corpro gm7123 is a drop-in replacement for the adv7123, let's
> > remove dumb-vga-dac from our compatible list.
> > 
> > Reviewed-by: Chen-Yu Tsai <wens@csie.org>
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Maxime Ripard Sept. 6, 2021, 1:39 p.m. UTC | #22
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
Maxime Ripard Sept. 6, 2021, 1:40 p.m. UTC | #23
On Sun, Sep 05, 2021 at 11:13:25PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:48 CEST je Maxime Ripard napisal(a):
> > While it doesn't really matter from a functional point of view in this
> > driver's case, it's usually a good practice to list the clocks in a
> > driver in the same driver across all its users.
> > 
> > The H6 is using the inverse order than all the other users, so let's
> > make it consistent.
> > 
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Maxime Ripard Sept. 6, 2021, 1:40 p.m. UTC | #24
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
Maxime Ripard Sept. 6, 2021, 1:41 p.m. UTC | #25
On Sun, Sep 05, 2021 at 11:14:08PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:50 CEST je Maxime Ripard napisal(a):
> > The fixed regulator clock name has a unit address, but no reg property,
> > which generates a warning in DTC. Change its name to remove its useless
> > unit address.
> > 
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Maxime Ripard Sept. 6, 2021, 1:41 p.m. UTC | #26
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
Maxime Ripard Sept. 6, 2021, 1:41 p.m. UTC | #27
On Sun, Sep 05, 2021 at 11:24:48PM +0200, Jernej Škrabec wrote:
> Dne sreda, 01. september 2021 ob 11:18:52 CEST je Maxime Ripard napisal(a):
> > Neither the binding nor the driver make any use of the wakeup-source
> > property for the AXP803. Remove it.
> > 
> > Signed-off-by: Maxime Ripard <maxime@cerno.tech>
> 
> Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com>

Applied, thanks
Maxime
Mark Brown Sept. 13, 2021, 10:53 a.m. UTC | #28
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