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=-5.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 A6E51C433DF for ; Thu, 2 Jul 2020 07:50:35 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 74C5020B1F for ; Thu, 2 Jul 2020 07:50:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="CLwIySaP"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="t2jT/Z09" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 74C5020B1F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=samsung.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+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=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:References:In-Reply-To:MIME-Version:Date:Message-ID: From:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+k78T+zOFWbxyjOq3YLxgDnlZR6pItDkqqa6BdjLURg=; b=CLwIySaP6rxXWAEaNLIEP+n7f g9OIv7a/25cGCzacYRGi8mt/gUaM4mbrDLwwuL6Iam0mXZ681hGzQ0cHbekzPzLmZyK556Vx9Tj6z VShlHjNl70Z/txIlBl44HB4DC8wB1PMuQlA7m/pvF1nkwYAw3GaBB4b9/IcrpVBuaX4U9u/7zAqfX NeJ9hdXrzjSVpVaO9QSgjWxUrwTZkNx7kbBkTYiLS4SB3G9x/CDwjks7tcKS/Y63LReGRSqFZDPnP eGig3D9j6hPP5ncfJqN1T8Xb05UL4ANYyDXbHnQ9LMSNWkrzelRhU9Yr5VlMTz1ByCddf6Yk1Ze8I knZsBG1VQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jqtxl-00011f-Iq; Thu, 02 Jul 2020 07:48:53 +0000 Received: from mailout1.w1.samsung.com ([210.118.77.11]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jqtxM-0000lw-3W for linux-arm-kernel@lists.infradead.org; Thu, 02 Jul 2020 07:48:29 +0000 Received: from eucas1p1.samsung.com (unknown [182.198.249.206]) by mailout1.w1.samsung.com (KnoxPortal) with ESMTP id 20200702074826euoutp0146ac88c491221475339f8cbc89c90861~d3-mgS3FN2609526095euoutp01W for ; Thu, 2 Jul 2020 07:48:26 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout1.w1.samsung.com 20200702074826euoutp0146ac88c491221475339f8cbc89c90861~d3-mgS3FN2609526095euoutp01W DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1593676106; bh=7YNN+9wShxeLt7CQ8t/PBpHhBfC5AdB9p0g33Egi1Xw=; h=Subject:To:Cc:From:Date:In-Reply-To:References:From; b=t2jT/Z097GlqnKV2yR8THlY2mbQsRyU/gpdvEaEDNXo8XgDkpLLpkxfnFWgwsO6IL rViWvZ6GrFkuZdEPKYnAcesvwq86FmGVkYjVCnwciQLQiB8GqboDwKjH4WfCVs7drL sd7/GEnnCbqjjxlAPg5YP5u2V4c67gQs7SeIbMlk= Received: from eusmges3new.samsung.com (unknown [203.254.199.245]) by eucas1p2.samsung.com (KnoxPortal) with ESMTP id 20200702074826eucas1p2bc51d4ea26094f6975b6132be0c06ede~d3-mTmFBh1699716997eucas1p2x; Thu, 2 Jul 2020 07:48:26 +0000 (GMT) Received: from eucas1p1.samsung.com ( [182.198.249.206]) by eusmges3new.samsung.com (EUCPMTA) with SMTP id D7.29.06318.A419DFE5; Thu, 2 Jul 2020 08:48:26 +0100 (BST) Received: from eusmtrp2.samsung.com (unknown [182.198.249.139]) by eucas1p2.samsung.com (KnoxPortal) with ESMTPA id 20200702074825eucas1p2791360695d10b4b3bd93046b42a7c0ec~d3-l6TOm11914919149eucas1p2r; Thu, 2 Jul 2020 07:48:25 +0000 (GMT) Received: from eusmgms2.samsung.com (unknown [182.198.249.180]) by eusmtrp2.samsung.com (KnoxPortal) with ESMTP id 20200702074825eusmtrp2eeb230027eabaa274d49d234abcbf527~d3-l5Y9mI0980709807eusmtrp2H; Thu, 2 Jul 2020 07:48:25 +0000 (GMT) X-AuditID: cbfec7f5-371ff700000018ae-e0-5efd914a9ac5 Received: from eusmtip1.samsung.com ( [203.254.199.221]) by eusmgms2.samsung.com (EUCPMTA) with SMTP id 51.1B.06017.9419DFE5; Thu, 2 Jul 2020 08:48:25 +0100 (BST) Received: from [106.210.88.143] (unknown [106.210.88.143]) by eusmtip1.samsung.com (KnoxPortal) with ESMTPA id 20200702074825eusmtip13d90dba610361392a6aa8d5d1724dae5~d3-lOUFuN1440914409eusmtip1N; Thu, 2 Jul 2020 07:48:24 +0000 (GMT) Subject: Re: [PATCH v2] ARM: dts: exynos: Fix missing empty reg/ranges property regulators on Trats To: Krzysztof Kozlowski From: Marek Szyprowski Message-ID: <757ee654-e087-d660-2a9c-801743e83567@samsung.com> Date: Thu, 2 Jul 2020 09:48:26 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20200702073913.GA1187@kozik-lap> Content-Language: en-US X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrPKsWRmVeSWpSXmKPExsWy7djPc7peE//GGTx9zWzxYN42NouNM9az Wlz/8pzVYv6Rc6wW/Y+BEufPb2C32PT4GqvF5V1z2CxmnN/HZLFo6xd2i9a9R9gt2p++ZHbg 8di0qpPNY/OSeo++LasYPT5vkgtgieKySUnNySxLLdK3S+DKmHXpAlPBZt6K9UsfsTYwLuPu YuTkkBAwkVi4aDtTFyMXh5DACkaJy0eeM0I4Xxglzp7fxgLhfGaUeHq2kxWupW8uVMtyRonP 59ezQjjvGSVuHN/JAlIlLJAqca1pLTOILSKgKXH973ewbmaBlcwSr3dagthsAoYSXW+72EBs XgE7ib1/DjCC2CwCKhLvm0+BzREViJNY/xLkQJAaQYmTM5+AxTkF9CT6Nr6Amikvsf3tHGYI W1zi1pP5YNdJCFxil9jXcYMd4mwXiekTZ7NB2MISr45vgYrLSPzfCdPQzCjx8NxadginBxgc TTMYIaqsJe6c+wXUzQG0QlNi/S59iLCjxJ2Jv9hBwhICfBI33gpCHMEnMWnbdGaIMK9ER5sQ RLWaxKzj6+DWHrxwiXkCo9IsJK/NQvLOLCTvzELYu4CRZRWjeGppcW56arFxXmq5XnFibnFp Xrpecn7uJkZgsjr97/jXHYz7/iQdYhTgYFTi4c2o+BMnxJpYVlyZe4hRgoNZSYTX6ezpOCHe lMTKqtSi/Pii0pzU4kOM0hwsSuK8xotexgoJpCeWpGanphakFsFkmTg4pRoYw9Iv6oVUO2py 8drks1u0yK++yGJYxXhs1fWrS9arfe/eq77thaOZWuWJtPQtlb12+jadBzpURSpnpPfpRJ3d 6ZJxr2TF92oVFuGKkJkWE777nHJxVzOpXqm0fHvUYTvBxV90e9dKX2jZskF58R1HXXeg495P Y0tf18jArll3ZgrXYUXuKCWW4oxEQy3mouJEABzL3vZSAwAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrEIsWRmVeSWpSXmKPExsVy+t/xu7qeE//GGfyZpGXxYN42NouNM9az Wlz/8pzVYv6Rc6wW/Y9fM1ucP7+B3WLT42usFpd3zWGzmHF+H5PFoq1f2C1a9x5ht2h/+pLZ gcdj06pONo/NS+o9+rasYvT4vEkugCVKz6Yov7QkVSEjv7jEVina0MJIz9DSQs/IxFLP0Ng8 1srIVEnfziYlNSezLLVI3y5BL2PWpQtMBZt5K9YvfcTawLiMu4uRk0NCwERiYd9cpi5GLg4h gaWMEsunzGKHSMhInJzWwAphC0v8udbFBlH0llHiw9PXzCAJYYFUiWtNa8FsEQFNiet/v7OC FDELrGSWeP2llRGio5lJ4v36y0wgVWwChhJdb0FGcXLwCthJ7P1zgBHEZhFQkXjffIoFxBYV iJNYvmU+O0SNoMTJmU/A4pwCehJ9G1+AncQsYCYxb/NDZghbXmL72zlQtrjErSfzmSYwCs1C 0j4LScssJC2zkLQsYGRZxSiSWlqcm55bbKRXnJhbXJqXrpecn7uJERif24793LKDsetd8CFG AQ5GJR7ejIo/cUKsiWXFlbmHGCU4mJVEeJ3Ono4T4k1JrKxKLcqPLyrNSS0+xGgK9NxEZinR 5Hxg6sgriTc0NTS3sDQ0NzY3NrNQEuftEDgYIySQnliSmp2aWpBaBNPHxMEp1cC4tUNmmd3K 7Zwf+brYDP9M3mXY/LlJu3iWe2y56fK5mSpbNIufnOLReyHpOO/StGmGc04+PcRw+PiCZe+z u4JWPo1TZfr2Na1iWbcK05Q+LRvl4uBK6U/zF5x4c+NWi3lJocw1qxVff5s15n50ZTopt2Hl ysa96p9Wd+g6xfk/SfivyCnitX+hEktxRqKhFnNRcSIAeG6IKuUCAAA= X-CMS-MailID: 20200702074825eucas1p2791360695d10b4b3bd93046b42a7c0ec X-Msg-Generator: CA X-RootMTR: 20200629210025eucas1p219a52e75ecce9e813aa80f0126780189 X-EPHeader: CA CMS-TYPE: 201P X-CMS-RootMailID: 20200629210025eucas1p219a52e75ecce9e813aa80f0126780189 References: <20200629205948.32250-1-krzk@kernel.org> <97651868-30f3-6b91-1ea2-551ee1ebad8f@samsung.com> <20200702061611.GC4175@kozik-lap> <20200702073913.GA1187@kozik-lap> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200702_034828_406650_7D8F209E X-CRM114-Status: GOOD ( 15.98 ) 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: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Bartlomiej Zolnierkiewicz , Sylwester Nawrocki , Pankaj Dubey , linux-kernel@vger.kernel.org, Chanwoo Choi , Rob Herring , Kukjin Kim , Alim Akhtar , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 02.07.2020 09:39, Krzysztof Kozlowski wrote: > On Thu, Jul 02, 2020 at 08:16:11AM +0200, Krzysztof Kozlowski wrote: >> On Tue, Jun 30, 2020 at 08:27:01AM +0200, Marek Szyprowski wrote: >>> On 29.06.2020 22:59, Krzysztof Kozlowski wrote: >>>> Remove the regulators node entirely because its children do not have any >>>> unit addresses. This fixes DTC warning: >>>> >>>> Warning (simple_bus_reg): /regulators/regulator-0: missing or empty reg/ranges property >>>> >>>> Signed-off-by: Krzysztof Kozlowski >>> Tested-by: Marek Szyprowski >>> >>> What about removing the regulators node from other boards: >>> exynos4412-origen.dts, exynos5420-smdk5420.dts and exynos5250-arndale.dts? >>> >>> On the other hand, maybe it would be really easier to add missing >>> address/size-cells properties to exynos4210-trats.dts/regulators node? >> Indeed let's keep it consistent so in such case better to add here >> proper address/size-cells. > Actually more of DTSes put fixed regulators directly in root node, not > under "regulators" node: > exynos3250-monk.dts > exynos4210-i9100.dts > exynos4210-origen.dts > exynos4210-universal_c210.dts > exynos4412-galaxy-s3.dtsi > exynos4412-midas.dtsi > exynos4412-n710x.dts > exynos4412-odroidx.dts > exynos5250-smdk5250.dts > exynos5250-snow-common.dtsi > exynos5420-peach-pit.dts > exynos5800-peach-pi.dts > > If we want it to be consistent, it's easier to remove the regulator > nodes from exynos4412-origen.dts, exynos5420-smdk5420.dts and > exynos5250-arndale.dts. Feel free, I'm fine with both approaches. Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel