[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/7] hw/mips: Express dependencies of the r4k
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/7] hw/mips: Express dependencies of the r4k platform with Kconfig |
Date: |
Mon, 11 Mar 2019 12:19:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.1 |
On 3/11/19 7:58 AM, Thomas Huth wrote:
> On 11/03/2019 01.56, Philippe Mathieu-Daudé wrote:
>> This platform use standard PC devices connected to an ISA bus.
>> Networking is provided by a ne2000 chipset.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> v3: remove CONFIG_NE2000_ISA (Thomas)
>> add PFLASH_CFI01
>> ---
>> default-configs/mips-softmmu-common.mak | 1 -
>> hw/mips/Kconfig | 11 +++++++++++
>> 2 files changed, 11 insertions(+), 1 deletion(-)
>>
>> diff --git a/default-configs/mips-softmmu-common.mak
>> b/default-configs/mips-softmmu-common.mak
>> index caebafd416..2ca57bfee1 100644
>> --- a/default-configs/mips-softmmu-common.mak
>> +++ b/default-configs/mips-softmmu-common.mak
>> @@ -24,7 +24,6 @@ CONFIG_I8257=y
>> CONFIG_PIIX4=y
>> CONFIG_IDE_ISA=y
>> CONFIG_IDE_PIIX=y
>> -CONFIG_NE2000_ISA=y
>> CONFIG_PFLASH_CFI01=y
>> CONFIG_I8259=y
>> CONFIG_MC146818RTC=y
>> diff --git a/hw/mips/Kconfig b/hw/mips/Kconfig
>> index ab006477c7..5678c9cf15 100644
>> --- a/hw/mips/Kconfig
>> +++ b/hw/mips/Kconfig
>> @@ -1,5 +1,16 @@
>> config R4K
>> bool
>> + select ISA_BUS
>> + select SERIAL_ISA
>> + select I8259
>> + select I8254
>> + select MC146818RTC
>> + select VGA_ISA
>> + select NE2000_ISA
>
> Since Paolo now also introduced the "imply" keyword, I think you should
> use "imply" instead of "select" for both, VGA_ISA and NE2000_ISA, since
> the r4k machine can work without both when you specify -vga none and
> -net none.
OK I'll use "imply", thanks.
>
> Thomas
>
- [Qemu-devel] [PATCH v3 0/7] Kconfig dependencies for MIPS machines (but Malta), Philippe Mathieu-Daudé, 2019/03/10
- [Qemu-devel] [PATCH v3 3/7] hw/mips: Express dependencies of the r4k platform with Kconfig, Philippe Mathieu-Daudé, 2019/03/10
- [Qemu-devel] [PATCH v3 4/7] hw/mips: Remove the redundant CONFIG_MIPS_ITU Makefile variable, Philippe Mathieu-Daudé, 2019/03/10
- [Qemu-devel] [PATCH v3 5/7] hw/mips: Express dependencies of the Boston machine with Kconfig, Philippe Mathieu-Daudé, 2019/03/10
- [Qemu-devel] [PATCH v3 6/7] hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge, Philippe Mathieu-Daudé, 2019/03/10
- [Qemu-devel] [PATCH v3 7/7] hw/mips: Express dependencies of the Fulong 2E machine with Kconfig, Philippe Mathieu-Daudé, 2019/03/10