* [U-Boot] v2011.12 and later do not boot on dreamplug.
2012-01-08 19:49 ` Ian Campbell
@ 2012-01-08 22:54 ` Jason
2012-01-12 8:01 ` Ian Campbell
2012-01-09 10:59 ` [U-Boot] Bug#655102: " Holger Brunck
2012-01-12 16:10 ` [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register Ian Campbell
2 siblings, 1 reply; 13+ messages in thread
From: Jason @ 2012-01-08 22:54 UTC (permalink / raw)
To: u-boot
On Sun, Jan 08, 2012 at 07:49:05PM +0000, Ian Campbell wrote:
> On Sun, 2012-01-08 at 15:30 +0000, Ian Campbell wrote:
> > v2011.12 hangs on boot on my dreamplug with:
> >
> > U-Boot 2011.12-dirty (Jan 08 2012 - 15:16:44)
> > Marvell-DreamPlug
> >
> > SoC: Kirkwood 88F6281_A1
> > DRAM: 512 MiB
> > WARNING: Caches not enabled
> >
> > I bisected it down to:
> > commit ca4b55800ed74207c35271bf7335a092d4955416
> > Author: Heiko Schocher <hs@denx.de>
> > Date: Wed Nov 9 20:06:23 2011 +0000
> >
> > arm, arm926ejs: always do cpu critical inits
> >
> > always do the cpu critical inits in cpu_init_crit,
> > and only jump to lowlevel_init, if CONFIG_SKIP_LOWLEVEL_INIT
> > is not defined.
> >
> > Signed-off-by: Heiko Schocher <hs@denx.de>
> > Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>
> > Cc: Wolfgang Denk <hs@denx.de>
> > Cc: Sandeep Paulraj <s-paulraj@ti.com>
> > Cc: Tom Rini <tom.rini@gmail.com>
> > Cc: Christian Riesch <christian.riesch@omicron.at>
> >
> > and in particular the line
> > orr r0, r0, #0x00000002 /* set bit 2 (A) Align */
> >
> > Commenting out just that one line causes things to work. Is the error
> > setting this bit or is it that there is a bug somewhere else which
> > prevents us setting this bit?
>
> Seem that the latter is the case. The following fixes it for me.
>
> (BTW, my mails to u-boot@ seem to be getting moderated, even though I am
> subscribed, is that normal?)
>
> 8<-------------------------------------------------------------
>
> From d6bc6e1a4f49314577e0be9af3cd020078593fdb Mon Sep 17 00:00:00 2001
> From: Ian Campbell <ijc@hellion.org.uk>
> Date: Sun, 8 Jan 2012 19:44:38 +0000
> 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 <ijc@hellion.org.uk>
> ---
> 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) {
> --
> 1.7.5.4
Acked-By: Jason Cooper <u-boot@lakedaemon.net>
Dammit. My apologies. I should've caught that. Thanks for the patch.
Jason.
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] v2011.12 and later do not boot on dreamplug.
2012-01-08 22:54 ` Jason
@ 2012-01-12 8:01 ` Ian Campbell
2012-01-12 15:11 ` Jason
0 siblings, 1 reply; 13+ messages in thread
From: Ian Campbell @ 2012-01-12 8:01 UTC (permalink / raw)
To: u-boot
On Sun, 2012-01-08 at 17:54 -0500, Jason wrote:
> On Sun, Jan 08, 2012 at 07:49:05PM +0000, Ian Campbell wrote:
> > On Sun, 2012-01-08 at 15:30 +0000, Ian Campbell wrote:
> > 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) {
> > --
> > 1.7.5.4
>
> Acked-By: Jason Cooper <u-boot@lakedaemon.net>
>
> Dammit. My apologies. I should've caught that. Thanks for the patch.
No problem, was interesting to dig into u-boot for the first time!
Did I get the CC list right to cover whoever will eventually apply this
patch?
You are also working on upstreaming dreamplug support into Linux, is
that right? What is the current state of the art? I'm currently using a
3.1 kernel + patches from
http://people.debian.org/~zumbi/dreamplug/patches/ . (BTW feel free to
reply off list or switch to e.g. linux-arm-kernel since I guess this is
strict speaking off-topic here).
Thanks,
Ian.
--
Ian Campbell
Revolution, n.:
A form of government abroad.
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] v2011.12 and later do not boot on dreamplug.
2012-01-12 8:01 ` Ian Campbell
@ 2012-01-12 15:11 ` Jason
0 siblings, 0 replies; 13+ messages in thread
From: Jason @ 2012-01-12 15:11 UTC (permalink / raw)
To: u-boot
Ian,
On Thu, Jan 12, 2012 at 08:01:45AM +0000, Ian Campbell wrote:
> On Sun, 2012-01-08 at 17:54 -0500, Jason wrote:
> > On Sun, Jan 08, 2012 at 07:49:05PM +0000, Ian Campbell wrote:
> > > On Sun, 2012-01-08 at 15:30 +0000, Ian Campbell wrote:
> > > 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) {
> > > --
> > > 1.7.5.4
> >
> > Acked-By: Jason Cooper <u-boot@lakedaemon.net>
> >
> > Dammit. My apologies. I should've caught that. Thanks for the patch.
>
> No problem, was interesting to dig into u-boot for the first time!
Well, at least my mistake created another u-boot hacker. ;-)
> Did I get the CC list right to cover whoever will eventually apply this
> patch?
Meh, I've been pondering this. The merge window is open right now, go
ahead and resubmit according to [1]. Make sure to put the Message-Id:
of the original patch email you sent as an In-Reply-To:, and mention in
the changelog that this 'v2' follows [1] for patch submissions.
Also, make sure Prafulla and Albert are in the CC if they aren't
already.
I think the original will get lost in the shuffle because it doesn't
have [PATCH 1/1] in the subject line.
> You are also working on upstreaming dreamplug support into Linux, is
> that right? What is the current state of the art? I'm currently using a
> 3.1 kernel + patches from
> http://people.debian.org/~zumbi/dreamplug/patches/ . (BTW feel free to
> reply off list or switch to e.g. linux-arm-kernel since I guess this is
> strict speaking off-topic here).
I use this [2] for 3.1, since it uses the same nor flash partitioning as
u-boot does. This way, you could edit the u-boot config from userspace
or upgrade u-boot from userspace.
hth,
Jason.
[1] http://www.denx.de/wiki/U-Boot/Patches
[2] http://permalink.gmane.org/gmane.linux.ports.arm.kernel/143116
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] Bug#655102: v2011.12 and later do not boot on dreamplug.
2012-01-08 19:49 ` Ian Campbell
2012-01-08 22:54 ` Jason
@ 2012-01-09 10:59 ` Holger Brunck
2012-01-12 16:10 ` [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register Ian Campbell
2 siblings, 0 replies; 13+ messages in thread
From: Holger Brunck @ 2012-01-09 10:59 UTC (permalink / raw)
To: u-boot
On 01/08/2012 09:00 PM, Ian Campbell wrote:
> On Sun, 2012-01-08 at 15:30 +0000, Ian Campbell wrote:
>> v2011.12 hangs on boot on my dreamplug with:
>>
>> U-Boot 2011.12-dirty (Jan 08 2012 - 15:16:44)
>> Marvell-DreamPlug
>>
>> SoC: Kirkwood 88F6281_A1
>> DRAM: 512 MiB
>> WARNING: Caches not enabled
>>
>> I bisected it down to:
>> commit ca4b55800ed74207c35271bf7335a092d4955416
>> Author: Heiko Schocher <hs@denx.de>
>> Date: Wed Nov 9 20:06:23 2011 +0000
>>
>> arm, arm926ejs: always do cpu critical inits
>>
>> always do the cpu critical inits in cpu_init_crit,
>> and only jump to lowlevel_init, if CONFIG_SKIP_LOWLEVEL_INIT
>> is not defined.
>>
>> Signed-off-by: Heiko Schocher <hs@denx.de>
>> Cc: Albert ARIBAUD <albert.u.boot@aribaud.net>
>> Cc: Wolfgang Denk <hs@denx.de>
>> Cc: Sandeep Paulraj <s-paulraj@ti.com>
>> Cc: Tom Rini <tom.rini@gmail.com>
>> Cc: Christian Riesch <christian.riesch@omicron.at>
>>
>> and in particular the line
>> orr r0, r0, #0x00000002 /* set bit 2 (A) Align */
>>
>> Commenting out just that one line causes things to work. Is the error
>> setting this bit or is it that there is a bug somewhere else which
>> prevents us setting this bit?
>
> Seem that the latter is the case. The following fixes it for me.
>
> (BTW, my mails to u-boot@ seem to be getting moderated, even though I am
> subscribed, is that normal?)
>
> 8<-------------------------------------------------------------
>
> From d6bc6e1a4f49314577e0be9af3cd020078593fdb Mon Sep 17 00:00:00 2001
> From: Ian Campbell <ijc@hellion.org.uk>
> Date: Sun, 8 Jan 2012 19:44:38 +0000
> 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 <ijc@hellion.org.uk>
> ---
> 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) {
we had also SPI problems on our km_kirkwood boards with u-boot v2011.12. This
patch fixes it. Thanks! So:
Tested-By: Holger Brunck <holger.brunck@keymile.com>
Regards
Holger
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register
2012-01-08 19:49 ` Ian Campbell
2012-01-08 22:54 ` Jason
2012-01-09 10:59 ` [U-Boot] Bug#655102: " Holger Brunck
@ 2012-01-12 16:10 ` Ian Campbell
2012-01-12 16:22 ` Prafulla Wadaskar
2 siblings, 1 reply; 13+ messages in thread
From: Ian Campbell @ 2012-01-12 16:10 UTC (permalink / raw)
To: u-boot
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 <ijc@hellion.org.uk>
Acked-By: Jason Cooper <u-boot@lakedaemon.net>
Tested-By: Holger Brunck <holger.brunck@keymile.com>
Cc: Prafulla Wadaskar <prafulla@marvell.com>
Cc: Albert ARIBAUD <albert.u.boot@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) {
--
1.7.5.4
^ permalink raw reply related [flat|nested] 13+ messages in thread* [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register
2012-01-12 16:10 ` [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register Ian Campbell
@ 2012-01-12 16:22 ` Prafulla Wadaskar
2012-01-12 16:56 ` Ian Campbell
2012-01-27 16:34 ` Ian Campbell
0 siblings, 2 replies; 13+ messages in thread
From: Prafulla Wadaskar @ 2012-01-12 16:22 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Ian Campbell [mailto:ijc at hellion.org.uk]
> Sent: 12 January 2012 21:40
> To: u-boot at 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 <ijc@hellion.org.uk>
> Acked-By: Jason Cooper <u-boot@lakedaemon.net>
> Tested-By: Holger Brunck <holger.brunck@keymile.com>
> Cc: Prafulla Wadaskar <prafulla@marvell.com>
> Cc: Albert ARIBAUD <albert.u.boot@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
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register
2012-01-12 16:22 ` Prafulla Wadaskar
@ 2012-01-12 16:56 ` Ian Campbell
2012-01-27 16:34 ` Ian Campbell
1 sibling, 0 replies; 13+ messages in thread
From: Ian Campbell @ 2012-01-12 16:56 UTC (permalink / raw)
To: u-boot
On Thu, 2012-01-12 at 08:22 -0800, Prafulla Wadaskar wrote:
>
> > -----Original Message-----
> > From: Ian Campbell [mailto:ijc at hellion.org.uk]
> > Sent: 12 January 2012 21:40
> > To: u-boot at 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 <ijc@hellion.org.uk>
> > Acked-By: Jason Cooper <u-boot@lakedaemon.net>
> > Tested-By: Holger Brunck <holger.brunck@keymile.com>
> > Cc: Prafulla Wadaskar <prafulla@marvell.com>
> > Cc: Albert ARIBAUD <albert.u.boot@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.
Thanks!
Ian.
>
> Regards..
> Prafulla . ..
>
> > --
> > 1.7.5.4
>
>
--
Ian Campbell
Current Noise: Monster Magnet - Cry
A kid'll eat the middle of an Oreo, eventually.
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register
2012-01-12 16:22 ` Prafulla Wadaskar
2012-01-12 16:56 ` Ian Campbell
@ 2012-01-27 16:34 ` Ian Campbell
2012-02-08 9:54 ` Prafulla Wadaskar
1 sibling, 1 reply; 13+ messages in thread
From: Ian Campbell @ 2012-01-27 16:34 UTC (permalink / raw)
To: u-boot
On Thu, 2012-01-12 at 08:22 -0800, Prafulla Wadaskar wrote:
>
> > -----Original Message-----
> > From: Ian Campbell [mailto:ijc at hellion.org.uk]
> > Sent: 12 January 2012 21:40
> > To: u-boot at 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 <ijc@hellion.org.uk>
> > Acked-By: Jason Cooper <u-boot@lakedaemon.net>
> > Tested-By: Holger Brunck <holger.brunck@keymile.com>
> > Cc: Prafulla Wadaskar <prafulla@marvell.com>
> > Cc: Albert ARIBAUD <albert.u.boot@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.
Ping?
This patch doesn't seem to be in git yet.
Ian.
--
Ian Campbell
Current Noise: Iron Monkey - 9 Joint Spiritual Whip
"Were there no women, men might live like gods."
-- Thomas Dekker
^ permalink raw reply [flat|nested] 13+ messages in thread* [U-Boot] [PATCH] kirkwood_spi: correct access to irq_mask register
2012-01-27 16:34 ` Ian Campbell
@ 2012-02-08 9:54 ` Prafulla Wadaskar
2012-02-09 6:30 ` Ian Campbell
0 siblings, 1 reply; 13+ messages in thread
From: Prafulla Wadaskar @ 2012-02-08 9:54 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Ian Campbell [mailto:ijc at hellion.org.uk]
> Sent: 27 January 2012 22:04
> To: Prafulla Wadaskar
> Cc: u-boot at lists.denx.de; Albert ARIBAUD
> Subject: RE: [PATCH] kirkwood_spi: correct access to irq_mask register
>
> On Thu, 2012-01-12 at 08:22 -0800, Prafulla Wadaskar wrote:
> >
> > > -----Original Message-----
> > > From: Ian Campbell [mailto:ijc at hellion.org.uk]
> > > Sent: 12 January 2012 21:40
> > > To: u-boot at 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 <ijc@hellion.org.uk>
> > > Acked-By: Jason Cooper <u-boot@lakedaemon.net>
> > > Tested-By: Holger Brunck <holger.brunck@keymile.com>
> > > Cc: Prafulla Wadaskar <prafulla@marvell.com>
> > > Cc: Albert ARIBAUD <albert.u.boot@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.
>
> Ping?
>
> This patch doesn't seem to be in git yet.
Applied to u-boot-marvell.git master branch
Regards..
Prafulla . . .
^ permalink raw reply [flat|nested] 13+ messages in thread