[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110203150432.40cd74c4@lxorguk.ukuu.org.uk>
Date: Thu, 3 Feb 2011 15:04:32 +0000
From: Alan Cox <alan@...rguk.ukuu.org.uk>
To: Mark Brown <broonie@...nsource.wolfsonmicro.com>
Cc: Russ Gorby <russ.gorby@...el.com>,
David Brownell <dbrownell@...rs.sourceforge.net>,
Grant Likely <grant.likely@...retlab.ca>,
"open list:SPI SUBSYSTEM" <spi-devel-general@...ts.sourceforge.net>,
open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/1] spi: intel_mid_ssp_spi: new SPI driver for intel
Medfield platform
On Thu, 3 Feb 2011 13:28:00 +0000
Mark Brown <broonie@...nsource.wolfsonmicro.com> wrote:
> On Wed, Feb 02, 2011 at 10:40:54PM +0000, Alan Cox wrote:
>
> > And this is the unified one that handles all the devices, but I gather
> > may need some fixing/test work on Medfield.
>
> I've got the same question here as I had with Russ' patch: it looks like
> there's some overlap with the SSP ports used for audio (it's just a
> generic programmable serial port so even if it's not normally used for
> audio that's a possiblity), how is that handled?
The SSP has PCI configuration indicating how it is being assigned, which
is in vendor capability byte 6. The low 3 bits indicte the mode, where
mode 1 is an SPI master/slave, and in that case bit 6 is set for a slave.
The SSP/SPI driver will only grab ports that have been assigned to that
purpose as part of the system design.
I'm just putting the other bits from the generic driver back into the
more featured/tested specific driver that Russ posted.
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists