Hi Linus, Please pull from 'master' branch of git://www.linux-watchdog.org/linux-watchdog.git This pull request contains: * addition of MOXA ART watchdog driver (moxart_wdt) * addition of CSR SiRFprimaII and SiRFatlasVI watchdog driver (sirfsoc_wdt) * addition of ralink watchdog driver (rt2880_wdt) * various fixes and cleanups (__user annotation, ioctl return codes, removal of redundant of_match_ptr, removal of unnecessary amba_set_drvdata(), use allocated buffer for usb_control_msg, ...) * removal of MODULE_ALIAS_MISCDEV statements * watchdog related DT bindings * first set of improvements on the w83627hf_wdt driver This will update the following files: Documentation/devicetree/bindings/gpio/men-a021-wdt.txt | 22 b/Documentation/devicetree/bindings/watchdog/dw_wdt.txt | 21 b/Documentation/devicetree/bindings/watchdog/men-a021-wdt.txt | 25 b/Documentation/devicetree/bindings/watchdog/moxa,moxart-watchdog.txt | 15 b/Documentation/devicetree/bindings/watchdog/rt2880-wdt.txt | 19 b/Documentation/devicetree/bindings/watchdog/sirfsoc_wdt.txt | 14 b/arch/arm/configs/prima2_defconfig | 1 b/drivers/watchdog/Kconfig | 27 b/drivers/watchdog/Makefile | 3 b/drivers/watchdog/acquirewdt.c | 4 b/drivers/watchdog/advantechwdt.c | 1 b/drivers/watchdog/alim1535_wdt.c | 1 b/drivers/watchdog/alim7101_wdt.c | 1 b/drivers/watchdog/ar7_wdt.c | 1 b/drivers/watchdog/at32ap700x_wdt.c | 1 b/drivers/watchdog/at91rm9200_wdt.c | 3 b/drivers/watchdog/ath79_wdt.c | 1 b/drivers/watchdog/bcm2835_wdt.c | 1 b/drivers/watchdog/bcm63xx_wdt.c | 1 b/drivers/watchdog/bfin_wdt.c | 1 b/drivers/watchdog/cpu5wdt.c | 1 b/drivers/watchdog/davinci_wdt.c | 1 b/drivers/watchdog/dw_wdt.c | 36 - b/drivers/watchdog/ep93xx_wdt.c | 1 b/drivers/watchdog/eurotechwdt.c | 1 b/drivers/watchdog/gef_wdt.c | 1 b/drivers/watchdog/geodewdt.c | 1 b/drivers/watchdog/hpwdt.c | 1 b/drivers/watchdog/i6300esb.c | 1 b/drivers/watchdog/iTCO_wdt.c | 5 b/drivers/watchdog/ib700wdt.c | 1 b/drivers/watchdog/ibmasr.c | 1 b/drivers/watchdog/ie6xx_wdt.c | 1 b/drivers/watchdog/imx2_wdt.c | 2 b/drivers/watchdog/indydog.c | 1 b/drivers/watchdog/intel_scu_watchdog.c | 1 b/drivers/watchdog/iop_wdt.c | 1 b/drivers/watchdog/it8712f_wdt.c | 1 b/drivers/watchdog/it87_wdt.c | 1 b/drivers/watchdog/ixp4xx_wdt.c | 2 b/drivers/watchdog/jz4740_wdt.c | 1 b/drivers/watchdog/kempld_wdt.c | 5 b/drivers/watchdog/ks8695_wdt.c | 1 b/drivers/watchdog/lantiq_wdt.c | 1 b/drivers/watchdog/m54xx_wdt.c | 1 b/drivers/watchdog/machzwd.c | 1 b/drivers/watchdog/max63xx_wdt.c | 1 b/drivers/watchdog/mixcomwd.c | 1 b/drivers/watchdog/moxart_wdt.c | 165 ++++ b/drivers/watchdog/mpc8xxx_wdt.c | 1 b/drivers/watchdog/mtx-1_wdt.c | 1 b/drivers/watchdog/mv64x60_wdt.c | 3 b/drivers/watchdog/nuc900_wdt.c | 1 b/drivers/watchdog/nv_tco.c | 1 b/drivers/watchdog/of_xilinx_wdt.c | 1 b/drivers/watchdog/omap_wdt.c | 38 - b/drivers/watchdog/orion_wdt.c | 3 b/drivers/watchdog/pc87413_wdt.c | 2 b/drivers/watchdog/pcwd.c | 4 b/drivers/watchdog/pcwd_pci.c | 4 b/drivers/watchdog/pcwd_usb.c | 16 b/drivers/watchdog/pika_wdt.c | 2 b/drivers/watchdog/pnx4008_wdt.c | 1 b/drivers/watchdog/pnx833x_wdt.c | 1 b/drivers/watchdog/rc32434_wdt.c | 4 b/drivers/watchdog/rdc321x_wdt.c | 3 b/drivers/watchdog/rt2880_wdt.c | 207 ++++++ b/drivers/watchdog/s3c2410_wdt.c | 2 b/drivers/watchdog/sa1100_wdt.c | 1 b/drivers/watchdog/sb_wdog.c | 1 b/drivers/watchdog/sbc60xxwdt.c | 1 b/drivers/watchdog/sbc7240_wdt.c | 2 b/drivers/watchdog/sbc8360.c | 1 b/drivers/watchdog/sbc_epx_c3.c | 1 b/drivers/watchdog/sbc_fitpc2_wdt.c | 2 b/drivers/watchdog/sc1200wdt.c | 1 b/drivers/watchdog/sc520_wdt.c | 1 b/drivers/watchdog/sch311x_wdt.c | 5 b/drivers/watchdog/scx200_wdt.c | 1 b/drivers/watchdog/shwdt.c | 1 b/drivers/watchdog/sirfsoc_wdt.c | 226 ++++++ b/drivers/watchdog/smsc37b787_wdt.c | 2 b/drivers/watchdog/softdog.c | 1 b/drivers/watchdog/sp5100_tco.c | 1 b/drivers/watchdog/sp805_wdt.c | 1 b/drivers/watchdog/stmp3xxx_rtc_wdt.c | 5 b/drivers/watchdog/sunxi_wdt.c | 2 b/drivers/watchdog/ts72xx_wdt.c | 38 - b/drivers/watchdog/txx9wdt.c | 1 b/drivers/watchdog/ux500_wdt.c | 3 b/drivers/watchdog/w83627hf_wdt.c | 341 +++------- b/drivers/watchdog/w83697hf_wdt.c | 1 b/drivers/watchdog/w83697ug_wdt.c | 1 b/drivers/watchdog/w83877f_wdt.c | 1 b/drivers/watchdog/w83977f_wdt.c | 1 b/drivers/watchdog/wafer5823wdt.c | 1 b/drivers/watchdog/watchdog_core.c | 2 b/drivers/watchdog/wdrtas.c | 2 b/drivers/watchdog/wdt.c | 2 b/drivers/watchdog/wdt285.c | 1 b/drivers/watchdog/wdt977.c | 1 b/drivers/watchdog/wdt_pci.c | 2 b/drivers/watchdog/wm831x_wdt.c | 8 b/drivers/watchdog/xen_wdt.c | 1 104 files changed, 930 insertions(+), 429 deletions(-) with these Changes: commit ef0c1a6b1b05f85f0a1cf6ef283512fd25d86ad4 Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Sat Aug 17 13:58:42 2013 -0700 watchdog: w83627hf: Use helper functions to access superio registers Use helper functions named similar to other drivers to access superio registers. Request memory region only when needed, and use request_muxed_region(). This lets other devices (hwmon, gpio) use the same region. Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit ac461103c57ac46cd6778f1fd0352ff0e4a2d38e Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Sat Aug 17 13:58:41 2013 -0700 watchdog: w83627hf: Enable watchdog device only if not already enabled There is no need to enable the watchdog device if it is already enabled. Also, when enabling the watchdog device, only set the watchdog device enable bit and do not touch other bits; depending on the chip type, those bits may enable other functionality. Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 8f526389599fc543b204c12a36de8345eb298085 Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Sat Aug 17 13:58:40 2013 -0700 watchdog: w83627hf: Enable watchdog only once It is unnecessary to enable the logical device and WDT0 each time the watchdog is accessed. Do it only once during initialization. Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 30a83695aa2ed3ffb1445d6daef1e620c3ca4e6d Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Mon Oct 28 19:43:57 2013 -0700 watchdog: w83627hf: Convert to watchdog infrastructure Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 4a7e94a0637da7e1ddce748da49ae780c370eeef Author: Victor Kamensky <victor.kamensky@xxxxxxxxxx> Date: Sat Nov 16 02:01:05 2013 +0200 watchdog: omap_wdt: raw read and write endian fix All OMAP IP blocks expect LE data, but CPU may operate in BE mode. Need to use endian neutral functions to read/write h/w registers. I.e instead of __raw_read[lw] and __raw_write[lw] functions code need to use read[lw]_relaxed and write[lw]_relaxed functions. If the first simply reads/writes register, the second will byteswap it if host operates in BE mode. Changes are trivial sed like replacement of __raw_xxx functions with xxx_relaxed variant. Signed-off-by: Victor Kamensky <victor.kamensky@xxxxxxxxxx> Signed-off-by: Taras Kondratiuk <taras.kondratiuk@xxxxxxxxxx> Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit b0df38dd3554d47fcfd5c1183951c2fe7dd07eda Author: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> Date: Mon Nov 11 21:33:44 2013 +0100 watchdog: sirf: don't depend on dummy value of CLOCK_TICK_RATE As CSR SiRF is converted to multi platform CLOCK_TICK_RATE is a dummy value that seems to match the right value is used. (arch/arm/mach-prima2/include/mach/timex.h which defined CLOCK_TICK_RATE to 1000000 was removed in commit cf82e0e (ARM: sirf: enable multiplatform support); marco used the same file.) To not depend on that dummy value use a local #define instead. Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit ab5bbdc7015bb35c364ad5f3c955736b17d1acb6 Author: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Date: Fri Nov 8 01:24:19 2013 -0800 watchdog: pcwd_usb: overflow in usb_pcwd_send_command() We changed "buf" from being an array of 6 chars to being a pointer this sizeof(buf) needs to be updated as well. Fixes: 2ddb8089a7e5 ('watchdog: pcwd_usb: Use allocated buffer for usb_control_msg') Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 0859ffc3b3cb673ac1d3f33b0265cf60d1e3836d Author: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> Date: Thu Oct 31 15:50:55 2013 +0800 watchdog: rt2880_wdt: fix return value check in rt288x_wdt_probe() In case of error, the function devm_request_and_ioremap() returns NULL pointer not ERR_PTR(). Fix it by using devm_ioremap_resource() instead of devm_request_and_ioremap(). Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 6ffcff931971ce78d08220a7a3bbaddaea207598 Author: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Date: Mon Oct 28 14:17:17 2013 +0530 watchdog: watchdog_core: Fix a trivial typo Fixed a trivial typo. Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 58e5637333afc2bb024ff8aef212311be0614d47 Author: Dinh Nguyen <dinguyen@xxxxxxxxxx> Date: Tue Oct 22 11:59:12 2013 -0500 watchdog: dw: Enable OF support for DW watchdog timer Add device tree support to the DW watchdog timer. Signed-off-by: Dinh Nguyen <dinguyen@xxxxxxxxxx> Acked-by: Jamie Iles <jamie@xxxxxxxxxxxxx> Acked-by: Mark Rutland <mark.rutland@xxxxxxx> Reviewed-by: Pavel Machek <pavel@xxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> Cc: Viresh Kumar <viresh.linux@xxxxxxxxx> Cc: Rob Herring <rob.herring@xxxxxxxxxxx> Cc: Pawel Moll <pawel.moll@xxxxxxx> Cc: Stephen Warren <swarren@xxxxxxxxxxxxx> Cc: Ian Campbell <ijc+devicetree@xxxxxxxxxxxxxx> Cc: devicetree@xxxxxxxxxxxxxxx Cc: linux-watchdog@xxxxxxxxxxxxxxx commit 487722cf2d66126338217896642bd5eec832c34b Author: Jean Delvare <jdelvare@xxxxxxx> Date: Mon Oct 21 17:38:49 2013 +0200 watchdog: Get rid of MODULE_ALIAS_MISCDEV statements I just can't find any value in MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR) and MODULE_ALIAS_MISCDEV(TEMP_MINOR) statements. Either the device is enumerated and the driver already has a module alias (e.g. PCI, USB etc.) that will get the right driver loaded automatically. Or the device is not enumerated and loading its driver will lead to more or less intrusive hardware poking. Such hardware poking should be limited to a bare minimum, so the user should really decide which drivers should be tried and in what order. Trying them all in arbitrary order can't do any good. On top of that, loading that many drivers at once bloats the kernel log. Also many drivers will stay loaded afterward, bloating the output of "lsmod" and wasting memory. Some modules (cs5535_mfgpt which gets loaded as a dependency) can't even be unloaded! If defining char-major-10-130 is needed then it should happen in user-space. Signed-off-by: Jean Delvare <jdelvare@xxxxxxx> Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> Cc: Stephen Warren <swarren@xxxxxxxxxxxxx> Cc: Mike Frysinger <vapier.adi@xxxxxxxxx> Cc: Wan ZongShun <mcuos.com@xxxxxxxxx> Cc: Ben Dooks <ben-linux@xxxxxxxxx> Cc: Kukjin Kim <kgene.kim@xxxxxxxxxxx> Cc: Zwane Mwaikambo <zwane@xxxxxxxxxxxxxxxx> Cc: Jim Cromie <jim.cromie@xxxxxxxxx> commit 67b9fbdf3602dfd525371d7b1f49346f1d11d37d Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Mon Oct 14 09:32:48 2013 -0700 watchdog: ts72xx_wdt: Propagate return value from timeout_to_regval timeout_to_regval() returns a valid error code. Might as well use it. Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 5412df0bda90a2bdcef2771849ad5472b9ef8100 Author: Guenter Roeck <linux@xxxxxxxxxxxx> Date: Mon Oct 14 09:29:34 2013 -0700 watchdog: pcwd_usb: Use allocated buffer for usb_control_msg usb_control_msg() must use a dma-capable buffer. This fixes the following error reported by smatch: drivers/watchdog/pcwd_usb.c:257 usb_pcwd_send_command() error: doing dma on the stack (buf) Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx> Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit b1f9cd3225ae2e8626fe7503ae7be92240873049 Author: Michal Simek <michal.simek@xxxxxxxxxx> Date: Thu Oct 3 11:46:30 2013 +0200 watchdog: sp805_wdt: Remove unnecessary amba_set_drvdata() Driver core clears the driver data to NULL after device_release or on probe failure, so just remove it from here. Driver core change: "device-core: Ensure drvdata = NULL when no driver is bound" (sha1: 0998d0631001288a5974afc0b2a5f568bcdecb4d) Signed-off-by: Michal Simek <michal.simek@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit f0fcbdbf202e2be36c8eb6d1f5c01f95805777de Author: Xianglong Du <Xianglong.Du@xxxxxxx> Date: Wed Oct 2 08:13:49 2013 +0800 watchdog: sirf: add watchdog driver of CSR SiRFprimaII and SiRFatlasVI On CSR SiRFprimaII and SiRFatlasVI, the 6th timer can act as a watchdog timer when the Watchdog mode is enabled. watchdog occur when TIMER watchdog counter matches the value software pre-set, when this event occurs, the effect is the same as the system software reset. Signed-off-by: Xianglong Du <Xianglong.Du@xxxxxxx> Signed-off-by: Barry Song <Baohua.Song@xxxxxxx> Cc: Romain Izard <romain.izard.pro@xxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 85eee81922399ec0be1b2c870f9333b4bca56650 Author: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Date: Mon Sep 30 10:12:51 2013 +0530 watchdog: Remove redundant of_match_ptr of_match_ptr() is a macro used to avoid undefined reference error if CONFIG_OF is used to selectively compile in or out the data structure. It is defined as follows: #ifdef CONFIG_OF #define of_match_ptr(ptr) ptr #else #define of_match_ptr(ptr) NULL #endif In the case of this series, none of the drivers use CONFIG_OF macro to compile out the data structure (i.e., the data structure is always defined). Hence the use of of_match_ptr() does not make any sense. Thus removing it to make the code look simpler for readability. Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit a20a99fbb88ac58c2e118e8759967d1cd7355322 Author: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Date: Fri Aug 23 12:38:32 2013 +0300 watchdog: ts72xx_wdt: cleanup return codes in ioctl There seems to be some confusion here which functions return positive numbers and which return negative error codes. copy_to_user() returns the number of bytes remaining to be copied but we want to return -EFAULT. The rest is just clean up. get_user() actually returns zero on success and -EFAULT on error so we can preserve the error code. The timeout_to_regval() function returns -EINVAL on failure, but we can propogate that back instead of hardcoding -EINVAL ourselves. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> -- commit cfff96e69f05176afd0e58d3a8c853a6d68ce544 Author: Johannes Thumshirn <johannes.thumshirn@xxxxxx> Date: Wed Aug 21 14:42:09 2013 +0200 documentation/devicetree: Move DT bindings from gpio to watchdog I accidently put the devicetree bindings for the MEN A21 watchdog driver in Documentation/devicetree/bindings/gpio instead of Documentation/devicetree/bindings/watchdog, this patch addresses this error. Signed-off-by: Johannes Thumshirn <johannes.thumshirn@xxxxxx> Acked-by: Stephen Warren <swarren@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> Cc: Rob Herring <rob.herring@xxxxxxxxxxx> Cc: Pawel Moll <pawel.moll@xxxxxxx> Cc: Mark Rutland <mark.rutland@xxxxxxx> Cc: Ian Campbell <ian.campbell@xxxxxxxxxx> Cc: Rob Landley <rob@xxxxxxxxxxx> commit 473cf939ff3442dc86d531f3bb152a2f129ea9d1 Author: John Crispin <blogic@xxxxxxxxxxx> Date: Thu Aug 8 11:31:43 2013 +0200 watchdog: add ralink watchdog driver Add a driver for the watchdog timer found on Ralink SoC Signed-off-by: John Crispin <blogic@xxxxxxxxxxx> Signed-off-by: Gabor Juhos <juhosg@xxxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> Cc: linux-watchdog@xxxxxxxxxxxxxxx Cc: linux-mips@xxxxxxxxxxxxxx Cc: devicetree-discuss@xxxxxxxxxxxxxxxx commit e14538e0db8d880291064348bda7d52ec8322675 Author: Jonas Jensen <jonas.jensen@xxxxxxxxx> Date: Fri Aug 2 16:40:45 2013 +0200 watchdog: Add MOXA ART watchdog driver This patch adds a watchdog driver for the main hardware watchdog timer found on MOXA ART SoCs. The MOXA ART SoC provides one writable timer register, restarting the hardware once it reaches zero. The register is auto decremented every APB clock cycle. Signed-off-by: Jonas Jensen <jonas.jensen@xxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit b3970bdebbb2901becca7cb82e44b5c5976f0fae Author: Jingoo Han <jg1.han@xxxxxxxxxxx> Date: Thu Aug 1 14:39:46 2013 +0900 watchdog: kempld_wdt: Add __user annotation Added __user annotation to fix the following sparse warnings. Also, it makes 'kempld_prescaler' static because it is used only in this file. drivers/watchdog/kempld_wdt.c:70:11: warning: symbol 'kempld_prescaler' was not declared. Should it be static? drivers/watchdog/kempld_wdt.c:364:23: warning: incorrect type in initializer (different address spaces) drivers/watchdog/kempld_wdt.c:364:23: expected int const [noderef] <asn:1>*register __p drivers/watchdog/kempld_wdt.c:364:23: got int *<noident> Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 60295507f6abf47a40d1fd758cd4401e7193f16e Author: Jingoo Han <jg1.han@xxxxxxxxxxx> Date: Thu Aug 1 14:38:36 2013 +0900 watchdog: dw_wdt: Add __user annotation Added __user annotation to fix the following sparse warnings. drivers/watchdog/dw_wdt.c:206:38: warning: incorrect type in argument 1 (different address spaces) drivers/watchdog/dw_wdt.c:206:38: expected void [noderef] <asn:1>*to drivers/watchdog/dw_wdt.c:206:38: got struct watchdog_info *<noident> drivers/watchdog/dw_wdt.c:211:24: warning: incorrect type in initializer (different address spaces) drivers/watchdog/dw_wdt.c:211:24: expected int const [noderef] <asn:1>*register __p drivers/watchdog/dw_wdt.c:211:24: got int *<noident> Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> Acked-by: Jamie Iles <jamie@xxxxxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit bc8fdfbe75058c6569dd6a08bdc2a411db533c47 Author: Jingoo Han <jg1.han@xxxxxxxxxxx> Date: Tue Jul 30 19:58:51 2013 +0900 watchdog: use dev_get_platdata() Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 813296a1a209baaf1471c360591946edd795bcbe Author: Niels de Vos <ndevos@xxxxxxxxxx> Date: Mon Jul 29 09:38:18 2013 +0200 watchdog: imx2_wdt: expose module alias for loading from device-tree Enable auto loading by udev when imx2_wdt is compiled as a module. Signed-off-by: Niels de Vos <ndevos@xxxxxxxxxx> Acked-by: Shawn Guo <shawn.guo@xxxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit 280103e6b5de6ae1e23d9067202cdd67dd06b490 Author: Heiko Stübner <heiko@xxxxxxxxx> Date: Wed Jun 26 20:04:31 2013 +0200 watchdog: dw_wdt: use clk_prepare_enable and clk_disable_unprepare This is necessary to make the driver work with platforms using the common clock framework. Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> commit ad83c6cbf30eba322fbc822453316a4954d9b607 Author: Heiko Stübner <heiko@xxxxxxxxx> Date: Wed Jun 26 20:03:52 2013 +0200 watchdog: dw_wdt: convert to SIMPLE_DEV_PM_OPS The dw_wdt only provides PM_SLEEP operations, so convert the driver to use SIMPLE_DEV_PM_OPS instead of populating the struct manually. This has the added effect of simplifying the CONFIG_PM ifdefs. Signed-off-by: Heiko Stuebner <heiko@xxxxxxxxx> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> Signed-off-by: Wim Van Sebroeck <wim@xxxxxxxxx> Greetings, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html