On Tue, Mar 27, 2012 at 01:42:23PM +1100, David Gibson wrote: > Currently dma_bdrv_io() takes a 'to_dev' boolean parameter to > determine the direction of DMA it is emulating. We already have a > DMADirection enum designed specifically to encode DMA directions. > This patch uses it for dma_bdrv_io() as well. This involves removing > the DMADirection definition from the #ifdef it was inside, but since that > only existed to protect the definition of dma_addr_t from places where > config.h is not included, there wasn't any reason for it to be there in > the first place. > > Cc: Kevin Wolf <kw...@redhat.com> > > Signed-off-by: David Gibson <da...@gibson.dropbear.id.au> > --- > dma-helpers.c | 20 ++++++++++++-------- > dma.h | 12 ++++++------ > hw/ide/core.c | 3 ++- > hw/ide/macio.c | 3 ++- > 4 files changed, 22 insertions(+), 16 deletions(-)
Reviewed-by: Stefan Hajnoczi <stefa...@linux.vnet.ibm.com>