please don't post patches upstream with '[u-boot-release]' in the
subject.
On Fri, 10 Aug 2012 17:01:33 -0500
Timur Tabi <ti...@freescale.com> wrote:
> +enum srds_prtcl serdes_device_from_fm_port(enum fm_port port)
> +{
> + switch (port) {
> + case FM1_DTSEC1:
> + return SGMII_FM1_DTSEC1;
> + case FM1_DTSEC2:
> + return SGMII_FM1_DTSEC2;
> + case FM1_DTSEC3:
> + return SGMII_FM1_DTSEC3;
> + case FM1_DTSEC4:
> + return SGMII_FM1_DTSEC4;
> + case FM1_DTSEC5:
> + return SGMII_FM1_DTSEC5;
> + case FM1_10GEC1:
> + return XAUI_FM1;
> + case FM2_DTSEC1:
> + return SGMII_FM2_DTSEC1;
> + case FM2_DTSEC2:
> + return SGMII_FM2_DTSEC2;
> + case FM2_DTSEC3:
> + return SGMII_FM2_DTSEC3;
> + case FM2_DTSEC4:
> + return SGMII_FM2_DTSEC4;
> + case FM2_DTSEC5:
> + return SGMII_FM2_DTSEC5;
> + case FM2_10GEC1:
> + return XAUI_FM2;
> + default:
> + return NONE;
> + }
> +}
shouldn't this be a static const array lookup?
Kim
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
