lkml.org 
[lkml]   [2008]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/5] ide: ->ide_dma_clear_irq() -> ->clear_irq()
On Tue, Sep 16, 2008 at 01:16:54PM +0400, Sergei Shtylyov wrote:
> >>>>Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> >>>>
> >>>Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
> >>>
> >> Not feeling sure about this patch -- ->waiting_for_dma probably
> >>should've been left where it was...
> >>
> >
> >Well, it doesn't change behavior and I think having ->clear_irq method
> >independent from the transfer mode is a preffered approach.
> >
>
> But its implementations will have to depend on it anyway. And
> clearing the IDE interrupt in general already depends on the transfer
> mode -- the BMIDE interrupt which is a (delayed) reflection of INTRQ is
> cleared implicitly by the dma_end() method -- except in at least sgiioc4
> driver.
> BTW, sgiioc4 seems another candidate for clear_irq() implementation
> -- currently clearing is done implicitly by the read_status() method (I
> don't quite understand why it clears DMA error interrupt there).
> However, since there's no documentation, I'm not sure how the IDE
> interrupt is latched by IOC4.

The person who originally wrote sgiioc4 has been gone for a long time.

I'm guessing that the DMA error is cleared there, because any DMA error
status has already been processed above, and it's easier to clear
unconditionally than to test and clear, and there's no ill effect to
clearing something that's already clear.

jeremy


\
 
 \ /
  Last update: 2008-11-12 03:05    [W:0.155 / U:1.708 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site