Hi Mike, On Wed, Oct 26, 2011 at 11:14 PM, Mike Frysinger <vap...@gentoo.org> wrote: > On Wed, Oct 26, 2011 at 18:54, Simon Glass wrote: >> --- /dev/null >> +++ b/drivers/gpio/sandbox.c >> >> +#include <asm/io.h> >> +#include <asm/bitops.h> > > not used > >> +#include <asm/arch/gpio.h> > > should be asm/gpio.h > >> +/* These functions make up what could be a public interface within U-Boot */ > > these are the public interface in u-boot. i wonder why you need the > sandbox_* names at all rather than just the public asm/gpio.h > interfaces. > >> +U_BOOT_CMD( >> + gpio, 5, 1, do_gpio, > > if you implemented the asm/gpio.h interface only, you should be able > to re-use the existing cmd_gpio.c file rather than implementing your > own from scratch. > -mike >
Yes I'm afraid this patch is against a slightly older tree. I rebased it but did not update it, but I will do so before sending it up for real. Thanks for the comments. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot