* [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM
@ 2014-10-04 9:39 Chen Gang
2014-10-04 9:45 ` Chen Gang
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Chen Gang @ 2014-10-04 9:39 UTC (permalink / raw)
To: wim, linux-watchdog, linux-kernel
They need HAS_IOMEM, so let them depend on it, the related error (with
allmodconfig under um):
MODPOST 1205 modules
ERROR: "syscall" [fs/hostfs/hostfs.ko] undefined!
ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined!
ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined!
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
drivers/watchdog/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index a51ccf3..d0ddf90 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -132,6 +132,7 @@ config WM8350_WATCHDOG
config XILINX_WATCHDOG
tristate "Xilinx Watchdog timer"
+ depends on HAS_IOMEM
select WATCHDOG_CORE
help
Watchdog driver for the xps_timebase_wdt ip core.
@@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG
config TEGRA_WATCHDOG
tristate "Tegra watchdog"
- depends on ARCH_TEGRA || COMPILE_TEST
+ depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM
select WATCHDOG_CORE
help
Say Y here to include support for the watchdog timer
--
1.9.3
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM 2014-10-04 9:39 [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM Chen Gang @ 2014-10-04 9:45 ` Chen Gang 2014-10-05 0:18 ` Guenter Roeck 2014-10-05 1:28 ` [PATCH v2] watchdog: " Chen Gang 2 siblings, 0 replies; 6+ messages in thread From: Chen Gang @ 2014-10-04 9:45 UTC (permalink / raw) To: wim, linux-watchdog, linux-kernel On 10/4/14 17:39, Chen Gang wrote: > They need HAS_IOMEM, so let them depend on it, the related error (with > allmodconfig under um): > > MODPOST 1205 modules > ERROR: "syscall" [fs/hostfs/hostfs.ko] undefined! ^ Sorry, the line above is useless for this issue, please skip it. Thanks. > ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! > ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined! > > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > drivers/watchdog/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index a51ccf3..d0ddf90 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -132,6 +132,7 @@ config WM8350_WATCHDOG > > config XILINX_WATCHDOG > tristate "Xilinx Watchdog timer" > + depends on HAS_IOMEM > select WATCHDOG_CORE > help > Watchdog driver for the xps_timebase_wdt ip core. > @@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG > > config TEGRA_WATCHDOG > tristate "Tegra watchdog" > - depends on ARCH_TEGRA || COMPILE_TEST > + depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM > select WATCHDOG_CORE > help > Say Y here to include support for the watchdog timer > -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM 2014-10-04 9:39 [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM Chen Gang 2014-10-04 9:45 ` Chen Gang @ 2014-10-05 0:18 ` Guenter Roeck 2014-10-05 1:25 ` Chen Gang 2014-10-05 1:28 ` [PATCH v2] watchdog: " Chen Gang 2 siblings, 1 reply; 6+ messages in thread From: Guenter Roeck @ 2014-10-05 0:18 UTC (permalink / raw) To: Chen Gang, wim, linux-watchdog, linux-kernel On 10/04/2014 02:39 AM, Chen Gang wrote: > They need HAS_IOMEM, so let them depend on it, the related error (with > allmodconfig under um): > > MODPOST 1205 modules > ERROR: "syscall" [fs/hostfs/hostfs.ko] undefined! > ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! > ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined! > Headline is a bit excessive ("watchdog: ..." would have been sufficient, the first error is irrelevant as you found out yourself, and you got an extra empty line here. Otherwise looks good. Can you resubmit with that fixed ? On a side note, guess the conclusion was to not provide a dummy for devm_ioremap_resource if HAS_IOMEM is not defined. Too bad. Guenter > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> > --- > drivers/watchdog/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index a51ccf3..d0ddf90 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -132,6 +132,7 @@ config WM8350_WATCHDOG > > config XILINX_WATCHDOG > tristate "Xilinx Watchdog timer" > + depends on HAS_IOMEM > select WATCHDOG_CORE > help > Watchdog driver for the xps_timebase_wdt ip core. > @@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG > > config TEGRA_WATCHDOG > tristate "Tegra watchdog" > - depends on ARCH_TEGRA || COMPILE_TEST > + depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM > select WATCHDOG_CORE > help > Say Y here to include support for the watchdog timer > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM 2014-10-05 0:18 ` Guenter Roeck @ 2014-10-05 1:25 ` Chen Gang 0 siblings, 0 replies; 6+ messages in thread From: Chen Gang @ 2014-10-05 1:25 UTC (permalink / raw) To: Guenter Roeck, wim, linux-watchdog, linux-kernel On 10/5/14 8:18, Guenter Roeck wrote: > On 10/04/2014 02:39 AM, Chen Gang wrote: >> They need HAS_IOMEM, so let them depend on it, the related error (with >> allmodconfig under um): >> >> MODPOST 1205 modules >> ERROR: "syscall" [fs/hostfs/hostfs.ko] undefined! >> ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! >> ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined! >> > Headline is a bit excessive ("watchdog: ..." would have been sufficient, > the first error is irrelevant as you found out yourself, and you got an > extra empty line here. Otherwise looks good. > > Can you resubmit with that fixed ? > OK, thank, I shall send patch v2 for it. > On a side note, guess the conclusion was to not provide a dummy for > devm_ioremap_resource if HAS_IOMEM is not defined. Too bad. > For me, it is checked during config time, so need report error during compiling or building time instead of dummy. And I'd like to let devm_ioremap* quoted by "#if HAS_IOMEM", so can find this issue during compiling time, not in linking time (which can save builder's time resource). Thanks. >> >> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> >> --- >> drivers/watchdog/Kconfig | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig >> index a51ccf3..d0ddf90 100644 >> --- a/drivers/watchdog/Kconfig >> +++ b/drivers/watchdog/Kconfig >> @@ -132,6 +132,7 @@ config WM8350_WATCHDOG >> >> config XILINX_WATCHDOG >> tristate "Xilinx Watchdog timer" >> + depends on HAS_IOMEM >> select WATCHDOG_CORE >> help >> Watchdog driver for the xps_timebase_wdt ip core. >> @@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG >> >> config TEGRA_WATCHDOG >> tristate "Tegra watchdog" >> - depends on ARCH_TEGRA || COMPILE_TEST >> + depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM >> select WATCHDOG_CORE >> help >> Say Y here to include support for the watchdog timer >> > -- Chen Gang Open, share, and attitude like air, water, and life which God blessed ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2] watchdog: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM 2014-10-04 9:39 [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM Chen Gang 2014-10-04 9:45 ` Chen Gang 2014-10-05 0:18 ` Guenter Roeck @ 2014-10-05 1:28 ` Chen Gang 2014-10-05 1:51 ` Guenter Roeck 2 siblings, 1 reply; 6+ messages in thread From: Chen Gang @ 2014-10-05 1:28 UTC (permalink / raw) To: wim, Guenter Roeck; +Cc: linux-watchdog, linux-kernel They need HAS_IOMEM, so let them depend on it, the related error (with allmodconfig under um): MODPOST 1205 modules ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined! Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> --- drivers/watchdog/Kconfig | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index a51ccf3..d0ddf90 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -132,6 +132,7 @@ config WM8350_WATCHDOG config XILINX_WATCHDOG tristate "Xilinx Watchdog timer" + depends on HAS_IOMEM select WATCHDOG_CORE help Watchdog driver for the xps_timebase_wdt ip core. @@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG config TEGRA_WATCHDOG tristate "Tegra watchdog" - depends on ARCH_TEGRA || COMPILE_TEST + depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM select WATCHDOG_CORE help Say Y here to include support for the watchdog timer -- 1.9.3 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] watchdog: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM 2014-10-05 1:28 ` [PATCH v2] watchdog: " Chen Gang @ 2014-10-05 1:51 ` Guenter Roeck 0 siblings, 0 replies; 6+ messages in thread From: Guenter Roeck @ 2014-10-05 1:51 UTC (permalink / raw) To: Chen Gang, wim; +Cc: linux-watchdog, linux-kernel On 10/04/2014 06:28 PM, Chen Gang wrote: > They need HAS_IOMEM, so let them depend on it, the related error (with > allmodconfig under um): > > MODPOST 1205 modules > ERROR: "devm_ioremap_resource" [drivers/watchdog/tegra_wdt.ko] undefined! > ERROR: "devm_ioremap_resource" [drivers/watchdog/of_xilinx_wdt.ko] undefined! > > Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> > --- > drivers/watchdog/Kconfig | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index a51ccf3..d0ddf90 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -132,6 +132,7 @@ config WM8350_WATCHDOG > > config XILINX_WATCHDOG > tristate "Xilinx Watchdog timer" > + depends on HAS_IOMEM > select WATCHDOG_CORE > help > Watchdog driver for the xps_timebase_wdt ip core. > @@ -472,7 +473,7 @@ config SIRFSOC_WATCHDOG > > config TEGRA_WATCHDOG > tristate "Tegra watchdog" > - depends on ARCH_TEGRA || COMPILE_TEST > + depends on (ARCH_TEGRA || COMPILE_TEST) && HAS_IOMEM > select WATCHDOG_CORE > help > Say Y here to include support for the watchdog timer > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-10-05 1:51 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-10-04 9:39 [PATCH] drivers/watchdog/Kconfig: Let XILINX_WATCHDOG and TEGRA_WATCHDOG depend on HAS_IOMEM Chen Gang 2014-10-04 9:45 ` Chen Gang 2014-10-05 0:18 ` Guenter Roeck 2014-10-05 1:25 ` Chen Gang 2014-10-05 1:28 ` [PATCH v2] watchdog: " Chen Gang 2014-10-05 1:51 ` Guenter Roeck
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).