On Sat, Mar 17, 2012 at 12:39 AM, Guennadi Liakhovetski <g.liakhovet...@gmx.de> wrote:
> The SPI subsystem core now manages message queues internally. Remove the > local message queue implementation from the spi-bitbang driver and > migrate to the common one. > > Signed-off-by: Guennadi Liakhovetski <g.liakhovet...@gmx.de> This is great stuff! Acked-by: Linus Walleij <linus.wall...@linaro.org> (Since I've never really used the bitbang driver I wouldn't trust me to do any deeper review.) Quick thought: > +static int spi_bitbang_dummy_prepare(struct spi_master *master) > +{ > + return 0; (...) > + master->prepare_transfer_hardware = spi_bitbang_dummy_prepare; > + master->unprepare_transfer_hardware = spi_bitbang_dummy_prepare; Maybe we should think about making these two hooks optional if this pattern turns up a lot. I'll try to fix some refactoring further down the road if nobody beats me to it. Yours, Linus Walleij ------------------------------------------------------------------------------ This SF email is sponsosred by: Try Windows Azure free for 90 days Click Here http://p.sf.net/sfu/sfd2d-msazure _______________________________________________ spi-devel-general mailing list spi-devel-general@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/spi-devel-general