From: Miguel Ojeda Sandonis - cfag12864b_work should be static - spelling - coding style Signed-off-by: Miguel Ojeda Sandonis Signed-off-by: Andrew Morton --- drivers/auxdisplay/cfag12864b.c | 9 ++++----- 1 files changed, 4 insertions(+), 5 deletions(-) diff -puN drivers/auxdisplay/cfag12864b.c~drivers-add-lcd-support-update-7 drivers/auxdisplay/cfag12864b.c --- a/drivers/auxdisplay/cfag12864b.c~drivers-add-lcd-support-update-7 +++ a/drivers/auxdisplay/cfag12864b.c @@ -63,10 +63,10 @@ unsigned int cfag12864b_getrate(void) * cfag12864b/ks0108 reads the command/data. * * CS1 = First ks0108controller. - * If high, the first ks0108 receives commands/data. + * If high, the first ks0108 controller receives commands/data. * * CS2 = Second ks0108 controller - * If high, the second ks0108 receives commands/data. + * If high, the second ks0108 controller receives commands/data. * * DI = Data/Instruction * If low, cfag12864b will expect commands. @@ -223,7 +223,7 @@ static DEFINE_MUTEX(cfag12864b_mutex); static unsigned char cfag12864b_updating; static void cfag12864b_update(void *arg); static struct workqueue_struct *cfag12864b_workqueue; -DECLARE_WORK(cfag12864b_work, cfag12864b_update, NULL); +static DECLARE_WORK(cfag12864b_work, cfag12864b_update, NULL); static void cfag12864b_queue(void) { @@ -241,8 +241,7 @@ unsigned char cfag12864b_enable(void) cfag12864b_updating = 1; cfag12864b_queue(); ret = 0; - } - else + } else ret = 1; mutex_unlock(&cfag12864b_mutex); _