From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: sata AHCI controller over non-PCI bus Date: Mon, 10 Aug 2009 13:20:36 +0400 Message-ID: <4A7FE664.5020401@ru.mvista.com> References: <4A7FE245.8020002@ru.mvista.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from h155.mvista.com ([63.81.120.155]:65188 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752856AbZHJJUk (ORCPT ); Mon, 10 Aug 2009 05:20:40 -0400 In-Reply-To: <4A7FE245.8020002@ru.mvista.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: "Subbrathnam, Swaminathan" Cc: "linux-ide@vger.kernel.org" Hello, I wrote: >> I would like to post support for OMAPL138 SATA controller >> that is AHCI 1.1 compliant. OMAPL138 is an ARM926 based platform. >> >> Current "ahci.c" assumes that AHCI is getting supported over PCI by >> default, whereas the SATA AHCI controller on OMAPL138 is connected to >> a non-PCI bus. Since I am in the process of posting the support I >> would like to get the opinion from the community on how to proceed >> on this front. > > Well, you're not the first with this problem, Marvell already had it. Er, not Marvell... Synopsis. :-< >> 2. Re-organize the ahci.c to abstract the AHCI functionalities >> from PCI, non-PCI bus dependencies. >> >> Currently we have implemented Option 1 in our internal tree to >> support SATA on OMAPL138 platform and it works well. Ideally I would >> want to proceed with Option 2 but would like get feedback from the >> community before proceeding in this direction. > > See > http://git.kernel.org/?p=linux/kernel/git/jgarzik/libata-dev.git;a=shortlog;h=libahci > for Jeff Garzik's work on separating AHCI support from PCI. Well, no, it doesn't separate PCI code yet (and the task for this patchset was different). Yet it was a step in that direction... I suggest that you somehow coordinate with Sagar Borikar, see his initial patch here: http://marc.info/?l=linux-ide&m=123807608819226 MBR, Sergei