Commit 0f1a277b authored by Mark Brown's avatar Mark Brown

spi: axi-spi-engine: small cleanups

Merge series from David Lechner <dlechner@baylibre.com>:

This series contains a few small cleanups to the axi-spi-engine driver,
mostly suggested from previous reviews.
parents ee09bb72 5c708541
...@@ -12,6 +12,7 @@ ...@@ -12,6 +12,7 @@
#include <linux/io.h> #include <linux/io.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/overflow.h>
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/spi/spi.h> #include <linux/spi/spi.h>
...@@ -75,15 +76,13 @@ ...@@ -75,15 +76,13 @@
struct spi_engine_program { struct spi_engine_program {
unsigned int length; unsigned int length;
uint16_t instructions[]; uint16_t instructions[] __counted_by(length);
}; };
/** /**
* struct spi_engine_message_state - SPI engine per-message state * struct spi_engine_message_state - SPI engine per-message state
*/ */
struct spi_engine_message_state { struct spi_engine_message_state {
/** @p: Instructions for executing this message. */
struct spi_engine_program *p;
/** @cmd_length: Number of elements in cmd_buf array. */ /** @cmd_length: Number of elements in cmd_buf array. */
unsigned cmd_length; unsigned cmd_length;
/** @cmd_buf: Array of commands not yet written to CMD FIFO. */ /** @cmd_buf: Array of commands not yet written to CMD FIFO. */
...@@ -117,9 +116,10 @@ struct spi_engine { ...@@ -117,9 +116,10 @@ struct spi_engine {
static void spi_engine_program_add_cmd(struct spi_engine_program *p, static void spi_engine_program_add_cmd(struct spi_engine_program *p,
bool dry, uint16_t cmd) bool dry, uint16_t cmd)
{ {
if (!dry)
p->instructions[p->length] = cmd;
p->length++; p->length++;
if (!dry)
p->instructions[p->length - 1] = cmd;
} }
static unsigned int spi_engine_get_config(struct spi_device *spi) static unsigned int spi_engine_get_config(struct spi_device *spi)
...@@ -503,15 +503,13 @@ static irqreturn_t spi_engine_irq(int irq, void *devid) ...@@ -503,15 +503,13 @@ static irqreturn_t spi_engine_irq(int irq, void *devid)
static int spi_engine_optimize_message(struct spi_message *msg) static int spi_engine_optimize_message(struct spi_message *msg)
{ {
struct spi_engine_program p_dry, *p; struct spi_engine_program p_dry, *p;
size_t size;
spi_engine_precompile_message(msg); spi_engine_precompile_message(msg);
p_dry.length = 0; p_dry.length = 0;
spi_engine_compile_message(msg, true, &p_dry); spi_engine_compile_message(msg, true, &p_dry);
size = sizeof(*p->instructions) * (p_dry.length + 1); p = kzalloc(struct_size(p, instructions, p_dry.length + 1), GFP_KERNEL);
p = kzalloc(sizeof(*p) + size, GFP_KERNEL);
if (!p) if (!p)
return -ENOMEM; return -ENOMEM;
...@@ -543,7 +541,6 @@ static int spi_engine_transfer_one_message(struct spi_controller *host, ...@@ -543,7 +541,6 @@ static int spi_engine_transfer_one_message(struct spi_controller *host,
/* reinitialize message state for this transfer */ /* reinitialize message state for this transfer */
memset(st, 0, sizeof(*st)); memset(st, 0, sizeof(*st));
st->p = p;
st->cmd_buf = p->instructions; st->cmd_buf = p->instructions;
st->cmd_length = p->length; st->cmd_length = p->length;
msg->state = st; msg->state = st;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment