[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/15] hw/omap_gpio.c: Don't complain about some
From: |
andrzej zaborowski |
Subject: |
Re: [Qemu-devel] [PATCH 02/15] hw/omap_gpio.c: Don't complain about some writes to r/o registers |
Date: |
Sat, 30 Jul 2011 06:53:05 +0200 |
Hi,
I went ahead and pushed the series with the exception of this patch
and 14/15 because I think these are the types of patches that should
remain in downstream as a reminder, is there an argument for not
fixing these things in Linux?
In patch 04 I renamed omap2_gpio_module_set to omap2_gpio_set because
the parameter is no longer the module pointer. By the way I think we
should also pass the target agent pointer on creation the same way
clocks are passed and use omap_l4_attach.
In patch 07 I bumped the vmstate version because the structure seems
to have changed.
In patch 12 I removed the
else {
s->bdrv_cur = s->bdrv;
} part because there seemed to be no reason to add it, please check
that I haven't broken something.
Cheers
On 29 July 2011 17:35, Peter Maydell <address@hidden> wrote:
> Don't complain about some writes to r/o OMAP2 GPIO registers, because the
> kernel will do them anyway.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> hw/omap_gpio.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/omap_gpio.c b/hw/omap_gpio.c
> index 478f7d9..b53b13b 100644
> --- a/hw/omap_gpio.c
> +++ b/hw/omap_gpio.c
> @@ -385,7 +385,7 @@ static void omap2_gpio_module_write(void *opaque,
> target_phys_addr_t addr,
> case 0x00: /* GPIO_REVISION */
> case 0x14: /* GPIO_SYSSTATUS */
> case 0x38: /* GPIO_DATAIN */
> - OMAP_RO_REG(addr);
> + /* read-only, ignore quietly */
> break;
>
> case 0x10: /* GPIO_SYSCONFIG */
> @@ -531,7 +531,7 @@ static void omap2_gpio_module_writep(void *opaque,
> target_phys_addr_t addr,
> case 0x00: /* GPIO_REVISION */
> case 0x14: /* GPIO_SYSSTATUS */
> case 0x38: /* GPIO_DATAIN */
> - OMAP_RO_REG(addr);
> + /* read-only, ignore quietly */
> break;
>
> case 0x10: /* GPIO_SYSCONFIG */
> --
> 1.7.1
>
>
- [Qemu-devel] [PULL 00/15] omap patches (gpio, nand, onenand, lm832x), Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 10/15] hw/nand: Writing to NAND can only clear bits, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 06/15] hw/nand: Pass block device state to init function, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 14/15] onenand: Ignore zero writes to boot command space, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 01/15] hw/omap_l4.c: Add helper function omap_l4_region_base, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 08/15] hw/nand: Support devices wider than 8 bits, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 02/15] hw/omap_gpio.c: Don't complain about some writes to r/o registers, Peter Maydell, 2011/07/29
- Re: [Qemu-devel] [PATCH 02/15] hw/omap_gpio.c: Don't complain about some writes to r/o registers,
andrzej zaborowski <=
- [Qemu-devel] [PATCH 11/15] hw/nand: qdevify, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 15/15] hw/onenand: program actions can only clear bits, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 09/15] hw/nand: Support multiple reads following READ STATUS, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 13/15] onenand: Handle various ID fields separately, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 04/15] hw/omap_gpio.c: Convert to qdev, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 07/15] hw/nand: Support large NAND devices, Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 05/15] lm832x: Take DeviceState pointer in lm832x_key_event(), Peter Maydell, 2011/07/29
- [Qemu-devel] [PATCH 12/15] onenand: Pass BlockDriverState to init function, Peter Maydell, 2011/07/29