Re: [PATCH v2 5/5] arm64: dts: add support for A5 based Amlogic AV400

[Xianwei Zhao]

Hi Krzysztof,
    Thanks for your reply.

On 2024/3/20 17:48, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 20/03/2024 10:44, Xianwei Zhao via B4 Relay wrote:
> > From: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> >
> > Amlogic A5 is an application processor designed for smart audio
> > and IoT applications.
> >
> > Add basic support for the A5 based Amlogic AV400 board, which describes
> > the following components: CPU, GIC, IRQ, Timer and UART.
> > These are capable of booting up into the serial console.
> >
> > Signed-off-by: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/amlogic/Makefile               |  1 +
> >   .../boot/dts/amlogic/amlogic-a5-a113x2-av400.dts   | 42 ++++++++++++++++++++++
> >   arch/arm64/boot/dts/amlogic/amlogic-a5.dtsi        | 40 +++++++++++++++++++++
> >   3 files changed, 83 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile
> > index 9a50ec11bb8d..154c9efb26e4 100644
> > --- a/arch/arm64/boot/dts/amlogic/Makefile
> > +++ b/arch/arm64/boot/dts/amlogic/Makefile
> > @@ -1,5 +1,6 @@
> >   # SPDX-License-Identifier: GPL-2.0
> >   dtb-$(CONFIG_ARCH_MESON) += amlogic-a4-a113l2-ba400.dtb
> > +dtb-$(CONFIG_ARCH_MESON) += amlogic-a5-a113x2-av400.dtb
> >   dtb-$(CONFIG_ARCH_MESON) += amlogic-c3-c302x-aw409.dtb
> >   dtb-$(CONFIG_ARCH_MESON) += amlogic-t7-a311d2-an400.dtb
> >   dtb-$(CONFIG_ARCH_MESON) += amlogic-t7-a311d2-khadas-vim4.dtb
> > diff --git a/arch/arm64/boot/dts/amlogic/amlogic-a5-a113x2-av400.dts b/arch/arm64/boot/dts/amlogic/amlogic-a5-a113x2-av400.dts
> > new file mode 100644
> > index 000000000000..2f5f5ea74bc9
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/amlogic/amlogic-a5-a113x2-av400.dts
> > @@ -0,0 +1,42 @@
> > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> > +/*
> > + * Copyright (c) 2024 Amlogic, Inc. All rights reserved.
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "amlogic-a5.dtsi"
> > +
> > +/ {
> > +     model = "Amlogic A113X2 av400 Development Board";
> > +     compatible = "amlogic,av400","amlogic,a5";
>
> Please fix missing space in all your patches.
Will fix.
> Best regards,
> Krzysztof