> -----Original Message----- > From: Ian Campbell [mailto:i...@hellion.org.uk] > Sent: 12 January 2012 21:40 > To: u-boot@lists.denx.de > Cc: Ian Campbell; Prafulla Wadaskar; Albert ARIBAUD > Subject: [PATCH] kirkwood_spi: correct access to irq_mask register > > Problem appears to have been present since day one but masked because > alignment > aborts were not enabled. ca4b55800ed7 "arm, arm926ejs: always do cpu > critical > inits" turned on alignment aborts and uncovered this latent problem. > > Signed-off-by: Ian Campbell <i...@hellion.org.uk> > Acked-By: Jason Cooper <u-b...@lakedaemon.net> > Tested-By: Holger Brunck <holger.bru...@keymile.com> > Cc: Prafulla Wadaskar <prafu...@marvell.com> > Cc: Albert ARIBAUD <albert.u.b...@aribaud.net> > --- > v2: Resending per patch submission guidelines. > --- > drivers/spi/kirkwood_spi.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/spi/kirkwood_spi.c b/drivers/spi/kirkwood_spi.c > index dfe542d..db8ba8b 100644 > --- a/drivers/spi/kirkwood_spi.c > +++ b/drivers/spi/kirkwood_spi.c > @@ -66,7 +66,7 @@ struct spi_slave *spi_setup_slave(unsigned int bus, > unsigned int cs, > debug("data = 0x%08x \n", data); > > writel(KWSPI_SMEMRDIRQ, &spireg->irq_cause); > - writel(KWSPI_IRQMASK, spireg->irq_mask); > + writel(KWSPI_IRQMASK, &spireg->irq_mask); > > /* program mpp registers to select SPI_CSn */ > if (cs) {
Hi Ian Thanks for the patch (resend :-) ) Actually it was in my pipeline. I will pull this ASAP as well as other patches in my plate. Regards.. Prafulla . .. > -- > 1.7.5.4 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot