On 2018-12-27 07:34, Yang Zhong wrote: > From: Paolo Bonzini <pbonz...@redhat.com> > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- [...] > diff --git a/hw/ide/Kconfig b/hw/ide/Kconfig > index 5ec449525f..091f3a81c9 100644 > --- a/hw/ide/Kconfig > +++ b/hw/ide/Kconfig > @@ -3,33 +3,44 @@ config IDE_CORE > > config IDE_QDEV > bool > + select IDE_CORE > > config IDE_PCI > bool > + select IDE_CORE > > config IDE_ISA > bool > + select IDE_QDEV > > config IDE_PIIX > bool > + select IDE_QDEV > > config IDE_CMD646 > bool > + select IDE_QDEV
PIIX and CMD646 seem to be derived from TYPE_PCI_IDE, so shouldn't these switches rather select IDE_PCI instead? (Or depend on IDE_PCI?) > config IDE_MACIO > bool > + select IDE_QDEV > > config IDE_MMIO > bool > + select IDE_QDEV > > config IDE_VIA > bool > + select IDE_QDEV dito, VIA is a PCI device, too. > config MICRODRIVE > bool > + select IDE_QDEV > > config AHCI > bool > + select IDE_QDEV > > config IDE_SII3112 > bool > + select IDE_QDEV > dito, SII3112 is a PCI device. Thomas