On Mon, 10 Feb 2014 09:11:56 +0100 (CET) Michal Kubecek <mkubecek@xxxxxxx> wrote: > +static const char *slave_state_tbl[] = { > + "active", > + "backup", > + NULL > +}; > + > +static const char *slave_mii_status_tbl[] = { > + "up", > + "going_down", > + "down", > + "going_back", > + NULL, > +}; > + Is there some correlation of these states to values in a header file. Something like static const char *slave_mii_status_tbl[] = { [BOND_LINK_UP] = "active", [BOND_LINK_FAIL] = "fail", [BOND_LINK_DOWN] = "down", [BOND_LINK_BACK] = "back", }; And don't use null terminated arrays, use ARRAY_SIZE() instead. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html