From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756856AbYIOWLk (ORCPT ); Mon, 15 Sep 2008 18:11:40 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753006AbYIOWL2 (ORCPT ); Mon, 15 Sep 2008 18:11:28 -0400 Received: from h155.mvista.com ([63.81.120.155]:65131 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751795AbYIOWL1 (ORCPT ); Mon, 15 Sep 2008 18:11:27 -0400 Message-ID: <48CEDD89.5060107@ru.mvista.com> Date: Tue, 16 Sep 2008 02:11:21 +0400 From: Sergei Shtylyov User-Agent: Thunderbird 2.0.0.16 (Windows/20080708) MIME-Version: 1.0 To: Bartlomiej Zolnierkiewicz Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/5] ide: ->ide_dma_clear_irq() -> ->clear_irq() References: <200808192031.40288.bzolnier@gmail.com> <48AB478B.5040209@ru.mvista.com> In-Reply-To: <48AB478B.5040209@ru.mvista.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, I wrote: >> * Rename ->ide_dma_clear_irq method to ->clear_irq >> and move it from ide_hwif_t to struct ide_port_ops. >> >> * Move ->waiting_for_dma check inside ->clear_irq method. >> >> * Move ->dma_base check inside ->clear_irq method. >> >> piix.c: >> * Add ich_port_ops and remove init_hwif_ich() wrapper. >> >> There should be no functional changes caused by this patch. >> > > Good. I think it's worth implementing this method in at least > cmd64x.c which actually reads the IDE interrupt latch bits > (independent from the DMA interrupt status) in the dma_test_irq() > methods but never clears them, so the latches may reflect a > non-current state of the IDE interrupt... I forgot that it does clear them in its dma_end() methods (which I myself have reworked :-). It seems however that at least for SFF-8038 compatibles, it makes sense to leave it that way since INTRQ might be asserted while BMIDE interrupt bit is not, so the interrupt latch would need clearing even on DMA timeout... > It may also be worth considering turning this method into > test-and-clear, so that we can get the actual IDE interrupt state on > the chips that implement this... Probably might add the test_irq() method to be called on !hwif->waiting_for_dma. Cleraing the status at once seems impractical... >> Signed-off-by: Bartlomiej Zolnierkiewicz > > Acked-by: Sergei Shtylyov Not feeling sure about this patch -- ->waiting_for_dma probably should've been left where it was... MBR, Sergei