Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967116AbaFTKyA (ORCPT ); Fri, 20 Jun 2014 06:54:00 -0400 Received: from mail-pb0-f52.google.com ([209.85.160.52]:55347 "EHLO mail-pb0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965650AbaFTKx7 (ORCPT ); Fri, 20 Jun 2014 06:53:59 -0400 From: Pramod Gurav To: linux-kernel@vger.kernel.org, inux-pm@vger.kernel.org Cc: "Rafael J. Wysocki" , Daniel Lezcano , Pramod Gurav Subject: [PATCH] cpuidle: Fix Coding style reported by checkpatch.pl Date: Fri, 20 Jun 2014 16:27:45 +0530 Message-Id: <1403261865-7472-1-git-send-email-pramod.gurav.etc@gmail.com> X-Mailer: git-send-email 1.7.9.5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Fixed: ERROR: do not use assignment in if condition Fixed: WARNING: EXPORT_SYMBOL should immediately follow its function Fixed: WARNING: line over 80 characters in a comment CC: Rafael J. Wysocki CC: Daniel Lezcano Signed-off-by: Pramod Gurav --- drivers/cpuidle/cpuidle.c | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c index 8236746..2c8752e 100644 --- a/drivers/cpuidle/cpuidle.c +++ b/drivers/cpuidle/cpuidle.c @@ -179,7 +179,9 @@ void cpuidle_reflect(struct cpuidle_device *dev, int index) void cpuidle_install_idle_handler(void) { if (enabled_devices) { - /* Make sure all changes finished before we switch to new idle */ + /* + * Make sure all changes finished before we switch to new idle + */ smp_wmb(); initialized = 1; } @@ -204,7 +206,6 @@ void cpuidle_pause_and_lock(void) mutex_lock(&cpuidle_lock); cpuidle_uninstall_idle_handler(); } - EXPORT_SYMBOL_GPL(cpuidle_pause_and_lock); /** @@ -215,7 +216,6 @@ void cpuidle_resume_and_unlock(void) cpuidle_install_idle_handler(); mutex_unlock(&cpuidle_lock); } - EXPORT_SYMBOL_GPL(cpuidle_resume_and_unlock); /* Currently used in suspend/resume path to suspend cpuidle */ @@ -267,9 +267,11 @@ int cpuidle_enable_device(struct cpuidle_device *dev) if (ret) return ret; - if (cpuidle_curr_governor->enable && - (ret = cpuidle_curr_governor->enable(drv, dev))) - goto fail_sysfs; + if (cpuidle_curr_governor->enable) { + ret = cpuidle_curr_governor->enable(drv, dev); + if (ret) + goto fail_sysfs; + } smp_wmb(); @@ -283,7 +285,6 @@ fail_sysfs: return ret; } - EXPORT_SYMBOL_GPL(cpuidle_enable_device); /** @@ -311,7 +312,6 @@ void cpuidle_disable_device(struct cpuidle_device *dev) cpuidle_remove_device_sysfs(dev); enabled_devices--; } - EXPORT_SYMBOL_GPL(cpuidle_disable_device); static void __cpuidle_unregister_device(struct cpuidle_device *dev) @@ -399,7 +399,6 @@ out_unregister: __cpuidle_unregister_device(dev); goto out_unlock; } - EXPORT_SYMBOL_GPL(cpuidle_register_device); /** @@ -423,7 +422,6 @@ void cpuidle_unregister_device(struct cpuidle_device *dev) cpuidle_resume_and_unlock(); } - EXPORT_SYMBOL_GPL(cpuidle_unregister_device); /** -- 1.7.9.5 -- 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/