[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4CC1E00A.3010109@ti.com>
Date: Fri, 22 Oct 2010 15:03:38 -0400
From: Cyril Chemparathy <cyril@...com>
To: Alan Cox <alan@...rguk.ukuu.org.uk>
CC: "davinci-linux-open-source@...ux.davincidsp.com"
<davinci-linux-open-source@...ux.davincidsp.com>,
"spi-devel-general@...ts.sourceforge.net"
<spi-devel-general@...ts.sourceforge.net>,
"broonie@...nsource.wolfsonmicro.com"
<broonie@...nsource.wolfsonmicro.com>,
"lrg@...mlogic.co.uk" <lrg@...mlogic.co.uk>,
"dbrownell@...rs.sourceforge.net" <dbrownell@...rs.sourceforge.net>,
"grant.likely@...retlab.ca" <grant.likely@...retlab.ca>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"rpurdie@...ys.net" <rpurdie@...ys.net>, sameo@...ux.intel.com
Subject: Re: [PATCH v3 01/12] misc: add driver for sequencer serial port
On 10/22/2010 10:20 AM, Alan Cox wrote:
> On Thu, 21 Oct 2010 17:01:02 -0400
> Cyril Chemparathy <cyril@...com> wrote:
>
>> TI's sequencer serial port (TI-SSP) is a jack-of-all-trades type of serial port
>> device. It has a built-in programmable execution engine that can be programmed
>> to operate as almost any serial bus (I2C, SPI, EasyScale, and others).
>>
>> This patch adds a driver for this controller device. The driver does not
>> expose a user-land interface. Protocol drivers built on top of this layer are
>> expected to remain in-kernel.
>
> I suspect the driver belongs with the mfd drivers not with drivers/misc ?
>
> otherwise it looks very nice.
Unlike MFDs, this device doesn't have cells with differing
functionality. Instead it has functionally identical ports that can
operate in a variety of modes. That said, does this still fit in with
other MFD drivers? If so, I don't see a problem with moving it there.
Thanks
Cyril.
--
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