Re: [PATCH,v2][1/2] sata-mv: enable HDD led blinking when NCQ is active for GenIIe

[Mark Lord <liml@rtr.ca>]

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?  :)

???