libata: reorganize ata_port_operations
authorTejun Heo <htejun@gmail.com>
Tue, 25 Mar 2008 03:22:47 +0000 (12:22 +0900)
committerJeff Garzik <jgarzik@redhat.com>
Thu, 17 Apr 2008 19:44:16 +0000 (15:44 -0400)
Over the time, ops in ata_port_operations has become a bit confusing.
Reorganize.  SFF/BMDMA ops are separated into separate a group as they
will be taken out of ata_port_operations later.

Signed-off-by: Tejun Heo <htejun@gmail.com>
include/linux/libata.h

index bc60132c7d339ffb0d7651295774b2b1a86bc72e..9476a479691b8ee6728f1da0d1d579b18c173f76 100644 (file)
@@ -665,69 +665,74 @@ struct ata_port {
 };
 
 struct ata_port_operations {
-       void (*dev_config) (struct ata_device *);
+       /*
+        * Command execution
+        */
+       int  (*qc_defer)(struct ata_queued_cmd *qc);
+       int  (*check_atapi_dma)(struct ata_queued_cmd *qc);
+       void (*qc_prep)(struct ata_queued_cmd *qc);
+       unsigned int (*qc_issue)(struct ata_queued_cmd *qc);
 
-       void (*set_piomode) (struct ata_port *, struct ata_device *);
-       void (*set_dmamode) (struct ata_port *, struct ata_device *);
-       unsigned long (*mode_filter) (struct ata_device *, unsigned long);
+       /*
+        * Configuration and exception handling
+        */
+       int  (*cable_detect)(struct ata_port *ap);
+       unsigned long (*mode_filter)(struct ata_device *dev, unsigned long xfer_mask);
+       void (*set_piomode)(struct ata_port *ap, struct ata_device *dev);
+       void (*set_dmamode)(struct ata_port *ap, struct ata_device *dev);
+       int  (*set_mode)(struct ata_link *link, struct ata_device **r_failed_dev);
 
-       void (*tf_load) (struct ata_port *ap, const struct ata_taskfile *tf);
-       void (*tf_read) (struct ata_port *ap, struct ata_taskfile *tf);
+       void (*dev_config)(struct ata_device *dev);
 
-       void (*exec_command)(struct ata_port *ap, const struct ata_taskfile *tf);
+       void (*freeze)(struct ata_port *ap);
+       void (*thaw)(struct ata_port *ap);
+       void (*error_handler)(struct ata_port *ap);
+       void (*post_internal_cmd)(struct ata_queued_cmd *qc);
+
+       /*
+        * Optional features
+        */
+       int  (*scr_read)(struct ata_port *ap, unsigned int sc_reg, u32 *val);
+       int  (*scr_write)(struct ata_port *ap, unsigned int sc_reg, u32 val);
+       void (*pmp_attach)(struct ata_port *ap);
+       void (*pmp_detach)(struct ata_port *ap);
+       int  (*enable_pm)(struct ata_port *ap, enum link_pm policy);
+       void (*disable_pm)(struct ata_port *ap);
+
+       /*
+        * Start, stop, suspend and resume
+        */
+       int  (*port_suspend)(struct ata_port *ap, pm_message_t mesg);
+       int  (*port_resume)(struct ata_port *ap);
+       int  (*port_start)(struct ata_port *ap);
+       void (*port_stop)(struct ata_port *ap);
+       void (*host_stop)(struct ata_host *host);
+
+       /*
+        * SFF / taskfile oriented ops
+        */
+       void (*dev_select)(struct ata_port *ap, unsigned int device);
        u8   (*check_status)(struct ata_port *ap);
        u8   (*check_altstatus)(struct ata_port *ap);
-       void (*dev_select)(struct ata_port *ap, unsigned int device);
-
-       void (*phy_reset) (struct ata_port *ap); /* obsolete */
-       int  (*set_mode) (struct ata_link *link, struct ata_device **r_failed_dev);
-
-       int (*cable_detect) (struct ata_port *ap);
-
-       int  (*check_atapi_dma) (struct ata_queued_cmd *qc);
-
-       void (*bmdma_setup) (struct ata_queued_cmd *qc);
-       void (*bmdma_start) (struct ata_queued_cmd *qc);
-
-       unsigned int (*data_xfer) (struct ata_device *dev, unsigned char *buf,
-                                  unsigned int buflen, int rw);
-
-       int (*qc_defer) (struct ata_queued_cmd *qc);
-       void (*qc_prep) (struct ata_queued_cmd *qc);
-       unsigned int (*qc_issue) (struct ata_queued_cmd *qc);
-
-       /* port multiplier */
-       void (*pmp_attach) (struct ata_port *ap);
-       void (*pmp_detach) (struct ata_port *ap);
-
-       /* Error handlers.  ->error_handler overrides ->eng_timeout and
-        * indicates that new-style EH is in place.
+       void (*tf_load)(struct ata_port *ap, const struct ata_taskfile *tf);
+       void (*tf_read)(struct ata_port *ap, struct ata_taskfile *tf);
+       void (*exec_command)(struct ata_port *ap, const struct ata_taskfile *tf);
+       unsigned int (*data_xfer)(struct ata_device *dev, unsigned char *buf,
+                                 unsigned int buflen, int rw);
+       u8   (*irq_on)(struct ata_port *);
+
+       void (*irq_clear)(struct ata_port *);
+       void (*bmdma_setup)(struct ata_queued_cmd *qc);
+       void (*bmdma_start)(struct ata_queued_cmd *qc);
+       void (*bmdma_stop)(struct ata_queued_cmd *qc);
+       u8   (*bmdma_status)(struct ata_port *ap);
+
+       /*
+        * Obsolete
         */
-       void (*eng_timeout) (struct ata_port *ap); /* obsolete */
-
-       void (*freeze) (struct ata_port *ap);
-       void (*thaw) (struct ata_port *ap);
-       void (*error_handler) (struct ata_port *ap);
-       void (*post_internal_cmd) (struct ata_queued_cmd *qc);
-
+       void (*phy_reset)(struct ata_port *ap);
+       void (*eng_timeout)(struct ata_port *ap);
        irq_handler_t irq_handler;
-       void (*irq_clear) (struct ata_port *);
-       u8 (*irq_on) (struct ata_port *);
-
-       int (*scr_read) (struct ata_port *ap, unsigned int sc_reg, u32 *val);
-       int (*scr_write) (struct ata_port *ap, unsigned int sc_reg, u32 val);
-
-       int (*port_suspend) (struct ata_port *ap, pm_message_t mesg);
-       int (*port_resume) (struct ata_port *ap);
-       int (*enable_pm) (struct ata_port *ap, enum link_pm policy);
-       void (*disable_pm) (struct ata_port *ap);
-       int (*port_start) (struct ata_port *ap);
-       void (*port_stop) (struct ata_port *ap);
-
-       void (*host_stop) (struct ata_host *host);
-
-       void (*bmdma_stop) (struct ata_queued_cmd *qc);
-       u8   (*bmdma_status) (struct ata_port *ap);
 };
 
 struct ata_port_info {