Re: [V2 Patch] nmi watchdog: do not use cpp symbol in Kconfig

From: Mike Frysinger
Date: Wed Feb 01 2012 - 13:55:27 EST


On Wed, Feb 1, 2012 at 07:31, Cong Wang <xiyou.wangcong@xxxxxxxxx> wrote:
> ARCH_HAS_NMI_WATCHDOG is a macro defined by arch,
> but config HARDLOCKUP_DETECTOR depends on it.
> This is wrong, ARCH_HAS_NMI_WATCHDOG has to be a Kconfig
> config, and arch's need it should select it explicitly.
>
> ---
> Âarch/Kconfig             Â|  Â3 +++
> Âarch/blackfin/Kconfig         |  Â1 +
> Âarch/blackfin/include/asm/irq.h    |  Â4 ----
> Âarch/mn10300/Kconfig         Â|  Â1 +
> Âarch/mn10300/include/asm/reset-regs.h | Â Â4 ----
> Âarch/sparc/Kconfig          Â|  Â1 +
> Âarch/sparc/include/asm/irq_64.h    |  Â1 -
> Âinclude/linux/nmi.h          |  Â2 +-
> Âlib/Kconfig.debug           |  Â2 +-
> Â9 files changed, 8 insertions(+), 11 deletions(-)

Acked-by: Mike Frysinger <vapier@xxxxxxxxxx>
-mike
N‹§²æìr¸›yúèšØb²X¬¶ÇvØ^–)Þ{.nÇ+‰·¥Š{±‘êçzX§¶›¡Ü}©ž²ÆzÚ&j:+v‰¨¾«‘êçzZ+€Ê+zf£¢·hšˆ§~†­†Ûiÿûàz¹®w¥¢¸?™¨è­Ú&¢)ßf”ù^jÇy§m…á@A«a¶Úÿ 0¶ìh®å’i