spi: spi cleanups

- Rearranged multi-line comment style.
- Add tabs.
- Add spaces.
Signed-off-by: NJagannadha Sutradharudu Teki <jaganna@xilinx.com>
上级 a5e8199a
...@@ -29,10 +29,11 @@ ...@@ -29,10 +29,11 @@
#define SPI_XFER_END 0x02 /* Deassert CS after transfer */ #define SPI_XFER_END 0x02 /* Deassert CS after transfer */
/* Header byte that marks the start of the message */ /* Header byte that marks the start of the message */
#define SPI_PREAMBLE_END_BYTE 0xec #define SPI_PREAMBLE_END_BYTE 0xec
/*----------------------------------------------------------------------- /**
* Representation of a SPI slave, i.e. what we're communicating with. * struct spi_slave: Representation of a SPI slave,
* i.e. what we're communicating with.
* *
* Drivers are expected to extend this with controller-specific data. * Drivers are expected to extend this with controller-specific data.
* *
...@@ -42,12 +43,12 @@ ...@@ -42,12 +43,12 @@
* be written at once, excluding command bytes. * be written at once, excluding command bytes.
*/ */
struct spi_slave { struct spi_slave {
unsigned int bus; unsigned int bus;
unsigned int cs; unsigned int cs;
unsigned int max_write_size; unsigned int max_write_size;
}; };
/*----------------------------------------------------------------------- /**
* Initialization, must be called once on start up. * Initialization, must be called once on start up.
* *
* TODO: I don't think we really need this. * TODO: I don't think we really need this.
...@@ -60,10 +61,10 @@ void spi_init(void); ...@@ -60,10 +61,10 @@ void spi_init(void);
* Allocate and zero all fields in the spi slave, and set the bus/chip * Allocate and zero all fields in the spi slave, and set the bus/chip
* select. Use the helper macro spi_alloc_slave() to call this. * select. Use the helper macro spi_alloc_slave() to call this.
* *
* @offset: Offset of struct spi_slave within slave structure * @offset: Offset of struct spi_slave within slave structure.
* @size: Size of slave structure * @size: Size of slave structure.
* @bus: Bus ID of the slave chip. * @bus: Bus ID of the slave chip.
* @cs: Chip select ID of the slave chip on the specified bus. * @cs: Chip select ID of the slave chip on the specified bus.
*/ */
void *spi_do_alloc_slave(int offset, int size, unsigned int bus, void *spi_do_alloc_slave(int offset, int size, unsigned int bus,
unsigned int cs); unsigned int cs);
...@@ -74,10 +75,10 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus, ...@@ -74,10 +75,10 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus,
* Allocate and zero all fields in the spi slave, and set the bus/chip * Allocate and zero all fields in the spi slave, and set the bus/chip
* select. * select.
* *
* @_struct: Name of structure to allocate (e.g. struct tegra_spi). This * @_struct: Name of structure to allocate (e.g. struct tegra_spi).
* structure must contain a member 'struct spi_slave *slave'. * This structure must contain a member 'struct spi_slave *slave'.
* @bus: Bus ID of the slave chip. * @bus: Bus ID of the slave chip.
* @cs: Chip select ID of the slave chip on the specified bus. * @cs: Chip select ID of the slave chip on the specified bus.
*/ */
#define spi_alloc_slave(_struct, bus, cs) \ #define spi_alloc_slave(_struct, bus, cs) \
spi_do_alloc_slave(offsetof(_struct, slave), \ spi_do_alloc_slave(offsetof(_struct, slave), \
...@@ -89,13 +90,13 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus, ...@@ -89,13 +90,13 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus,
* Allocate and zero all fields in the spi slave, and set the bus/chip * Allocate and zero all fields in the spi slave, and set the bus/chip
* select. * select.
* *
* @bus: Bus ID of the slave chip. * @bus: Bus ID of the slave chip.
* @cs: Chip select ID of the slave chip on the specified bus. * @cs: Chip select ID of the slave chip on the specified bus.
*/ */
#define spi_alloc_slave_base(bus, cs) \ #define spi_alloc_slave_base(bus, cs) \
spi_do_alloc_slave(0, sizeof(struct spi_slave), bus, cs) spi_do_alloc_slave(0, sizeof(struct spi_slave), bus, cs)
/*----------------------------------------------------------------------- /**
* Set up communications parameters for a SPI slave. * Set up communications parameters for a SPI slave.
* *
* This must be called once for each slave. Note that this function * This must be called once for each slave. Note that this function
...@@ -103,10 +104,10 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus, ...@@ -103,10 +104,10 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus,
* contents of spi_slave so that the hardware can be easily * contents of spi_slave so that the hardware can be easily
* initialized later. * initialized later.
* *
* bus: Bus ID of the slave chip. * @bus: Bus ID of the slave chip.
* cs: Chip select ID of the slave chip on the specified bus. * @cs: Chip select ID of the slave chip on the specified bus.
* max_hz: Maximum SCK rate in Hz. * @max_hz: Maximum SCK rate in Hz.
* mode: Clock polarity, clock phase and other parameters. * @mode: Clock polarity, clock phase and other parameters.
* *
* Returns: A spi_slave reference that can be used in subsequent SPI * Returns: A spi_slave reference that can be used in subsequent SPI
* calls, or NULL if one or more of the parameters are not supported. * calls, or NULL if one or more of the parameters are not supported.
...@@ -114,14 +115,14 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus, ...@@ -114,14 +115,14 @@ void *spi_do_alloc_slave(int offset, int size, unsigned int bus,
struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs, struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs,
unsigned int max_hz, unsigned int mode); unsigned int max_hz, unsigned int mode);
/*----------------------------------------------------------------------- /**
* Free any memory associated with a SPI slave. * Free any memory associated with a SPI slave.
* *
* slave: The SPI slave * @slave: The SPI slave
*/ */
void spi_free_slave(struct spi_slave *slave); void spi_free_slave(struct spi_slave *slave);
/*----------------------------------------------------------------------- /**
* Claim the bus and prepare it for communication with a given slave. * Claim the bus and prepare it for communication with a given slave.
* *
* This must be called before doing any transfers with a SPI slave. It * This must be called before doing any transfers with a SPI slave. It
...@@ -130,25 +131,25 @@ void spi_free_slave(struct spi_slave *slave); ...@@ -130,25 +131,25 @@ void spi_free_slave(struct spi_slave *slave);
* allowed to claim the same bus for several slaves without releasing * allowed to claim the same bus for several slaves without releasing
* the bus in between. * the bus in between.
* *
* slave: The SPI slave * @slave: The SPI slave
* *
* Returns: 0 if the bus was claimed successfully, or a negative value * Returns: 0 if the bus was claimed successfully, or a negative value
* if it wasn't. * if it wasn't.
*/ */
int spi_claim_bus(struct spi_slave *slave); int spi_claim_bus(struct spi_slave *slave);
/*----------------------------------------------------------------------- /**
* Release the SPI bus * Release the SPI bus
* *
* This must be called once for every call to spi_claim_bus() after * This must be called once for every call to spi_claim_bus() after
* all transfers have finished. It may disable any SPI hardware as * all transfers have finished. It may disable any SPI hardware as
* appropriate. * appropriate.
* *
* slave: The SPI slave * @slave: The SPI slave
*/ */
void spi_release_bus(struct spi_slave *slave); void spi_release_bus(struct spi_slave *slave);
/*----------------------------------------------------------------------- /**
* SPI transfer * SPI transfer
* *
* This writes "bitlen" bits out the SPI MOSI port and simultaneously clocks * This writes "bitlen" bits out the SPI MOSI port and simultaneously clocks
...@@ -161,19 +162,19 @@ void spi_release_bus(struct spi_slave *slave); ...@@ -161,19 +162,19 @@ void spi_release_bus(struct spi_slave *slave);
* temporary variables, this is OK). * temporary variables, this is OK).
* *
* spi_xfer() interface: * spi_xfer() interface:
* slave: The SPI slave which will be sending/receiving the data. * @slave: The SPI slave which will be sending/receiving the data.
* bitlen: How many bits to write and read. * @bitlen: How many bits to write and read.
* dout: Pointer to a string of bits to send out. The bits are * @dout: Pointer to a string of bits to send out. The bits are
* held in a byte array and are sent MSB first. * held in a byte array and are sent MSB first.
* din: Pointer to a string of bits that will be filled in. * @din: Pointer to a string of bits that will be filled in.
* flags: A bitwise combination of SPI_XFER_* flags. * @flags: A bitwise combination of SPI_XFER_* flags.
* *
* Returns: 0 on success, not 0 on failure * Returns: 0 on success, not 0 on failure
*/ */
int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout, int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
void *din, unsigned long flags); void *din, unsigned long flags);
/*----------------------------------------------------------------------- /**
* Determine if a SPI chipselect is valid. * Determine if a SPI chipselect is valid.
* This function is provided by the board if the low-level SPI driver * This function is provided by the board if the low-level SPI driver
* needs it to determine if a given chipselect is actually valid. * needs it to determine if a given chipselect is actually valid.
...@@ -183,7 +184,7 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout, ...@@ -183,7 +184,7 @@ int spi_xfer(struct spi_slave *slave, unsigned int bitlen, const void *dout,
*/ */
int spi_cs_is_valid(unsigned int bus, unsigned int cs); int spi_cs_is_valid(unsigned int bus, unsigned int cs);
/*----------------------------------------------------------------------- /**
* Activate a SPI chipselect. * Activate a SPI chipselect.
* This function is provided by the board code when using a driver * This function is provided by the board code when using a driver
* that can't control its chipselects automatically (e.g. * that can't control its chipselects automatically (e.g.
...@@ -192,7 +193,7 @@ int spi_cs_is_valid(unsigned int bus, unsigned int cs); ...@@ -192,7 +193,7 @@ int spi_cs_is_valid(unsigned int bus, unsigned int cs);
*/ */
void spi_cs_activate(struct spi_slave *slave); void spi_cs_activate(struct spi_slave *slave);
/*----------------------------------------------------------------------- /**
* Deactivate a SPI chipselect. * Deactivate a SPI chipselect.
* This function is provided by the board code when using a driver * This function is provided by the board code when using a driver
* that can't control its chipselects automatically (e.g. * that can't control its chipselects automatically (e.g.
...@@ -201,18 +202,18 @@ void spi_cs_activate(struct spi_slave *slave); ...@@ -201,18 +202,18 @@ void spi_cs_activate(struct spi_slave *slave);
*/ */
void spi_cs_deactivate(struct spi_slave *slave); void spi_cs_deactivate(struct spi_slave *slave);
/*----------------------------------------------------------------------- /**
* Set transfer speed. * Set transfer speed.
* This sets a new speed to be applied for next spi_xfer(). * This sets a new speed to be applied for next spi_xfer().
* slave: The SPI slave * @slave: The SPI slave
* hz: The transfer speed * @hz: The transfer speed
*/ */
void spi_set_speed(struct spi_slave *slave, uint hz); void spi_set_speed(struct spi_slave *slave, uint hz);
/*----------------------------------------------------------------------- /**
* Write 8 bits, then read 8 bits. * Write 8 bits, then read 8 bits.
* slave: The SPI slave we're communicating with * @slave: The SPI slave we're communicating with
* byte: Byte to be written * @byte: Byte to be written
* *
* Returns: The value that was read, or a negative value on error. * Returns: The value that was read, or a negative value on error.
* *
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册