Subject: Re: [PATCH v2 3/4] arm64: dts: cn913x: add device trees for topology B boards

From: Andrew Lunn <[email-masked]>
Date: Thu, 17 Dec 2020 16:41:02 +0100

> diff --git a/arch/arm64/boot/dts/marvell/Makefile b/arch/arm64/boot/dts/marvell/Makefile
> index 3e5f2e7a040c..d9b924a63d89 100644
> --- a/arch/arm64/boot/dts/marvell/Makefile
> +++ b/arch/arm64/boot/dts/marvell/Makefile
> @@ -13,6 +13,9 @@ dtb-$(CONFIG_ARCH_MVEBU) += armada-8040-db.dtb
>  dtb-$(CONFIG_ARCH_MVEBU) += armada-8040-mcbin.dtb
>  dtb-$(CONFIG_ARCH_MVEBU) += armada-8040-mcbin-singleshot.dtb
>  dtb-$(CONFIG_ARCH_MVEBU) += armada-8080-db.dtb
> -dtb-$(CONFIG_ARCH_MVEBU) += cn9130-db.dtb
> -dtb-$(CONFIG_ARCH_MVEBU) += cn9131-db.dtb
> -dtb-$(CONFIG_ARCH_MVEBU) += cn9132-db.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9130-db-A.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9130-db-B.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9131-db-A.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9131-db-B.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9132-db-A.dtb
> +dtb-$(CONFIG_ARCH_MVEBU) += cn9132-db-B.dtb

Are the filenames consider ABI? I wonder if cn9130-db A should
continue to be called cn9130-db.dtb? Are bootloaders and image
installers going to break because the filename changed? 

The rest of the changes look fine.

    Andrew

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel