[Qemu-devel] Re: [PATCH 01/13] ide: split ide command interpretation off

[Stefan Hajnoczi]

On Fri, Nov 26, 2010 at 7:17 PM, Alexander Graf <address@hidden> wrote:
> @@ -1853,17 +1850,29 @@ void ide_ioport_write(void *opaque, uint32_t addr, 
> uint32_t val)
>     default:
>     case 7:
>         /* command */
> +        ide_exec_cmd(opaque, val);

Slightly clearer but only worth changing if you need to respin the patch series:
ide_exec_cmd(bus, val);

Reviewed-by: Stefan Hajnoczi <address@hidden>