Kim Phillips wrote:
please don't post patches upstream with '[u-boot-release]' in the subject.
I didn't.
http://lists.denx.de/pipermail/u-boot/2012-August/130618.html
Your mailer is confused. I bcc the u-boot-release mailing list, and I presume your mailer (or our mail server) sent you that copy of the mail instead of the "real" one.
shouldn't this be a static const array lookup?
The compiler should convert it into an array lookup automatically, but I can change it if you insist. Since I don't like writing code that depends on the values of an enum, the array will look like this:
static const enum srds_prtcl srds_table[] = { [FM1_DTSEC1] = SGMII_FM1_DTSEC1, [FM1_DTSEC2] = SGMII_FM1_DTSEC2, [FM1_DTSEC3] = SGMII_FM1_DTSEC3, [FM1_DTSEC4] = SGMII_FM1_DTSEC4, [FM1_DTSEC5] = SGMII_FM1_DTSEC5, [FM1_10GEC1] = XAUI_FM1, [FM2_DTSEC1] = SGMII_FM2_DTSEC1, [FM2_DTSEC2] = SGMII_FM2_DTSEC2, [FM2_DTSEC3] = SGMII_FM2_DTSEC3, [FM2_DTSEC4] = SGMII_FM2_DTSEC4, [FM2_DTSEC5] = SGMII_FM2_DTSEC5, [FM2_10GEC1] = XAUI_FM2, };
if ((port < FM1_DTSEC1) || (port > FM2_10GEC1)) return NONE; else return srds_table[port];
I'm not sure that's an improvement.