Platform: OLPC: move global variables into priv struct
authorAndres Salomon <dilinger@queued.net>
Sat, 14 Jul 2012 00:10:45 +0000 (17:10 -0700)
committerAndres Salomon <dilinger@queued.net>
Wed, 1 Aug 2012 03:27:31 +0000 (23:27 -0400)
Populate olpc_ec_priv with variables that were previously global.  This
makes things a tad bit clearer, IMO.

Signed-off-by: Andres Salomon <dilinger@queued.net>
Acked-by: Paul Fox <pgf@laptop.org>
Reviewed-by: Thomas Gleixner <tglx@linutronix.de>
drivers/platform/olpc/olpc-ec.c

index 1a15a79..0f9f859 100644 (file)
@@ -31,6 +31,12 @@ struct ec_cmd_desc {
 
 struct olpc_ec_priv {
        struct olpc_ec_driver *drv;
+       struct work_struct worker;
+       struct mutex cmd_lock;
+
+       /* Pending EC commands */
+       struct list_head cmd_q;
+       spinlock_t cmd_q_lock;
 
        struct dentry *dbgfs_dir;
 
@@ -46,16 +52,9 @@ struct olpc_ec_priv {
        bool suspended;
 };
 
-static void olpc_ec_worker(struct work_struct *w);
-
-static DECLARE_WORK(ec_worker, olpc_ec_worker);
-static LIST_HEAD(ec_cmd_q);
-static DEFINE_SPINLOCK(ec_cmd_q_lock);
-
 static struct olpc_ec_driver *ec_driver;
 static struct olpc_ec_priv *ec_priv;
 static void *ec_cb_arg;
-static DEFINE_MUTEX(ec_cb_lock);
 
 void olpc_ec_driver_register(struct olpc_ec_driver *drv, void *arg)
 {
@@ -66,49 +65,51 @@ EXPORT_SYMBOL_GPL(olpc_ec_driver_register);
 
 static void olpc_ec_worker(struct work_struct *w)
 {
+       struct olpc_ec_priv *ec = container_of(w, struct olpc_ec_priv, worker);
        struct ec_cmd_desc *desc = NULL;
        unsigned long flags;
 
        /* Grab the first pending command from the queue */
-       spin_lock_irqsave(&ec_cmd_q_lock, flags);
-       if (!list_empty(&ec_cmd_q)) {
-               desc = list_first_entry(&ec_cmd_q, struct ec_cmd_desc, node);
+       spin_lock_irqsave(&ec->cmd_q_lock, flags);
+       if (!list_empty(&ec->cmd_q)) {
+               desc = list_first_entry(&ec->cmd_q, struct ec_cmd_desc, node);
                list_del(&desc->node);
        }
-       spin_unlock_irqrestore(&ec_cmd_q_lock, flags);
+       spin_unlock_irqrestore(&ec->cmd_q_lock, flags);
 
        /* Do we actually have anything to do? */
        if (!desc)
                return;
 
        /* Protect the EC hw with a mutex; only run one cmd at a time */
-       mutex_lock(&ec_cb_lock);
+       mutex_lock(&ec->cmd_lock);
        desc->err = ec_driver->ec_cmd(desc->cmd, desc->inbuf, desc->inlen,
                        desc->outbuf, desc->outlen, ec_cb_arg);
-       mutex_unlock(&ec_cb_lock);
+       mutex_unlock(&ec->cmd_lock);
 
        /* Finished, wake up olpc_ec_cmd() */
        complete(&desc->finished);
 
        /* Run the worker thread again in case there are more cmds pending */
-       schedule_work(&ec_worker);
+       schedule_work(&ec->worker);
 }
 
 /*
  * Throw a cmd descripter onto the list.  We now have SMP OLPC machines, so
  * locking is pretty critical.
  */
-static void queue_ec_descriptor(struct ec_cmd_desc *desc)
+static void queue_ec_descriptor(struct ec_cmd_desc *desc,
+               struct olpc_ec_priv *ec)
 {
        unsigned long flags;
 
        INIT_LIST_HEAD(&desc->node);
 
-       spin_lock_irqsave(&ec_cmd_q_lock, flags);
-       list_add_tail(&desc->node, &ec_cmd_q);
-       spin_unlock_irqrestore(&ec_cmd_q_lock, flags);
+       spin_lock_irqsave(&ec->cmd_q_lock, flags);
+       list_add_tail(&desc->node, &ec->cmd_q);
+       spin_unlock_irqrestore(&ec->cmd_q_lock, flags);
 
-       schedule_work(&ec_worker);
+       schedule_work(&ec->worker);
 }
 
 int olpc_ec_cmd(u8 cmd, u8 *inbuf, size_t inlen, u8 *outbuf, size_t outlen)
@@ -137,7 +138,7 @@ int olpc_ec_cmd(u8 cmd, u8 *inbuf, size_t inlen, u8 *outbuf, size_t outlen)
        desc.err = 0;
        init_completion(&desc.finished);
 
-       queue_ec_descriptor(&desc);
+       queue_ec_descriptor(&desc, ec);
 
        /* Timeouts must be handled in the platform-specific EC hook */
        wait_for_completion(&desc.finished);
@@ -266,7 +267,14 @@ static int olpc_ec_probe(struct platform_device *pdev)
        ec = kzalloc(sizeof(*ec), GFP_KERNEL);
        if (!ec)
                return -ENOMEM;
+
        ec->drv = ec_driver;
+       INIT_WORK(&ec->worker, olpc_ec_worker);
+       mutex_init(&ec->cmd_lock);
+
+       INIT_LIST_HEAD(&ec->cmd_q);
+       spin_lock_init(&ec->cmd_q_lock);
+
        ec_priv = ec;
        platform_set_drvdata(pdev, ec);