powerpc/opal: Make __opal_async_{get, release}_token() static
authorCyril Bur <cyrilbur@gmail.com>
Fri, 3 Nov 2017 02:41:41 +0000 (13:41 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 6 Nov 2017 09:20:26 +0000 (20:20 +1100)
There are no callers of both __opal_async_get_token() and
__opal_async_release_token().

This patch also removes the possibility of "emergency through
synchronous call to __opal_async_get_token()" as such it makes more
sense to initialise opal_sync_sem for the maximum number of async
tokens.

Signed-off-by: Cyril Bur <cyrilbur@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/include/asm/opal.h
arch/powerpc/platforms/powernv/opal-async.c

index 04c32b0..00aae31 100644 (file)
@@ -306,9 +306,7 @@ extern void opal_notifier_enable(void);
 extern void opal_notifier_disable(void);
 extern void opal_notifier_update_evt(uint64_t evt_mask, uint64_t evt_val);
 
-extern int __opal_async_get_token(void);
 extern int opal_async_get_token_interruptible(void);
-extern int __opal_async_release_token(int token);
 extern int opal_async_release_token(int token);
 extern int opal_async_wait_response(uint64_t token, struct opal_msg *msg);
 extern int opal_get_sensor_data(u32 sensor_hndl, u32 *sensor_data);
index 45b3feb..64255d3 100644 (file)
@@ -33,7 +33,7 @@ static struct semaphore opal_async_sem;
 static struct opal_msg *opal_async_responses;
 static unsigned int opal_max_async_tokens;
 
-int __opal_async_get_token(void)
+static int __opal_async_get_token(void)
 {
        unsigned long flags;
        int token;
@@ -73,7 +73,7 @@ int opal_async_get_token_interruptible(void)
 }
 EXPORT_SYMBOL_GPL(opal_async_get_token_interruptible);
 
-int __opal_async_release_token(int token)
+static int __opal_async_release_token(int token)
 {
        unsigned long flags;
 
@@ -199,11 +199,7 @@ int __init opal_async_comp_init(void)
                goto out_opal_node;
        }
 
-       /* Initialize to 1 less than the maximum tokens available, as we may
-        * require to pop one during emergency through synchronous call to
-        * __opal_async_get_token()
-        */
-       sema_init(&opal_async_sem, opal_max_async_tokens - 1);
+       sema_init(&opal_async_sem, opal_max_async_tokens);
 
 out_opal_node:
        of_node_put(opal_node);