[PATCH 0/2] Try to fix DT schema problems for V3-series DTs

28 views
Skip to first unread message

Icenowy Zheng

unread,
Oct 3, 2020, 7:49:15 PM10/3/20
to Rob Herring, Maxime Ripard, Chen-Yu Tsai, Clément Péron, devic...@vger.kernel.org, linux-ar...@lists.infradead.org, linux-...@vger.kernel.org, linux...@googlegroups.com, Icenowy Zheng
This patchset tries to fix DT schema verification errors that exist in
V3-series device trees.

The first patch drops bogus properties that is not needed in PineCube
DT, and the second one adds compatible to the binding.

Icenowy Zheng (2):
ARM: dts: sun8i: s3: drop bogus cells for CSI subnode on PineCube
dt-bindings: sram: sunxi-sram: add V3s compatible string

.../bindings/sram/allwinner,sun4i-a10-system-control.yaml | 3 +++
arch/arm/boot/dts/sun8i-s3-pinecube.dts | 3 ---
2 files changed, 3 insertions(+), 3 deletions(-)

--
2.28.0

Icenowy Zheng

unread,
Oct 3, 2020, 7:50:43 PM10/3/20
to Rob Herring, Maxime Ripard, Chen-Yu Tsai, Clément Péron, devic...@vger.kernel.org, linux-ar...@lists.infradead.org, linux-...@vger.kernel.org, linux...@googlegroups.com, Icenowy Zheng
The address and size cells infomation in CSI subnode is not necessary
(because the camera subnode has no reg property).

Drop them.

Signed-off-by: Icenowy Zheng <ice...@aosc.io>
---
arch/arm/boot/dts/sun8i-s3-pinecube.dts | 3 ---
1 file changed, 3 deletions(-)

diff --git a/arch/arm/boot/dts/sun8i-s3-pinecube.dts b/arch/arm/boot/dts/sun8i-s3-pinecube.dts
index 9bab6b7f4014..76c8a29e355b 100644
--- a/arch/arm/boot/dts/sun8i-s3-pinecube.dts
+++ b/arch/arm/boot/dts/sun8i-s3-pinecube.dts
@@ -64,9 +64,6 @@ &csi1 {
status = "okay";

port {
- #address-cells = <1>;
- #size-cells = <0>;
-
csi1_ep: endpoint {
remote-endpoint = <&ov5640_ep>;
bus-width = <8>;
--
2.28.0

Icenowy Zheng

unread,
Oct 3, 2020, 7:50:50 PM10/3/20
to Rob Herring, Maxime Ripard, Chen-Yu Tsai, Clément Péron, devic...@vger.kernel.org, linux-ar...@lists.infradead.org, linux-...@vger.kernel.org, linux...@googlegroups.com, Icenowy Zheng
Add compatible string for V3s, with H3 one as fallback.

This is used in device tree now, but not standardized in DT binding.

Signed-off-by: Icenowy Zheng <ice...@aosc.io>
---
.../bindings/sram/allwinner,sun4i-a10-system-control.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml b/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml
index 6ebcbc153691..b66a07e21d1e 100644
--- a/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml
+++ b/Documentation/devicetree/bindings/sram/allwinner,sun4i-a10-system-control.yaml
@@ -33,6 +33,9 @@ properties:
- const: allwinner,sun4i-a10-system-control
- const: allwinner,sun8i-a23-system-control
- const: allwinner,sun8i-h3-system-control
+ - items:
+ - const: allwinner,sun8i-v3s-system-control
+ - const: allwinner,sun8i-h3-system-control
- items:
- const: allwinner,sun8i-r40-system-control
- const: allwinner,sun4i-a10-system-control
--
2.28.0

Maxime Ripard

unread,
Oct 5, 2020, 4:22:52 AM10/5/20
to Icenowy Zheng, Rob Herring, Chen-Yu Tsai, Clément Péron, devic...@vger.kernel.org, linux-ar...@lists.infradead.org, linux-...@vger.kernel.org, linux...@googlegroups.com
On Sun, Oct 04, 2020 at 07:48:42AM +0800, Icenowy Zheng wrote:
> This patchset tries to fix DT schema verification errors that exist in
> V3-series device trees.
>
> The first patch drops bogus properties that is not needed in PineCube
> DT, and the second one adds compatible to the binding.
>
> Icenowy Zheng (2):
> ARM: dts: sun8i: s3: drop bogus cells for CSI subnode on PineCube
> dt-bindings: sram: sunxi-sram: add V3s compatible string

Squashed the first one and applied the second, thanks!
Maxime
signature.asc
Reply all
Reply to author
Forward
0 new messages