On Mon, 4 May 2020, Philippe Mathieu-Daudé wrote:
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).
I've meant splitting the internal header into a public one only containing
the EHCISysBusState struct definition and TYPE_* #defines (or those could
be replaced with string value where used as done at several other places
to reduce dependence on this header). These are the only parts which used
outside hcd-ehci. All other internal stuff would remain in internal
header where they belong.
AwA10State also uses OHCISysBusState, why doesn't that have the same
problem? Why no other models need this? could this be really a problem
with these two models?
Regards,
BALATON Zoltan
