Re: [PATCH 4/6] * Register platform interface

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jul 23, 2010 at 09:43:17AM -0700, Dmitry Torokhov wrote:
> On Fri, Jul 23, 2010 at 04:38:59PM +0100, Alan Cox wrote:
> > > I am confused here... Are you having a separate SPI driver create a
> > > platform device and then you have mrstouch to bind to this
> > > intermediate platform device? Are you doing that so you can introduce
> > > I2C interface later? If so I think I prefer how adxl34x and ad7879
> > > drivers are structured - they are split into core and interface parts
> > > but do not require extra devices/drivers (see in my 'next' brnach).
> > 
> > There is no SPI interface to the device. It ended up in the kernel SPI
> > because old versions of the device firmware listed it in the firmware
> > tables as SPI and rather than doing the right thing (correcting the
> > type) the x86 code created an SPI device for it.
> > 
> > At a certain level it may be SPI, but it's all hidden behind the
> > firmware on the SCU and nothing to do with Linux SPI at all.
> > 
> 
> Ah, OK then.
> 
> So, what is the plan of action? Greg, are you going to apply the patches
> to staging and move over to input or shoudl I do that (apply Alan's
> patches to staging _in my tree_ and move over to drivers.input)?

I already have the driver in my staging-next tree, so I can apply these
6 patches as well, and then move it to the drivers/input subdirectory if
you want.

Or I can leave it all in drivers/staging/ and then after 2.6.36-rc1
comes, you can move the driver into drivers/input/ subdir.

Which do you prefer?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux