Commit 1a7ed5be authored by Matt Porter's avatar Matt Porter Committed by Felipe Balbi

usb: gadget: s3c-hsotg: fix build on x86 and other architectures

The readsl and writesl I/O accessors are only defined on some
architectures. The driver currently depends on CONFIG_ARM because
the build breaks on x86, in particular. Switch to use of ioread32_rep
and iowrite32_rep to fix build on all architectures and remove the
CONFIG_ARM dependency.

Also update printk formatting to handle a long long dma_addr_t to avoid
warnings on !32-bit architectures.
Signed-off-by: default avatarMatt Porter <mporter@linaro.org>
Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
parent 1af54b7a
...@@ -301,7 +301,6 @@ config USB_PXA27X ...@@ -301,7 +301,6 @@ config USB_PXA27X
gadget drivers to also be dynamically linked. gadget drivers to also be dynamically linked.
config USB_S3C_HSOTG config USB_S3C_HSOTG
depends on ARM
tristate "Designware/S3C HS/OtG USB Device controller" tristate "Designware/S3C HS/OtG USB Device controller"
help help
The Designware USB2.0 high-speed gadget controller The Designware USB2.0 high-speed gadget controller
......
...@@ -617,7 +617,7 @@ static int s3c_hsotg_write_fifo(struct s3c_hsotg *hsotg, ...@@ -617,7 +617,7 @@ static int s3c_hsotg_write_fifo(struct s3c_hsotg *hsotg,
to_write = DIV_ROUND_UP(to_write, 4); to_write = DIV_ROUND_UP(to_write, 4);
data = hs_req->req.buf + buf_pos; data = hs_req->req.buf + buf_pos;
writesl(hsotg->regs + EPFIFO(hs_ep->index), data, to_write); iowrite32_rep(hsotg->regs + EPFIFO(hs_ep->index), data, to_write);
return (to_write >= can_write) ? -ENOSPC : 0; return (to_write >= can_write) ? -ENOSPC : 0;
} }
...@@ -720,8 +720,8 @@ static void s3c_hsotg_start_req(struct s3c_hsotg *hsotg, ...@@ -720,8 +720,8 @@ static void s3c_hsotg_start_req(struct s3c_hsotg *hsotg,
ureq->length, ureq->actual); ureq->length, ureq->actual);
if (0) if (0)
dev_dbg(hsotg->dev, dev_dbg(hsotg->dev,
"REQ buf %p len %d dma 0x%08x noi=%d zp=%d snok=%d\n", "REQ buf %p len %d dma 0x%08llx noi=%d zp=%d snok=%d\n",
ureq->buf, length, ureq->dma, ureq->buf, length, (unsigned long long)ureq->dma,
ureq->no_interrupt, ureq->zero, ureq->short_not_ok); ureq->no_interrupt, ureq->zero, ureq->short_not_ok);
maxreq = get_ep_limit(hs_ep); maxreq = get_ep_limit(hs_ep);
...@@ -789,8 +789,8 @@ static void s3c_hsotg_start_req(struct s3c_hsotg *hsotg, ...@@ -789,8 +789,8 @@ static void s3c_hsotg_start_req(struct s3c_hsotg *hsotg,
dma_reg = dir_in ? DIEPDMA(index) : DOEPDMA(index); dma_reg = dir_in ? DIEPDMA(index) : DOEPDMA(index);
writel(ureq->dma, hsotg->regs + dma_reg); writel(ureq->dma, hsotg->regs + dma_reg);
dev_dbg(hsotg->dev, "%s: 0x%08x => 0x%08x\n", dev_dbg(hsotg->dev, "%s: 0x%08llx => 0x%08x\n",
__func__, ureq->dma, dma_reg); __func__, (unsigned long long)ureq->dma, dma_reg);
} }
ctrl |= DxEPCTL_EPEna; /* ensure ep enabled */ ctrl |= DxEPCTL_EPEna; /* ensure ep enabled */
...@@ -1488,7 +1488,7 @@ static void s3c_hsotg_rx_data(struct s3c_hsotg *hsotg, int ep_idx, int size) ...@@ -1488,7 +1488,7 @@ static void s3c_hsotg_rx_data(struct s3c_hsotg *hsotg, int ep_idx, int size)
* note, we might over-write the buffer end by 3 bytes depending on * note, we might over-write the buffer end by 3 bytes depending on
* alignment of the data. * alignment of the data.
*/ */
readsl(fifo, hs_req->req.buf + read_ptr, to_read); ioread32_rep(fifo, hs_req->req.buf + read_ptr, to_read);
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment