diff options
author | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-10-10 16:39:19 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2008-10-10 16:39:19 -0400 |
commit | 48fb2688aa67baba373531cc4ed2d9e695983c3f (patch) | |
tree | 9382e5175e43b914fd06bb778cc6f37a4967ab9d /include/linux/ide.h | |
parent | 4dde4492d850a4c9bcaa92e5bd7f4eebe3e2f5ab (diff) |
ide: remove drive->driveid
* Factor out HDIO_[OBSOLETE,GET]_IDENTITY ioctls handling
to ide_get_identity_ioctl().
* Use temporary buffer in ide_get_identity_ioctl() instead
of accessing drive->id directly.
* Add ide_id_to_hd_driveid() inline to convert raw id into
struct hd_driveid format (needed on big-endian).
* Use ide_id_to_hd_driveid() in ide_get_identity_ioctl(),
cleanup ide_fix_driveid() and switch ide to use use raw id.
* Remove no longer needed drive->driveid.
This leaves us with 3 users of struct hd_driveid in tree:
- arch/um/drivers/ubd_kern.c
- drivers/block/xsysace.c
- drivers/usb/storage/isd200.c
While at it:
* Use ata_id_u{32,64}() and ata_id_has_{dma,lba,iordy}() macros.
There should be no functional changes caused by this patch.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'include/linux/ide.h')
-rw-r--r-- | include/linux/ide.h | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/include/linux/ide.h b/include/linux/ide.h index 0c85aff3edf1..e887927e00e6 100644 --- a/include/linux/ide.h +++ b/include/linux/ide.h | |||
@@ -380,11 +380,7 @@ struct ide_drive_s { | |||
380 | struct request *rq; /* current request */ | 380 | struct request *rq; /* current request */ |
381 | struct ide_drive_s *next; /* circular list of hwgroup drives */ | 381 | struct ide_drive_s *next; /* circular list of hwgroup drives */ |
382 | void *driver_data; /* extra driver data */ | 382 | void *driver_data; /* extra driver data */ |
383 | union { | 383 | u16 *id; /* identification info */ |
384 | /* identification info */ | ||
385 | struct hd_driveid *driveid; | ||
386 | u16 *id; | ||
387 | }; | ||
388 | #ifdef CONFIG_IDE_PROC_FS | 384 | #ifdef CONFIG_IDE_PROC_FS |
389 | struct proc_dir_entry *proc; /* /proc/ide/ directory entry */ | 385 | struct proc_dir_entry *proc; /* /proc/ide/ directory entry */ |
390 | struct ide_settings_s *settings;/* /proc/ide/ drive settings */ | 386 | struct ide_settings_s *settings;/* /proc/ide/ drive settings */ |
@@ -1353,8 +1349,7 @@ extern int ide_set_xfer_rate(ide_drive_t *drive, u8 rate); | |||
1353 | 1349 | ||
1354 | static inline int ide_dev_has_iordy(u16 *id) | 1350 | static inline int ide_dev_has_iordy(u16 *id) |
1355 | { | 1351 | { |
1356 | return ((id[ATA_ID_FIELD_VALID] & 2) && | 1352 | return ((id[ATA_ID_FIELD_VALID] & 2) && ata_id_has_iordy(id)) ? 1 : 0; |
1357 | (((struct hd_driveid *)id)->capability & 8)) ? 1 : 0; | ||
1358 | } | 1353 | } |
1359 | 1354 | ||
1360 | static inline int ide_dev_is_sata(u16 *id) | 1355 | static inline int ide_dev_is_sata(u16 *id) |