On Wed, 5 Jul 2023, Daniel Henrique Barboza wrote:
Zoltan,
Patches 1-9 are queued. Don't need to re-send those.
Thanks, the last two patches are also reviewed and they don't depend on
the ones before so you could queue those too.
The only outstanding patches are those 3 that rename the type defines to
match their string values. We could come up with better names but those
suggested by Philippe are too long IMO so at least the patches in this
series clean up the current mess and we could rename these later. I'd
rather not change the string values too much as those are what QOM
actually uses to ideintify the types but we're free to change the defines.
Currently we have:
#define TYPE_PPC4xx_PCI_HOST_BRIDGE "ppc4xx-pcihost"
and then a "ppc4xx-host-bridge" type without a define which is another
type which is quite confusing. I may have partly created this mess back
when I first tried to add sam460ex and did not know much about this but at
least I'd like to improve it a little and resolve some of it now.
Regards,
BALATON Zoltan
