From: Frans Pop <elendil@planet.nl>
To: Grant Coady <grant_lkml@dodo.com.au>
Subject: Re: Query: should 'Marvell SATA support' depend on PCI?
Date: Sun, 10 Aug 2008 14:24:58 +0200
User-Agent: KMail/1.9.9
Cc: alan@lxorguk.ukuu.org.uk, linux-kernel@vger.kernel.org
References: <vg3t941ebudmtb8afj82thuldug82ak1fs@4ax.com>
In-reply-To: <vg3t941ebudmtb8afj82thuldug82ak1fs@4ax.com>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200808101424.59011.elendil@planet.nl>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1202
Lines: 32

Grant Coady wrote:
> I was reworking the ATA driver menu patch to shrink it (merge 'depends
> on' lines) and noticed that the:
> 
> config SATA_MV
>         tristate "Marvell SATA support (HIGHLY EXPERIMENTAL)"
>         depends on EXPERIMENTAL
>         help
>           This option enables support for the Marvell Serial ATA family.
>           Currently supports 88SX[56]0[48][01] chips.
> 
> doesn't have 'depends on PCI && EXPERIMENTAL' like other drivers around
> it, this is okay or should it also depend on PCI?

AFAICT it should not as it can also be implemented as a platform device.

I have SATA_MV on a QNAP TS-109 NAS dervice (arm) and lspci on that gives me:
$ lspci
00:00.0 Memory controller: Marvell Technology Group Ltd. Device 5182 (rev 02)
01:00.0 Memory controller: Marvell Technology Group Ltd. Device 5182 (rev 02)

So the disk controller is not on the PCI bus.
And in /sys/ I see sata_mv listed under devices/platform/.


Cheers,
FJP
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/