Commit b9b9af11 authored by Anton Vorontsov's avatar Anton Vorontsov Committed by Linus Torvalds

spi_mpc83xx: split mpc83xx_spi_work() into two routines

mpc83xx_spi_work() is quite large, with up to five indentation levels and
is quite difficult to read.

So, split the function in two parts:
1. mpc83xx_spi_work() now only traverse queued spi messages;
2. mpc83xx_spi_do_one_msg() only manages single messages.

There should be no functional changes.
Signed-off-by: default avatarAnton Vorontsov <avorontsov@ru.mvista.com>
Cc: Kumar Gala <galak@gate.crashing.org>
Cc: David Brownell <david-b@pacbell.net>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 9effb959
...@@ -350,71 +350,76 @@ static int mpc83xx_spi_bufs(struct spi_device *spi, struct spi_transfer *t) ...@@ -350,71 +350,76 @@ static int mpc83xx_spi_bufs(struct spi_device *spi, struct spi_transfer *t)
return mpc83xx_spi->count; return mpc83xx_spi->count;
} }
static void mpc83xx_spi_work(struct work_struct *work) static void mpc83xx_spi_do_one_msg(struct spi_message *m)
{ {
struct mpc83xx_spi *mpc83xx_spi = struct spi_device *spi = m->spi;
container_of(work, struct mpc83xx_spi, work); struct spi_transfer *t;
unsigned int cs_change;
spin_lock_irq(&mpc83xx_spi->lock); const int nsecs = 50;
mpc83xx_spi->busy = 1; int status;
while (!list_empty(&mpc83xx_spi->queue)) {
struct spi_message *m; cs_change = 1;
struct spi_device *spi; status = 0;
struct spi_transfer *t = NULL; list_for_each_entry(t, &m->transfers, transfer_list) {
unsigned cs_change; if (t->bits_per_word || t->speed_hz) {
int status, nsecs = 50; /* Don't allow changes if CS is active */
status = -EINVAL;
m = container_of(mpc83xx_spi->queue.next,
struct spi_message, queue); if (cs_change)
list_del_init(&m->queue); status = mpc83xx_spi_setup_transfer(spi, t);
spin_unlock_irq(&mpc83xx_spi->lock); if (status < 0)
spi = m->spi;
cs_change = 1;
status = 0;
list_for_each_entry(t, &m->transfers, transfer_list) {
if (t->bits_per_word || t->speed_hz) {
/* Don't allow changes if CS is active */
status = -EINVAL;
if (cs_change)
status = mpc83xx_spi_setup_transfer(spi, t);
if (status < 0)
break;
}
if (cs_change) {
mpc83xx_spi_chipselect(spi, BITBANG_CS_ACTIVE);
ndelay(nsecs);
}
cs_change = t->cs_change;
if (t->len)
status = mpc83xx_spi_bufs(spi, t);
if (status) {
status = -EMSGSIZE;
break; break;
} }
m->actual_length += t->len;
if (t->delay_usecs)
udelay(t->delay_usecs);
if (cs_change) { if (cs_change) {
ndelay(nsecs); mpc83xx_spi_chipselect(spi, BITBANG_CS_ACTIVE);
mpc83xx_spi_chipselect(spi, BITBANG_CS_INACTIVE); ndelay(nsecs);
ndelay(nsecs); }
} cs_change = t->cs_change;
if (t->len)
status = mpc83xx_spi_bufs(spi, t);
if (status) {
status = -EMSGSIZE;
break;
} }
m->actual_length += t->len;
m->status = status; if (t->delay_usecs)
m->complete(m->context); udelay(t->delay_usecs);
if (status || !cs_change) { if (cs_change) {
ndelay(nsecs); ndelay(nsecs);
mpc83xx_spi_chipselect(spi, BITBANG_CS_INACTIVE); mpc83xx_spi_chipselect(spi, BITBANG_CS_INACTIVE);
ndelay(nsecs);
} }
}
m->status = status;
m->complete(m->context);
if (status || !cs_change) {
ndelay(nsecs);
mpc83xx_spi_chipselect(spi, BITBANG_CS_INACTIVE);
}
mpc83xx_spi_setup_transfer(spi, NULL);
}
static void mpc83xx_spi_work(struct work_struct *work)
{
struct mpc83xx_spi *mpc83xx_spi = container_of(work, struct mpc83xx_spi,
work);
spin_lock_irq(&mpc83xx_spi->lock);
mpc83xx_spi->busy = 1;
while (!list_empty(&mpc83xx_spi->queue)) {
struct spi_message *m = container_of(mpc83xx_spi->queue.next,
struct spi_message, queue);
list_del_init(&m->queue);
spin_unlock_irq(&mpc83xx_spi->lock);
mpc83xx_spi_setup_transfer(spi, NULL); mpc83xx_spi_do_one_msg(m);
spin_lock_irq(&mpc83xx_spi->lock); spin_lock_irq(&mpc83xx_spi->lock);
} }
......
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