Hi, On Thu, Jun 28, 2018 at 8:15 AM, Antoine Tenart <antoine.tenart@xxxxxxxxxxx> wrote: > New compatibles are now supported by the Inside Secure SafeXcel driver. > As they are more specific than the old ones, they should be used > whenever possible. This patch updates the Marvell cp110 device tree > accordingly. > > Signed-off-by: Antoine Tenart <antoine.tenart@xxxxxxxxxxx> > --- > arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > index 2bf083272a87..bb2914f90048 100644 > --- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi > @@ -427,7 +427,7 @@ > }; > > CP110_LABEL(crypto): crypto@800000 { > - compatible = "inside-secure,safexcel-eip197"; > + compatible = "inside-secure,safexcel-eip197b"; So the device is still compatible with the less-specific binding, right? If so, it should probably have both compatible properties in there, not just the more specific one. (Same for another patch in this series). -Olof