lkml.org 
[lkml]   [2005]   [Feb]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[patch ide-dev 4/9] add ide_task_init_flush() helper

* add ide_task_init_flush() helper
* use it in do_idedisk_cacheflush() and ide_start_power_step()
* inline do_idedisk_cacheflush() into ide_cacheflush_p()

diff -Nru a/drivers/ide/ide-disk.c b/drivers/ide/ide-disk.c
--- a/drivers/ide/ide-disk.c 2005-02-19 17:22:58 +01:00
+++ b/drivers/ide/ide-disk.c 2005-02-19 17:22:58 +01:00
@@ -723,24 +723,6 @@
return 0;
}

-static int do_idedisk_flushcache (ide_drive_t *drive)
-{
- ide_task_t args;
- struct ata_taskfile *tf = &args.tf;
-
- memset(&args, 0, sizeof(ide_task_t));
-
- if (ide_id_has_flush_cache_ext(drive->id)) {
- tf->command = WIN_FLUSH_CACHE_EXT;
- tf->flags |= ATA_TFLAG_LBA48;
- } else
- tf->command = WIN_FLUSH_CACHE;
-
- args.command_type = IDE_DRIVE_TASK_NO_DATA;
- args.handler = &task_no_data_intr;
- return ide_raw_taskfile(drive, &args, NULL);
-}
-
static int set_acoustic (ide_drive_t *drive, int arg)
{
ide_task_t args;
@@ -931,10 +913,14 @@

static void ide_cacheflush_p(ide_drive_t *drive)
{
+ ide_task_t task;
+
if (!drive->wcache || !ide_id_has_flush_cache(drive->id))
return;

- if (do_idedisk_flushcache(drive))
+ ide_task_init_flush(drive, &task);
+
+ if (ide_raw_taskfile(drive, &task, NULL))
printk(KERN_INFO "%s: wcache flush failed!\n", drive->name);
}

diff -Nru a/drivers/ide/ide-io.c b/drivers/ide/ide-io.c
--- a/drivers/ide/ide-io.c 2005-02-19 17:22:58 +01:00
+++ b/drivers/ide/ide-io.c 2005-02-19 17:22:58 +01:00
@@ -55,6 +55,24 @@
#include <asm/io.h>
#include <asm/bitops.h>

+void ide_task_init_flush(ide_drive_t *drive, ide_task_t *task)
+{
+ struct ata_taskfile *tf = &task->tf;
+
+ memset(task, 0, sizeof(*task));
+
+ if (ide_id_has_flush_cache_ext(drive->id)) {
+ tf->command = WIN_FLUSH_CACHE_EXT;
+ tf->flags |= ATA_TFLAG_LBA48;
+ } else
+ tf->command = WIN_FLUSH_CACHE;
+
+ task->command_type = IDE_DRIVE_TASK_NO_DATA;
+ task->handler = &task_no_data_intr;
+}
+
+EXPORT_SYMBOL_GPL(ide_task_init_flush);
+
static void ide_fill_flush_cmd(ide_drive_t *drive, struct request *rq)
{
char *buf = rq->cmd;
@@ -247,14 +265,8 @@
return ide_stopped;
}

- if (ide_id_has_flush_cache_ext(drive->id)) {
- tf->command = WIN_FLUSH_CACHE_EXT;
- tf->flags |= ATA_TFLAG_LBA48;
- } else
- tf->command = WIN_FLUSH_CACHE;
+ ide_task_init_flush(drive, args);

- args->command_type = IDE_DRIVE_TASK_NO_DATA;
- args->handler = &task_no_data_intr;
return do_rw_taskfile(drive, args);

case idedisk_pm_standby: /* Suspend step 2 (standby) */
diff -Nru a/include/linux/ide.h b/include/linux/ide.h
--- a/include/linux/ide.h 2005-02-19 17:22:58 +01:00
+++ b/include/linux/ide.h 2005-02-19 17:22:58 +01:00
@@ -924,6 +924,19 @@
typedef ide_startstop_t (ide_handler_t)(ide_drive_t *);
typedef int (ide_expiry_t)(ide_drive_t *);

+typedef struct ide_task_s {
+ struct ata_taskfile tf;
+ u16 data;
+ ide_reg_valid_t tf_out_flags;
+ ide_reg_valid_t tf_in_flags;
+ int data_phase;
+ int command_type;
+ ide_pre_handler_t *prehandler;
+ ide_handler_t *handler;
+ struct request *rq; /* copy of request */
+ void *special; /* valid_t generally */
+} ide_task_t;
+
typedef struct hwgroup_s {
/* irq handler, if active */
ide_startstop_t (*handler)(ide_drive_t *);
@@ -1189,6 +1202,8 @@
*/
extern void ide_init_drive_cmd (struct request *rq);

+void ide_task_init_flush(ide_drive_t *, ide_task_t *);
+
u64 ide_tf_get_address(struct ata_taskfile *);

/*
@@ -1250,19 +1265,6 @@
* (ide_drive_t *drive, u8 cmd, u8 nsect, u8 feature, u8 sectors, u8 *buf)
*/
extern int ide_wait_cmd(ide_drive_t *, u8, u8, u8, u8, u8 *);
-
-typedef struct ide_task_s {
- struct ata_taskfile tf;
- u16 data;
- ide_reg_valid_t tf_out_flags;
- ide_reg_valid_t tf_in_flags;
- int data_phase;
- int command_type;
- ide_pre_handler_t *prehandler;
- ide_handler_t *handler;
- struct request *rq; /* copy of request */
- void *special; /* valid_t generally */
-} ide_task_t;

extern u32 ide_read_24(ide_drive_t *);

-
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/
\
 
 \ /
  Last update: 2005-03-22 14:10    [W:0.050 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site