From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 3/7] ide: merge ide_disk_special() into do_special() Date: Thu, 14 May 2009 16:23:32 +0400 Message-ID: <4A0C0D44.9020308@ru.mvista.com> References: <20090513205139.28638.87226.sendpatchset@localhost.localdomain> <20090513205200.28638.90827.sendpatchset@localhost.localdomain> <1242248102.4639.17.camel@Joe-AcerAspire> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from h155.mvista.com ([63.81.120.155]:3937 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1760944AbZENMWx (ORCPT ); Thu, 14 May 2009 08:22:53 -0400 In-Reply-To: <1242248102.4639.17.camel@Joe-AcerAspire> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Bartlomiej Zolnierkiewicz Cc: Joe Perches , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Hello. Joe Perches wrote: >>While at it: >>- change debug printk() level to KERN_DEBUG >>- update documentation >>There should be no functional changes caused by this patch. >>Signed-off-by: Bartlomiej Zolnierkiewicz >>--- >> drivers/ide/ide-io.c | 44 ++++++++++++++++++-------------------------- >> 1 file changed, 18 insertions(+), 26 deletions(-) >>Index: b/drivers/ide/ide-io.c >>=================================================================== >>--- a/drivers/ide/ide-io.c >>+++ b/drivers/ide/ide-io.c >>@@ -184,11 +184,28 @@ static void ide_tf_set_setmult_cmd(ide_d > [] >>+#ifdef DEBUG >>+ printk(KERN_DEBUG "%s: %s: 0x%02x\n", drive->name, s->all); >>+#endif > pr_debug? Yes, and the number of arguments doesn't correspond to the number of the format specifiers. WBR, Sergei