[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-4.1] hw/arm/nseries: Deprecate the n800 and
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH for-4.1] hw/arm/nseries: Deprecate the n800 and n810 machines |
Date: |
Fri, 12 Apr 2019 20:25:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 12/04/2019 18.21, Philippe Mathieu-Daudé wrote:
> On 4/12/19 5:26 PM, Thomas Huth wrote:
>> These machines depend on the deprecated bluetooth subsystem. In case we
>> finally decide to remove the bluetooth subsystem, we likely need to
>> remove the n800 and n810 machines, too, so mark them now as deprecated.
>>
>> Signed-off-by: Thomas Huth <address@hidden>
>> ---
>> hw/arm/nseries.c | 3 +++
>> qemu-deprecated.texi | 6 ++++++
>> 2 files changed, 9 insertions(+)
>>
>> diff --git a/hw/arm/nseries.c b/hw/arm/nseries.c
>> index 906b7ca22d..66e50b850c 100644
>> --- a/hw/arm/nseries.c
>> +++ b/hw/arm/nseries.c
>> @@ -1309,6 +1309,9 @@ static void n8x0_init(MachineState *machine,
>> struct n800_s *s = (struct n800_s *) g_malloc0(sizeof(*s));
>> int sdram_size = binfo->ram_size;
>>
>> + warn_report("The n800 and n810 machines are deprecated. Please use a "
>> + "different Arm machine instead.");
>> +
>
> Unrelated to this patch, but can you add a similar warning in
> uart_hci_init() to make obvious that the bluetooth subsystem is now
> deprecated?
It's already there, in vl.c, case QEMU_OPTION_bt.
Thomas