[v1] hw/arm: Unconditionally map MMIO-based USB host controllers

[PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers

Philippe Mathieu-Daudé posted 2 patches 10 months, 2 weeks ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240115165615.78323-1-philmd@linaro.org

Maintainers: Beniamino Galvani <b.galvani@gmail.com>, Peter Maydell <peter.maydell@linaro.org>, Strahinja Jankovic <strahinja.p.jankovic@gmail.com>

There is a newer version of this series

hw/arm/allwinner-a10.c | 49 +++++++++++++++++-------------------------
hw/arm/nseries.c       |  4 +---
2 files changed, 21 insertions(+), 32 deletions(-)

Expand all Fold all

[PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers

Posted by Philippe Mathieu-Daudé 10 months, 2 weeks ago

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-0336a20a9ebf@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

 hw/arm/allwinner-a10.c | 49 +++++++++++++++++-------------------------
 hw/arm/nseries.c       |  4 +---
 2 files changed, 21 insertions(+), 32 deletions(-)

-- 
2.41.0

Re: [PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers

Posted by Peter Maydell 10 months, 1 week ago

On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé <philmd@linaro.org> 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-0336a20a9ebf@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

Posted by Philippe Mathieu-Daudé 10 months, 1 week ago

On 19/1/24 17:51, Peter Maydell wrote:
> On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé <philmd@linaro.org> 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-0336a20a9ebf@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-philmd@linaro.org/

Re: [PATCH 0/2] hw/arm: Unconditionally map MMIO-based USB host controllers

Posted by Peter Maydell 10 months, 2 weeks ago

On Mon, 15 Jan 2024 at 16:56, Philippe Mathieu-Daudé <philmd@linaro.org> 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-0336a20a9ebf@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