Re: [PATCH] watchdog: Fix omap watchdogs to enable the magic close bit

2015-02-02 Thread Wim Van Sebroeck
Hi Tony, * Tony Lindgren t...@atomide.com [141014 12:27]: This allows testing the watchdog easily with distros just by doing pkill -9 watchdog. Reported-by: Thomas Dziedzic gos...@gmail.com Signed-off-by: Tony Lindgren t...@atomide.com Wim, still not seeing this applied, did you

Re: [RFC 02/23] watchdog: omap_wdt: raw read and write endian fix

2013-11-17 Thread Wim Van Sebroeck
Hi Victor, Taras, From: Victor Kamensky victor.kamen...@linaro.org 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

Re: [PATCH v5] watchdog: introduce retu_wdt driver

2013-02-07 Thread Wim Van Sebroeck
Hi Aaro, Introduce Retu watchdog driver. Cc: linux-watch...@vger.kernel.org Acked-by: Felipe Balbi ba...@ti.com Acked-by: Tony Lindgren t...@atomide.com Signed-off-by: Aaro Koskinen aaro.koski...@iki.fi Cc: Wim Van Sebroeck w...@iguana.be Added tolinux-watchdog-next. Kind regards, Wim

Re: [PATCH 5/7] ARM: OMAP2+: WDT: move init; add read_reset_sources pdata function pointer

2012-10-23 Thread Wim Van Sebroeck
-by: Paul Walmsley p...@pwsan.com Cc: Wim Van Sebroeck w...@iguana.be Signed-off-by: Wim Van Sebroeck w...@iguana.be Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [PATCH 6/7] watchdog: OMAP: use standard GETBOOTSTATUS interface; use platform_data fn ptr

2012-10-23 Thread Wim Van Sebroeck
can be removed. In the short term, this patch is needed to allow the PRM code to be removed from arch/arm/mach-omap2 (it is being moved to a driver). Signed-off-by: Paul Walmsley p...@pwsan.com Cc: Wim Van Sebroeck w...@iguana.be Signed-off-by: Wim Van Sebroeck w...@iguana.be Kind regards

Re: [PATCH 5/7] ARM: OMAP2+: WDT: move init; add read_reset_sources pdata function pointer (fwd)

2012-10-23 Thread Wim Van Sebroeck
Hi Paul, When you have the opportunity, could you take a look at this patch, and the subsequent patch 6/7, and ack them if you're okay with them? Signed them off, Acked-by would probably have been better :-). We'd like to merge thse as part of a larger cleanup series through the arm-soc

Re: [PATCH_v2] watchdog: Convert twl4030_wdt to watchdog core

2012-09-27 Thread Wim Van Sebroeck
Hi, On 09/11/2012 09:01 AM, Jarkko Nikula wrote: Convert the twl4030_wdt watchdog driver to watchdog core. While at there use devm_kzalloc and set the default timeout in order to be able test this driver with a simple shell script. Signed-off-by: Jarkko

Re: [PATCH 15/17] ARM: OMAP: Split plat/hardware.h, use local soc.h for omap2+

2012-09-11 Thread Wim Van Sebroeck
. Also only minimal changes are done to sound/soc/omap/* to remove the unneeded includes and to define OMAP44XX_MCPDM_L3_BASE locally so there's no need to include omap44xx.h. While at it, also sort some of the includes in the standard way. Cc: linux-watch...@vger.kernel.org Cc: Wim Van

Re: [PATCH] Watchdog: OMAP: Fix the runtime pm code to avoid module getting stuck intransition state.

2012-07-07 Thread Wim Van Sebroeck
only during probe/shutdown and suspend/resume. The patch fixes issue 1 and 2 Signed-off-by: Lokesh Vutla lokeshvu...@ti.com Acked-by: Santosh Shilimkar santosh.shilim...@ti.com Cc: Wim Van Sebroeck w...@iguana.be --- Any comments on this patch ? If not, can you please queue this up

Re: [PATCHv2 3/3] watchdog: omap_wdt: add device tree support

2012-07-07 Thread Wim Van Sebroeck
Hi Tony, Hi Wim, * jgq...@gmail.com jgq...@gmail.com [120531 20:56]: From: Xiao Jiang jgq...@gmail.com Add device table for omap_wdt to support dt. Care to ack this patch in the series? Yep. Acked-by: Wim Van Sebroeck w...@iguana.be Kind regards, Wim. -- To unsubscribe from

Re: [PATCH] watchdog: twl4030_wdt: Watchdog device registration issue fix

2011-03-28 Thread Wim Van Sebroeck
Hi All, Hello All, Any comments on this patch? Regards, Keerthy On Wed, Mar 2, 2011 at 2:17 AM, Keerthy j-keer...@ti.com wrote: twl4030_wdt driver and omap_wdt driver are registering as misc_device name as watchdog and the same minor number WATCHDOG_MINOR( value = 130). There

Re: [PATCH v2 0/2] OMAP3: wdtimer: Fix CORE idle

2011-03-10 Thread Wim Van Sebroeck
runtime-pm OMAP3: wdtimer: Fix CORE idle transition arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |1 + drivers/watchdog/omap_wdt.c| 25 +++-- 2 files changed, 24 insertions(+), 2 deletions(-) Acked-by: Wim Van Sebroeck w...@iguana.be linux-omap-pm

Re: Handling multiple watchdogs

2011-02-22 Thread Wim Van Sebroeck
Hi Alan, 1) make sure that we have the new watchdog core infrastructure going in for 2.6.32. This new core integrates the common code that we use over and over again. I once wrote code for it and then Alan had different ideas and thoughts and wrote his updated code. I reviewed

Re: [PATCH v5 6/6] OMAP: WDT: Use PM runtime APIs instead of clk FW APIs

2010-09-15 Thread Wim Van Sebroeck
Hi All, Call runtime pm APIs pm_runtime_put_sync() and pm_runtime_get_sync() for enabling/disabling the clocks, sysconfig settings instead of using clock FW APIs. Signed-off-by: Charulatha V ch...@ti.com This is all omap specific code. So if Kevin and Tony are fine with the code then tou

Re: [PATCH] twl4030_wdt: Disable watchdog while probing

2010-05-20 Thread Wim Van Sebroeck
Hi Timo, It seems that in this patch, you disable the watchdog regardless whether the registration fails or not. I wonder if this is a good thing to do. What if the bootloader have already enabled the watchdog and then user space booting fails for some reason before any process is able to

Re: [PATCH] twl4030_wdt: Disable watchdog while probing

2010-05-19 Thread Wim Van Sebroeck
Hi Ameya, If we are not able to register then it is better to have watchdog in disabled state than noticing a system reboot. Signed-off-by: Ameya Palande ameya.pala...@nokia.com This was added in my linux-2.6-watchdog-next tree. Will be sent to Linus for mainline inclusion in a couple of

[RFC] [PATCH] watchdog_info separation and constify

2010-01-19 Thread Wim Van Sebroeck
Hi All, please comment on following patch. Kind regards, Wim. commit 88d0b1a9c071d26e7b4831320067c84b04ea04a8 Author: Wim Van Sebroeck w...@iguana.be Date: Sat Dec 26 18:55:22 2009 + [WATCHDOG] watchdog_info separation and constify make sure that the watchdog_info struct

Re: [PATCH 0/2] watchdog:OMAP3:Add support for IVA2, SECURE WDTs

2009-07-22 Thread Wim Van Sebroeck
Hi Ulrik, This patch series enables support for IVA2 and SECURE WDTs, available on omap34xx. The WDTs will be accessible (when present on device) through: MPU: /dev/watchdog SECURE: /dev/watchdog_secure IVA2: /dev/watchdog_iva2 Tested on Zoom1 OMAP3 platform, compile-tested for

Re: Handling multiple watchdogs

2009-07-22 Thread Wim Van Sebroeck
Hi Simon, I have two watchdogs on my board that I both want to handle. What would be the proper approach in this case? Fixing the watchdog core to create a class of watchdog drivers and treating the existing /dev/watchdog as a back compatibility hack. It's been talked about for a very

Re: [PATCH 1/1] watchdog: OMAP fixes: enable clock in probe, trigger timer reload

2009-06-18 Thread Wim Van Sebroeck
Hi Ulrik, This patch contains two bugfixes: 1)In omap_wdt_probe() the watchdog is reset and disabled. This requires register access and the clks needs to be enabled temporarily 2)In omap_wdt_open() the timer register needs to be reloaded to trigger a new timer value (the default of 60s)

Re: [PATCH 1/2] omap: twl4030_wdt: twl4030 watchdog driver

2009-06-04 Thread Wim Van Sebroeck
Signed-off-by: Wim Van Sebroeck w...@iguana.be diff --git a/drivers/mfd/twl4030-core.c b/drivers/mfd/twl4030-core.c index ec90e95..b8df518 100644 --- a/drivers/mfd/twl4030-core.c +++ b/drivers/mfd/twl4030-core.c @@ -101,6 +101,12 @@ #define twl_has_usb() false #endif +#if defined

Re: [PATCH 35/58] move omap_wdt's probe function to .devinit.text

2009-03-29 Thread Wim Van Sebroeck
Hi Uwe, A pointer to omap_wdt_probe is passed to the core via platform_driver_register and so the function must not disappear when the .init sections are discarded. Otherwise (if also having HOTPLUG=y) unbinding and binding a device to the driver via sysfs will result in an oops as does a

Re: [PATCH] watchdog: another ioremap() fix

2008-09-19 Thread Wim Van Sebroeck
Hi All, On Thu, Sep 18, 2008 at 11:02:58PM +0300, Felipe Balbi wrote: Well, what can I say. I'll try to get rid of those cpu conditional code in the driver and sync omap_wdt.c with mainline. Send all the patches via Wim, so Tony can get them later. BTW, I'd also suggest copying Andrew

Re: [PATCH 1/5] watchdog: sync linux-omap changes

2008-09-19 Thread Wim Van Sebroeck
Hi Balbi, And thanks for the review. I'll fix your comments and resend the series ;-) Thanks because I had the same objections then Russell. Greetings, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of a message to [EMAIL PROTECTED] More majordomo

Re: [PATCH 5/5] watchdog: introduce platform_data and remove cpu conditional code

2008-09-19 Thread Wim Van Sebroeck
Hi Balbi, Well, patches 4 and 5 should be ignored. Should I resend or could I rely on the fact that people won't pick them up ? I will not pick them up (to add them in the watchdog tree). Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-omap in the body of