From: Bernhard Beschow Date: Sat, 15 Mar 2025 10:49:35 +0000 (+0100) Subject: hw/ppc/e500: Use TYPE_FSL_ESDHC_BE X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=3b8ea2c193b3c07a46b94efe557e2a390a85e9de;p=thirdparty%2Fqemu.git hw/ppc/e500: Use TYPE_FSL_ESDHC_BE TYPE_FSL_ESDHC_BE maches real hardware more closely by reusing code of TYPE_IMX_USDHC. For example, it fixes Linux to flood the guest console with "mmc0: Internal clock never stabilised" messages in the QEMU advent calendar 2018 day 19 image. Reported-by: Thomas Huth Signed-off-by: Bernhard Beschow Reviewed-by: Philippe Mathieu-Daudé Tested-by: BALATON Zoltan Message-ID: <20260112145418.220506-10-shentey@gmail.com> Signed-off-by: Philippe Mathieu-Daudé --- diff --git a/hw/ppc/e500.c b/hw/ppc/e500.c index eb0d3a418e..d6ca2e8563 100644 --- a/hw/ppc/e500.c +++ b/hw/ppc/e500.c @@ -1036,15 +1036,7 @@ void ppce500_init(MachineState *machine) memory_region_add_subregion(ccsr_addr_space, MPC85XX_ESDHC_REGS_OFFSET, sysbus_mmio_get_region(s, 0)); - /* - * Compatible with: - * - SD Host Controller Specification Version 2.0 Part A2 - * (See MPC8569E Reference Manual) - */ - dev = qdev_new(TYPE_SYSBUS_SDHCI); - qdev_prop_set_uint8(dev, "sd-spec-version", 2); - qdev_prop_set_uint8(dev, "endianness", DEVICE_BIG_ENDIAN); - qdev_prop_set_uint8(dev, "vendor", SDHCI_VENDOR_FSL); + dev = qdev_new(TYPE_FSL_ESDHC_BE); s = SYS_BUS_DEVICE(dev); sysbus_realize_and_unref(s, &error_fatal); sysbus_connect_irq(s, 0, qdev_get_gpio_in(mpicdev, MPC85XX_ESDHC_IRQ));