AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20070708125047.71bf4d83@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<maxim.kozlovsky@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	0	BB375AF679D4A34E9CA8DFA650E2B04E046A8A5B@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Sun, 8 Jul 2007 12:51:20 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Maxim Kozlovsky" <maxim.kozlovsky@onstor.com>
Subject: Re: please review
Message-ID: <20070708125120.71ee1286@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E046A8A5B@onstor-exch02.onstor.net>
References: <20070705175356.5ff0f5df@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E046A87CA@onstor-exch02.onstor.net>
	<20070706145649.5bb21b56@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E046A8A54@onstor-exch02.onstor.net>
	<20070708122802.215789a2@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E046A8A59@onstor-exch02.onstor.net>
	<20070708123638.58cc3587@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E046A8A5B@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Sun, 8 Jul 2007 12:47:18 -0700 "Maxim Kozlovsky"
<maxim.kozlovsky@onstor.com> wrote:

> This works:
> 
> 	else if (pending & STATUSF_IP2)
>         do_IRQ(2);
> 	else if (pending & STATUSF_IP3)
>         ll_bc_mv64440_irq();
> 
> This does not:
> 
> 	else if (pending & STATUSF_IP2)
>         ll_bc_mv64440_irq();
> 	else if (pending & STATUSF_IP3)
>         do_IRQ(3);

are you also changing the corresponding line in arch_init_irq?

> 
> >-----Original Message-----
> >From: Andy Sharp
> >Sent: Sunday, July 08, 2007 12:37 PM
> >To: Maxim Kozlovsky
> >Subject: Re: please review
> >
> >On Sun, 8 Jul 2007 12:33:56 -0700 "Maxim Kozlovsky"
> ><maxim.kozlovsky@onstor.com> wrote:
> >
> >> Ok, 3 as in the original. I've tried you modified code and I did
> >> not get the interrupts for the mgmt bus driver.
> >
> >OK, so before I go a-changin' it, did you try it with 3? ~:^)
> >
> >> >-----Original Message-----
> >> >From: Andy Sharp
> >> >Sent: Sunday, July 08, 2007 12:28 PM
> >> >To: Maxim Kozlovsky
> >> >Subject: Re: please review
> >> >
> >> >On Sun, 8 Jul 2007 10:56:28 -0700 "Maxim Kozlovsky"
> >> ><maxim.kozlovsky@onstor.com> wrote:
> >> >
> >> >> Irq.c:
> >> >>
> >> >> You are using the wrong line, should be 2 as in the original.
> >> >
> >> >.
> >> >.
> >> >.
> >> >	else if (pending & STATUSF_IP2)
> >> >		ll_bc_mv64440_irq();
> >> >
> >> >.
> >> >.
> >> >.
> >> >
> >> >	/* set up the cascading interrupts */
> >> >	setup_irq(2, &cascade_mv64440);
> >> >.
> >> >.
> >> >.
> >> >
> >> >It looks to me like I am using 2.  ?
