Hi Stefano and Alexander, On Sun, May 24, 2015 at 4:43 AM, Stefano Babic <sba...@denx.de> wrote:
>> Wouldn't it be better to just enable CONFIG_CMD_GPIO and then change the >> boot-script in the config to something like "if gpio ..." instead of >> adding something special? >> >> Assuming the gpio command works on imx, which I haven't tested or looked >> up. > > gpio works - this is really a good idea, moving the check into the > script. Fabio, what do you mind ? I think the idea is good, thanks. I wanted to keep consistency with the mx6cuboxi implementation (which was based on TI's implementation suggested by Tom during the review of the mx6cuboxi patches). Also, the gpio script idea would work fine for selecting the dtb file, but not inside checkboard() function, where I print the board revision name. Other aspect I thought is the fact that in case we have another revision of the board in the future, I think that C code is more flexible for handling it. So I like the idea of gpio script, but I would prefer to keep the current implementation if possible due to the reasons stated above. Thanks for the suggestion. Fabio Estevam _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
