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.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 3D26FC433DF for ; Thu, 2 Jul 2020 07:39:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 1D8E420899 for ; Thu, 2 Jul 2020 07:39:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1593675566; bh=wWiFkYyYMP2WwvSYeZw5ASaMoJ6PQY8vOQ0lKc9zjE8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=K30WtQHBjLheBs9Im9Bk/2L0xkamnZw9OGhflxjXcVr1oVuFnlqLDfvgtuoXzQaPQ JyZ2h95Lyf7ipEJz7WPha41TczrxyDWkEVEU9mROlLaFwoUwAjClmg5cCuYXZ9K7G9 kRVGTIUzxL3kcegK1jyZgIVKB+aDDRP16/NfEKBc= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726963AbgGBHjT (ORCPT ); Thu, 2 Jul 2020 03:39:19 -0400 Received: from mail-ej1-f66.google.com ([209.85.218.66]:42993 "EHLO mail-ej1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726362AbgGBHjS (ORCPT ); Thu, 2 Jul 2020 03:39:18 -0400 Received: by mail-ej1-f66.google.com with SMTP id f12so1879027eja.9; 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=Z+X5rHOyWUGQQOzN0IvKFPjnX2Va0khS9K5AeUUIiLQ/mTihQNddWS5PsFeaw8w2Dj otOu36jRIrb3uEW0elavruCI/tXU9YqrUNR1mIEOeeNBwASQ/n8ylR8CFNGWH/DXvU40 5yH825HY6KUpkRKaqLaEl+oG8DQOctBreFN0GCBJk83B3ih1UwPwal7NflnHc/VQqSg0 PJLfBS1CaszE6y0S0otolOydQw3cHrF9W4ZZoKxq/XEHjxLXCt1xBqoStByBXcFbuWjd 0av0J2/66e8UcSIzZVSDSA7x8wh+JmUyi6z+JSpB0mllt0XbFJi0vLR3joX+7wy6uNt5 SU6Q== X-Gm-Message-State: AOAM53334Ybj0QrPVr6luojHQgFgicSwbXCQds5OluFntIMSrg+/OoFg yUCJ/whlFtLZ/TAI8hEJYvk= 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 Cc: Sylwester Nawrocki , Rob Herring , Kukjin Kim , Alim Akhtar , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, Bartlomiej Zolnierkiewicz , Chanwoo Choi , Pankaj Dubey 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-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200702061611.GC4175@kozik-lap> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.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 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