Re: [PATCH 02/17] spi/spi.h: Add configure from slave support.

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

 



On Tuesday 26 November 2013 05:51 PM, Mark Brown wrote:
On Tue, Nov 26, 2013 at 04:51:53PM +0530, Sourav Poddar wrote:
On Tuesday 26 November 2013 03:59 PM, Mark Brown wrote:
+	void	(*configure_from_slave)(struct spi_device *spi, u8 *val);
Clearly this is far too unstructured to be useful, this is only ever
going to work with one specific combination of slave and master since
there's absolutely no semantics defined.
I thought "val" might be good enuf, and this can be filled with whatever
data that need to be used from slave into master(irrespective of
slave and master?).
The name isn't the issue, it's the fact that you're passing a completely
unspecified blob of data.
May be, what you are suggesting here is to we should define a flash information
structure and should pass that structure here ?

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




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux