Hi!
Suspend/resume support in ide seems to work okay these days, so this
should be applied...
Pavel
--- clean/include/linux/ide.h 2004-03-11 18:11:23.000000000 +0100
+++ linux/include/linux/ide.h 2004-03-26 01:08:28.000000000 +0100
@@ -24,8 +24,6 @@
#include <asm/io.h>
#include <asm/semaphore.h>
-#define DEBUG_PM
-
/*
* This is the multiple IDE interface driver, as evolved from hd.c.
* It supports up to four IDE interfaces, on one or more IRQs (usually 14 & 15).
--
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]
On Fri, Mar 26, 2004 at 01:11:54AM +0100, Pavel Machek wrote:
> Hi!
>
> Suspend/resume support in ide seems to work okay these days, so this
> should be applied...
> Pavel
>
> --- clean/include/linux/ide.h 2004-03-11 18:11:23.000000000 +0100
> +++ linux/include/linux/ide.h 2004-03-26 01:08:28.000000000 +0100
> @@ -24,8 +24,6 @@
> #include <asm/io.h>
> #include <asm/semaphore.h>
>
> -#define DEBUG_PM
> -
> /*
> * This is the multiple IDE interface driver, as evolved from hd.c.
> * It supports up to four IDE interfaces, on one or more IRQs (usually 14 & 15).
>
> --
> When do you have a heart between your knees?
> [Johanka's followup: and *two* hearts?]
What about these stale #ifdefs?
drivers/ide/ide-io.c:126:#ifdef DEBUG_PM
drivers/ide/ide-io.c:220:#ifdef DEBUG_PM
drivers/ide/ide-io.c:638:#ifdef DEBUG_PM
drivers/ide/ide-io.c:662:#ifdef DEBUG_PM
This patch sweeps up both DEBUG_PM and DEBUG #ifdefs in favour of pr_debug()
ide-io.c | 74 ++++++++++++++++++++++++++-------------------------------------
1 files changed, 31 insertions(+), 43 deletions(-)
--- a/drivers/ide/ide-io.c 2004-03-22 12:30:36.000000000 +0100
+++ b/drivers/ide/ide-io.c 2004-03-28 19:24:09.000000000 +0200
@@ -23,7 +23,8 @@
* are deemed to be part of the source code.
*/
-
+#undef DEBUG
+
#include <linux/config.h>
#include <linux/module.h>
#include <linux/types.h>
@@ -123,10 +124,8 @@
{
unsigned long flags;
-#ifdef DEBUG_PM
- printk("%s: completing PM request, %s\n", drive->name,
- blk_pm_suspend_request(rq) ? "suspend" : "resume");
-#endif
+ pr_debug("%s: completing PM request, %s\n", drive->name,
+ blk_pm_suspend_request(rq) ? "suspend" : "resume");
spin_lock_irqsave(&ide_lock, flags);
if (blk_pm_suspend_request(rq)) {
blk_stop_queue(drive->queue);
@@ -217,10 +216,8 @@
}
}
} else if (blk_pm_request(rq)) {
-#ifdef DEBUG_PM
- printk("%s: complete_power_step(step: %d, stat: %x, err: %x)\n",
- drive->name, rq->pm->pm_step, stat, err);
-#endif
+ pr_debug("%s: complete_power_step(step: %d, stat: %x, err: %x)\n",
+ drive->name, rq->pm->pm_step, stat, err);
DRIVER(drive)->complete_power_step(drive, rq, stat, err);
if (rq->pm->pm_step == ide_pm_state_completed)
ide_complete_pm_request(drive, rq);
@@ -469,9 +466,7 @@
{
special_t *s = &drive->special;
-#ifdef DEBUG
- printk("%s: do_special: 0x%02x\n", drive->name, s->all);
-#endif
+ pr_debug("%s: do_special: 0x%02x\n", drive->name, s->all);
if (s->b.set_tune) {
s->b.set_tune = 0;
if (HWIF(drive)->tuneproc != NULL)
@@ -514,16 +509,16 @@
if (!args)
goto done;
-#ifdef DEBUG
- printk("%s: DRIVE_TASK_CMD ", drive->name);
- printk("cmd=0x%02x ", args[0]);
- printk("fr=0x%02x ", args[1]);
- printk("ns=0x%02x ", args[2]);
- printk("sc=0x%02x ", args[3]);
- printk("lcyl=0x%02x ", args[4]);
- printk("hcyl=0x%02x ", args[5]);
- printk("sel=0x%02x\n", args[6]);
-#endif
+
+ pr_debug("%s: DRIVE_TASK_CMD ", drive->name);
+ pr_debug("cmd=0x%02x ", args[0]);
+ pr_debug("fr=0x%02x ", args[1]);
+ pr_debug("ns=0x%02x ", args[2]);
+ pr_debug("sc=0x%02x ", args[3]);
+ pr_debug("lcyl=0x%02x ", args[4]);
+ pr_debug("hcyl=0x%02x ", args[5]);
+ pr_debug("sel=0x%02x\n", args[6]);
+
hwif->OUTB(args[1], IDE_FEATURE_REG);
hwif->OUTB(args[3], IDE_SECTOR_REG);
hwif->OUTB(args[4], IDE_LCYL_REG);
@@ -539,13 +534,13 @@
if (!args)
goto done;
-#ifdef DEBUG
- printk("%s: DRIVE_CMD ", drive->name);
- printk("cmd=0x%02x ", args[0]);
- printk("sc=0x%02x ", args[1]);
- printk("fr=0x%02x ", args[2]);
- printk("xx=0x%02x\n", args[3]);
-#endif
+
+ pr_debug("%s: DRIVE_CMD ", drive->name);
+ pr_debug("cmd=0x%02x ", args[0]);
+ pr_debug("sc=0x%02x ", args[1]);
+ pr_debug("fr=0x%02x ", args[2]);
+ pr_debug("xx=0x%02x\n", args[3]);
+
if (args[0] == WIN_SMART) {
hwif->OUTB(0x4f, IDE_LCYL_REG);
hwif->OUTB(0xc2, IDE_HCYL_REG);
@@ -564,9 +559,7 @@
* NULL is actually a valid way of waiting for
* all current requests to be flushed from the queue.
*/
-#ifdef DEBUG
- printk("%s: DRIVE_CMD (null)\n", drive->name);
-#endif
+ pr_debug("%s: DRIVE_CMD (null)\n", drive->name);
ide_end_drive_cmd(drive,
hwif->INB(IDE_STATUS_REG),
hwif->INB(IDE_ERROR_REG));
@@ -593,10 +586,8 @@
BUG_ON(!(rq->flags & REQ_STARTED));
-#ifdef DEBUG
- printk("%s: start_request: current=0x%08lx\n",
- HWIF(drive)->name, (unsigned long) rq);
-#endif
+ pr_debug("%s: start_request: current=0x%08lx\n",
+ HWIF(drive)->name, (unsigned long) rq);
/* bail early if we've exceeded max_failures */
if (drive->max_failures && (drive->failures > drive->max_failures)) {
@@ -635,9 +626,8 @@
* point.
*/
int rc;
-#ifdef DEBUG_PM
- printk("%s: Wakeup request inited, waiting for !BSY...\n", drive->name);
-#endif
+
+ pr_debug("%s: Wakeup request inited, waiting for !BSY...\n", drive->name);
rc = ide_wait_not_busy(HWIF(drive), 35000);
if (rc)
printk(KERN_WARNING "%s: bus not ready on wakeup\n", drive->name);
@@ -659,10 +649,8 @@
else if (rq->flags & REQ_DRIVE_TASKFILE)
return execute_drive_cmd(drive, rq);
else if (blk_pm_request(rq)) {
-#ifdef DEBUG_PM
- printk("%s: start_power_step(step: %d)\n",
- drive->name, rq->pm->pm_step);
-#endif
+ pr_debug("%s: start_power_step(step: %d)\n",
+ drive->name, rq->pm->pm_step);
startstop = DRIVER(drive)->start_power_step(drive, rq);
if (startstop == ide_stopped &&
rq->pm->pm_step == ide_pm_state_completed)
Hi!
> What about these stale #ifdefs?
>
> drivers/ide/ide-io.c:126:#ifdef DEBUG_PM
> drivers/ide/ide-io.c:220:#ifdef DEBUG_PM
> drivers/ide/ide-io.c:638:#ifdef DEBUG_PM
> drivers/ide/ide-io.c:662:#ifdef DEBUG_PM
I wanted to leave debugging possible...
> This patch sweeps up both DEBUG_PM and DEBUG #ifdefs in favour of pr_debug()
...but your patch looks better.
Pavel
> ide-io.c | 74 ++++++++++++++++++++++++++-------------------------------------
--
When do you have a heart between your knees?
[Johanka's followup: and *two* hearts?]
On Monday 29 of March 2004 00:25, Pavel Machek wrote:
> Hi!
>
> > What about these stale #ifdefs?
> >
> > drivers/ide/ide-io.c:126:#ifdef DEBUG_PM
> > drivers/ide/ide-io.c:220:#ifdef DEBUG_PM
> > drivers/ide/ide-io.c:638:#ifdef DEBUG_PM
> > drivers/ide/ide-io.c:662:#ifdef DEBUG_PM
>
> I wanted to leave debugging possible...
>
> > This patch sweeps up both DEBUG_PM and DEBUG #ifdefs in favour of
> > pr_debug()
Well, I doubt anybody wants to debug taskfile IOs and PM at the same time.
[ pm_debug() ? ]
BTW adding -DDEBUG to EXTRA_CFLAGS in Makefile is cleaner
than defining/undefining it inside .c or .h.
> ...but your patch looks better.
> Pavel
>
> > ide-io.c | 74
> > ++++++++++++++++++++++++++-------------------------------------