qemu-arm
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] hw/usb: Make "hcd-ehci.h" header public


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH] hw/usb: Make "hcd-ehci.h" header public
Date: Mon, 4 May 2020 12:21:21 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0

On 5/4/20 12:12 PM, Philippe Mathieu-Daudé wrote:
On 5/4/20 10:48 AM, BALATON Zoltan wrote:
On Mon, 4 May 2020, Philippe Mathieu-Daudé wrote:
As target-specific code use this header, move it to the publicly
accessible include/ folder.

 $ git grep hw/usb/hcd-ehci.h
 hw/arm/allwinner-h3.c:31:#include "hw/usb/hcd-ehci.h"
 hw/arm/exynos4210.c:38:#include "hw/usb/hcd-ehci.h"
 hw/ppc/sam460ex.c:38:#include "hw/usb/hcd-ehci.h"
 include/hw/arm/allwinner-a10.h:13:#include "hw/usb/hcd-ehci.h"
 include/hw/arm/aspeed_soc.h:29:#include "hw/usb/hcd-ehci.h"

All of these only need either the type #define or EHCISysBusState so splitting only those off to a public header should be enough and better than making public all of ehci's internal header.

Ah you mean forward-declare EHCISysBusState in "qemu/typedefs.h", OK.

It won't work because AspeedSoCState and AwA10State use the full structure (not a pointer to it).


Regards,
BALATON Zoltan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]