RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Zhang, Rui > Sent: Thursday, April 25, 2013 12:32 AM > To: Eduardo Valentin; amit.kach...@linaro.org > Cc: linux...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers > > > > > -Original Message- > > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > > ow...@vger.kernel.org] On Behalf Of Eduardo Valentin > > Sent: Thursday, April 18, 2013 1:12 AM > > To: amit.kach...@linaro.org > > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > > ker...@vger.kernel.org > > Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers > > Importance: High > > > > As this code is not a module nor a platform device driver, this patch > > removes some unused header files. > > > > Cc: Zhang Rui > > Cc: linux...@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Eduardo Valentin > > --- > > drivers/thermal/cpu_cooling.c | 3 --- > > 1 file changed, 3 deletions(-) > > > > diff --git a/drivers/thermal/cpu_cooling.c > > b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 > > --- a/drivers/thermal/cpu_cooling.c > > +++ b/drivers/thermal/cpu_cooling.c > > @@ -20,10 +20,7 @@ > > * > > * > > > ~~ > > ~ > > ~~~ > > */ > > -#include > > -#include > > This is still needed because of EXPORT_SYMBOL. > Or else, I get: > drivers/thermal/cpu_cooling.c:225:1: warning: data definition has no > type or storage class [enabled by default] > drivers/thermal/cpu_cooling.c:225:1: warning: type defaults to int in > declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] > drivers/thermal/cpu_cooling.c:225:1: warning: parameter names (without > types) in function declaration [enabled by default] > drivers/thermal/cpu_cooling.c:479:1: warning: data definition has no > type or storage class [enabled by default] > drivers/thermal/cpu_cooling.c:479:1: warning: type defaults to int in > declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] > drivers/thermal/cpu_cooling.c:479:1: warning: parameter names (without > types) in function declaration [enabled by default] > drivers/thermal/cpu_cooling.c:505:1: warning: data definition has no > type or storage class [enabled by default] > drivers/thermal/cpu_cooling.c:505:1: warning: type defaults to int in > declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] > drivers/thermal/cpu_cooling.c:505:1: warning: parameter names (without > types) in function declaration [enabled by default] > Patch refreshed with keeping "#include " and applied. > thanks, > rui > > > #include > > -#include > > #include > > #include > > #include > > -- > > 1.8.2.1.342.gfa7285d > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-pm" > in > > the body of a message to majord...@vger.kernel.org More majordomo > info > > at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majord...@vger.kernel.org More majordomo info > at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Eduardo Valentin > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers > Importance: High > > As this code is not a module nor a platform device driver, this patch > removes some unused header files. > > Cc: Zhang Rui > Cc: linux...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Eduardo Valentin > --- > drivers/thermal/cpu_cooling.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/thermal/cpu_cooling.c > b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 > --- a/drivers/thermal/cpu_cooling.c > +++ b/drivers/thermal/cpu_cooling.c > @@ -20,10 +20,7 @@ > * > * > ~~~ > ~~~ > */ > -#include > -#include This is still needed because of EXPORT_SYMBOL. Or else, I get: drivers/thermal/cpu_cooling.c:225:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:225:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:225:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:479:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:505:1: warning: parameter names (without types) in function declaration [enabled by default] thanks, rui > #include > -#include > #include > #include > #include > -- > 1.8.2.1.342.gfa7285d > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majord...@vger.kernel.org More majordomo info > at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Eduardo Valentin > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers > Importance: High > > As this code is not a module nor a platform device driver, this patch > removes some unused header files. > > Cc: Zhang Rui > Cc: linux...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Eduardo Valentin Applied. Thanks, rui > --- > drivers/thermal/cpu_cooling.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/thermal/cpu_cooling.c > b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 > --- a/drivers/thermal/cpu_cooling.c > +++ b/drivers/thermal/cpu_cooling.c > @@ -20,10 +20,7 @@ > * > * > ~~~ > ~~~ > */ > -#include > -#include > #include > -#include > #include > #include > #include > -- > 1.8.2.1.342.gfa7285d > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pm" in > the body of a message to majord...@vger.kernel.org More majordomo info > at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers Importance: High As this code is not a module nor a platform device driver, this patch removes some unused header files. Cc: Zhang Rui rui.zh...@intel.com Cc: linux...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com Applied. Thanks, rui --- drivers/thermal/cpu_cooling.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -20,10 +20,7 @@ * * ~~~ ~~~ */ -#include linux/kernel.h -#include linux/module.h #include linux/thermal.h -#include linux/platform_device.h #include linux/cpufreq.h #include linux/err.h #include linux/slab.h -- 1.8.2.1.342.gfa7285d -- To unsubscribe from this list: send the line unsubscribe linux-pm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers Importance: High As this code is not a module nor a platform device driver, this patch removes some unused header files. Cc: Zhang Rui rui.zh...@intel.com Cc: linux...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com --- drivers/thermal/cpu_cooling.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -20,10 +20,7 @@ * * ~~~ ~~~ */ -#include linux/kernel.h -#include linux/module.h This is still needed because of EXPORT_SYMBOL. Or else, I get: drivers/thermal/cpu_cooling.c:225:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:225:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:225:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:479:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:505:1: warning: parameter names (without types) in function declaration [enabled by default] thanks, rui #include linux/thermal.h -#include linux/platform_device.h #include linux/cpufreq.h #include linux/err.h #include linux/slab.h -- 1.8.2.1.342.gfa7285d -- To unsubscribe from this list: send the line unsubscribe linux-pm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Zhang, Rui Sent: Thursday, April 25, 2013 12:32 AM To: Eduardo Valentin; amit.kach...@linaro.org Cc: linux...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers -Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 01/30] thermal: cpu_cooling: remove unused headers Importance: High As this code is not a module nor a platform device driver, this patch removes some unused header files. Cc: Zhang Rui rui.zh...@intel.com Cc: linux...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com --- drivers/thermal/cpu_cooling.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -20,10 +20,7 @@ * * ~~ ~ ~~~ */ -#include linux/kernel.h -#include linux/module.h This is still needed because of EXPORT_SYMBOL. Or else, I get: drivers/thermal/cpu_cooling.c:225:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:225:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:225:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:479:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:479:1: warning: parameter names (without types) in function declaration [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: data definition has no type or storage class [enabled by default] drivers/thermal/cpu_cooling.c:505:1: warning: type defaults to int in declaration of EXPORT_SYMBOL_GPL [-Wimplicit-int] drivers/thermal/cpu_cooling.c:505:1: warning: parameter names (without types) in function declaration [enabled by default] Patch refreshed with keeping #include linux/module.h and applied. thanks, rui #include linux/thermal.h -#include linux/platform_device.h #include linux/cpufreq.h #include linux/err.h #include linux/slab.h -- 1.8.2.1.342.gfa7285d -- To unsubscribe from this list: send the line unsubscribe linux-pm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-pm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
Hi, On Wed, Apr 17, 2013 at 10:41 PM, Eduardo Valentin wrote: > As this code is not a module nor a platform device driver, > this patch removes some unused header files. > > Cc: Zhang Rui > Cc: linux...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Eduardo Valentin Reviewed-by: Amit Daniel Kachhap Thanks, Amit Daniel > --- > drivers/thermal/cpu_cooling.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c > index 5f5c780..778c43d 100644 > --- a/drivers/thermal/cpu_cooling.c > +++ b/drivers/thermal/cpu_cooling.c > @@ -20,10 +20,7 @@ > * > * ~~ > */ > -#include > -#include > #include > -#include > #include > #include > #include > -- > 1.8.2.1.342.gfa7285d > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 01/30] thermal: cpu_cooling: remove unused headers
Hi, On Wed, Apr 17, 2013 at 10:41 PM, Eduardo Valentin eduardo.valen...@ti.com wrote: As this code is not a module nor a platform device driver, this patch removes some unused header files. Cc: Zhang Rui rui.zh...@intel.com Cc: linux...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com Reviewed-by: Amit Daniel Kachhapamit.dan...@samsung.com Thanks, Amit Daniel --- drivers/thermal/cpu_cooling.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index 5f5c780..778c43d 100644 --- a/drivers/thermal/cpu_cooling.c +++ b/drivers/thermal/cpu_cooling.c @@ -20,10 +20,7 @@ * * ~~ */ -#include linux/kernel.h -#include linux/module.h #include linux/thermal.h -#include linux/platform_device.h #include linux/cpufreq.h #include linux/err.h #include linux/slab.h -- 1.8.2.1.342.gfa7285d -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
