On Monday, August 04, 2014 at 04:02:42 PM, Tom Rini wrote:
[...]
> Eeep. That seems very likely to be gotten incorrect by users.
>
> Can we do something like:
> mxc_spi.c:
> __weak int board_map_spi_cs_value(int desired_cs) { return -EINVAL; }
>
> fooboard.c:
> board_map_spi_cs_value(int desired_cs) {
> if (desired_cs == 0)
> return IMX_GPIO_NR(2, 30);
> else
> return -EINVAL;
> }
>
> I think it'll be very bad if the user has to type 'sf probe 0:15872' or
> 'sf probe 15872' since that's a programming detail rather than saying
> bank 2, gpio 30 (which I assume is what IMX_GPIO_NR means).
I'm worried about the user scripts which depend on this original behavior. Do
we
care about compatibility there ?
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
