On 06/28/2015 10:18 PM, kbuild test robot wrote:
tree:   git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   e3984319760987beee59b1d017416671ebb89b4c
commit: 387ee4d26e5f530e8a6a850cc0a817fad6324686 [13253/13461] watchdog: 
omap_wdt: early_enable module parameter
config: arm-omap2plus_defconfig (attached as .config)
reproduce:
   wget 
https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross
 -O ~/bin/make.cross
   chmod +x ~/bin/make.cross
   git checkout 387ee4d26e5f530e8a6a850cc0a817fad6324686
   # save the attached .config to linux build tree
   make.cross ARCH=arm

All error/warnings (new ones prefixed by >>):

    drivers/watchdog/omap_wdt.c: In function 'omap_wdt_probe':
drivers/watchdog/omap_wdt.c:288:18: error: 'omap_wdt' undeclared (first use in 
this function)
       omap_wdt_start(omap_wdt);
                      ^
    drivers/watchdog/omap_wdt.c:288:18: note: each undeclared identifier is 
reported only once for each function it appears in

vim +/omap_wdt +288 drivers/watchdog/omap_wdt.c

    282                 readl_relaxed(wdev->base + OMAP_WATCHDOG_REV) & 0xFF,
    283                 wdev->wdog.timeout);
    284 
    285         pm_runtime_put_sync(wdev->dev);
    286 
    287         if (early_enable)
  > 288                      omap_wdt_start(omap_wdt);
    289 
    290         return 0;
    291 }


Commit 'watchdog: omap_wdt: early_enable module parameter' interfers with
'watchdog: omap: put struct watchdog_device into driver data'. Wonder why
my test build didn't catch this ... ah, I didn't push that commit into my
test repository. Oh well :-(.

Wim, do you want to fix it up in the commit, or should Lars send a patch
to fix it on top of -next ? Should be something like

-                       omap_wdt_start(omap_wdt);
+                       omap_wdt_start(&wdev->wdog);

Thanks,
Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to