Skip to content

Commit

Permalink
dmaengine: shdma: Remove sh_dmae_slave_chan_id enum
Browse files Browse the repository at this point in the history
This patch replaces the sh_dmae_slave_chan_id enum
with an unsigned int. The purpose of this chainge is
to make it possible to separate the slave id enums
from the dmaengine header.

The slave id enums varies with processor model, so in
the future it makes sense to put these in the processor
specific headers together with the pinmux enums.

Signed-off-by: Magnus Damm <damm@opensource.se>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
  • Loading branch information
Magnus Damm authored and Paul Mundt committed Mar 23, 2010
1 parent ae6be51 commit 4bab9d4
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 15 deletions.
6 changes: 3 additions & 3 deletions arch/sh/include/asm/dmaengine.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

#define SH_DMAC_MAX_CHANNELS 6

enum sh_dmae_slave_chan_id {
enum {
SHDMA_SLAVE_SCIF0_TX,
SHDMA_SLAVE_SCIF0_RX,
SHDMA_SLAVE_SCIF1_TX,
Expand All @@ -38,7 +38,7 @@ enum sh_dmae_slave_chan_id {
};

struct sh_dmae_slave_config {
enum sh_dmae_slave_chan_id slave_id;
unsigned int slave_id;
dma_addr_t addr;
u32 chcr;
char mid_rid;
Expand Down Expand Up @@ -68,7 +68,7 @@ struct device;

/* Used by slave DMA clients to request DMA to/from a specific peripheral */
struct sh_dmae_slave {
enum sh_dmae_slave_chan_id slave_id; /* Set by the platform */
unsigned int slave_id; /* Set by the platform */
struct device *dma_dev; /* Set by the platform */
struct sh_dmae_slave_config *config; /* Set by the driver */
};
Expand Down
8 changes: 4 additions & 4 deletions arch/sh/include/asm/siu.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ struct device;

struct siu_platform {
struct device *dma_dev;
enum sh_dmae_slave_chan_id dma_slave_tx_a;
enum sh_dmae_slave_chan_id dma_slave_rx_a;
enum sh_dmae_slave_chan_id dma_slave_tx_b;
enum sh_dmae_slave_chan_id dma_slave_rx_b;
unsigned int dma_slave_tx_a;
unsigned int dma_slave_rx_a;
unsigned int dma_slave_tx_b;
unsigned int dma_slave_rx_b;
};

#endif /* ASM_SIU_H */
8 changes: 4 additions & 4 deletions drivers/dma/shdma.c
Original file line number Diff line number Diff line change
Expand Up @@ -266,19 +266,19 @@ static struct sh_desc *sh_dmae_get_desc(struct sh_dmae_chan *sh_chan)
}

static struct sh_dmae_slave_config *sh_dmae_find_slave(
struct sh_dmae_chan *sh_chan, enum sh_dmae_slave_chan_id slave_id)
struct sh_dmae_chan *sh_chan, struct sh_dmae_slave *param)
{
struct dma_device *dma_dev = sh_chan->common.device;
struct sh_dmae_device *shdev = container_of(dma_dev,
struct sh_dmae_device, common);
struct sh_dmae_pdata *pdata = shdev->pdata;
int i;

if ((unsigned)slave_id >= SHDMA_SLAVE_NUMBER)
if (param->slave_id >= SHDMA_SLAVE_NUMBER)
return NULL;

for (i = 0; i < pdata->slave_num; i++)
if (pdata->slave[i].slave_id == slave_id)
if (pdata->slave[i].slave_id == param->slave_id)
return pdata->slave + i;

return NULL;
Expand All @@ -299,7 +299,7 @@ static int sh_dmae_alloc_chan_resources(struct dma_chan *chan)
if (param) {
struct sh_dmae_slave_config *cfg;

cfg = sh_dmae_find_slave(sh_chan, param->slave_id);
cfg = sh_dmae_find_slave(sh_chan, param);
if (!cfg)
return -EINVAL;

Expand Down
4 changes: 2 additions & 2 deletions drivers/serial/sh-sci.c
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ struct sci_port {
struct dma_chan *chan_rx;
#ifdef CONFIG_SERIAL_SH_SCI_DMA
struct device *dma_dev;
enum sh_dmae_slave_chan_id slave_tx;
enum sh_dmae_slave_chan_id slave_rx;
unsigned int slave_tx;
unsigned int slave_rx;
struct dma_async_tx_descriptor *desc_tx;
struct dma_async_tx_descriptor *desc_rx[2];
dma_cookie_t cookie_tx;
Expand Down
4 changes: 2 additions & 2 deletions include/linux/serial_sci.h
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ struct plat_sci_port {
char *clk; /* clock string */
struct device *dma_dev;
#ifdef CONFIG_SERIAL_SH_SCI_DMA
enum sh_dmae_slave_chan_id dma_slave_tx;
enum sh_dmae_slave_chan_id dma_slave_rx;
unsigned int dma_slave_tx;
unsigned int dma_slave_rx;
#endif
};

Expand Down

0 comments on commit 4bab9d4

Please sign in to comment.