On Wednesday 25 January 2012 10:10:18 Matthias Fuchs wrote: > Can you also add a config option for the SPI bus number? I think these > two need to handled in the same patch. > > So you could add this stuff: > > static int do_spi_flash_probe(int argc, char * const argv[]) > { > +#ifdef CONFIG_SF_DEFAULT_BUS > + unsigned int bus = CONFIG_SF_DEFAULT_BUS; > +#else > unsigned int bus = 0; > +#endif
move the default logic to the top of the file like the existing CONFIG_SF_DEFAULT_SPEED, then the code later on only needs to do: unsigned int bus = CONFIG_SF_DEFAULT_BUS; otherwise, this suggestion sounds good too -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot