Hi Wolfgang, On Tue, May 21, 2013 at 6:24 PM, Wolfgang Denk <w...@denx.de> wrote:
> Can we _please_ remove all this code? A _runtime_ check for a _build_ > _time_ _detectable_ situation makes no sense to me. > > For such a misconfiguration, the build should fail. > > Handling this at runtime is the wrong approach. > > > This comment applies for the whole series. For the wandboard file, would the code bellow be better? for (index = 0; index < CONFIG_SYS_FSL_USDHC_NUM; ++index) { switch (index) { case 0: imx_iomux_v3_setup_multiple_pads( usdhc3_pads, ARRAY_SIZE(usdhc3_pads)); usdhc_cfg[0].sdhc_clk = mxc_get_clock(MXC_ESDHC3_CLK); usdhc_cfg[0].max_bus_width = 4; gpio_direction_input(USDHC3_CD_GPIO); break; case 1: imx_iomux_v3_setup_multiple_pads( usdhc1_pads, ARRAY_SIZE(usdhc1_pads)); usdhc_cfg[1].sdhc_clk = mxc_get_clock(MXC_ESDHC_CLK); usdhc_cfg[1].max_bus_width = 4; gpio_direction_input(USDHC1_CD_GPIO); break; } status = fsl_esdhc_initialize(bis, &usdhc_cfg[index]); if (status) return status; } return 0; > > > Actually - are you not surprised that you have to fix the same issue > for all boards? This is a clear indication of duplicated code that > needs to be factored out. I will try factor out. Regards, Fabio Estevam _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot