Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932284AbVLMOGB (ORCPT ); Tue, 13 Dec 2005 09:06:01 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932319AbVLMOGB (ORCPT ); Tue, 13 Dec 2005 09:06:01 -0500 Received: from rtsoft2.corbina.net ([85.21.88.2]:3052 "HELO mail.dev.rtsoft.ru") by vger.kernel.org with SMTP id S932284AbVLMOGA (ORCPT ); Tue, 13 Dec 2005 09:06:00 -0500 Date: Tue, 13 Dec 2005 17:06:29 +0300 From: Vitaly Wool To: linux-kernel@vger.kernel.org Cc: david-b@pacbell.net, dpervushin@gmail.com, akpm@osdl.org, greg@kroah.com, basicmark@yahoo.com, komal_shah802003@yahoo.com, stephen@streetfiresound.com, spi-devel-general@lists.sourceforge.net, Joachim_Jaeger@digi.com Subject: [PATCH/RFC] SPI: add async message handing library to David Brownell's core Message-Id: <20051213170629.7240d211.vwool@ru.mvista.com> In-Reply-To: <20051212182026.4e393d5a.vwool@ru.mvista.com> References: <20051212182026.4e393d5a.vwool@ru.mvista.com> X-Mailer: Sylpheed version 1.0.4 (GTK+ 1.2.10; i386-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 10178 Lines: 319 Greetings, as a part of the convergence process between two SPI cores (one is developed by David Brownell and the other one -- by Dmitry Pervushin and me) I'd like to post here one more result of mindwork :). Despite our previous activities of that kind which were more related to porting drivers from one core to the other, this is a port of the library for handling async SPI messages using kernel threads, one per each SPI controller, from our core to David's. This thingie hasn't been thoroughly tested yet, but it's lightweight and easy to understand so I don't think solving the problems that may arise will take long. Though I haven't actually done that yet, I'm sure that Stephen's PXA SSP driver will become easier to understand and less in footprint and will work faster when it's rewritten using this library. (Yes, I do expect performance improvement here as the current implementation schedules multiple tasklets, so scheduling penalty is high.) This patch performes just minimal intrusion to the SPI core itself, adding a 'void *context' field to the spi_master structure which seems to be a good thing anyway. drivers/spi/Kconfig | 9 + drivers/spi/Makefile | 3 drivers/spi/spi-thread.c | 198 +++++++++++++++++++++++++++++++++++++++++ include/linux/spi/spi-thread.h | 27 +++++ include/linux/spi/spi.h | 3 5 files changed, 240 insertions(+) Signed-off-by: Vitaly Wool Index: linux-2.6.orig/drivers/spi/Kconfig =================================================================== --- linux-2.6.orig.orig/drivers/spi/Kconfig +++ linux-2.6.orig/drivers/spi/Kconfig @@ -13,6 +13,7 @@ config SPI_ARCH_HAS_MASTER default y if ARCH_AT91 default y if ARCH_OMAP default y if ARCH_PXA + default y if ARCH_PNX4008 default y if X86 # devel hack only!! (ICH7 can...) config SPI_ARCH_HAS_SLAVE @@ -63,6 +64,14 @@ config SPI_MASTER controller and the protocol drivers for the SPI slave chips that are connected. +config SPI_THREAD + boolean "Library for threaded asynchronous message processing" + depends on SPI_MASTER + help + Choose this if you want to use thread-based asynchronous + message processing library. Using kernel threads for + asynchronous data processing is the most common option. + comment "SPI Master Controller Drivers" depends on SPI_MASTER Index: linux-2.6.orig/drivers/spi/Makefile =================================================================== --- linux-2.6.orig.orig/drivers/spi/Makefile +++ linux-2.6.orig/drivers/spi/Makefile @@ -10,6 +10,9 @@ endif # config declarations into driver model code obj-$(CONFIG_SPI_MASTER) += spi.o +# thread-based async message handling library +obj-$(CONFIG_SPI_THREAD) += spi-thread.o + # SPI master controller drivers (bus) obj-$(CONFIG_SPI_BITBANG) += spi_bitbang.o # ... add above this line ... Index: linux-2.6.orig/drivers/spi/spi-thread.c =================================================================== --- /dev/null +++ linux-2.6.orig/drivers/spi/spi-thread.c @@ -0,0 +1,198 @@ +/* + * drivers/spi/spi-thread.c + * + * Authors: + * Vitaly Wool + * Dmitry Pervushin + * + * Copyright (C) 2005 MontaVista Software, Inc + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License. + * + */ +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +static int spi_thread(void *); +static int spi_queue(struct spi_device *, struct spi_message *); + +struct threaded_async_data { + atomic_t exiting; + struct spi_master *master; + struct task_struct *thread; + wait_queue_head_t wq; + struct list_head msgs; + struct semaphore lock; + int (*xfer) (struct spi_master *, struct spi_message *); +}; + +/** + * spi_start_async - start the thread + * @master: SPI controller structure which the thread is related to + * @return: abstract pointer to the thread context + */ +int spi_start_async (struct spi_master *master, int (*xfer)(struct spi_master *, struct spi_message *)) +{ + struct threaded_async_data *td = kzalloc (sizeof (struct threaded_async_data), GFP_KERNEL); + int rc = 0; + + if (!td) { + rc = -ENOMEM; + goto out; + } + + if (master->context) { + rc = -EEXIST; + goto out; + } + + td->master = master; + atomic_set(&td->exiting, 0); + td->thread = kthread_run(spi_thread, td, "%s-work", master->cdev.dev->bus_id); + init_waitqueue_head(&td->wq); + init_MUTEX(&td->lock); + INIT_LIST_HEAD(&td->msgs); + master->transfer = spi_queue; + td->xfer = xfer; + master->context = td; + +out: + return rc; +} +EXPORT_SYMBOL_GPL(spi_start_async); + +/** + * spi_stop_async - stop the thread + * @master: SPI controller structure which the thread is related to + */ +void spi_stop_async (struct spi_master *master) +{ + struct threaded_async_data *td = master->context; + + if (td) { + /* TODO: free messages, if any */ + atomic_inc (&td->exiting); + kthread_stop(td->thread); + kfree(td); + master->context = NULL; + } +} +EXPORT_SYMBOL_GPL(spi_stop_async); + +/** + * spi_thread_awake - function that called to determine if thread needs to process any messages + * @td: pointer to struct threaded_async_data + * Description: + * Thread wakes up if there is signal to exit (bd->exiting is set) + * or there are any messages in bus' queue. + */ +static int spi_thread_awake(struct threaded_async_data *td) +{ + int ret = -EINVAL; + + if (!td || atomic_read(&td->exiting)) { + goto out; + } + + down(&td->lock); + ret = !list_empty(&td->msgs); + up(&td->lock); +out: + return ret; +} + +/** + * spi_get_next_msg - retrieve the next message + * @data: pointer to threaded_async_data structure associated with + * SPI controller thread + */ +static inline struct spi_message *spi_get_next_msg(struct threaded_async_data *data) +{ + return list_entry(data->msgs.next, struct spi_message, queue); +} + +/** + * spi_thread - the thread that calls bus functions to perform actual transfers + * @context: pointer to struct spi_bus_data with bus-specific data + * Description: + * This function is started as separate thread to perform actual + * transfers on SPI bus + */ +static int spi_thread(void *context) +{ + struct threaded_async_data *td = context; + struct spi_message *message = NULL; + + while (!kthread_should_stop()) { + + wait_event_interruptible(td->wq, spi_thread_awake(td)); + + if (atomic_read(&td->exiting)) + goto thr_exit; + + down(&td->lock); + while (!list_empty(&td->msgs)) { + /* + * this part is locked by td->lock, + * to protect spi_message extraction + */ + message = spi_get_next_msg(td); + list_del (&message->queue); + + up(&td->lock); + + message->status = td->xfer(td->master, message); + if (message->complete) + message->complete(context); + + /* lock the data again... */ + down(&td->lock); + } + up(&td->lock); + } + +thr_exit: + return 0; +} + +/** + * spi_queue - (internal) queue the message to be processed asynchronously + * @spi: SPI device to perform transfer to/from + * @msg: message to be sent + * Description: + * This function queues the message to SPI controller's queue. + */ +static int spi_queue(struct spi_device *spi, struct spi_message *msg) +{ + struct threaded_async_data *td = spi->master->context; + int rc = 0; + + if (!td) { + rc = -EINVAL; + goto out; + } + + msg->spi = spi; + down(&td->lock); + list_add_tail(&msg->queue, &td->msgs); + dev_dbg(spi->dev.parent, "message has been queued\n"); + up(&td->lock); + wake_up_interruptible(&td->wq); + +out: + return rc; +} + Index: linux-2.6.orig/include/linux/spi/spi-thread.h =================================================================== --- /dev/null +++ linux-2.6.orig/include/linux/spi/spi-thread.h @@ -0,0 +1,27 @@ +/* + * linux/drivers/spi/spi-thread.h + * + * Copyright (C) 2005 MontaVista Software, Inc + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License. + * + */ +#ifndef __SPI_THREAD_H +#define __SPI_THREAD_H + + +#if defined (CONFIG_SPI_THREAD) +extern int spi_start_async (struct spi_master *, int (*)(struct spi_master *, struct spi_message *)); +extern void spi_stop_async (struct spi_master *); +#else +static inline int spi_start_async (struct spi_master *master, int (*xfer)(struct spi_master *, struct spi_message *)) +{ + return -EINVAL; +} +static inline void spi_stop_async (struct spi_master *master) +{ +} +#endif /* CONFIG_SPI_THREAD */ +#endif /* __SPI_THREAD_H */ Index: linux-2.6.orig/include/linux/spi/spi.h =================================================================== --- linux-2.6.orig.orig/include/linux/spi/spi.h +++ linux-2.6.orig/include/linux/spi/spi.h @@ -152,6 +152,7 @@ static inline void spi_unregister_driver * device's SPI controller; protocol code may call this. * @transfer: adds a message to the controller's transfer queue. * @cleanup: frees controller-specific state + * @context: controller-specific data * * Each SPI master controller can communicate with one or more spi_device * children. These make a small bus, sharing MOSI, MISO and SCK signals @@ -207,6 +208,8 @@ struct spi_master { /* called on release() to free memory provided by spi_master */ void (*cleanup)(const struct spi_device *spi); + + void *context; }; /* the spi driver core manages memory for the spi_master classdev */ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/