This is the mail archive of the
ecos-patches@sources.redhat.com
mailing list for the eCos project.
Re: [patch] xscale irq-table
- From: "Daniel Schmidt" <Daniel dot Schmidt3 at gmx dot de>
- To: Andrew Lunn <andrew at lunn dot ch>
- Cc: ecos-patches at sources dot redhat dot com
- Date: Thu, 30 Sep 2004 11:30:29 +0200 (MEST)
- Subject: Re: [patch] xscale irq-table
- References: <20040930090417.GD25373@lunn.ch>
Hello,
after reading a littlebit more in the code:
the GPIOS 81..85 are missing; I'll do this (I've used offsets only GPIOx+i)
But then the the problem is not solved, which was the original start for my
patch.(http://ecos.sourceware.org/ml/ecos-discuss/2004-08/msg00099.html)
In:
arm/arch/current/include/hal_intr.h:
HAL_INTERRUPT_ATTACH(_vector_,_isr_,_data_,_object_)
...
hal_interrupt_handlers[_vector_]=...
vector is the CYGNUM_HAL_INTERRUPT_GPIO81; this means it is with the offset
CYGNUM_HAL_ISR_MIN (=8)
I don't know how to express but:
hal_interrupt_handlers is an arry:
0..(CYGNUM_HAL_ISR_MAX-CYGNUM_HAL_ISR_MIN)
but what it should be an array: CYGNUM_HAL_ISR_MIN..CYGNUM_HAL_ISR_MAX
bye
daniel
> On Wed, Sep 29, 2004 at 03:57:58PM +0200, Daniel Schmidt wrote:
> > hello,
> >
> > as written before the interrupt-vector-table in
> > arm/arch/current/include/hal_intr.h:
> hal_interrupt_{handlers|data|objects}
> > is to short for the pxa2x0. So you can not append interrupts for the
> upper
> > 8 GPIOs (78..85).
> >
> > I've attached the patch; please apply.
>
> This is not the correct way to do it. You should add the extra
>
> #define CYGNUM_HAL_INTERRUPT_GPIO81 (96+17)
> ...
> #define CYGNUM_HAL_INTERRUPT_GPIO85 (95+...
>
> Then change
>
> #define CYGNUM_HAL_ISR_MAX
>
> Please could you send a patch doing it this way.
>
> Thanks
> Andrew
>
--
+++ GMX DSL Premiumtarife 3 Monate gratis* + WLAN-Router 0,- EUR* +++
Clevere DSL-Nutzer wechseln jetzt zu GMX: http://www.gmx.net/de/go/dsl