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,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 43440C433E0 for ; Thu, 2 Jul 2020 07:40:52 +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 1209620899 for ; Thu, 2 Jul 2020 07:40:52 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="p8cy+pkw" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1209620899 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=JnqR1flfD2bvPS0u5posZyl9mgSUGsao+hJ0uI+mDtg=; b=p8cy+pkw1cqn6OCoOSuSaP6Lo 6QbCyytwR4TDciRNEBW6/jdtWY5ccgRDJNA9xM+5YyWFXoKlF4P4TPCeF9NdkkgtDP4GgzRZrgFgu oZkdLPzjeBBBzarbYOTy+/8YLL2hnHj2hMjjVppk23S4UmZt1t7PhN6N8hkOY/v92+6ThOsAwOmi6 tCRSv3SBXkNru1FmflJQHRtKQBOtlRRibMOcvSXRFRHUjp24uQlrcucTEfmVgdmUhcHllveTxKoic SqWpUajYge1xwq7AAg5+4wc6/y4a06zs08lwmk78oOEBJpq7UJEHuxmfZrRulmT59lldIAkbKzZya qeEKDtyEg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jqtoW-0008HQ-1Q; Thu, 02 Jul 2020 07:39:20 +0000 Received: from mail-ej1-f68.google.com ([209.85.218.68]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jqtoT-0008H5-Sb for linux-arm-kernel@lists.infradead.org; Thu, 02 Jul 2020 07:39:18 +0000 Received: by mail-ej1-f68.google.com with SMTP id lx13so9428888ejb.4 for ; Thu, 02 Jul 2020 00:39:17 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=L+qtrKeUknobDsiDsqcsCQWN3hO9VKyL68T69UrtNic=; b=dTUzwYLOTp5mn00x8zpohhUK7Ao/4Vk+z0adC+HD4hLZk5lW4zYgu1+EUJ1KBZvgQA E4g4chIFxF9+ZqKkZ8hDiShToEZiwOBpgBA+8meIazAxxfi7WuWt2sr1LDU6J2U2GUKc Q/24RyX/DN+l/x1CaMx8wFRxNoLF+mM5rNA3yGtN+Exzh2fYt9qiB/9GF+6EtVuYy944 8IksexXoCKkp53tLuldYxB7gPni9EvTCwYPPxIWOy97bjxIjQ+jc0CVOLkoolmC+F0DT SxNLwK22j3tCgeKW+Dw+vI35kirB7xxh+ZGc2zWK7hctYjOgO1OK/N0Ath7fHWJ3+i5U Vj8A== X-Gm-Message-State: AOAM530Ue8EUAOnZ4lCEGZoExAFrGTnNKtzdKy4inGydJcLIXbH2nvt+ rPIBTj1vZHDZYNVLLqJ+i1c= X-Google-Smtp-Source: ABdhPJwAvhXIPghcOcmZpMa7MJoAcsGQ9JyVGobacO92qgnYbbeRf3sNWsQ+8MdyqKFuV4Agxd2xTA== X-Received: by 2002:a17:906:899:: with SMTP id n25mr24916943eje.298.1593675556682; Thu, 02 Jul 2020 00:39:16 -0700 (PDT) Received: from kozik-lap ([194.230.155.195]) by smtp.googlemail.com with ESMTPSA id w8sm6284760ejb.10.2020.07.02.00.39.14 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Thu, 02 Jul 2020 00:39:15 -0700 (PDT) Date: Thu, 2 Jul 2020 09:39:13 +0200 From: Krzysztof Kozlowski To: Marek Szyprowski Subject: Re: [PATCH v2] ARM: dts: exynos: Fix missing empty reg/ranges property regulators on Trats Message-ID: <20200702073913.GA1187@kozik-lap> References: <20200629205948.32250-1-krzk@kernel.org> <97651868-30f3-6b91-1ea2-551ee1ebad8f@samsung.com> <20200702061611.GC4175@kozik-lap> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200702061611.GC4175@kozik-lap> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200702_033917_932605_1ED7E134 X-CRM114-Status: GOOD ( 15.24 ) 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 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: > > Hi Krzysztof, > > > > 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. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel