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

[Frans Pop <elendil@planet.nl>]

On Wednesday 11 March 2009, Mark Lord wrote:
> > +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.

Fun, as it's also used in mv6_ops and mv_iie_ops. That's where I copied it 
from. I really don't have anywhere near the C fu to think up such 
constructs all by myself :-D

What it does (I've been assuming) is include any ops defined in the struct 
that is being referred to. So mv6xxx_iie_ops gets all the ops defined for 
mv6xxx_ops plus mv_iie_enable_led_blink.

> (or did I miss an update to the C programming language at some point? 
> :)
>

You're still way ahead of me.