Re: [PATCH v7 7/9] ipic: clean up unsupported ack operations

2007-11-29 Thread Kumar Gala

On Oct 19, 2007, at 6:38 AM, Li Yang wrote:

> IPIC controller doesn't support ack operations.  The pending registers
> are read-only.  The patch removes ack operations which are not needed.
>
> Signed-off-by: Li Yang <[EMAIL PROTECTED]>
> ---
> arch/powerpc/sysdev/ipic.c |   40 + 
> +--
> 1 files changed, 2 insertions(+), 38 deletions(-)

applied.

- 
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


RE: [PATCH v7 7/9] ipic: clean up unsupported ack operations

2007-11-30 Thread Li Yang
> -Original Message-
> From: Kumar Gala [mailto:[EMAIL PROTECTED] 
> Sent: Friday, November 30, 2007 8:36 AM
> To: Li Yang
> Cc: linuxppc-dev@ozlabs.org
> Subject: Re: [PATCH v7 7/9] ipic: clean up unsupported ack operations
> 
> 
> On Oct 19, 2007, at 6:38 AM, Li Yang wrote:
> 
> > IPIC controller doesn't support ack operations.  The 
> pending registers 
> > are read-only.  The patch removes ack operations which are 
> not needed.
> >
> > Signed-off-by: Li Yang <[EMAIL PROTECTED]>
> > ---
> > arch/powerpc/sysdev/ipic.c |   40 + 
> > +--
> > 1 files changed, 2 insertions(+), 38 deletions(-)
> 
> applied.

Hi Kumar,

Please hold on this one.  Actually external interrupts in edge mode need
this ack operation.  But in most cases (level triggered) ack is not
needed.  I will provide an updated patch later on to take care both
trigger modes.  Thanks.

- Leo
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev


Re: [PATCH v7 7/9] ipic: clean up unsupported ack operations

2007-11-30 Thread Kumar Gala

On Nov 30, 2007, at 4:03 AM, Li Yang wrote:

>> -Original Message-
>> From: Kumar Gala [mailto:[EMAIL PROTECTED]
>> Sent: Friday, November 30, 2007 8:36 AM
>> To: Li Yang
>> Cc: linuxppc-dev@ozlabs.org
>> Subject: Re: [PATCH v7 7/9] ipic: clean up unsupported ack operations
>>
>>
>> On Oct 19, 2007, at 6:38 AM, Li Yang wrote:
>>
>>> IPIC controller doesn't support ack operations.  The
>> pending registers
>>> are read-only.  The patch removes ack operations which are
>> not needed.
>>>
>>> Signed-off-by: Li Yang <[EMAIL PROTECTED]>
>>> ---
>>> arch/powerpc/sysdev/ipic.c |   40 +
>>> +--
>>> 1 files changed, 2 insertions(+), 38 deletions(-)
>>
>> applied.
>
> Hi Kumar,
>
> Please hold on this one.  Actually external interrupts in edge mode  
> need
> this ack operation.  But in most cases (level triggered) ack is not
> needed.  I will provide an updated patch later on to take care both
> trigger modes.  Thanks.

Ok, I'll drop this patch.

- k
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev