mtd/ps3vram: Use msleep in waits

Replace the use of udelay() with msleep() in the looping wait routines
ps3vram_notifier_wait() and ps3vram_wait_ring().

Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
Geoff Levand 2009-01-07 17:22:02 -08:00 committed by Benjamin Herrenschmidt
parent 993e62e674
commit 60c0c5987b

View file

@ -109,22 +109,19 @@ static void ps3vram_notifier_reset(struct mtd_info *mtd)
notify[i] = 0xffffffff; notify[i] = 0xffffffff;
} }
static int ps3vram_notifier_wait(struct mtd_info *mtd, int timeout_ms) static int ps3vram_notifier_wait(struct mtd_info *mtd, unsigned int timeout_ms)
{ {
struct ps3vram_priv *priv = mtd->priv; struct ps3vram_priv *priv = mtd->priv;
u32 *notify = ps3vram_get_notifier(priv->reports, NOTIFIER); u32 *notify = ps3vram_get_notifier(priv->reports, NOTIFIER);
unsigned long timeout = jiffies + msecs_to_jiffies(timeout_ms);
timeout_ms *= 1000;
do { do {
if (notify[3] == 0) if (!notify[3])
return 0; return 0;
msleep(1);
} while (time_before(jiffies, timeout));
if (timeout_ms) return -ETIMEDOUT;
udelay(1);
} while (timeout_ms--);
return -1;
} }
static void ps3vram_init_ring(struct mtd_info *mtd) static void ps3vram_init_ring(struct mtd_info *mtd)
@ -135,25 +132,22 @@ static void ps3vram_init_ring(struct mtd_info *mtd)
priv->ctrl[CTRL_GET] = FIFO_BASE + FIFO_OFFSET; priv->ctrl[CTRL_GET] = FIFO_BASE + FIFO_OFFSET;
} }
static int ps3vram_wait_ring(struct mtd_info *mtd, int timeout) static int ps3vram_wait_ring(struct mtd_info *mtd, unsigned int timeout_ms)
{ {
struct ps3vram_priv *priv = mtd->priv; struct ps3vram_priv *priv = mtd->priv;
unsigned long timeout = jiffies + msecs_to_jiffies(timeout_ms);
/* wait until setup commands are processed */ do {
timeout *= 1000;
while (--timeout) {
if (priv->ctrl[CTRL_PUT] == priv->ctrl[CTRL_GET]) if (priv->ctrl[CTRL_PUT] == priv->ctrl[CTRL_GET])
break; return 0;
udelay(1); msleep(1);
} } while (time_before(jiffies, timeout));
if (timeout == 0) {
dev_dbg(priv->dev, "%s:%d: FIFO timeout (%08x/%08x/%08x)\n",
__func__, __LINE__, priv->ctrl[CTRL_PUT],
priv->ctrl[CTRL_GET], priv->ctrl[CTRL_TOP]);
return -ETIMEDOUT;
}
return 0; dev_dbg(priv->dev, "%s:%d: FIFO timeout (%08x/%08x/%08x)\n", __func__,
__LINE__, priv->ctrl[CTRL_PUT], priv->ctrl[CTRL_GET],
priv->ctrl[CTRL_TOP]);
return -ETIMEDOUT;
} }
static void ps3vram_out_ring(struct ps3vram_priv *priv, u32 data) static void ps3vram_out_ring(struct ps3vram_priv *priv, u32 data)