On 31/03/2015 13:19, Andreas Färber wrote:
> Am 30.03.2015 um 13:45 schrieb Paolo Bonzini:
>> ioport.c is already able to split a 16-bit access into two 8-bit
>> accesses to consecutive ports. Tested with Epic Pinball.
>>
>> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
>> ---
>> hw/audio/sb16.c | 7 -------
>> 1 file changed, 7 deletions(-)
>>
>> diff --git a/hw/audio/sb16.c b/hw/audio/sb16.c
>> index 444eb9e..b052de5 100644
>> --- a/hw/audio/sb16.c
>> +++ b/hw/audio/sb16.c
>> @@ -1121,12 +1121,6 @@ static IO_WRITE_PROTO (mixer_write_datab)
>> s->mixer_regs[s->mixer_nreg] = val;
>> }
>>
>> -static IO_WRITE_PROTO (mixer_write_indexw)
>> -{
>> - mixer_write_indexb (opaque, nport, val & 0xff);
>> - mixer_write_datab (opaque, nport, (val >> 8) & 0xff);
>> -}
>
> I assume you've verified the endianness handling, too?
Yes, it's little-endian in either case:
mrp->write(mrpio->portio_opaque, mrp->base + addr, data & 0xff);
mrp->write(mrpio->portio_opaque, mrp->base + addr + 1, data >> 8);
for the sb16, index is 0x224 and data is 0x225.
Paolo
