On Thu, May 4, 2017 at 7:45 PM, Geert Uytterhoeven <geert+renesas@xxxxxxxxx> wrote: > From: Hisashi Nakamura <hisashi.nakamura.ak@xxxxxxxxxxx> > > Add slave mode support to the MSIOF driver, in both PIO and DMA mode. > > For now this only supports the transmission of messages with a size > that is known in advance. > > Signed-off-by: Hisashi Nakamura <hisashi.nakamura.ak@xxxxxxxxxxx> > Signed-off-by: Hiromitsu Yamasaki <hiromitsu.yamasaki.ym@xxxxxxxxxxx> > [geert: Timeout handling cleanup, spi core integration, cancellation, > rewording] > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > index 2ce15ca977828668..7c4e8c4f3a9bddfd 100644 > --- a/drivers/spi/spi-sh-msiof.c > +++ b/drivers/spi/spi-sh-msiof.c > +static int sh_msiof_slave_abort(struct spi_master *master) > +{ > + struct sh_msiof_spi_priv *p = spi_master_get_devdata(master); > + unsigned long flags; > + > + spin_lock_irqsave(&p->done.wait.lock, flags); > + if (!p->done.done) { > + wait_queue_t *curr, *next; > + > + list_for_each_entry_safe(curr, next, &p->done.wait.task_list, > + task_list) { > + signal_wake_up(curr->private, 1); 0day reported a build failure in the modular case (thanks!): ERROR: "signal_wake_up_state" [drivers/spi/spi-sh-msiof.ko] undefined! signal_wake_up() is a static inline function calling signal_wake_up_state(), but the latter is not exported to modules. I'll bring it up with the scheduler people... > + break; > + } > + } > + spin_unlock_irqrestore(&p->done.wait.lock, flags); > + return 0; > +} Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html