Re: [PATCH 3/3] spi: create an optional message queueing infrastructure

2012-02-03 Thread Linus Walleij
On Thu, Feb 2, 2012 at 5:05 PM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: [Me] + * This function checks if there is any spi message in the queue that + * needs processing and delegate control to appropriate function + * do_polling_transfer()/do_interrupt_dma_transfer() + * based on

Re: [PATCH 3/3] spi: create an optional message queueing infrastructure

2012-02-03 Thread Linus Walleij
On Thu, Feb 2, 2012 at 7:52 PM, Grant Likely grant.lik...@secretlab.ca wrote: This looks really good.  In fact, can you think of any reason why we wouldn't force all spi drivers to use central queueing?  I don't see any advantage in leaving the old mechanism in place other than to allow driver

[PATCH 3/3] spi: create an optional message queueing infrastructure

2012-02-02 Thread Linus Walleij
From: Linus Walleij linus.wall...@linaro.org This rips the message queue in the PL022 driver out and pushes it into (optional) common infrastructure. Drivers that want to use the message pumping thread will need to select the symbol SPI_MASTER_QUEUE and implement three methods in place for the

Re: [PATCH 3/3] spi: create an optional message queueing infrastructure

2012-02-02 Thread Grant Likely
On Thu, Feb 02, 2012 at 01:59:59PM +0100, Linus Walleij wrote: From: Linus Walleij linus.wall...@linaro.org This rips the message queue in the PL022 driver out and pushes it into (optional) common infrastructure. Drivers that want to use the message pumping thread will need to select the

Re: [PATCH 3/3] spi: create an optional message queueing infrastructure

2012-02-02 Thread Grant Likely
On Thu, Feb 2, 2012 at 3:47 PM, Mark Brown broo...@opensource.wolfsonmicro.com wrote: On Thu, Feb 02, 2012 at 11:52:47AM -0700, Grant Likely wrote: This looks really good.  In fact, can you think of any reason why we wouldn't force all spi drivers to use central queueing?  I don't see any