Re: [PATCH 1/3] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain

From: Nishanth Menon <nm@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: devicetree@vger.kernel.org,
	Grygorii Strashko <grygorii.strashko@ti.com>,
	Lokesh Vutla <lokeshvutla@ti.com>,
	linux-kernel@vger.kernel.org, robh+dt@kernel.org,
	t-kristo@ti.com, Faiz Abbas <faiz_abbas@ti.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/3] arm64: dts: ti: k3-j7200-main: Add gpio nodes in main domain
Date: Fri, 13 Nov 2020 14:55:52 -0600	[thread overview]
Message-ID: <20201113205552.p63u2gxgw4pfxil5@stunning> (raw)
In-Reply-To: <c6267288-581e-3b34-cec5-d5879f3d1283@ti.com>

On 00:39-20201114, Sekhar Nori wrote:
> 
> I was using the latest schema from master. But I changed to 2020.08.1
> also, and still don't see the warning.
> 
> $ dt-doc-validate --version
> 2020.12.dev1+gab5a73fcef26
> 
> I dont have a system-wide dtc installed. One in kernel tree is updated.
> 
> $ scripts/dtc/dtc --version
> Version: DTC 1.6.0-gcbca977e
> 
> Looking at your logs, it looks like you have more patches than just this
> applied. I wonder if thats making a difference. Can you check with just
> these patches applied to linux-next or share your tree which includes
> other patches?
> 
> In your logs, you have such error for other interrupt controller nodes
> as well. For example:
> 
>  arch/arm64/boot/dts/ti/k3-j7200-main.dtsi:
> /bus@100000/bus@30000000/interrupt-controller1: Missing #address-cells
> in interrupt provider
> 
> Which I don't see in my logs. My guess is some other patch(es) in your
> patch stack either uncovers this warning or causes it.

Oh boy! I sent you and myself on wild goose chase! Really sorry about
messing up in the report of bug.

It is not dtbs_check, it is building dtbs with W=2 that generates this
warning. dtc 1.6.0 is sufficient to reproduce this behavior.

Using v5.10-rc1 as baseline (happens the same with next-20201113 as
		well.

v5.10-rc1: https://pastebin.ubuntu.com/p/Pn9HDqRjQ4/ (recording:
    https://asciinema.org/a/55YVpql9Bq8rh8fePTxI2xObO)

v5.10-rc1 + 1st patch in the series(since we are testing):
	https://pastebin.ubuntu.com/p/QWQRMSv565/ (recording:
https://asciinema.org/a/ZSKZkOY13l4lmZ2xWH34jMlM1)

Diff: https://pastebin.ubuntu.com/p/239sYYT2QY/

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel