From patchwork Fri May 4 01:58:48 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Shawn Guo X-Patchwork-Id: 10379657 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id D7B8260353 for ; Fri, 4 May 2018 01:59:27 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id B53ED292E3 for ; Fri, 4 May 2018 01:59:27 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id A7168292E6; Fri, 4 May 2018 01:59:27 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id CD0EB292E3 for ; Fri, 4 May 2018 01:59:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:To :From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=zFpFp5p2PVQ2erb4BqvTEf60MhMgj+tHXBncIEeBfUY=; b=DWpwQr7HBmDPuf +WxOd01xNUFi4SZ9x8Q12XCon66n2O7HROAn5ko/McdbvC+IGXlmBQjvO3egZCsaAiwidPAC9WLDe 3aTeiZi14zzKi2UbL84O2/sflSMz1tXk6nrItX2wQYeUoXVHVQ85n7IC9++X47eAoeDus9QAm/m2g YSth0aKrTDIAplPQgwZt4IB1GUmvhsqpEFgaI5uXKYTFLh5iRGHg00XVL9oAE47vOposgAMLohbKK fMmap34abY+UR7hIht0jNhD/2D1K75NVkfw3aK/4cqLwrHFPfM5BTJFeoW1kyx9H/hS6ilyb+px6v OcOF/tPJA7taPuanpX2g==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fEQ0I-0002TM-AT; Fri, 04 May 2018 01:59:22 +0000 Received: from mail-pg0-f68.google.com ([74.125.83.68]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1fEQ0E-0002SM-PD for linux-arm-kernel@lists.infradead.org; Fri, 04 May 2018 01:59:20 +0000 Received: by mail-pg0-f68.google.com with SMTP id m21-v6so14405677pgv.8 for ; Thu, 03 May 2018 18:59:07 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=vyl1f41HqH2EbK0XFOqv/8yWskSmLyleqAzimiTFfGI=; b=hIte//ZdDlOxgkUn2RH2b4lh5ZsFteBcJAU/f6GLlo0eZWwMEUIdHJXcZeFvI6ZzIq 9XK5EcoFWYNzpe6j7LdeRvf3Q8rdnBx+D/keQ3Vt98qPOrA/sbNOBpFB1U0eLBTlda+a PuFNKneX0OK5u/0d9BjLVHnh8KKnN1TsJutdmMZghbokrgG8lfvW0QuZAjnpDiO/PaIQ KcGX0/nTMRLYtqGIXd828aeoFDBFBftVr/VxJP5S1Xl7TTvfXAXV8Pa0DvpPfJDwccTQ 8giYUghJlNiScFVg80QUV7vGeT6Jodd13PHEE29LfSGQsMEI8W1WlU+Ynpn8TwS4cIzJ B9Fg== X-Gm-Message-State: ALQs6tA2pciOCi6oq1T+kFdYkrPIifkLS4E/mm1vbda9uYIrpdwE23PR AHP7hHCDXSivcv0dpT95tNLF6g== X-Google-Smtp-Source: AB8JxZqykCjXC9pcAj2znBMYyozcP99kTLg/TsoGR0kpCChxMrY5BSminUI4PROU55tFXQNGfLOjOg== X-Received: by 10.98.182.16 with SMTP id j16mr24989908pff.17.1525399147456; Thu, 03 May 2018 18:59:07 -0700 (PDT) Received: from localhost.localdomain ([45.56.152.81]) by smtp.gmail.com with ESMTPSA id o10-v6sm27793370pgc.80.2018.05.03.18.59.02 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 03 May 2018 18:59:06 -0700 (PDT) From: Shawn Guo To: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: [RFC PATCH] ARM: dts: imx: replace underscore with hyphen in aliases name Date: Fri, 4 May 2018 09:58:48 +0800 Message-Id: <1525399128-11587-1-git-send-email-shawnguo@kernel.org> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20180503_185918_832333_CC0D5C75 X-CRM114-Status: GOOD ( 15.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Gary Bisson , Rob Herring , linux-imx@nxp.com, kernel@pengutronix.de, Fabio Estevam , Shawn Guo , =?UTF-8?q?Lothar=20Wa=C3=9Fmann?= Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP It replaces underscore with hyphen in aliases name to fix DTC alias_paths warning below, which is seen with various i.MX board dts files when W=1 switch is on. Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-' Cc: Lothar Waßmann Cc: Gary Bisson Signed-off-by: Shawn Guo Acked-By: Lothar Waßmann Reviewed-by: Gary Bisson --- Hi Lothar, Gary, Per my search on kernel code, this change will not break anything on kernel support. But I'm not really sure about the case of bootloaders running on the boards. That's why I'm sending it as RFC. We really want i.MX to be the first-class citizen in the community. After a few cycles efforts on cleaning up DTC warnings, this alias_paths warning is the last one we see with i.MX dts files. We really want to get rid of it and make i.MX dts be DTC warning free. So please comment whether we can merge this patch. Shawn arch/arm/boot/dts/imx53-tx53.dtsi | 2 +- arch/arm/boot/dts/imx6qdl-tx6.dtsi | 6 +++--- arch/arm/boot/dts/imx6sx-nitrogen6sx.dts | 4 ++-- arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dts | 2 +- arch/arm/boot/dts/imx6ul-tx6ul.dtsi | 6 +++--- arch/arm/boot/dts/imx7d-nitrogen7.dts | 4 ++-- 6 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/arm/boot/dts/imx53-tx53.dtsi b/arch/arm/boot/dts/imx53-tx53.dtsi index 5dd3dd3a2fd0..54cf3e67069a 100644 --- a/arch/arm/boot/dts/imx53-tx53.dtsi +++ b/arch/arm/boot/dts/imx53-tx53.dtsi @@ -58,7 +58,7 @@ can0 = &can2; /* Make the can interface indices consistent with TX28/TX48 modules */ can1 = &can1; ipu = &ipu; - reg_can_xcvr = ®_can_xcvr; + reg-can-xcvr = ®_can_xcvr; usbh1 = &usbh1; usbotg = &usbotg; }; diff --git a/arch/arm/boot/dts/imx6qdl-tx6.dtsi b/arch/arm/boot/dts/imx6qdl-tx6.dtsi index f015e2d1cf35..a98fb2564c63 100644 --- a/arch/arm/boot/dts/imx6qdl-tx6.dtsi +++ b/arch/arm/boot/dts/imx6qdl-tx6.dtsi @@ -50,11 +50,11 @@ can0 = &can2; can1 = &can1; ethernet0 = &fec; - lcdif_23bit_pins_a = &pinctrl_disp0_1; - lcdif_24bit_pins_a = &pinctrl_disp0_2; + lcdif-23bit-pins-a = &pinctrl_disp0_1; + lcdif-24bit-pins-a = &pinctrl_disp0_2; pwm0 = &pwm1; pwm1 = &pwm2; - reg_can_xcvr = ®_can_xcvr; + reg-can-xcvr = ®_can_xcvr; stk5led = &user_led; usbotg = &usbotg; sdhc0 = &usdhc1; diff --git a/arch/arm/boot/dts/imx6sx-nitrogen6sx.dts b/arch/arm/boot/dts/imx6sx-nitrogen6sx.dts index b58f770c40d9..59e52f504922 100644 --- a/arch/arm/boot/dts/imx6sx-nitrogen6sx.dts +++ b/arch/arm/boot/dts/imx6sx-nitrogen6sx.dts @@ -48,8 +48,8 @@ compatible = "boundary,imx6sx-nitrogen6sx", "fsl,imx6sx"; aliases { - fb_lcd = &lcdif1; - t_lcd = &t_lcd; + fb-lcd = &lcdif1; + t-lcd = &t_lcd; }; memory@80000000 { diff --git a/arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dts b/arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dts index 2d80f7b50bc0..97686097a86e 100644 --- a/arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dts +++ b/arch/arm/boot/dts/imx6ul-tx6ul-mainboard.dts @@ -48,7 +48,7 @@ compatible = "karo,imx6ul-tx6ul", "fsl,imx6ul"; aliases { - lcdif_24bit_pins_a = &pinctrl_disp0_3; + lcdif-24bit-pins-a = &pinctrl_disp0_3; mmc0 = &usdhc1; /delete-property/ mmc1; serial2 = &uart3; diff --git a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi index f678d18ad44a..02b5ba42cd59 100644 --- a/arch/arm/boot/dts/imx6ul-tx6ul.dtsi +++ b/arch/arm/boot/dts/imx6ul-tx6ul.dtsi @@ -53,10 +53,10 @@ i2c2 = &i2c1; i2c3 = &i2c3; i2c4 = &i2c4; - lcdif_23bit_pins_a = &pinctrl_disp0_1; - lcdif_24bit_pins_a = &pinctrl_disp0_2; + lcdif-23bit-pins-a = &pinctrl_disp0_1; + lcdif-24bit-pins-a = &pinctrl_disp0_2; pwm0 = &pwm5; - reg_can_xcvr = ®_can_xcvr; + reg-can-xcvr = ®_can_xcvr; serial2 = &uart5; serial4 = &uart3; spi0 = &ecspi2; diff --git a/arch/arm/boot/dts/imx7d-nitrogen7.dts b/arch/arm/boot/dts/imx7d-nitrogen7.dts index 52167298984d..b8e73b46cddb 100644 --- a/arch/arm/boot/dts/imx7d-nitrogen7.dts +++ b/arch/arm/boot/dts/imx7d-nitrogen7.dts @@ -49,8 +49,8 @@ compatible = "boundary,imx7d-nitrogen7", "fsl,imx7d"; aliases { - fb_lcd = &lcdif; - t_lcd = &t_lcd; + fb-lcd = &lcdif; + t-lcd = &t_lcd; }; memory@80000000 {