On Fri, 14 Mar 2008, Roberto Riggio wrote:

> Signed-off-by: Roberto Riggio <[EMAIL PROTECTED]>
>
> --
> Index: package/kernel/modules/other.mk
> ===================================================================
> --- package/kernel/modules/other.mk   (revisione 10602)
> +++ package/kernel/modules/other.mk   (copia locale)
> @@ -419,7 +419,7 @@
>    TITLE:=Natsemi SCX200 Watchdog support
>    DEPENDS:[EMAIL PROTECTED]
>    KCONFIG:=CONFIG_SC1200_WDT
> -  FILES:=$(LINUX_DIR)/drivers/char/watchdog/scx200_wdt.$(LINUX_KMOD_SUFFIX)
> +  FILES:=$(LINUX_DIR)/drivers/watchdog/scx200_wdt.$(LINUX_KMOD_SUFFIX)
>    AUTOLOAD:=$(call AutoLoad,50,scx200_wdt)
>  endef

i'm confused ... doesn't that makefile already take the watchdog path
change into account with the following:

===
# XXX: added a workaround for watchdog path changes
ifeq ($(KERNEL),2.4)
  WATCHDOG_DIR=char
endif
ifeq ($(KERNEL_PATCHVER),2.6.24)
  WATCHDOG_DIR=watchdog
endif
WATCHDOG_DIR?=char/watchdog
===

  so don't hardcode that change, use $(WATCHDOG_DIR) instead, no?

rday
--



========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry:
    Have classroom, will lecture.

http://crashcourse.ca                          Waterloo, Ontario, CANADA
========================================================================
_______________________________________________
openwrt-devel mailing list
[email protected]
http://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to