From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@free-electrons.com (Gregory CLEMENT) Date: Fri, 12 Jan 2018 11:10:03 +0100 Subject: [PATCH] arm64: dts: marvell: armada-80x0: Fix pinctrl compatible string In-Reply-To: <20180112110612.223da2d9@windsurf.lan> (Thomas Petazzoni's message of "Fri, 12 Jan 2018 11:06:12 +0100") References: <20180112100002.3967-1-gregory.clement@free-electrons.com> <20180112110612.223da2d9@windsurf.lan> Message-ID: <871siv75hg.fsf@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Thomas, On ven., janv. 12 2018, Thomas Petazzoni wrote: > Hello, > > On Fri, 12 Jan 2018 11:00:02 +0100, Gregory CLEMENT wrote: >> When replacing the cpm by cp0 and cps by cp1 [1] not only the label and >> the alias were replaced but also the compatible string which was wrong. >> >> Due to this the pinctrl driver was no more probed. >> >> This patch fix it by reverting this change for the pinctrl compatible >> string on Armada 8K. >> >> [1]: "arm64: dts: marvell: replace cpm by cp0, cps by cp1" > > It is normally recommended to give the commit ID, i.e: > > In commit 01b451ed3bd2 ("arm64: dts: marvell: replace cpm by cp0, cps > by cp1"), ... But this commit in not yet in Linus tree, the ID you point is in linux-next, so once this patch will be merged the commit ID would become meaningless. Gregory > >> >> Signed-off-by: Gregory CLEMENT > > Reviewed-by: Thomas Petazzoni > > Thanks for fixing this mistake! > > Thomas > -- > Thomas Petazzoni, CTO, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com