From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 73B91C433FF for ; Tue, 30 Jul 2019 01:09:49 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 418FE2070B for ; Tue, 30 Jul 2019 01:09:49 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="C9qe2jAg"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=aj.id.au header.i=@aj.id.au header.b="f9JjiX8o"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="RSnK+zp5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 418FE2070B Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=aj.id.au Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:Subject:To:From:Date:References: In-Reply-To:Message-Id:Mime-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=23DoLvwsYScAlIIPDnjvfZagDSXdHTM/TrRo9vJ/x+s=; b=C9qe2jAg/HWmnU 9bQ3flToy5Bpjo4Wo60s8cUYn8Y4EjhB1PKYz7IyBVhS5F2sGLYyr3kK47vR8ptEz1uXiLaeiku6P GVREw36xuwngzqJrgBSWFGKD0Azw7b3baIVd3mBEkfiCACM8PiSj3ULn5qrt1cGD9zf8aqQWjk2xH qUo+yN7z9AumQ1n8nDCjfsbmUwQCHIOVWZaw3yKV5Fi9iUfgh6mdpSqf2A2QXkzZQRLc/dk9ccWOX jY3Y0cW4kJe7Ac02fTbzPvEag2gRdCOZbPNhJnyHkNppK2peSv6GNImOxRAUMr0qcVLjpEIB+7J+Z dAtf3aFmF6nam3/3/ewg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1hsGe4-0000lu-1w; Tue, 30 Jul 2019 01:09:40 +0000 Received: from new3-smtp.messagingengine.com ([66.111.4.229]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1hsGdz-0000lT-Vm for linux-arm-kernel@lists.infradead.org; Tue, 30 Jul 2019 01:09:37 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 3E9B01BF6; Mon, 29 Jul 2019 21:09:34 -0400 (EDT) Received: from imap2 ([10.202.2.52]) by compute4.internal (MEProxy); Mon, 29 Jul 2019 21:09:34 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=aj.id.au; h= mime-version:message-id:in-reply-to:references:date:from:to:cc :subject:content-type; s=fm3; bh=eN6StjwqcbFvBGm255dFKEDbANWqniw k/QHr4l2B8Yk=; b=f9JjiX8oQOMQXV/cuozHUCB4pzOqt3MgGW5j+ieLKBluIxW 9qTwq16SPmkbKZ2pU0uVnnexkGeGhPnlufG7wmA9iOB42se/Uf/FR/SHRk5vwHMb rQ0Kj3wnv5HEKZnzXLIyQSCyTI2BpBfwflzcNYzctlZqmI+CiNJ4l4JgrgYy6r1m NJ9miZpWKnckdH5OlyCDvoQhsp3Kvkw/0tg7IdjqtutG3J+VFry8r4NtTXuQMCpM Y6Et7QDm66hADgvC81l39Y6KDmUmhQwayD2oFZUbxxLT5/9WVXrd39o8b0C+4VIy iVWvG6oFint2AoFBApfI8FzvRiWseYT3VPb+7zg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=eN6Stj wqcbFvBGm255dFKEDbANWqniwk/QHr4l2B8Yk=; b=RSnK+zp5XQ2Pj5z+B23X3U FxtVThX/Pze3piInoA1RfkTO/LJ0EGEq98eNVSd1eayua0j3DZTD7yqo/zuzj9Hz BJWf2XEEi2wSi2MjH8aXEb8hcpxabAx/M0QK+2q9n9lpZ9NWpF3alykQ+Bv6xtgx /XWZAievp2xWgqhiB2Ym8D5RZ+YPNw4BrNm2whyqOcknssCzFWm/Z3P1yuBzQNoj 11ctvzcg61B2PnNNPi7X5dSMMmi9/DyZdlAGJb8biY1URabakNUt7sRc807uNwMX itWgUD4bHFSaAjDddJCQ1vLncAc4VDDt0s59RTrrMKeF8c9RXtnLxt12QtMWj8fg == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduvddrledvgdegvdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvufgtsehttdertderredtnecuhfhrohhmpedftehnughr vgifucflvghffhgvrhihfdcuoegrnhgurhgvfiesrghjrdhiugdrrghuqeenucfrrghrrg hmpehmrghilhhfrhhomheprghnughrvgifsegrjhdrihgurdgruhenucevlhhushhtvghr ufhiiigvpedt X-ME-Proxy: Received: by mailuser.nyi.internal (Postfix, from userid 501) id C726FE00A2; Mon, 29 Jul 2019 21:09:30 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.1.6-736-gdfb8e44-fmstable-20190718v2 Mime-Version: 1.0 Message-Id: In-Reply-To: References: <20190726053959.2003-1-andrew@aj.id.au> Date: Tue, 30 Jul 2019 10:39:50 +0930 From: "Andrew Jeffery" To: "Rob Herring" Subject: Re: [RFC-ish PATCH 00/17] Clean up ASPEED devicetree warnings X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190729_180936_163752_1BA655CD X-CRM114-Status: GOOD ( 23.02 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , linux-aspeed@lists.ozlabs.org, Linus Walleij , Corey Minyard , Joel Stanley , devicetree@vger.kernel.org, Xo Wang , Arnd Bergmann , Ken Chen , Adriana Kobylak , "open list:GPIO SUBSYSTEM" , =?UTF-8?Q?YangBrianC=2EW_=E6=A5=8A=E5=98=89=E5=81=89_TAO?= , openipmi-developer@lists.sourceforge.net, "Alexander A. Filippov" , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" , Tao Ren , "linux-kernel@vger.kernel.org" , yao.yuan@linaro.org, Patrick Venture , John Wang , Greg Kroah-Hartman , Stefan M Schaeckeler , Haiyue Wang Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, 30 Jul 2019, at 10:23, Rob Herring wrote: > On Thu, Jul 25, 2019 at 11:40 PM Andrew Jeffery wrote: > > > > Hello, > > > > The aim of this series is to minimise/eliminate all the warnings from the > > ASPEED devicetrees. It mostly achieves its goal, as outlined below. > > > > Using `aspeed_g5_defconfig` we started with the follow warning count: > > > > $ make dtbs 2>&1 >/dev/null | wc -l > > 218 > > > > and after the full series is applied we have: > > > > $ make dtbs 2>&1 >/dev/null | wc -l > > 2 > > > > for a 100x reduction. > > > > Getting there though isn't without some potential controversy, which I've saved > > for the last half of the series. The following patches I think are in pretty > > good shape: > > > > ARM: dts: aspeed-g5: Move EDAC node to APB > > ARM: dts: aspeed-g5: Use recommended generic node name for SDMC > > ARM: dts: aspeed-g5: Fix aspeed,external-nodes description > > ARM: dts: vesnin: Add unit address for memory node > > ARM: dts: fp5280g2: Cleanup gpio-keys-polled properties > > ARM: dts: swift: Cleanup gpio-keys-polled properties > > ARM: dts: witherspoon: Cleanup gpio-keys-polled properties > > ARM: dts: aspeed: Cleanup lpc-ctrl and snoop regs > > ARM: dts: ibm-power9-dual: Add a unit address for OCC nodes > > > > With these patches applied we get to: > > > > $ make dtbs 2>&1 >/dev/null | wc -l > > 144 > > > > So they make a dent, but fail to clean up the bulk of the issues. From here > > I've mixed in some binding and driver changes with subsequent updates to the > > devicetrees: > > > > dt-bindings: pinctrl: aspeed: Add reg property as a hint > > dt-bindings: misc: Document reg for aspeed,p2a-ctrl nodes > > ARM: dts: aspeed: Add reg hints to syscon children > > dt-bindings: ipmi: aspeed: Introduce a v2 binding for KCS > > ipmi: kcs: Finish configuring ASPEED KCS device before enable > > ipmi: kcs: aspeed: Implement v2 bindings > > ARM: dts: aspeed-g5: Change KCS nodes to v2 binding > > ARM: dts: aspeed-g5: Sort LPC child nodes by unit address > > > > By `dt-bindings: ipmi: aspeed: Introduce a v2 binding for KCS` the warnings are > > reduced to: > > > > $ make dtbs 2>&1 >/dev/null | wc -l > > 125 > > > > The bang-for-buck is in fixing up the KCS bindings which removes all-but-two of > > the remaining warnings (which we can't feasibly remove), but doing so forces > > code changes (which I'd avoided up until this point). > > > > Reflecting broadly on the fixes, I think I've made a mistake way back by using > > syscon/simple-mfds to expose the innards of the SCU and LPC controllers in the > > devicetree. This series cleans up what's currently there, but I have half a > > mind to rev the SCU and LPC bindings to not use simple-mfd and instead have a > > driver implementation that uses `platform_device_register_full()` or similar to > > deal with the mess. > > > > Rob - I'm looking for your thoughts here and on the series, I've never felt > > entirely comfortable with what I cooked up. Your advice would be appreciated. > > The series generally looks fine to me from a quick scan. As far as > dropping 'simple-mfd', having less fine grained description in DT is > generally my preference. It comes down to whether what you have > defined is maintainable. As most of it is just additions, I think what > you have is fine. Maybe keep all this in mind for the next chip > depending how the SCU and LPC change. Okay, I think the timing of that suggestion is good given where things are with the AST2600. I'll keep that in mind. Consensus so far seems to be that the series is fine. I'll split it up and send out the sub-series to the relevant lists with the acks accumulated here. Thanks all for taking a look. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel