[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/15] hw/sparc64/niagara: Replace 'empty_slot'
From: |
Artyom Tarasenko |
Subject: |
Re: [Qemu-devel] [PATCH 10/15] hw/sparc64/niagara: Replace 'empty_slot' by 'unimplemented_device' |
Date: |
Tue, 2 Oct 2018 17:50:59 +0200 |
On Tue, Oct 2, 2018 at 3:24 PM Peter Maydell <address@hidden> wrote:
>
> On 1 October 2018 at 23:09, Philippe Mathieu-Daudé <address@hidden> wrote:
> > The TYPE_EMPTY_SLOT and TYPE_UNIMPLEMENTED_DEVICE are identical devices,
> > however the later use more recent APIs and is more widely used.
> >
> > Replace 'empty_slot' by 'unimplemented_device' to simplify devices code
> > maintenance.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
>
> > @@ -161,7 +161,7 @@ static void niagara_init(MachineState *machine)
> > serial_mm_init(sysmem, NIAGARA_UART_BASE, 0, NULL, 115200,
> > serial_hd(0), DEVICE_BIG_ENDIAN);
> > }
> > - empty_slot_init(NIAGARA_IOBBASE, NIAGARA_IOBSIZE);
> > + create_unimplemented_device("sun4v-iob", NIAGARA_IOBBASE,
> > NIAGARA_IOBSIZE);
> > sun4v_rtc_init(NIAGARA_RTC_BASE);
> > }
>
> Is this actually an unimplemented (missing) device, or are we
> implementing hardware-defined "no bus errors when this range is
> touched" behaviour ?
In this case it's really an unimplemented device. But in sun4m (patch
11/15) it's
"no bus errors when this range is touched" behaviour.
Artyom
--
Regards,
Artyom Tarasenko
SPARC and PPC PReP under qemu blog: http://tyom.blogspot.com/search/label/qemu
- Re: [Qemu-devel] [PATCH 02/15] hw/timer/sun4v-rtc: Convert from DPRINTF() macro to trace events, (continued)