From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 07 Mar 2018 22:49:46 +0100 (CET) Received: from mail.kernel.org ([198.145.29.99]:35872 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S23994727AbeCGVticGcsf (ORCPT ); Wed, 7 Mar 2018 22:49:38 +0100 Received: from saruman (jahogan.plus.com [212.159.75.221]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id C471721770; Wed, 7 Mar 2018 21:49:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C471721770 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=jhogan@kernel.org Date: Wed, 7 Mar 2018 21:49:26 +0000 From: James Hogan To: Rob Herring Cc: Alexandre Belloni , Ralf Baechle , Allan Nielsen , Linux-MIPS , "linux-kernel@vger.kernel.org" , devicetree@vger.kernel.org, =?utf-8?B?w4FsdmFybyBGZXJuw6FuZGV6?= Rojas , Kevin Cernekee , Florian Fainelli Subject: MIPS DT W=1 warnings (was Re: [PATCH v5 2/5] MIPS: mscc: add ocelot dtsi) Message-ID: <20180307214925.GU4197@saruman> References: <20180306121607.1567-1-alexandre.belloni@bootlin.com> <20180306121607.1567-3-alexandre.belloni@bootlin.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="MhP8cYafZlTESjGT" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.2 (2016-11-26) Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 62849 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: jhogan@kernel.org Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips --MhP8cYafZlTESjGT Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Rob, On Wed, Mar 07, 2018 at 10:08:28AM -0600, Rob Herring wrote: > Please compile with W=3D1 and fix any issues like this one which is a > unit-address without a reg property. Drop the unit-address. I was just giving the BMIPS W=3D1 DT warnings a look, and a few look spurious. I'd value your opinion on their legitimacy (its hard to care about W=3D1 if spurious or seemingly pedantic warnings are going to be common). e.g. 1) arch/mips/boot/dts/brcm/bcm9ejtagprb.dtb: Warning (unit_address_vs_reg): No= de /ubus/syscon-reboot@10000068 has a unit name, but no reg property = =20 due to: periph_cntl: syscon@fff8c000 { compatible =3D "syscon"; reg =3D <0xfff8c000 0xc>; native-endian; }; reboot: syscon-reboot@fff8c008 { compatible =3D "syscon-reboot"; regmap =3D <&periph_cntl>; offset =3D <0x8>; mask =3D <0x1>; }; That doesn't seem to take regmap into account. Would you strictly drop the unit-address in this case, or is there a way the DT compiler can be fixed (i presume offset and mask are binding specific, so the best it could do is probably to allow the unit-address due to the regmap without checking the actual address)? 2) arch/mips/boot/dts/brcm/bcm9ejtagprb.dtb: Warning (simple_bus_reg): Node /u= bus/syscon-reboot@10000068 missing or empty reg/ranges property Same code as above. Should syscon-reboot be outside of the simple-bus that both nodes are in, or is it fine there? There's a similar warning =66rom a DTS which has a syscon property instead of regmap. 3) arch/mips/boot/dts/brcm/bcm97425svmb.dtb: Warning (simple_bus_reg): Node /r= db@10000000/spi@41c000 simple-bus unit address format error, expected "4199= 20" qspi: spi@41c000 { #address-cells =3D <0x1>; #size-cells =3D <0x0>; compatible =3D "brcm,spi-bcm-qspi", "brcm,spi-brcmstb-qspi"; clocks =3D <&upg_clk>; reg =3D <0x419920 0x4 0x41c200 0x188 0x41c000 0x50>; reg-names =3D "cs_reg", "hif_mspi", "bspi"; ... Well 41c000 is one of the reg entries, just not the first. I presume bspi is the "main" one, perhaps that should come first since we have reg-names, but even that could potentially confuse driver code if it didn't find reg resources by name (in this case it does appear to, so perhaps that would fine)? Thanks James --MhP8cYafZlTESjGT Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEd80NauSabkiESfLYbAtpk944dnoFAlqgXmUACgkQbAtpk944 dnoh8BAAm19Ncrz8I87sU9634jgN/y/RC3UzZL2JEG9TEWo7VJWqZuLddt//yZ9I OdgedWa5If1CSKc7RGCMpa0mhLE8xAHRmeQnzIygx0Nl9uoQpWoh9Y6epNNKHJ67 sB3kgATY0zpICtOebTwB/0VsYErbigVBzEtKVC/xXNRu+90qVHMr4HZiOVwx9g7E Uc/Yq7r5iUExjtu+MdP12ldEcS3Kq1Djz2sLpOG2mQWoKT4UEwlneQIFAJWgjm0y 6ozsOZL/6JTp5edKvxOmVi8tfBCYodk22iwW61AsELPVMBzzIr4sy7Do3r9sCH5B HDpCIQQTS4eZ7Pqi+6sD7uLSP05mFyP4otejO4eMcySR9gT29x2s6Qj+Z2WTxgBg 51qIOAOEb4eoDHt+8bQjOTrYdwkSazr95Tyg8bXMj6YPGfiYV7cDPTQ8Fd3ToH36 ZisYGKMhqsX5racfIQkpRvVWgC61ZsF/nEsBAlj1C5JeviyEWdGmMKMQRiIUyL/U UCrRWHJVrm7jMJrkZ3CBGqCzyXOgGXQpUrC24pUSapk+4NJFZyEGjTxLAXrFm4i6 e6yWNXC2YNkBrO6lVp5bUpYMtxzepew+BhG6YXRFAKyLkxJZqoKzYOztaiCP5SJL S46mFLXB+qIqz4cUEAkbLcDshGiPF/bLEJ6f/+ZE6PiLs4DssB4= =0Oql -----END PGP SIGNATURE----- --MhP8cYafZlTESjGT--