On Mon, 2015-08-17 at 10:31 +0200, Hans de Goede wrote:
> Hi,
>
> On 17-08-15 10:29, Ian Campbell wrote:
> > On Sat, 2015-08-15 at 22:02 +0200, Hans de Goede wrote:
> > > Other then having a few less chip-select lines the nand
> > > controller
> > > on sun4i, sun5i and sun7i is identical.
> > >
> > > Signed-off-by: Hans de Goede <hdego...@redhat.com>
> > > ---
> > > board/sunxi/board.c | 12 +++++++++---
> > > drivers/mtd/nand/Kconfig | 4 ++--
> > > 2 files changed, 11 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/board/sunxi/board.c b/board/sunxi/board.c
> > > index 1ebd0a4..d411e96 100644
> > > --- a/board/sunxi/board.c
> > > +++ b/board/sunxi/board.c
> > > @@ -112,13 +112,19 @@ int dram_init(void)
> > > static void nand_pinmux_setup(void)
> > > {
> > > unsigned int pin;
> > > - for (pin = SUNXI_GPC(0); pin <= SUNXI_GPC(6); pin++)
> > > - sunxi_gpio_set_cfgpin(pin, SUNXI_GPC_NAND);
> > >
> > > - for (pin = SUNXI_GPC(8); pin <= SUNXI_GPC(22); pin++)
> > > + for (pin = SUNXI_GPC(0); pin <= SUNXI_GPC(19); pin++)
> >
> > This looks to have added GPC(7) to all platforms, was that
> > intentional?
>
> Yes, as GPC7 is a nand pin on all platforms, no idea why the original
> code was skipping it. I should probably mention this in the commit
> msg though.
Please, then: Acked-by: Ian Campbell <i...@hellion.org.uk>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
