Date: Sat, 11 Nov 2006 17:21:12 +0100
From: Nicolas Kaiser <nikai@nikai.net>
To: Alexey Dobriyan <adobriyan@gmail.com>
Cc: linux-ide@vger.kernel.org, trivial@kernel.org,
       linux-kernel@vger.kernel.org
Subject: Re: [PATCH][TRIVIAL] drivers/ide: stray bracket
Message-ID: <20061111172112.7e1aba49@lucky.kitzblitz>
In-Reply-To: <20061111131612.GA4974@martell.zuzino.mipt.ru>
References: <20061111014756.3467d7ee@lucky.kitzblitz>
	<20061111131612.GA4974@martell.zuzino.mipt.ru>
Organization: -
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1201
Lines: 35

* Alexey Dobriyan <adobriyan@gmail.com>:
> On Sat, Nov 11, 2006 at 01:47:56AM +0100, Nicolas Kaiser wrote:
> > Stray bracket in debug code.

> Just remove whole printk. It was broken for a looong time.

If you prefer it that way, here we go:
Remove debug code that was broken for long time.

Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
---

 drivers/ide/legacy/hd.c |    5 -----
 1 file changed, 5 deletions(-)

diff -uprN a/drivers/ide/legacy/hd.c b/drivers/ide/legacy/hd.c
--- a/drivers/ide/legacy/hd.c	2006-09-20 05:42:06.000000000 +0200
+++ b/drivers/ide/legacy/hd.c	2006-11-11 17:11:28.000000000 +0100
@@ -456,11 +456,6 @@ ok_to_read:
 	req->errors = 0;
 	i = --req->nr_sectors;
 	--req->current_nr_sectors;
-#ifdef DEBUG
-	printk("%s: read: sector %ld, remaining = %ld, buffer=%p\n",
-		req->rq_disk->disk_name, req->sector, req->nr_sectors,
-		req->buffer+512));
-#endif
 	if (req->current_nr_sectors <= 0)
 		end_request(req, 1);
 	if (i > 0) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/