* [PATCH] sc1200: remove redundant warning message from sc1200_tune_chipset()
@ 2007-07-13 21:10 Bartlomiej Zolnierkiewicz
2007-07-14 16:38 ` Sergei Shtylyov
0 siblings, 1 reply; 3+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2007-07-13 21:10 UTC (permalink / raw)
To: linux-ide
ide_config_drive_speed() called by sc1200_set_xfer_mode() already gives
a meaningful warning message in case of failure.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
---
drivers/ide/pci/sc1200.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
Index: b/drivers/ide/pci/sc1200.c
===================================================================
--- a/drivers/ide/pci/sc1200.c
+++ b/drivers/ide/pci/sc1200.c
@@ -149,10 +149,8 @@ static int sc1200_tune_chipset(ide_drive
/*
* Tell the drive to switch to the new mode; abort on failure.
*/
- if (sc1200_set_xfer_mode(drive, mode)) {
- printk("SC1200: set xfer mode failure\n");
+ if (sc1200_set_xfer_mode(drive, mode))
return 1; /* failure */
- }
switch (mode) {
case XFER_PIO_4:
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sc1200: remove redundant warning message from sc1200_tune_chipset()
2007-07-13 21:10 [PATCH] sc1200: remove redundant warning message from sc1200_tune_chipset() Bartlomiej Zolnierkiewicz
@ 2007-07-14 16:38 ` Sergei Shtylyov
2007-07-18 21:06 ` Bartlomiej Zolnierkiewicz
0 siblings, 1 reply; 3+ messages in thread
From: Sergei Shtylyov @ 2007-07-14 16:38 UTC (permalink / raw)
To: Bartlomiej Zolnierkiewicz; +Cc: linux-ide
Hello.
Bartlomiej Zolnierkiewicz wrote:
> ide_config_drive_speed() called by sc1200_set_xfer_mode() already gives
> a meaningful warning message in case of failure.
> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
MBR, Sergei
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sc1200: remove redundant warning message from sc1200_tune_chipset()
2007-07-14 16:38 ` Sergei Shtylyov
@ 2007-07-18 21:06 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 3+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2007-07-18 21:06 UTC (permalink / raw)
To: Sergei Shtylyov; +Cc: linux-ide
On Saturday 14 July 2007, Sergei Shtylyov wrote:
> Hello.
>
> Bartlomiej Zolnierkiewicz wrote:
>
> > ide_config_drive_speed() called by sc1200_set_xfer_mode() already gives
> > a meaningful warning message in case of failure.
>
> > Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
>
> Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
added
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-07-18 21:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-13 21:10 [PATCH] sc1200: remove redundant warning message from sc1200_tune_chipset() Bartlomiej Zolnierkiewicz
2007-07-14 16:38 ` Sergei Shtylyov
2007-07-18 21:06 ` Bartlomiej Zolnierkiewicz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).