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=-14.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,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 305F8C43461 for ; Wed, 16 Sep 2020 01:48:00 +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 AEE0E2076C for ; Wed, 16 Sep 2020 01:47:59 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="hhCfHeQS" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AEE0E2076C Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=hisilicon.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:List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe :List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From:References:To:Subject: Reply-To:Cc:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=9ZoU0wKlnNbs2XaktxsB4brkQexRPil2i2R4zlfPGJ0=; b=hhCfHeQSI/f2+NHjI22+DwhIxE 9s+aamXQV33GyNAqwABzHPXjCdYPRYb/VH9ojbFLXc9kBsjb2SB/QRqd6j6RxjFt5qnqCYARFD5GO ZCLdEUIn8TlACCAqpUE+RYqZIknasO018d2m55V9Fs60zoLtzuPqTqsL4/ZnYRugh+fN/LFlz86tk hRJQ2djIcwXsQahCluzmftnz1RvBh51+dlEK9SqKdVXjnC36A+wMSDu552bqL2HQaN+LMmCknJ+Kq lkc/isYZNenkNJCkotw87SSHHO1PhE2uRc0pzFxmhJHyo/tN8LUFKjMYwIRp7vh7yWFFG5wLg+jOj e5n+eNlA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kIMWu-0006DW-7f; Wed, 16 Sep 2020 01:46:40 +0000 Received: from szxga04-in.huawei.com ([45.249.212.190] helo=huawei.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kIMWp-0006Ba-P0 for linux-arm-kernel@lists.infradead.org; Wed, 16 Sep 2020 01:46:37 +0000 Received: from DGGEMS407-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 9BB58704D1F6EFDE3521; Wed, 16 Sep 2020 09:46:30 +0800 (CST) Received: from [10.57.101.250] (10.57.101.250) by DGGEMS407-HUB.china.huawei.com (10.3.19.207) with Microsoft SMTP Server id 14.3.487.0; Wed, 16 Sep 2020 09:46:27 +0800 Subject: Re: [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay" To: Adrian Schmutzler , Rob Herring , , References: <20200830200551.1474-1-freifunk@adrianschmutzler.de> From: Wei Xu Message-ID: <5F616E73.4080707@hisilicon.com> Date: Wed, 16 Sep 2020 09:46:27 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20200830200551.1474-1-freifunk@adrianschmutzler.de> X-Originating-IP: [10.57.101.250] X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200915_214636_622135_7110430E X-CRM114-Status: GOOD ( 14.30 ) 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: , 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 Hi Adrian, On 2020/8/31 4:05, Adrian Schmutzler wrote: > While the DT parser recognizes "ok" as a valid value for the > "status" property, it is actually mentioned nowhere. Use the > proper value "okay" instead, as done in the majority of files > already. > > Signed-off-by: Adrian Schmutzler Thanks! Applied to the hisilicon arm64 dt tree. Best Regards, Wei > --- > .../boot/dts/hisilicon/hi3660-hikey960.dts | 6 +++--- > .../boot/dts/hisilicon/hi3670-hikey970.dts | 2 +- > .../arm64/boot/dts/hisilicon/hi6220-hikey.dts | 18 ++++++++--------- > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 2 +- > arch/arm64/boot/dts/hisilicon/hip05-d02.dts | 6 +++--- > arch/arm64/boot/dts/hisilicon/hip06-d03.dts | 18 ++++++++--------- > arch/arm64/boot/dts/hisilicon/hip07-d05.dts | 20 +++++++++---------- > 7 files changed, 36 insertions(+), 36 deletions(-) > > diff --git a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > index c1b614dabb8e..963300eede17 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > +++ b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > @@ -530,7 +530,7 @@ > rt1711h: rt1711h@4e { > compatible = "richtek,rt1711h"; > reg = <0x4e>; > - status = "ok"; > + status = "okay"; > interrupt-parent = <&gpio27>; > interrupts = <3 IRQ_TYPE_LEVEL_LOW>; > pinctrl-names = "default"; > @@ -570,7 +570,7 @@ > }; > > adv7533: adv7533@39 { > - status = "ok"; > + status = "okay"; > compatible = "adi,adv7533"; > reg = <0x39>; > adi,dsi-lanes = <4>; > @@ -656,7 +656,7 @@ > &sdio_cfg_func>; > /* WL_EN */ > vmmc-supply = <&wlan_en>; > - status = "ok"; > + status = "okay"; > > wlcore: wlcore@2 { > compatible = "ti,wl1837"; > diff --git a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > index 7dac33d4fd5c..7f9f9886c349 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > +++ b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts > @@ -418,7 +418,7 @@ > &sdio_cfg_func>; > /* WL_EN */ > vmmc-supply = <&wlan_en>; > - status = "ok"; > + status = "okay"; > > wlcore: wlcore@2 { > compatible = "ti,wl1837"; > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > index 533ed523888d..91d08673c02e 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts > @@ -267,7 +267,7 @@ > &uart1 { > assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>; > assigned-clock-rates = <150000000>; > - status = "ok"; > + status = "okay"; > > bluetooth { > compatible = "ti,wl1835-st"; > @@ -278,21 +278,21 @@ > }; > > &uart2 { > - status = "ok"; > + status = "okay"; > label = "LS-UART0"; > }; > > &uart3 { > - status = "ok"; > + status = "okay"; > label = "LS-UART1"; > }; > > &ade { > - status = "ok"; > + status = "okay"; > }; > > &dsi { > - status = "ok"; > + status = "okay"; > > ports { > /* 1 for output port */ > @@ -489,17 +489,17 @@ > > > &i2c0 { > - status = "ok"; > + status = "okay"; > }; > > &i2c1 { > - status = "ok"; > + status = "okay"; > }; > > &i2c2 { > #address-cells = <1>; > #size-cells = <0>; > - status = "ok"; > + status = "okay"; > > adv7533: adv7533@39 { > compatible = "adi,adv7533"; > @@ -541,5 +541,5 @@ > }; > > &spi0 { > - status = "ok"; > + status = "okay"; > }; > diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > index 3d189d9f0d24..3bab4bc6ff99 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi > @@ -371,7 +371,7 @@ > clocks = <&sys_ctrl HI6220_EDMAC_ACLK>; > dma-no-cci; > dma-type = "hi6220_dma"; > - status = "ok"; > + status = "okay"; > }; > > dual_timer0: timer@f8008000 { > diff --git a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts > index e93c65ede06c..369b69b17b91 100644 > --- a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts > +++ b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts > @@ -42,15 +42,15 @@ > }; > > &uart0 { > - status = "ok"; > + status = "okay"; > }; > > &peri_gpio0 { > - status = "ok"; > + status = "okay"; > }; > > &lbc { > - status = "ok"; > + status = "okay"; > #address-cells = <2>; > #size-cells = <1>; > ranges = <0 0 0x0 0x90000000 0x08000000>, > diff --git a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts > index 677862beebef..9f4a930e734d 100644 > --- a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts > +++ b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts > @@ -22,37 +22,37 @@ > }; > > &ipmi0 { > - status = "ok"; > + status = "okay"; > }; > > &uart0 { > - status = "ok"; > + status = "okay"; > }; > > ð0 { > - status = "ok"; > + status = "okay"; > }; > > ð1 { > - status = "ok"; > + status = "okay"; > }; > > ð2 { > - status = "ok"; > + status = "okay"; > }; > > ð3 { > - status = "ok"; > + status = "okay"; > }; > > &sas1 { > - status = "ok"; > + status = "okay"; > }; > > &usb_ohci { > - status = "ok"; > + status = "okay"; > }; > > &usb_ehci { > - status = "ok"; > + status = "okay"; > }; > diff --git a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts > index fcbdffe0868b..81a2312c8a26 100644 > --- a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts > +++ b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts > @@ -50,41 +50,41 @@ > }; > > &uart0 { > - status = "ok"; > + status = "okay"; > }; > > &ipmi0 { > - status = "ok"; > + status = "okay"; > }; > > &usb_ohci { > - status = "ok"; > + status = "okay"; > }; > > &usb_ehci { > - status = "ok"; > + status = "okay"; > }; > > ð0 { > - status = "ok"; > + status = "okay"; > }; > > ð1 { > - status = "ok"; > + status = "okay"; > }; > > ð2 { > - status = "ok"; > + status = "okay"; > }; > > ð3 { > - status = "ok"; > + status = "okay"; > }; > > &sas1 { > - status = "ok"; > + status = "okay"; > }; > > &p0_pcie2_a { > - status = "ok"; > + status = "okay"; > }; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel