Re: [PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers
On 19/1/24 17:51, Peter Maydell wrote: On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé wrote: When a chipset contain a USB controller, we can not simply remove it. We could disable it, but that requires more changes this series isn't aiming for. For more context: https://lore.kernel.org/qemu-devel/56fde49f-7dc6-4f8e-9bbf-0336a20a9...@roeck-us.net/ Philippe Mathieu-Daudé (2): hw/arm/allwinner-a10: Unconditionally map the USB Host controllers hw/arm/nseries: Unconditionally map the TUSB6010 USB Host controller Applied to target-arm.next, thanks. Thanks! I posted a v2 with the requested changes (mention migration compat break): https://lore.kernel.org/qemu-devel/20240119215106.45776-1-phi...@linaro.org/
Re: [PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers
On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé wrote: > > When a chipset contain a USB controller, we can not simply > remove it. We could disable it, but that requires more changes > this series isn't aiming for. For more context: > https://lore.kernel.org/qemu-devel/56fde49f-7dc6-4f8e-9bbf-0336a20a9...@roeck-us.net/ > > Philippe Mathieu-Daudé (2): > hw/arm/allwinner-a10: Unconditionally map the USB Host controllers > hw/arm/nseries: Unconditionally map the TUSB6010 USB Host controller > Applied to target-arm.next, thanks. -- PMM
Re: [PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers
On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé wrote: > > When a chipset contain a USB controller, we can not simply > remove it. We could disable it, but that requires more changes > this series isn't aiming for. For more context: > https://lore.kernel.org/qemu-devel/56fde49f-7dc6-4f8e-9bbf-0336a20a9...@roeck-us.net/ > > Philippe Mathieu-Daudé (2): > hw/arm/allwinner-a10: Unconditionally map the USB Host controllers > hw/arm/nseries: Unconditionally map the TUSB6010 USB Host controller Yes, I think this makes sense. It is technically a migration compat break for the affected machines if the user was passing the '-usb none' option, but I think we're OK with that; worth mentioning in the commit messages I guess. -- PMM
