Frans Pop wrote:
..
@@ -480,11 +487,19 @@ struct mv_hw_ops {
unsigned int port);
void (*enable_leds)(struct mv_host_priv *hpriv, void __iomem *mmio);
void (*read_preamp)(struct mv_host_priv *hpriv, int idx,
- void __iomem *mmio);
+ void __iomem *mmio);
int (*reset_hc)(struct mv_host_priv *hpriv, void __iomem *mmio,
unsigned int n_hc);
void (*reset_flash)(struct mv_host_priv *hpriv, void __iomem *mmio);
void (*reset_bus)(struct ata_host *host, void __iomem *mmio);
+ void (*enable_led_blink)(struct mv_host_priv *hpriv,
+ void __iomem *mmio, int enable_blink);
+
+ /*
+ * ->inherits must be the last field and all the preceding
+ * fields must be pointers.
+ */
+ const struct mv_hw_ops *inherits;
};
..
+static const struct mv_hw_ops mv6xxx_iie_ops = {
+ .inherits = &mv6xxx_ops,
+ .enable_led_blink = mv_iie_enable_led_blink,
+};
+
I'm afraid I just don't understand the purpose of "inherits" above.
This field appears to never be referenced anywhere.
(or did I miss an update to the C programming language at some point? :)
???
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
