Re: [PATCH v4 00/21] arm64: sunxi: Initial Allwinner H616 SoC support

From: Maxime Ripard <maxime@cerno.tech>
To: Andre Przywara <andre.przywara@arm.com>
Cc: "Chen-Yu Tsai" <wens@csie.org>,
	"Jernej Skrabec" <jernej.skrabec@siol.net>,
	"Samuel Holland" <samuel@sholland.org>,
	"Icenowy Zheng" <icenowy@aosc.io>,
	"Rob Herring" <robh@kernel.org>,
	"Clément Péron" <peron.clem@gmail.com>,
	"Shuosheng Huang" <huangshuosheng@allwinnertech.com>,
	"Yangtao Li" <tiny.windzz@gmail.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com,
	"Linus Walleij" <linus.walleij@linaro.org>,
	"Ulf Hansson" <ulf.hansson@linaro.org>
Subject: Re: [PATCH v4 00/21] arm64: sunxi: Initial Allwinner H616 SoC support
Date: Tue, 26 Jan 2021 13:07:47 +0100	[thread overview]
Message-ID: <20210126120747.zvsk2ttowruemefu@gilmour> (raw)
In-Reply-To: <20210125151811.11871-1-andre.przywara@arm.com>

[-- Attachment #1: Type: text/plain, Size: 4552 bytes --]

Hi,

On Mon, Jan 25, 2021 at 03:17:50PM +0000, Andre Przywara wrote:
> Hi,
> 
> an update from the v3 last week, to add support for the Allwinner H616
> SoC. Still based on the (updated) sunxi/for-next branch.
> I am omitting the MMC and pinctrl patches now, as they have been taken
> by Ulf and LinusW already into their trees.
> 
> I addressed the comments from the list, though not sure if the HOSC clock
> coming from the RTC is really a good idea (didn't change it).
> 
> I was hoping that at least the clock and AXP patches (01/21-06/21) could
> make it into 5.12 still? That would allow booting boards from SD card.
> I am happy to strip the .dtsi and .dts patch to only contain the devices
> that we actually support then, if needed.
> 
> For the changes: I split the joint DT bindings patch into subsystem
> specific ones, hopefully this simplifies merging. There is a new fix
> for the axp20x-pek driver, and the AXP IRQ fix has been amended as well
> (according to Samuel's comments).
> 
> I eventually got USB to work, by requiring PHY 2 for [EO]HCI 1 & 3 as well.
> Not sure this is the right fix, it might be just one clock or reset
> line from that PHY that is needed for the others as well. Will do further
> experiments.
> 
> For a more detailed changelog, see below.

With your series applied:

ARCH=arm64 make -j18 dtbs_check  
arch/arm64/Makefile:25: ld does not support --fix-cortex-a53-843419; kernel may be susceptible to erratum
arch/arm64/Makefile:33: LSE atomics not supported by binutils
arch/arm64/Makefile:44: Detected assembler with broken .inst; disassembly will be unreliable
  DTC     arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dtb
  DTC     arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml
  CHECK   arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: clock@3001000: clocks: [[7], [8, 0], [8, 2]] is too long
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: clock@3001000: clock-names: ['hosc', 'losc', 'iosc'] is too long
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: ethernet@5020000: compatible: 'oneOf' conditional failed, one must be fixed:
	['allwinner,sun50i-h616-emac', 'allwinner,sun50i-a64-emac'] is too long
	Additional items are not allowed ('allwinner,sun50i-a64-emac' was unexpected)
	'allwinner,sun8i-a83t-emac' was expected
	'allwinner,sun8i-h3-emac' was expected
	'allwinner,sun8i-r40-emac' was expected
	'allwinner,sun8i-v3s-emac' was expected
	'allwinner,sun50i-a64-emac' was expected
	'allwinner,sun50i-h6-emac' was expected
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: clock@7010000: clocks: [[7], [8, 0], [8, 2], [2, 4]] is too long
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: clock@7010000: clock-names: ['hosc', 'losc', 'iosc', 'pll-periph'] is too long
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/clock/allwinner,sun4i-a10-ccu.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: rsb@7083000: compatible: 'oneOf' conditional failed, one must be fixed:
	['allwinner,sun50i-h616-rsb', 'allwinner,sun8i-a23-rsb'] is too long
	Additional items are not allowed ('allwinner,sun8i-a23-rsb' was unexpected)
	'allwinner,sun8i-a23-rsb' was expected
	'allwinner,sun8i-a83t-rsb' was expected
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/bus/allwinner,sun8i-a23-rsb.yaml
/home/max/Work/allwinner/build/arm64/arch/arm64/boot/dts/allwinner/sun50i-h616-orangepi-zero2.dt.yaml: leds: led-0:default-state:0: 'on' is not of type 'array'
	From schema: /home/max/Work/repos/linux/Documentation/devicetree/bindings/leds/leds-gpio.yaml

The last one is not related to your changes, but the rest must be fixed

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]