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=-10.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 D0CDCC4363A for ; Tue, 27 Oct 2020 10:39:19 +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 4C43120829 for ; Tue, 27 Oct 2020 10:39:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="nu1lzn+T" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4C43120829 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=vJ2roEa6t28Fo6PHrNBFlUWQF7y1YuuQdOqx7zSmqXY=; b=nu1lzn+TehdJn0SiQq/fWYATW y4VZNNrSW3zAGhVTWWPhhunjXTjjYQOWCTNp/atAeu6szL5LGY3VFtiQiYaRiXqkkr2nQgaTTYcXU nmjfslLDbEROdUqgMKkepVtHoBCf6feWD74iPndrNNDEG0GWxod6V18Zsjn7pFrZAYgsRd2k/lzEf LKgvij9u1aOu7Jw5F6HwJWaQyHv31/bzFkLSyMD6q/89Sgmp8JyoIzF+3YNyJgAHVL7tmBDDvonTv D7N19hcnJAOtimu849hQ8gAds8wwSErvimmj/IhkuYkKibpY8wyMHfeCkxWxl7I1Z8GH8r/VK37LJ KEWHJKQpA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kXMMJ-0008Is-5Q; Tue, 27 Oct 2020 10:37:43 +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 1kXMME-0008FR-Jm for linux-arm-kernel@lists.infradead.org; Tue, 27 Oct 2020 10:37:40 +0000 Received: by mail-ej1-f68.google.com with SMTP id p9so1501625eji.4 for ; Tue, 27 Oct 2020 03:37:36 -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; bh=UCQxlKOKzQn6npHWL+chgiqrZv7e5KPLyCbYaVbyysg=; b=Dn98F+5+sLC0iieHcCW9ITzxg+pu1Jg3woWjF8CxauCFb/5yjKyOB0cS4URHy3jNq9 fqOOvpkv7Wi/PaeGFjCyQ91vNimwfhGK5TkPxakHgxC4xEWPE2PR05Q7X06/u1rIvxZz yzETIP3s13Zbwp25+L1hgxFMxJbVlD2r9Y0Ymrk99YqKar9mQ2nwKXTxpU+GIBrS7aST xb2PkdDJpKJQkhllj2InrNXrWFCJhMbz69klqGeJnQ2s8ralmw7trilT477H9Nl10NaE JK7PbiUSGI8l4pA9Pp3R61tLS94o27iWSRWzyU36cZaf3E8jAxKYQdCtPjk/kK1HzuHh xyww== X-Gm-Message-State: AOAM533RsvNwkdPGjbWM4OHqKda5yPaN1RXSuSgI5ZS9wCgCo6ExLoEy HdiRGsgdFEutnIaC7tJbYR/+GaGxZpB6mWTf X-Google-Smtp-Source: ABdhPJyDHPgPUFF8HO24yXsm5O+2et3SJH7AaZyv5j2kp7V8iKd5OFxvy1aK9EXyS3oSAdQ0tMhWJQ== X-Received: by 2002:a17:906:ca54:: with SMTP id jx20mr1775790ejb.541.1603795055225; Tue, 27 Oct 2020 03:37:35 -0700 (PDT) Received: from kozik-lap ([194.230.155.184]) by smtp.googlemail.com with ESMTPSA id pw17sm762269ejb.116.2020.10.27.03.37.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Oct 2020 03:37:34 -0700 (PDT) Date: Tue, 27 Oct 2020 11:37:32 +0100 From: Krzysztof Kozlowski To: Marek Szyprowski Subject: Re: [PATCH 08/12] ARM: dts: exynos: adjust node names to DT spec in Exynos542x boards Message-ID: <20201027103732.GA20429@kozik-lap> References: <20201026181528.163143-1-krzk@kernel.org> <20201026181528.163143-9-krzk@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201027_063738_734918_56C2EC09 X-CRM114-Status: GOOD ( 21.23 ) 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 , 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 Mon, Oct 26, 2020 at 10:47:47PM +0100, Marek Szyprowski wrote: > Hi Krzysztof, > > On 26.10.2020 19:15, Krzysztof Kozlowski wrote: > > The Devicetree specification expects device node names to have a generic > > name, representing the class of a device. Also the convention for node > > names is to use hyphens, not underscores. > > > > No functional changes. > > > > Signed-off-by: Krzysztof Kozlowski > > --- > > arch/arm/boot/dts/exynos5420-arndale-octa.dts | 4 ++-- > > arch/arm/boot/dts/exynos5420-peach-pit.dts | 4 ++-- > > arch/arm/boot/dts/exynos5420-smdk5420.dts | 2 +- > > arch/arm/boot/dts/exynos5422-odroid-core.dtsi | 2 +- > > arch/arm/boot/dts/exynos5422-odroidxu3-audio.dtsi | 2 +- > > arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 4 ++-- > > arch/arm/boot/dts/exynos5422-odroidxu3.dts | 8 ++++---- > > arch/arm/boot/dts/exynos5800-peach-pi.dts | 4 ++-- > > 8 files changed, 15 insertions(+), 15 deletions(-) > > > > diff --git a/arch/arm/boot/dts/exynos5420-arndale-octa.dts b/arch/arm/boot/dts/exynos5420-arndale-octa.dts > > index dd7f8385d81e..bf457d0c02eb 100644 > > --- a/arch/arm/boot/dts/exynos5420-arndale-octa.dts > > +++ b/arch/arm/boot/dts/exynos5420-arndale-octa.dts > > @@ -39,7 +39,7 @@ oscclk { > > }; > > }; > > > > - gpio_keys { > > + gpio-keys { > > compatible = "gpio-keys"; > > > > wakeup { > > @@ -344,7 +344,7 @@ &hdmi { > > &hsi2c_4 { > > status = "okay"; > > > > - s2mps11_pmic@66 { > > + pmic@66 { > > compatible = "samsung,s2mps11-pmic"; > > reg = <0x66>; > > > > diff --git a/arch/arm/boot/dts/exynos5420-peach-pit.dts b/arch/arm/boot/dts/exynos5420-peach-pit.dts > > index 2bcbdf8a39bf..d54392fe6260 100644 > > --- a/arch/arm/boot/dts/exynos5420-peach-pit.dts > > +++ b/arch/arm/boot/dts/exynos5420-peach-pit.dts > > @@ -138,7 +138,7 @@ panel_in: endpoint { > > }; > > }; > > > > - mmc1_pwrseq: mmc1_pwrseq { > > + mmc1_pwrseq: mmc1-pwrseq { > > compatible = "mmc-pwrseq-simple"; > > reset-gpios = <&gpx0 0 GPIO_ACTIVE_LOW>; /* WIFI_EN */ > > clocks = <&max77802 MAX77802_CLK_32K_CP>; > > @@ -205,7 +205,7 @@ &hsi2c_4 { > > status = "okay"; > > clock-frequency = <400000>; > > > > - max77802: max77802-pmic@9 { > > + max77802: pmic@9 { > > compatible = "maxim,max77802"; > > interrupt-parent = <&gpx3>; > > interrupts = <1 IRQ_TYPE_NONE>; > > diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts > > index 4e49d8095b29..d506da9fa661 100644 > > --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts > > +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts > > @@ -129,7 +129,7 @@ &hdmi { > > &hsi2c_4 { > > status = "okay"; > > > > - s2mps11_pmic@66 { > > + pmic@66 { > > compatible = "samsung,s2mps11-pmic"; > > reg = <0x66>; > > > > diff --git a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi > > index b1cf9414ce17..25fb6331c75e 100644 > > --- a/arch/arm/boot/dts/exynos5422-odroid-core.dtsi > > +++ b/arch/arm/boot/dts/exynos5422-odroid-core.dtsi > > @@ -503,7 +503,7 @@ &dmc { > > &hsi2c_4 { > > status = "okay"; > > > > - s2mps11_pmic@66 { > > + pmic@66 { > > compatible = "samsung,s2mps11-pmic"; > > reg = <0x66>; > > samsung,s2mps11-acokb-ground; > > diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3-audio.dtsi b/arch/arm/boot/dts/exynos5422-odroidxu3-audio.dtsi > > index b5ec4f47eb3a..f5f9c077df74 100644 > > --- a/arch/arm/boot/dts/exynos5422-odroidxu3-audio.dtsi > > +++ b/arch/arm/boot/dts/exynos5422-odroidxu3-audio.dtsi > > @@ -40,7 +40,7 @@ codec { > > > > &hsi2c_5 { > > status = "okay"; > > - max98090: max98090@10 { > > + max98090: codec@10 { > > compatible = "maxim,max98090"; > > reg = <0x10>; > > interrupt-parent = <&gpx3>; > > diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > > index 5da2d81e3be2..e35af40a55cb 100644 > > --- a/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > > +++ b/arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi > > @@ -13,12 +13,12 @@ > > #include "exynos5422-odroid-core.dtsi" > > > > / { > > - gpio_keys { > > + gpio-keys { > > compatible = "gpio-keys"; > > pinctrl-names = "default"; > > pinctrl-0 = <&power_key>; > > > > - power_key { > > + power-key { > > /* > > * The power button (SW2) is connected to the PWRON > > * pin (active high) of the S2MPS11 PMIC, which acts > > diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3.dts b/arch/arm/boot/dts/exynos5422-odroidxu3.dts > > index d0f6ac5fa79d..5ff493ecb6ff 100644 > > --- a/arch/arm/boot/dts/exynos5422-odroidxu3.dts > > +++ b/arch/arm/boot/dts/exynos5422-odroidxu3.dts > > @@ -21,28 +21,28 @@ &i2c_0 { > > status = "okay"; > > > > /* A15 cluster: VDD_ARM */ > > - ina231@40 { > > + sensor@40 { > power-sensor? Good point, thanks! Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel