[PATCH u-boot, v2019.04-aspeed-openbmc v2] ARM: dts: Aspeed: Add Facebook Harma DTS

Andrew Jeffery andrew at codeconstruct.com.au
Thu Jun 6 12:41:48 AEST 2024


On Wed, 2024-06-05 at 21:54 +0800, Peter Yin wrote:
> Initial introduction of Facebook Harma
> equipped with Aspeed 2600 BMC SoC.
> 
> Signed-off-by: Peter Yin <peteryin.openbmc at gmail.com>
> ---
> Change log:
> v1 -> v2
>  - Revise common to harma
> 
> v1
>  - Create facebook common dts
> ---
>  arch/arm/dts/Makefile          |  1 +
>  arch/arm/dts/ast2600-harma.dts | 66 ++++++++++++++++++++++++++++++++++

The discussion I was having with Patrick was we could make this e.g.
ast2600-facebook.dts ...

>  2 files changed, 67 insertions(+)
>  create mode 100644 arch/arm/dts/ast2600-harma.dts
> 
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 6f4b4d8a17..cb96c0a673 100755
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -688,6 +688,7 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
>  	ast2600-evb.dtb \
>  	ast2600-fpga.dtb \
>  	ast2600-greatlakes.dtb \
> +	ast2600-harma.dtb \
>  	ast2600-intel.dtb \
>  	ast2600-intel.dtb \
>  	ast2600-ncsi.dtb \
> diff --git a/arch/arm/dts/ast2600-harma.dts b/arch/arm/dts/ast2600-harma.dts
> new file mode 100644
> index 0000000000..11d6d5d60c
> --- /dev/null
> +++ b/arch/arm/dts/ast2600-harma.dts
> @@ -0,0 +1,66 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +// Copyright (c) 2024 Meta Platforms Inc.
> +/dts-v1/;
> +
> +#include "ast2600-u-boot.dtsi"
> +
> +/ {
> +	model = "Facebook Harma BMC";
> +	compatible = "facebook,harma-bmc", "aspeed,ast2600";

... and then just list the compatibles for the concrete platforms here,
like IBM does for P10.

That way your patch for Minerva becomes a 1-line change here.

Can you please send a series doing as much?

Cheers,

Andrew


More information about the openbmc mailing list