[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board |
Date: |
Tue, 09 Dec 2014 00:20:28 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 |
On 12/08/14 22:34, Peter Maydell wrote:
> On 8 December 2014 at 21:19, Laszlo Ersek <address@hidden> wrote:
>> So the following in addition makes it work on TCG (x86_64) too:
>>
>> -----------------
>> diff --git a/hw/nvram/fw_cfg.c b/hw/nvram/fw_cfg.c
>> index 7147fea..c2bc44c 100644
>> --- a/hw/nvram/fw_cfg.c
>> +++ b/hw/nvram/fw_cfg.c
>> @@ -31,7 +31,7 @@
>> #include "qemu/config-file.h"
>>
>> #define FW_CFG_SIZE 2
>> -#define FW_CFG_DATA_SIZE 1
>> +#define FW_CFG_DATA_SIZE 8
>> #define TYPE_FW_CFG "fw_cfg"
>> #define FW_CFG_NAME "fw_cfg"
>> #define FW_CFG_PATH "/machine/" FW_CFG_NAME
>> -----------------
>>
>> It affects the memory_region_init_io() call in fw_cfg_initfn().
>>
>> I hope to submit a small v3 series soon.
>
> If you do that don't you now try to define an ioport on
> x86 that's 8 bytes wide? You probably also need to check
> whether the ppc and sparc boards that use mmio fw_cfg
> can handle the wider data register.
The above was just a quick PoC. The series I'm about to post takes care
not to change anything (not even gdb experience) for clients that don't
request the wider register.
Thanks
Laszlo
Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Christopher Covington, 2014/12/08
Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Gerd Hoffmann, 2014/12/09
- Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Laszlo Ersek, 2014/12/09
- Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Peter Maydell, 2014/12/09
- Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Richard W.M. Jones, 2014/12/09
- Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Paolo Bonzini, 2014/12/10
- Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Andrew Jones, 2014/12/10
Re: [Qemu-devel] [PATCH v2] arm: add fw_cfg to "virt" board, Laszlo Ersek, 2014/12/08