Thanks a lot for your help
________________________________
寄件者: Peter Maydell <peter.maydell@linaro.org>
寄件日期: 2026年8月12日 下午 06:43
收件者: Kuan-Jui Chiu <kchiu@axiado.com>
副本: qemu-devel@nongnu.org <qemu-devel@nongnu.org>; philmd@oss.qualcomm.com <philmd@oss.qualcomm.com>
主旨: Re: [PATCH v7 0/8] Add Axiado SoC AX3000 and EVK board

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.


On Mon, 10 Aug 2026 at 16:31, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Mon, 13 Jul 2026 at 08:31, Kuan-Jui Chiu <kchiu@axiado.com> wrote:
> >
> > This patchset introduces Axiado SoC AX3000 and a EVK SCM3003 for emulation
> > The model for Axiado SoC AX3000 supports
> >   4 Cortex-A53 CPUs
> >   Arm Generic Interrupt Controller v3
> >   4 Cadence UARTs
> >   1 SDHCI controller with PHY
> >   8 Cadence GPIOs
>
> Thanks; I've applied this to target-arm.next ready for when
> the tree reopens for 11.2.
>
> I made a couple of minor fixes to save you doing another respin:
>
> (1) you can't use %lx in logging printf format strings for printing
> a variable of type 'hwaddr' (it isn't a "long" on all hosts, and
> this won't compile on some systems) -- you need HWADDR_PRIx . I
> fixed some cases of this in patches 2 and 4.
>
> (2) we don't require the "<public>" and "<private>" marker comments
> any more, so I removed them from patch 7.

I also found that an issue with the Kconfig info caused a build
failure when any non-arm target is enabled in configure. This
fixes it (and makes the kconfig match how we do other board
and SoC kconfig stanzas in hw/arm/Kconfig):

diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig
index d0ae131628..e22c34a2e1 100644
--- a/hw/arm/Kconfig
+++ b/hw/arm/Kconfig
@@ -728,7 +728,6 @@ config ARMSSE

 config AXIADO_SOC
     bool
-    depends on ARM
     select ARM_GIC
     select CADENCE # UART
     select AXIADO_CLK
@@ -739,4 +738,5 @@ config AXIADO_SOC
 config AXIADO_EVK
     bool
     default y
+    depends on TCG && ARM
     select AXIADO_SOC

so I am squashing that in accordingly.

-- PMM