smc37c669: remove useless is_enabled functions
Calls to is_enabled are bounded to indices that actually exist in the SuperIO device. Therefore, the is_enabled functions in smc37c669 are not doing anything and they can be removed. Reviewed-by: Bernhard Beschow <shentey@gmail.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
41514c0a77
commit
2fdc20f9d4
@ -14,11 +14,6 @@
|
||||
|
||||
/* UARTs (compatible with NS16450 or PC16550) */
|
||||
|
||||
static bool is_serial_enabled(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return index < 2;
|
||||
}
|
||||
|
||||
static uint16_t get_serial_iobase(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return index ? 0x2f8 : 0x3f8;
|
||||
@ -31,11 +26,6 @@ static unsigned int get_serial_irq(ISASuperIODevice *sio, uint8_t index)
|
||||
|
||||
/* Parallel port */
|
||||
|
||||
static bool is_parallel_enabled(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return index < 1;
|
||||
}
|
||||
|
||||
static uint16_t get_parallel_iobase(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return 0x378;
|
||||
@ -53,11 +43,6 @@ static unsigned int get_parallel_dma(ISASuperIODevice *sio, uint8_t index)
|
||||
|
||||
/* Diskette controller (Software compatible with the Intel PC8477) */
|
||||
|
||||
static bool is_fdc_enabled(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return index < 1;
|
||||
}
|
||||
|
||||
static uint16_t get_fdc_iobase(ISASuperIODevice *sio, uint8_t index)
|
||||
{
|
||||
return 0x3f0;
|
||||
@ -79,20 +64,17 @@ static void smc37c669_class_init(ObjectClass *klass, void *data)
|
||||
|
||||
sc->parallel = (ISASuperIOFuncs){
|
||||
.count = 1,
|
||||
.is_enabled = is_parallel_enabled,
|
||||
.get_iobase = get_parallel_iobase,
|
||||
.get_irq = get_parallel_irq,
|
||||
.get_dma = get_parallel_dma,
|
||||
};
|
||||
sc->serial = (ISASuperIOFuncs){
|
||||
.count = 2,
|
||||
.is_enabled = is_serial_enabled,
|
||||
.get_iobase = get_serial_iobase,
|
||||
.get_irq = get_serial_irq,
|
||||
};
|
||||
sc->floppy = (ISASuperIOFuncs){
|
||||
.count = 1,
|
||||
.is_enabled = is_fdc_enabled,
|
||||
.get_iobase = get_fdc_iobase,
|
||||
.get_irq = get_fdc_irq,
|
||||
.get_dma = get_fdc_dma,
|
||||
|
Loading…
Reference in New Issue
Block a user