lkml.org 
[lkml]   [2013]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] SPI: SSP SPI Controller driver v3
From
Date
On Mon, 2013-01-07 at 00:36 +0100, Linus Walleij wrote:
> On Wed, Dec 19, 2012 at 10:56 AM, Mika Westerberg
> <mika.westerberg@linux.intel.com> wrote:
> > On Tue, Dec 18, 2012 at 04:11:36PM +0800, chao bi wrote:
> >>
> >> This patch is to implement SSP SPI controller driver, which has been applied and
> >> validated on intel Moorestown & Medfield platform. The patch are originated by
> >> Ken Mills <ken.k.mills@intel.com> and Sylvain Centelles <sylvain.centelles@intel.com>,
> >> migrating to lateset Linux mainline SPI framework by Channing <chao.bi@intel.com>
> >> and Chen Jun <jun.d.chen@intel.com> according to their integration & validation
> >> on Medfield platform.
> >
> > This is the same IP block as used in PXA, right? With few modifications
> > here and there. Is there a reason not to use spi-pxa2xx.c?
>
> This needs to be investigated. Two drivers for the same or closely related
> hardware block is never a good sign...
>
> Yours,
> Linus Walleij

Dear Linus ,Mika and Grant,

Thanks for your remind.

Frankly I'm currently not sure whether they share same IP.. per your reminds, I tried to find but get
limited info about PXA SSP's IP, from the code, looks like they have part of registers the same.

As far as I know, spi-pxa2xx.c is specific for SSP controller of PXA2XX/PXA3XX core, right? While Medfield
platform is embedded with ATOM core, the SSP driver we upload is validated on SSP controller of ATOM. In my
view, they're specific for different AP & Platforms, if compare the 2 files, there are still many difference
in how they works, if to choose a driver for Intel Medfild/Moorestown platform, I believe spi-intel-mid-ssp.c
driver could be a more mature solution.

What do you think? please correct me if I'm mistaken.

-chao



\
 
 \ /
  Last update: 2013-01-07 09:21    [W:1.623 / U:0.152 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site