From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH] pci: use pci_ioremap_bar() in drivers/ide Date: Wed, 22 Oct 2008 21:40:37 +0400 Message-ID: <48FF6595.5040203@ru.mvista.com> References: <20081020214830.1586704c@infradead.org> <48FDA8BF.20206@ru.mvista.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from gateway-1237.mvista.com ([63.81.120.155]:44570 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1758922AbYJVRk7 (ORCPT ); Wed, 22 Oct 2008 13:40:59 -0400 In-Reply-To: <48FDA8BF.20206@ru.mvista.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Arjan van de Ven Cc: linux-ide@vger.kernel.org, Bartlomiej Zolnierkiewicz Hello, I wrote: >> Use the newly introduced pci_ioremap_bar() function in drivers/ide. >> pci_ioremap_bar() just takes a pci device and a bar number, with the goal >> of making it really hard to get wrong, while also having a central place >> to stick sanity checks. >> Signed-off-by: Arjan van de Ven > Acked-by: Sergei Shtylyov No, I'm takign back my ACK. Since the patch intends to address all drivers/ide/, it's clearly no compolete -- siimage.c and scc_pata.c beg for the alike change (it will even permit to kill 4 local variables in the latter driver). MBR, Sergei