From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Subject: Re: [PATCH 0/4] drivers/ata: add low-level I/O calls Date: Fri, 12 Jan 2007 10:24:06 +0000 Message-ID: <20070112102406.6eb9374f@localhost.localdomain> References: <200701120958.l0C9wICB019343@toshiba.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from outpipe-village-512-1.bc.nu ([81.2.110.250]:53136 "EHLO lxorguk.ukuu.org.uk" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1161048AbXALKNk (ORCPT ); Fri, 12 Jan 2007 05:13:40 -0500 In-Reply-To: <200701120958.l0C9wICB019343@toshiba.co.jp> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Akira Iguchi Cc: linux-ide@vger.kernel.org, linuxppc-dev@ozlabs.org, kou.ishizaki@toshiba.co.jp, benh@kernel.crashing.org, arnd@arndb.de, paulus@samba.org, linas@austin.ibm.com On Fri, 12 Jan 2007 19:00:45 +0900 Akira Iguchi wrote: > Dear everyone, > > This is the patchset (based on 2.6.20-rc4) to add low-level I/O calls > which access the taskfile registers. The idea comes from drivers/ide > IN*/OUT* calls. I think this is the wrong approach. The existing code provides methods for things like "read the status", "write a taskfile" and this is the style that should be kept. Alan From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lxorguk.ukuu.org.uk (unknown [81.2.110.250]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTP id BAA3BDDF4B for ; Fri, 12 Jan 2007 21:12:59 +1100 (EST) Date: Fri, 12 Jan 2007 10:24:06 +0000 From: Alan To: Akira Iguchi Subject: Re: [PATCH 0/4] drivers/ata: add low-level I/O calls Message-ID: <20070112102406.6eb9374f@localhost.localdomain> In-Reply-To: <200701120958.l0C9wICB019343@toshiba.co.jp> References: <200701120958.l0C9wICB019343@toshiba.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Cc: arnd@arndb.de, linuxppc-dev@ozlabs.org, linux-ide@vger.kernel.org, paulus@samba.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 12 Jan 2007 19:00:45 +0900 Akira Iguchi wrote: > Dear everyone, > > This is the patchset (based on 2.6.20-rc4) to add low-level I/O calls > which access the taskfile registers. The idea comes from drivers/ide > IN*/OUT* calls. I think this is the wrong approach. The existing code provides methods for things like "read the status", "write a taskfile" and this is the style that should be kept. Alan