On Fri, May 04, 2018 at 08:52:48AM +0200, Lothar Waßmann wrote: > Hi, > > On Fri, 4 May 2018 09:58:48 +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=1 switch is on. > > > > Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-' > > > > Cc: Lothar Waßmann <LW@xxxxxxxxxxxxxxxxxxx> > > Cc: Gary Bisson <gary.bisson@xxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Shawn Guo <shawnguo@xxxxxxxxxx> > > --- > > 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. > > > This breaks our U-Boot which searches for the alias names with '_'. > But I think it's not worth keeping the old names alive, since the > problem can easily be fixed by adding the old alias name to the FDT > on affected boards from within U-Boot, if necessary. > > Alternatively we could add a: > reg_can_xcvr = "reg-can-xcvr"; > [...] > so that the respective aliases can be resolved using both names for a > while. Appreciate it. > > So: > Acked-By: Lothar Waßmann <LW@xxxxxxxxxxxxxxxxxxx> Thanks, Lothar. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
