On Fri, Nov 26, 2010 at 7:17 PM, Alexander Graf <ag...@suse.de> 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 <stefa...@linux.vnet.ibm.com>