* [PATCH 0/3] randconfig warning fixes for thermal
@ 2016-01-25 16:44 ` Arnd Bergmann
0 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: Zhang Rui, Eduardo Valentin; +Cc: linux-arm-kernel, linux-pm, spear-devel
These are three simple fixes for ARM randconfig builds:
[PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
[PATCH 2/3] thermal: allow spear-thermal driver to be a module
[PATCH 3/3] thermal: allow u8500-thermal driver to be a module
All three bugs are on configurationsn that are not important and
that rarely happen even with randconfig builds.
Arnd
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 0/3] randconfig warning fixes for thermal
@ 2016-01-25 16:44 ` Arnd Bergmann
0 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: linux-arm-kernel
These are three simple fixes for ARM randconfig builds:
[PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
[PATCH 2/3] thermal: allow spear-thermal driver to be a module
[PATCH 3/3] thermal: allow u8500-thermal driver to be a module
All three bugs are on configurationsn that are not important and
that rarely happen even with randconfig builds.
Arnd
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
2016-01-25 16:44 ` Arnd Bergmann
@ 2016-01-25 16:44 ` Arnd Bergmann
-1 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: Zhang Rui, Eduardo Valentin
Cc: linux-arm-kernel, linux-pm, spear-devel, Arnd Bergmann, linux-kernel
The spear thermal driver hides its suspend/resume function conditionally
based on CONFIG_PM, but references them based on CONFIG_PM_SLEEP, so
we get a warning if the former is set but the latter is not:
thermal/spear_thermal.c:58:12: warning: 'spear_thermal_suspend' defined but not used [-Wunused-function]
thermal/spear_thermal.c:75:12: warning: 'spear_thermal_resume' defined but not used [-Wunused-function]
This removes the #ifdef and instead uses a __maybe_uninitialized
annotation to avoid the warning and improve compile-time coverage.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/thermal/spear_thermal.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/thermal/spear_thermal.c b/drivers/thermal/spear_thermal.c
index 534dd9136662..81b35aace9de 100644
--- a/drivers/thermal/spear_thermal.c
+++ b/drivers/thermal/spear_thermal.c
@@ -54,8 +54,7 @@ static struct thermal_zone_device_ops ops = {
.get_temp = thermal_get_temp,
};
-#ifdef CONFIG_PM
-static int spear_thermal_suspend(struct device *dev)
+static int __maybe_unused spear_thermal_suspend(struct device *dev)
{
struct platform_device *pdev = to_platform_device(dev);
struct thermal_zone_device *spear_thermal = platform_get_drvdata(pdev);
@@ -72,7 +71,7 @@ static int spear_thermal_suspend(struct device *dev)
return 0;
}
-static int spear_thermal_resume(struct device *dev)
+static int __maybe_unused spear_thermal_resume(struct device *dev)
{
struct platform_device *pdev = to_platform_device(dev);
struct thermal_zone_device *spear_thermal = platform_get_drvdata(pdev);
@@ -94,7 +93,6 @@ static int spear_thermal_resume(struct device *dev)
return 0;
}
-#endif
static SIMPLE_DEV_PM_OPS(spear_thermal_pm_ops, spear_thermal_suspend,
spear_thermal_resume);
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
@ 2016-01-25 16:44 ` Arnd Bergmann
0 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: linux-arm-kernel
The spear thermal driver hides its suspend/resume function conditionally
based on CONFIG_PM, but references them based on CONFIG_PM_SLEEP, so
we get a warning if the former is set but the latter is not:
thermal/spear_thermal.c:58:12: warning: 'spear_thermal_suspend' defined but not used [-Wunused-function]
thermal/spear_thermal.c:75:12: warning: 'spear_thermal_resume' defined but not used [-Wunused-function]
This removes the #ifdef and instead uses a __maybe_uninitialized
annotation to avoid the warning and improve compile-time coverage.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/thermal/spear_thermal.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/thermal/spear_thermal.c b/drivers/thermal/spear_thermal.c
index 534dd9136662..81b35aace9de 100644
--- a/drivers/thermal/spear_thermal.c
+++ b/drivers/thermal/spear_thermal.c
@@ -54,8 +54,7 @@ static struct thermal_zone_device_ops ops = {
.get_temp = thermal_get_temp,
};
-#ifdef CONFIG_PM
-static int spear_thermal_suspend(struct device *dev)
+static int __maybe_unused spear_thermal_suspend(struct device *dev)
{
struct platform_device *pdev = to_platform_device(dev);
struct thermal_zone_device *spear_thermal = platform_get_drvdata(pdev);
@@ -72,7 +71,7 @@ static int spear_thermal_suspend(struct device *dev)
return 0;
}
-static int spear_thermal_resume(struct device *dev)
+static int __maybe_unused spear_thermal_resume(struct device *dev)
{
struct platform_device *pdev = to_platform_device(dev);
struct thermal_zone_device *spear_thermal = platform_get_drvdata(pdev);
@@ -94,7 +93,6 @@ static int spear_thermal_resume(struct device *dev)
return 0;
}
-#endif
static SIMPLE_DEV_PM_OPS(spear_thermal_pm_ops, spear_thermal_suspend,
spear_thermal_resume);
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 2/3] thermal: allow spear-thermal driver to be a module
2016-01-25 16:44 ` Arnd Bergmann
@ 2016-01-25 16:44 ` Arnd Bergmann
-1 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: Zhang Rui, Eduardo Valentin
Cc: linux-arm-kernel, linux-pm, spear-devel, Arnd Bergmann, linux-kernel
When the thermal subsystem is a loadable module, the spear driver
fails to build:
drivers/thermal/built-in.o: In function `spear_thermal_exit':
spear_thermal.c:(.text+0xf8): undefined reference to `thermal_zone_device_unregister'
drivers/thermal/built-in.o: In function `spear_thermal_probe':
spear_thermal.c:(.text+0x230): undefined reference to `thermal_zone_device_register'
This changes the symbol to a tristate, so Kconfig can track the
dependency correctly.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/thermal/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 8cc4ac64a91c..0852fd028dc9 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -195,7 +195,7 @@ config IMX_THERMAL
passive trip is crossed.
config SPEAR_THERMAL
- bool "SPEAr thermal sensor driver"
+ tristate "SPEAr thermal sensor driver"
depends on PLAT_SPEAR || COMPILE_TEST
depends on OF
help
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 2/3] thermal: allow spear-thermal driver to be a module
@ 2016-01-25 16:44 ` Arnd Bergmann
0 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: linux-arm-kernel
When the thermal subsystem is a loadable module, the spear driver
fails to build:
drivers/thermal/built-in.o: In function `spear_thermal_exit':
spear_thermal.c:(.text+0xf8): undefined reference to `thermal_zone_device_unregister'
drivers/thermal/built-in.o: In function `spear_thermal_probe':
spear_thermal.c:(.text+0x230): undefined reference to `thermal_zone_device_register'
This changes the symbol to a tristate, so Kconfig can track the
dependency correctly.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/thermal/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 8cc4ac64a91c..0852fd028dc9 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -195,7 +195,7 @@ config IMX_THERMAL
passive trip is crossed.
config SPEAR_THERMAL
- bool "SPEAr thermal sensor driver"
+ tristate "SPEAr thermal sensor driver"
depends on PLAT_SPEAR || COMPILE_TEST
depends on OF
help
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
2016-01-25 16:44 ` Arnd Bergmann
@ 2016-01-25 16:44 ` Arnd Bergmann
-1 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: Zhang Rui, Eduardo Valentin
Cc: linux-arm-kernel, linux-pm, spear-devel, Arnd Bergmann,
Linus Walleij, Lee Jones, linux-kernel
When the thermal subsystem is a loadable module, the u8500 driver
fails to build:
drivers/thermal/built-in.o: In function `db8500_thermal_probe':
db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
drivers/thermal/built-in.o: In function `db8500_thermal_work':
db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
This changes the symbol to a tristate, so Kconfig can track the
dependency correctly.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/mfd/db8500-prcmu.c | 3 +++
drivers/thermal/Kconfig | 4 ++--
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
index 2f4a1279b094..c0a86aeb1733 100644
--- a/drivers/mfd/db8500-prcmu.c
+++ b/drivers/mfd/db8500-prcmu.c
@@ -2045,6 +2045,7 @@ int db8500_prcmu_config_hotmon(u8 low, u8 high)
return 0;
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_config_hotmon);
static int config_hot_period(u16 val)
{
@@ -2071,11 +2072,13 @@ int db8500_prcmu_start_temp_sense(u16 cycles32k)
return config_hot_period(cycles32k);
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_start_temp_sense);
int db8500_prcmu_stop_temp_sense(void)
{
return config_hot_period(0xFFFF);
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_stop_temp_sense);
static int prcmu_a9wdog(u8 cmd, u8 d0, u8 d1, u8 d2, u8 d3)
{
diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0852fd028dc9..7c92c09be213 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -237,8 +237,8 @@ config DOVE_THERMAL
framework.
config DB8500_THERMAL
- bool "DB8500 thermal management"
- depends on ARCH_U8500
+ tristate "DB8500 thermal management"
+ depends on MFD_DB8500_PRCMU
default y
help
Adds DB8500 thermal management implementation according to the thermal
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
@ 2016-01-25 16:44 ` Arnd Bergmann
0 siblings, 0 replies; 17+ messages in thread
From: Arnd Bergmann @ 2016-01-25 16:44 UTC (permalink / raw)
To: linux-arm-kernel
When the thermal subsystem is a loadable module, the u8500 driver
fails to build:
drivers/thermal/built-in.o: In function `db8500_thermal_probe':
db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
drivers/thermal/built-in.o: In function `db8500_thermal_work':
db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
This changes the symbol to a tristate, so Kconfig can track the
dependency correctly.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/mfd/db8500-prcmu.c | 3 +++
drivers/thermal/Kconfig | 4 ++--
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
index 2f4a1279b094..c0a86aeb1733 100644
--- a/drivers/mfd/db8500-prcmu.c
+++ b/drivers/mfd/db8500-prcmu.c
@@ -2045,6 +2045,7 @@ int db8500_prcmu_config_hotmon(u8 low, u8 high)
return 0;
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_config_hotmon);
static int config_hot_period(u16 val)
{
@@ -2071,11 +2072,13 @@ int db8500_prcmu_start_temp_sense(u16 cycles32k)
return config_hot_period(cycles32k);
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_start_temp_sense);
int db8500_prcmu_stop_temp_sense(void)
{
return config_hot_period(0xFFFF);
}
+EXPORT_SYMBOL_GPL(db8500_prcmu_stop_temp_sense);
static int prcmu_a9wdog(u8 cmd, u8 d0, u8 d1, u8 d2, u8 d3)
{
diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 0852fd028dc9..7c92c09be213 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -237,8 +237,8 @@ config DOVE_THERMAL
framework.
config DB8500_THERMAL
- bool "DB8500 thermal management"
- depends on ARCH_U8500
+ tristate "DB8500 thermal management"
+ depends on MFD_DB8500_PRCMU
default y
help
Adds DB8500 thermal management implementation according to the thermal
--
2.7.0
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
2016-01-25 16:44 ` Arnd Bergmann
@ 2016-01-27 7:33 ` Lee Jones
-1 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2016-01-27 7:33 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Zhang Rui, Eduardo Valentin, linux-arm-kernel, linux-pm,
spear-devel, Linus Walleij, linux-kernel
On Mon, 25 Jan 2016, Arnd Bergmann wrote:
> When the thermal subsystem is a loadable module, the u8500 driver
> fails to build:
>
> drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> drivers/thermal/built-in.o: In function `db8500_thermal_work':
> db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
>
> This changes the symbol to a tristate, so Kconfig can track the
> dependency correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> drivers/mfd/db8500-prcmu.c | 3 +++
> drivers/thermal/Kconfig | 4 ++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
As this touches a core Thermal file, I think it's better off going in
via that tree. If that's a problem, I can take it with the relevant
Acks.
Acked-by: Lee Jones <lee.jones@linaro.org>
> diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
> index 2f4a1279b094..c0a86aeb1733 100644
> --- a/drivers/mfd/db8500-prcmu.c
> +++ b/drivers/mfd/db8500-prcmu.c
> @@ -2045,6 +2045,7 @@ int db8500_prcmu_config_hotmon(u8 low, u8 high)
>
> return 0;
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_config_hotmon);
>
> static int config_hot_period(u16 val)
> {
> @@ -2071,11 +2072,13 @@ int db8500_prcmu_start_temp_sense(u16 cycles32k)
>
> return config_hot_period(cycles32k);
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_start_temp_sense);
>
> int db8500_prcmu_stop_temp_sense(void)
> {
> return config_hot_period(0xFFFF);
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_stop_temp_sense);
>
> static int prcmu_a9wdog(u8 cmd, u8 d0, u8 d1, u8 d2, u8 d3)
> {
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 0852fd028dc9..7c92c09be213 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -237,8 +237,8 @@ config DOVE_THERMAL
> framework.
>
> config DB8500_THERMAL
> - bool "DB8500 thermal management"
> - depends on ARCH_U8500
> + tristate "DB8500 thermal management"
> + depends on MFD_DB8500_PRCMU
> default y
> help
> Adds DB8500 thermal management implementation according to the thermal
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
@ 2016-01-27 7:33 ` Lee Jones
0 siblings, 0 replies; 17+ messages in thread
From: Lee Jones @ 2016-01-27 7:33 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 25 Jan 2016, Arnd Bergmann wrote:
> When the thermal subsystem is a loadable module, the u8500 driver
> fails to build:
>
> drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> drivers/thermal/built-in.o: In function `db8500_thermal_work':
> db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
>
> This changes the symbol to a tristate, so Kconfig can track the
> dependency correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> drivers/mfd/db8500-prcmu.c | 3 +++
> drivers/thermal/Kconfig | 4 ++--
> 2 files changed, 5 insertions(+), 2 deletions(-)
As this touches a core Thermal file, I think it's better off going in
via that tree. If that's a problem, I can take it with the relevant
Acks.
Acked-by: Lee Jones <lee.jones@linaro.org>
> diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
> index 2f4a1279b094..c0a86aeb1733 100644
> --- a/drivers/mfd/db8500-prcmu.c
> +++ b/drivers/mfd/db8500-prcmu.c
> @@ -2045,6 +2045,7 @@ int db8500_prcmu_config_hotmon(u8 low, u8 high)
>
> return 0;
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_config_hotmon);
>
> static int config_hot_period(u16 val)
> {
> @@ -2071,11 +2072,13 @@ int db8500_prcmu_start_temp_sense(u16 cycles32k)
>
> return config_hot_period(cycles32k);
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_start_temp_sense);
>
> int db8500_prcmu_stop_temp_sense(void)
> {
> return config_hot_period(0xFFFF);
> }
> +EXPORT_SYMBOL_GPL(db8500_prcmu_stop_temp_sense);
>
> static int prcmu_a9wdog(u8 cmd, u8 d0, u8 d1, u8 d2, u8 d3)
> {
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 0852fd028dc9..7c92c09be213 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -237,8 +237,8 @@ config DOVE_THERMAL
> framework.
>
> config DB8500_THERMAL
> - bool "DB8500 thermal management"
> - depends on ARCH_U8500
> + tristate "DB8500 thermal management"
> + depends on MFD_DB8500_PRCMU
> default y
> help
> Adds DB8500 thermal management implementation according to the thermal
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
2016-01-25 16:44 ` Arnd Bergmann
(?)
@ 2016-02-05 14:44 ` Linus Walleij
-1 siblings, 0 replies; 17+ messages in thread
From: Linus Walleij @ 2016-02-05 14:44 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Zhang Rui, Eduardo Valentin, linux-arm-kernel, linux-pm,
spear-devel, Lee Jones, linux-kernel
On Mon, Jan 25, 2016 at 5:44 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> When the thermal subsystem is a loadable module, the u8500 driver
> fails to build:
>
> drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> drivers/thermal/built-in.o: In function `db8500_thermal_work':
> db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
>
> This changes the symbol to a tristate, so Kconfig can track the
> dependency correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
@ 2016-02-05 14:44 ` Linus Walleij
0 siblings, 0 replies; 17+ messages in thread
From: Linus Walleij @ 2016-02-05 14:44 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Zhang Rui, Eduardo Valentin, linux-arm-kernel, linux-pm,
spear-devel, Lee Jones, linux-kernel
On Mon, Jan 25, 2016 at 5:44 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> When the thermal subsystem is a loadable module, the u8500 driver
> fails to build:
>
> drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> drivers/thermal/built-in.o: In function `db8500_thermal_work':
> db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
>
> This changes the symbol to a tristate, so Kconfig can track the
> dependency correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
@ 2016-02-05 14:44 ` Linus Walleij
0 siblings, 0 replies; 17+ messages in thread
From: Linus Walleij @ 2016-02-05 14:44 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jan 25, 2016 at 5:44 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> When the thermal subsystem is a loadable module, the u8500 driver
> fails to build:
>
> drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> drivers/thermal/built-in.o: In function `db8500_thermal_work':
> db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
>
> This changes the symbol to a tristate, so Kconfig can track the
> dependency correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
2016-01-27 7:33 ` Lee Jones
@ 2016-02-09 22:10 ` Eduardo Valentin
-1 siblings, 0 replies; 17+ messages in thread
From: Eduardo Valentin @ 2016-02-09 22:10 UTC (permalink / raw)
To: Lee Jones
Cc: Arnd Bergmann, Zhang Rui, linux-arm-kernel, linux-pm,
spear-devel, Linus Walleij, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1100 bytes --]
On Wed, Jan 27, 2016 at 07:33:00AM +0000, Lee Jones wrote:
> On Mon, 25 Jan 2016, Arnd Bergmann wrote:
>
> > When the thermal subsystem is a loadable module, the u8500 driver
> > fails to build:
> >
> > drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> > db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> > drivers/thermal/built-in.o: In function `db8500_thermal_work':
> > db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
> >
> > This changes the symbol to a tristate, so Kconfig can track the
> > dependency correctly.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > ---
> > drivers/mfd/db8500-prcmu.c | 3 +++
> > drivers/thermal/Kconfig | 4 ++--
> > 2 files changed, 5 insertions(+), 2 deletions(-)
>
> As this touches a core Thermal file, I think it's better off going in
> via that tree. If that's a problem, I can take it with the relevant
> Acks.
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Thanks, I am adding to linux-next to see how it goes.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
@ 2016-02-09 22:10 ` Eduardo Valentin
0 siblings, 0 replies; 17+ messages in thread
From: Eduardo Valentin @ 2016-02-09 22:10 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jan 27, 2016 at 07:33:00AM +0000, Lee Jones wrote:
> On Mon, 25 Jan 2016, Arnd Bergmann wrote:
>
> > When the thermal subsystem is a loadable module, the u8500 driver
> > fails to build:
> >
> > drivers/thermal/built-in.o: In function `db8500_thermal_probe':
> > db8500_thermal.c:(.text+0x96c): undefined reference to `thermal_zone_device_register'
> > drivers/thermal/built-in.o: In function `db8500_thermal_work':
> > db8500_thermal.c:(.text+0xab4): undefined reference to `thermal_zone_device_update'
> >
> > This changes the symbol to a tristate, so Kconfig can track the
> > dependency correctly.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > ---
> > drivers/mfd/db8500-prcmu.c | 3 +++
> > drivers/thermal/Kconfig | 4 ++--
> > 2 files changed, 5 insertions(+), 2 deletions(-)
>
> As this touches a core Thermal file, I think it's better off going in
> via that tree. If that's a problem, I can take it with the relevant
> Acks.
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Thanks, I am adding to linux-next to see how it goes.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160209/3f8a8e06/attachment.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH 0/3] randconfig warning fixes for thermal
2016-01-25 16:44 ` Arnd Bergmann
@ 2016-02-09 22:18 ` Eduardo Valentin
-1 siblings, 0 replies; 17+ messages in thread
From: Eduardo Valentin @ 2016-02-09 22:18 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: Zhang Rui, linux-arm-kernel, linux-pm, spear-devel
[-- Attachment #1: Type: text/plain, Size: 548 bytes --]
On Mon, Jan 25, 2016 at 05:44:09PM +0100, Arnd Bergmann wrote:
> These are three simple fixes for ARM randconfig builds:
>
> [PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
> [PATCH 2/3] thermal: allow spear-thermal driver to be a module
> [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
>
> All three bugs are on configurationsn that are not important and
> that rarely happen even with randconfig builds.
>
> Arnd
Thanks a lot for your cleanups Arnd,
queueing all three.
Eduardo Valentin
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH 0/3] randconfig warning fixes for thermal
@ 2016-02-09 22:18 ` Eduardo Valentin
0 siblings, 0 replies; 17+ messages in thread
From: Eduardo Valentin @ 2016-02-09 22:18 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jan 25, 2016 at 05:44:09PM +0100, Arnd Bergmann wrote:
> These are three simple fixes for ARM randconfig builds:
>
> [PATCH 1/3] thermal: spear: use __maybe_unused for PM functions
> [PATCH 2/3] thermal: allow spear-thermal driver to be a module
> [PATCH 3/3] thermal: allow u8500-thermal driver to be a module
>
> All three bugs are on configurationsn that are not important and
> that rarely happen even with randconfig builds.
>
> Arnd
Thanks a lot for your cleanups Arnd,
queueing all three.
Eduardo Valentin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160209/57f41a2f/attachment.sig>
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2016-02-09 22:18 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-25 16:44 [PATCH 0/3] randconfig warning fixes for thermal Arnd Bergmann
2016-01-25 16:44 ` Arnd Bergmann
2016-01-25 16:44 ` [PATCH 1/3] thermal: spear: use __maybe_unused for PM functions Arnd Bergmann
2016-01-25 16:44 ` Arnd Bergmann
2016-01-25 16:44 ` [PATCH 2/3] thermal: allow spear-thermal driver to be a module Arnd Bergmann
2016-01-25 16:44 ` Arnd Bergmann
2016-01-25 16:44 ` [PATCH 3/3] thermal: allow u8500-thermal " Arnd Bergmann
2016-01-25 16:44 ` Arnd Bergmann
2016-01-27 7:33 ` Lee Jones
2016-01-27 7:33 ` Lee Jones
2016-02-09 22:10 ` Eduardo Valentin
2016-02-09 22:10 ` Eduardo Valentin
2016-02-05 14:44 ` Linus Walleij
2016-02-05 14:44 ` Linus Walleij
2016-02-05 14:44 ` Linus Walleij
2016-02-09 22:18 ` [PATCH 0/3] randconfig warning fixes for thermal Eduardo Valentin
2016-02-09 22:18 ` Eduardo Valentin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.