On Wed, 30 Aug 2023, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau <marcandre.lur...@redhat.com>

SM501 is going to depend on PIXMAN next.

Why is this patch needed when SM501 is the one that depends on PIXMAN and should pull in the dependency? Also what's the change in default.mak? ati-vga also uses pixman and currently has no fall back. The sm501 already has fallback when pixman fails so could work without pixman too, see x-pixman property in sm501.c.

Regards,
BALATON Zoltan

Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
---
configs/devices/ppc-softmmu/default.mak | 2 +-
hw/ppc/Kconfig                          | 2 ++
2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/configs/devices/ppc-softmmu/default.mak 
b/configs/devices/ppc-softmmu/default.mak
index a887f5438b..603ed701ed 100644
--- a/configs/devices/ppc-softmmu/default.mak
+++ b/configs/devices/ppc-softmmu/default.mak
@@ -8,7 +8,7 @@ CONFIG_PPC440=y
CONFIG_VIRTEX=y

# For Sam460ex
-CONFIG_SAM460EX=y
+#CONFIG_SAM460EX=n

# For Macs
CONFIG_MAC_OLDWORLD=y
diff --git a/hw/ppc/Kconfig b/hw/ppc/Kconfig
index 5dfbf47ef5..86d3f5cfca 100644
--- a/hw/ppc/Kconfig
+++ b/hw/ppc/Kconfig
@@ -58,6 +58,8 @@ config PPC4XX

config SAM460EX
    bool
+    default y
+    depends on PPC && PIXMAN
    select PFLASH_CFI01
    select IDE_SII3112
    select M41T80

Reply via email to