* [PATCH] watchdog: fix ARCH_BCM_MOBILE dependency
@ 2014-03-12 22:12 Alex Elder
2014-03-13 0:00 ` Guenter Roeck
0 siblings, 1 reply; 3+ messages in thread
From: Alex Elder @ 2014-03-12 22:12 UTC (permalink / raw)
To: wim; +Cc: linux-watchdog, linux-kernel
Starting with this commit:
047ef2fa rename ARCH_BCM to ARCH_BCM_MOBILE (clocksource)
the meaning of the ARCH_BCM config option is changed to represent
all Broadcom chips with code in the mach-bcm directory.
Configuration options related to specific Broadcom platforms should
now use another symbol (currently ARCH_BCM_MOBILE, ARCH_BCM2835, or
ARCH_BCM_5301X).
The BCM_KONA_WDT config option indicates a dependency on ARCH_BCM,
but it should be ARCH_BCM_MOBILE instead. Fix that.
Signed-off-by: Alex Elder <elder@linaro.org>
---
drivers/watchdog/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
index 79d2589..47a6f10 100644
--- a/drivers/watchdog/Kconfig
+++ b/drivers/watchdog/Kconfig
@@ -1160,7 +1160,7 @@ config BCM2835_WDT
config BCM_KONA_WDT
tristate "BCM Kona Watchdog"
- depends on ARCH_BCM
+ depends on ARCH_BCM_MOBILE
select WATCHDOG_CORE
help
Support for the watchdog timer on the following Broadcom BCM281xx
--
1.7.9.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] watchdog: fix ARCH_BCM_MOBILE dependency
2014-03-12 22:12 [PATCH] watchdog: fix ARCH_BCM_MOBILE dependency Alex Elder
@ 2014-03-13 0:00 ` Guenter Roeck
2014-03-13 1:41 ` Alex Elder
0 siblings, 1 reply; 3+ messages in thread
From: Guenter Roeck @ 2014-03-13 0:00 UTC (permalink / raw)
To: Alex Elder, wim; +Cc: linux-watchdog, linux-kernel
On 03/12/2014 03:12 PM, Alex Elder wrote:
> Starting with this commit:
> 047ef2fa rename ARCH_BCM to ARCH_BCM_MOBILE (clocksource)
> the meaning of the ARCH_BCM config option is changed to represent
> all Broadcom chips with code in the mach-bcm directory.
>
> Configuration options related to specific Broadcom platforms should
> now use another symbol (currently ARCH_BCM_MOBILE, ARCH_BCM2835, or
> ARCH_BCM_5301X).
>
> The BCM_KONA_WDT config option indicates a dependency on ARCH_BCM,
> but it should be ARCH_BCM_MOBILE instead. Fix that.
>
> Signed-off-by: Alex Elder <elder@linaro.org>
Hi Alex,
Markus Mayer already submitted a similar patch:
http://patchwork.roeck-us.net/patch/1263/
Guenter
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] watchdog: fix ARCH_BCM_MOBILE dependency
2014-03-13 0:00 ` Guenter Roeck
@ 2014-03-13 1:41 ` Alex Elder
0 siblings, 0 replies; 3+ messages in thread
From: Alex Elder @ 2014-03-13 1:41 UTC (permalink / raw)
To: Guenter Roeck, wim; +Cc: linux-watchdog, linux-kernel
On 03/12/2014 07:00 PM, Guenter Roeck wrote:
> On 03/12/2014 03:12 PM, Alex Elder wrote:
>> Starting with this commit:
>> 047ef2fa rename ARCH_BCM to ARCH_BCM_MOBILE (clocksource)
>> the meaning of the ARCH_BCM config option is changed to represent
>> all Broadcom chips with code in the mach-bcm directory.
>>
>> Configuration options related to specific Broadcom platforms should
>> now use another symbol (currently ARCH_BCM_MOBILE, ARCH_BCM2835, or
>> ARCH_BCM_5301X).
>>
>> The BCM_KONA_WDT config option indicates a dependency on ARCH_BCM,
>> but it should be ARCH_BCM_MOBILE instead. Fix that.
>>
>> Signed-off-by: Alex Elder <elder@linaro.org>
>
> Hi Alex,
>
> Markus Mayer already submitted a similar patch:
> http://patchwork.roeck-us.net/patch/1263/
Yes someone else pointed that out to me shortly
after I sent mine. I wasn't aware he had done it.
I discovered the problem independently today and
just sent out the fix. Thanks.
-Alex
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-03-13 1:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-12 22:12 [PATCH] watchdog: fix ARCH_BCM_MOBILE dependency Alex Elder
2014-03-13 0:00 ` Guenter Roeck
2014-03-13 1:41 ` Alex Elder
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox