DMA: Refactor the DMA device and align timing and atomic
authorAndreas Hansson <andreas.hansson@arm.com>
Wed, 22 Aug 2012 15:40:01 +0000 (11:40 -0400)
committerAndreas Hansson <andreas.hansson@arm.com>
Wed, 22 Aug 2012 15:40:01 +0000 (11:40 -0400)
commit2c1052cd4d1bef2965e449ad08130acec9b2ce38
tree4d9e692fece2fbe513017995af2b6a18a1769a50
parentc60db56741631b03e3431d03c26c9114c27ba6c6
DMA: Refactor the DMA device and align timing and atomic

This patch does a bunch of house-keeping updates on the DMA, including
indentation, and formatting, but most importantly breaks out the
response handling such that it can be shared between the atomic and
timing modes. It also removes a potential bug caused by the atomic
handling of responses only deleting the allocated request (pkt->req)
once the DMA action completes instead of doing so for every packet.

Before this patch, the handling of responses was near identical for
atomic and timing, but the code was simply duplicated. With this
patch, the handleResp method deals with the responses in both cases.

There are further updates to make after removing the NACKs, but that
will be part of a separate follow-up patch. This patch does not change
the behaviour of any regression.
src/dev/dma_device.cc
src/dev/dma_device.hh