You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
363 lines
12 KiB
Diff
363 lines
12 KiB
Diff
From f259d74e030faab15b95fb4bb56d7f424773c278 Mon Sep 17 00:00:00 2001
|
|
From: Geoff Levand <geoffrey.levand@am.sony.com>
|
|
Date: Tue, 6 Jan 2009 11:32:21 +0000
|
|
Subject: [PATCH] mtd/ps3vram: Cleanup ps3vram driver messages
|
|
|
|
Cleanup the ps3vram driver messages. Add a new struct device pointer
|
|
variable dev to struct ps3vram_priv and use dev_dbg(), pr_dbg(), etc.
|
|
where appropriate.
|
|
|
|
Signed-off-by: Geoff Levand <geoffrey.levand@am.sony.com>
|
|
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
|
|
---
|
|
drivers/mtd/devices/ps3vram.c | 116 ++++++++++++++++++++++++-----------------
|
|
1 files changed, 68 insertions(+), 48 deletions(-)
|
|
|
|
diff --git a/drivers/mtd/devices/ps3vram.c b/drivers/mtd/devices/ps3vram.c
|
|
index 18ca521..f5cc290 100644
|
|
--- a/drivers/mtd/devices/ps3vram.c
|
|
+++ b/drivers/mtd/devices/ps3vram.c
|
|
@@ -53,9 +53,6 @@ struct mtd_info ps3vram_mtd;
|
|
#define CACHE_PAGE_PRESENT 1
|
|
#define CACHE_PAGE_DIRTY 2
|
|
|
|
-#define dbg(fmt, args...) \
|
|
- pr_debug("%s:%d " fmt "\n", __func__, __LINE__, ## args)
|
|
-
|
|
struct ps3vram_tag {
|
|
unsigned int address;
|
|
unsigned int flags;
|
|
@@ -78,6 +75,7 @@ struct ps3vram_priv {
|
|
uint32_t *fifo_base;
|
|
uint32_t *fifo_ptr;
|
|
|
|
+ struct device *dev;
|
|
struct ps3vram_cache cache;
|
|
|
|
/* Used to serialize cache/DMA operations */
|
|
@@ -148,8 +146,9 @@ static int ps3vram_wait_ring(struct mtd_info *mtd, int timeout)
|
|
udelay(1);
|
|
}
|
|
if (timeout == 0) {
|
|
- pr_err("FIFO timeout (%08x/%08x/%08x)\n", priv->ctrl[CTRL_PUT],
|
|
- priv->ctrl[CTRL_GET], priv->ctrl[CTRL_TOP]);
|
|
+ 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;
|
|
}
|
|
|
|
@@ -181,7 +180,8 @@ static void ps3vram_rewind_ring(struct mtd_info *mtd)
|
|
L1GPU_CONTEXT_ATTRIBUTE_FB_BLIT,
|
|
0, 0, 0, 0);
|
|
if (status)
|
|
- pr_err("ps3vram: lv1_gpu_context_attribute FB_BLIT failed\n");
|
|
+ dev_err(priv->dev, "%s:%d: lv1_gpu_context_attribute failed\n",
|
|
+ __func__, __LINE__);
|
|
|
|
priv->fifo_ptr = priv->fifo_base;
|
|
}
|
|
@@ -201,11 +201,13 @@ static void ps3vram_fire_ring(struct mtd_info *mtd)
|
|
L1GPU_CONTEXT_ATTRIBUTE_FB_BLIT,
|
|
0, 0, 0, 0);
|
|
if (status)
|
|
- pr_err("ps3vram: lv1_gpu_context_attribute FB_BLIT failed\n");
|
|
+ dev_err(priv->dev, "%s:%d: lv1_gpu_context_attribute failed\n",
|
|
+ __func__, __LINE__);
|
|
|
|
if ((priv->fifo_ptr - priv->fifo_base) * sizeof(uint32_t) >
|
|
FIFO_SIZE - 1024) {
|
|
- dbg("fifo full, rewinding");
|
|
+ dev_dbg(priv->dev, "%s:%d: fifo full, rewinding\n", __func__,
|
|
+ __LINE__);
|
|
ps3vram_wait_ring(mtd, 200);
|
|
ps3vram_rewind_ring(mtd);
|
|
}
|
|
@@ -258,7 +260,8 @@ static int ps3vram_upload(struct mtd_info *mtd, unsigned int src_offset,
|
|
ps3vram_out_ring(priv, 0);
|
|
ps3vram_fire_ring(mtd);
|
|
if (ps3vram_notifier_wait(mtd, 200) < 0) {
|
|
- pr_err("notifier timeout\n");
|
|
+ dev_dbg(priv->dev, "%s:%d: notifier timeout\n", __func__,
|
|
+ __LINE__);
|
|
return -1;
|
|
}
|
|
|
|
@@ -289,7 +292,8 @@ static int ps3vram_download(struct mtd_info *mtd, unsigned int src_offset,
|
|
ps3vram_out_ring(priv, 0);
|
|
ps3vram_fire_ring(mtd);
|
|
if (ps3vram_notifier_wait(mtd, 200) < 0) {
|
|
- pr_err("notifier timeout\n");
|
|
+ dev_dbg(priv->dev, "%s:%d: notifier timeout\n", __func__,
|
|
+ __LINE__);
|
|
return -1;
|
|
}
|
|
|
|
@@ -302,16 +306,17 @@ static void ps3vram_cache_evict(struct mtd_info *mtd, int entry)
|
|
struct ps3vram_cache *cache = &priv->cache;
|
|
|
|
if (cache->tags[entry].flags & CACHE_PAGE_DIRTY) {
|
|
- dbg("flushing %d : 0x%08x", entry, cache->tags[entry].address);
|
|
+ dev_dbg(priv->dev, "%s:%d: flushing %d : 0x%08x\n", __func__,
|
|
+ __LINE__, entry, cache->tags[entry].address);
|
|
if (ps3vram_upload(mtd,
|
|
CACHE_OFFSET + entry * cache->page_size,
|
|
cache->tags[entry].address,
|
|
DMA_PAGE_SIZE,
|
|
cache->page_size / DMA_PAGE_SIZE) < 0) {
|
|
- pr_err("failed to upload from 0x%x to 0x%x size 0x%x\n",
|
|
- entry * cache->page_size,
|
|
- cache->tags[entry].address,
|
|
- cache->page_size);
|
|
+ dev_dbg(priv->dev, "%s:%d: failed to upload from "
|
|
+ "0x%x to 0x%x size 0x%x\n", __func__, __LINE__,
|
|
+ entry * cache->page_size,
|
|
+ cache->tags[entry].address, cache->page_size);
|
|
}
|
|
cache->tags[entry].flags &= ~CACHE_PAGE_DIRTY;
|
|
}
|
|
@@ -323,16 +328,16 @@ static void ps3vram_cache_load(struct mtd_info *mtd, int entry,
|
|
struct ps3vram_priv *priv = mtd->priv;
|
|
struct ps3vram_cache *cache = &priv->cache;
|
|
|
|
- dbg("fetching %d : 0x%08x", entry, address);
|
|
+ dev_dbg(priv->dev, "%s:%d: fetching %d : 0x%08x\n", __func__, __LINE__,
|
|
+ entry, address);
|
|
if (ps3vram_download(mtd,
|
|
address,
|
|
CACHE_OFFSET + entry * cache->page_size,
|
|
DMA_PAGE_SIZE,
|
|
cache->page_size / DMA_PAGE_SIZE) < 0) {
|
|
- pr_err("failed to download from 0x%x to 0x%x size 0x%x\n",
|
|
- address,
|
|
- entry * cache->page_size,
|
|
- cache->page_size);
|
|
+ dev_err(priv->dev, "%s:%d: failed to download from "
|
|
+ "0x%x to 0x%x size 0x%x\n", __func__, __LINE__, address,
|
|
+ entry * cache->page_size, cache->page_size);
|
|
}
|
|
|
|
cache->tags[entry].address = address;
|
|
@@ -346,7 +351,7 @@ static void ps3vram_cache_flush(struct mtd_info *mtd)
|
|
struct ps3vram_cache *cache = &priv->cache;
|
|
int i;
|
|
|
|
- dbg("FLUSH");
|
|
+ dev_dbg(priv->dev, "%s:%d: FLUSH\n", __func__, __LINE__);
|
|
for (i = 0; i < cache->page_count; i++) {
|
|
ps3vram_cache_evict(mtd, i);
|
|
cache->tags[i].flags = 0;
|
|
@@ -369,15 +374,15 @@ static unsigned int ps3vram_cache_match(struct mtd_info *mtd, loff_t address)
|
|
for (i = 0; i < cache->page_count; i++) {
|
|
if ((cache->tags[i].flags & CACHE_PAGE_PRESENT) &&
|
|
cache->tags[i].address == base) {
|
|
- dbg("found entry %d : 0x%08x",
|
|
- i, cache->tags[i].address);
|
|
+ dev_dbg(priv->dev, "%s:%d: found entry %d : 0x%08x\n",
|
|
+ __func__, __LINE__, i, cache->tags[i].address);
|
|
return i;
|
|
}
|
|
}
|
|
|
|
/* choose a random entry */
|
|
i = (jiffies + (counter++)) % cache->page_count;
|
|
- dbg("using cache entry %d", i);
|
|
+ dev_dbg(priv->dev, "%s:%d: using entry %d\n", __func__, __LINE__, i);
|
|
|
|
ps3vram_cache_evict(mtd, i);
|
|
ps3vram_cache_load(mtd, i, base);
|
|
@@ -389,18 +394,19 @@ static int ps3vram_cache_init(struct mtd_info *mtd)
|
|
{
|
|
struct ps3vram_priv *priv = mtd->priv;
|
|
|
|
- pr_info("creating cache: %d entries, %d bytes pages\n",
|
|
- CACHE_PAGE_COUNT, CACHE_PAGE_SIZE);
|
|
-
|
|
priv->cache.page_count = CACHE_PAGE_COUNT;
|
|
priv->cache.page_size = CACHE_PAGE_SIZE;
|
|
priv->cache.tags = kzalloc(sizeof(struct ps3vram_tag) *
|
|
CACHE_PAGE_COUNT, GFP_KERNEL);
|
|
if (priv->cache.tags == NULL) {
|
|
- pr_err("could not allocate cache tags\n");
|
|
+ dev_err(priv->dev, "%s:%d: could not allocate cache tags\n",
|
|
+ __func__, __LINE__);
|
|
return -ENOMEM;
|
|
}
|
|
|
|
+ dev_info(priv->dev, "created ram cache: %d entries, %d KiB each\n",
|
|
+ CACHE_PAGE_COUNT, CACHE_PAGE_SIZE / 1024);
|
|
+
|
|
return 0;
|
|
}
|
|
|
|
@@ -434,14 +440,14 @@ static int ps3vram_erase(struct mtd_info *mtd, struct erase_info *instr)
|
|
return 0;
|
|
}
|
|
|
|
-
|
|
static int ps3vram_read(struct mtd_info *mtd, loff_t from, size_t len,
|
|
size_t *retlen, u_char *buf)
|
|
{
|
|
struct ps3vram_priv *priv = mtd->priv;
|
|
unsigned int cached, count;
|
|
|
|
- dbg("from = 0x%08x len = 0x%zx", (unsigned int) from, len);
|
|
+ dev_dbg(priv->dev, "%s:%d: from=0x%08x len=0x%zx\n", __func__, __LINE__,
|
|
+ (unsigned int)from, len);
|
|
|
|
if (from >= mtd->size)
|
|
return -EINVAL;
|
|
@@ -463,8 +469,9 @@ static int ps3vram_read(struct mtd_info *mtd, loff_t from, size_t len,
|
|
entry = ps3vram_cache_match(mtd, from);
|
|
cached = CACHE_OFFSET + entry * priv->cache.page_size + offset;
|
|
|
|
- dbg("from=%08x cached=%08x offset=%08x avail=%08x count=%08x",
|
|
- (unsigned)from, cached, offset, avail, count);
|
|
+ dev_dbg(priv->dev, "%s:%d: from=%08x cached=%08x offset=%08x "
|
|
+ "avail=%08x count=%08x\n", __func__, __LINE__,
|
|
+ (unsigned int)from, cached, offset, avail, count);
|
|
|
|
if (avail > count)
|
|
avail = count;
|
|
@@ -507,8 +514,9 @@ static int ps3vram_write(struct mtd_info *mtd, loff_t to, size_t len,
|
|
entry = ps3vram_cache_match(mtd, to);
|
|
cached = CACHE_OFFSET + entry * priv->cache.page_size + offset;
|
|
|
|
- dbg("to=%08x cached=%08x offset=%08x avail=%08x count=%08x",
|
|
- (unsigned) to, cached, offset, avail, count);
|
|
+ dev_dbg(priv->dev, "%s:%d: to=%08x cached=%08x offset=%08x "
|
|
+ "avail=%08x count=%08x\n", __func__, __LINE__,
|
|
+ (unsigned int)to, cached, offset, avail, count);
|
|
|
|
if (avail > count)
|
|
avail = count;
|
|
@@ -544,12 +552,14 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
priv = ps3vram_mtd.priv;
|
|
|
|
mutex_init(&priv->lock);
|
|
+ priv->dev = &dev->core;
|
|
|
|
/* Allocate XDR buffer (1MiB aligned) */
|
|
priv->xdr_buf = (uint8_t *) __get_free_pages(GFP_KERNEL,
|
|
get_order(XDR_BUF_SIZE));
|
|
if (priv->xdr_buf == NULL) {
|
|
- pr_err("ps3vram: could not allocate XDR buffer\n");
|
|
+ dev_dbg(&dev->core, "%s:%d: could not allocate XDR buffer\n",
|
|
+ __func__, __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_free_priv;
|
|
}
|
|
@@ -560,7 +570,8 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
|
|
/* XXX: Need to open GPU, in case ps3fb or snd_ps3 aren't loaded */
|
|
if (ps3_open_hv_device(dev)) {
|
|
- pr_err("ps3vram: ps3_open_hv_device failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: ps3_open_hv_device failed\n",
|
|
+ __func__, __LINE__);
|
|
ret = -EAGAIN;
|
|
goto out_close_gpu;
|
|
}
|
|
@@ -572,7 +583,8 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
ddr_size -= ps3fb_videomemory.size;
|
|
ddr_size = ALIGN(ddr_size, 1024*1024);
|
|
if (ddr_size <= 0) {
|
|
- printk(KERN_ERR "ps3vram: specified size is too small\n");
|
|
+ dev_err(&dev->core, "%s:%d: specified size is too small\n",
|
|
+ __func__, __LINE__);
|
|
ret = -EINVAL;
|
|
goto out_close_gpu;
|
|
}
|
|
@@ -586,12 +598,11 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
ddr_size -= 1024*1024;
|
|
}
|
|
if (status != 0 || ddr_size <= 0) {
|
|
- pr_err("ps3vram: lv1_gpu_memory_allocate failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: lv1_gpu_memory_allocate failed\n",
|
|
+ __func__, __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_free_xdr_buf;
|
|
}
|
|
- pr_info("ps3vram: allocated %u MiB of DDR memory\n",
|
|
- (unsigned int) (ddr_size / 1024 / 1024));
|
|
|
|
/* Request context */
|
|
status = lv1_gpu_context_allocate(priv->memory_handle,
|
|
@@ -602,7 +613,8 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
&reports_lpar,
|
|
&reports_size);
|
|
if (status) {
|
|
- pr_err("ps3vram: lv1_gpu_context_allocate failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: lv1_gpu_context_allocate failed\n",
|
|
+ __func__, __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_free_memory;
|
|
}
|
|
@@ -612,28 +624,32 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
ps3_mm_phys_to_lpar(__pa(priv->xdr_buf)),
|
|
XDR_BUF_SIZE, 0);
|
|
if (status) {
|
|
- pr_err("ps3vram: lv1_gpu_context_iomap failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: lv1_gpu_context_iomap failed\n",
|
|
+ __func__, __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_free_context;
|
|
}
|
|
|
|
priv->base = ioremap(ddr_lpar, ddr_size);
|
|
if (!priv->base) {
|
|
- pr_err("ps3vram: ioremap failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: ioremap failed\n", __func__,
|
|
+ __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_free_context;
|
|
}
|
|
|
|
priv->ctrl = ioremap(ctrl_lpar, 64 * 1024);
|
|
if (!priv->ctrl) {
|
|
- pr_err("ps3vram: ioremap failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: ioremap failed\n", __func__,
|
|
+ __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_unmap_vram;
|
|
}
|
|
|
|
priv->reports = ioremap(reports_lpar, reports_size);
|
|
if (!priv->reports) {
|
|
- pr_err("ps3vram: ioremap failed\n");
|
|
+ dev_err(&dev->core, "%s:%d: ioremap failed\n", __func__,
|
|
+ __LINE__);
|
|
ret = -ENOMEM;
|
|
goto out_unmap_ctrl;
|
|
}
|
|
@@ -661,7 +677,8 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
ret = ps3vram_wait_ring(&ps3vram_mtd, 100);
|
|
mutex_unlock(&ps3_gpu_mutex);
|
|
if (ret < 0) {
|
|
- pr_err("failed to initialize channels\n");
|
|
+ dev_err(&dev->core, "%s:%d: failed to initialize channels\n",
|
|
+ __func__, __LINE__);
|
|
ret = -ETIMEDOUT;
|
|
goto out_unmap_reports;
|
|
}
|
|
@@ -669,12 +686,15 @@ static int __devinit ps3vram_probe(struct ps3_system_bus_device *dev)
|
|
ps3vram_cache_init(&ps3vram_mtd);
|
|
|
|
if (add_mtd_device(&ps3vram_mtd)) {
|
|
- pr_err("ps3vram: failed to register device\n");
|
|
+ dev_err(&dev->core, "%s:%d: add_mtd_device failed\n",
|
|
+ __func__, __LINE__);
|
|
ret = -EAGAIN;
|
|
goto out_cache_cleanup;
|
|
}
|
|
|
|
- pr_info("ps3vram mtd device registered, %lu bytes\n", ddr_size);
|
|
+ dev_info(&dev->core, "reserved %u MiB of gpu memory\n",
|
|
+ (unsigned int)(ddr_size / 1024 / 1024));
|
|
+
|
|
return 0;
|
|
|
|
out_cache_cleanup:
|
|
--
|
|
1.6.0.4
|
|
|