From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gary Bisson Subject: Re: [RFC PATCH] ARM: dts: imx: replace underscore with hyphen in aliases name Date: Fri, 4 May 2018 09:40:25 +0200 Message-ID: <20180504074025.GA11186@g751.home> References: <1525399128-11587-1-git-send-email-shawnguo@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <1525399128-11587-1-git-send-email-shawnguo@kernel.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Shawn Guo Cc: devicetree@vger.kernel.org, Rob Herring , linux-imx@nxp.com, kernel@pengutronix.de, Fabio Estevam , linux-arm-kernel@lists.infradead.org, Lothar =?iso-8859-1?Q?Wa=DFmann?= List-Id: devicetree@vger.kernel.org Hi Shawn, On Fri, May 04, 2018 at 09:58:48AM +0800, Shawn Guo wrote: > 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=3D1 switch is on. > = > Warning (alias_paths): /aliases: aliases property name must include only= lowercase and '-' > = > Cc: Lothar Wa=DFmann > Cc: Gary Bisson > Signed-off-by: Shawn Guo > --- > 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. Yes that is fine. Reviewed-by: Gary Bisson Thanks, Gary