On 20/11/17 00:47, Philippe Mathieu-Daudé wrote:
> On 11/17/2017 10:42 AM, Mark Cave-Ayland wrote:
>> This belongs in the PCI-ISA bridge rather than at the machine level.
>
> nice, this helps me in another series (clean out i386/pc, refactor
> superio devices).
Awesome!
>> Signed-off-by: Mark Cave-Ayland <mark.cave-ayl...@ilande.co.uk>
>> ---
>> hw/sparc64/sun4u.c | 78
>> +++++++++++++++++++++++++++++++---------------------
>> 1 file changed, 46 insertions(+), 32 deletions(-)
>>
>> diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
>> index f3203ea..b441f1e 100644
>> --- a/hw/sparc64/sun4u.c
>> +++ b/hw/sparc64/sun4u.c
>> @@ -85,6 +85,7 @@ typedef struct EbusState {
>> PCIDevice parent_obj;
>>
>> ISABus *isa_bus;
>> + uint64_t console_serial_base;
>
> I'd rather use hwaddr here, anyway:
> Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>
Yeah it could potentially be either, however to me it feels nicer to
have the value as uint64_t so it "matches" qdev_prop_set_uint64().
ATB,
Mark.
