Re: [PATCH v2 3/5] of-bindings: Add binding documentation for SPI busses and devices
From: Segher Boessenkool
Date: Fri Jul 04 2008 - 19:39:07 EST
+ The SPI master node requires the following properties:
+ - #address-cells - number of cells required to define a chip
select
+ address on the SPI bus.
Hrm. Should this (and "reg" in the child node) be required for SPI
masters that have only one chip select?
+ - max-speed - (required) Maximum SPI clocking speed of
device in Hz
The property name should include something "SPI", it's way too generic
otherwise.
+ - spi,cpol - (optional) Device requires inverse clock
polarity
+ - spi,cpha - (optional) Device requires shifted clock phase
Don't abbr the property names, there's nothing wrong with longer names.
The names shouldn't start with "spi," either, "spi" isn't a vendor;
how about "spi-inverse-clock-polarity" or similar?
+ - linux,modalias - (optional, Linux specific) Force binding of
SPI device
+ to a particular spi_device driver. Useful for changing
+ driver binding between spidev and a kernel SPI driver.
This is a temporary workaround I hope?
Segher
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/