On Fri, 14 Mar 2008, Roberto Riggio wrote:
> > i'm confused ... doesn't that makefile already take the watchdog path
> > change into account with the following:
>
> Ok, I didn't known of the $(WATCHDOG_DIR) because it was not used for
> that module.
>
> > so don't hardcode that change, use $(WATCHDOG_DIR) instead, no?
>
> Ok, so the real patch would be:
>
> 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_DIR)/scx200_wdt.$(LINUX_KMOD_SUFFIX)
> AUTOLOAD:=$(call AutoLoad,50,scx200_wdt)
> endef
that seems like the obvious fix, yes.
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
