<feed xmlns='http://www.w3.org/2005/Atom'>
<title>pm24.git/drivers/watchdog, branch v3.5</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<id>https://git.kobert.dev/pm24.git/atom?h=v3.5</id>
<link rel='self' href='https://git.kobert.dev/pm24.git/atom?h=v3.5'/>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/'/>
<updated>2012-06-28T18:40:56Z</updated>
<entry>
<title>watchdog: core: fix WDIOC_GETSTATUS return value</title>
<updated>2012-06-28T18:40:56Z</updated>
<author>
<name>Wim Van Sebroeck</name>
<email>wim@iguana.be</email>
</author>
<published>2012-06-26T18:07:21Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=8b9468d496abd357cc797b27a79d4402f5e0e94d'/>
<id>urn:sha1:8b9468d496abd357cc797b27a79d4402f5e0e94d</id>
<content type='text'>
In commit 7a87982420e5e126bfefeb42232d1fd92052794e we added
a wrapper for the WDIOC_GETSTATUS ioctl call. The code results
however in a different behaviour: it returns an error if the
driver doesn't support the status operation. This is not
according to the API that says that when we don't support
the status operation, that we just should return a 0 value.
Only when the device isn't there anymore, we should return an
error.

Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: hpwdt: Unregister NMI events on exit.</title>
<updated>2012-06-28T18:40:31Z</updated>
<author>
<name>Mingarelli, Thomas</name>
<email>Thomas.Mingarelli@hp.com</email>
</author>
<published>2012-06-26T08:27:00Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=a089361cf5f1d6a5295aa5385238bd044998e1e9'/>
<id>urn:sha1:a089361cf5f1d6a5295aa5385238bd044998e1e9</id>
<content type='text'>
This patch is to unregister for NMI events upon exit. Also we are now
making the default setting for allow_kdump enabled.

Signed-off-by: Thomas Mingarelli &lt;thomas.mingarelli@hp.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: iTCO_wdt: add platform driver module alias</title>
<updated>2012-06-28T18:40:10Z</updated>
<author>
<name>Jan Beulich</name>
<email>JBeulich@suse.com</email>
</author>
<published>2012-06-22T15:41:00Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=e5de32e3ec9d4d5a355659760d5045b80c0a05d8'/>
<id>urn:sha1:e5de32e3ec9d4d5a355659760d5045b80c0a05d8</id>
<content type='text'>
The recent conversion of iTCO_wdt resulted in the driver no longer
getting loaded automatically, since it no longer has a
MODULE_DEVICE_TABLE() included. As the lpc_ich driver now creates a
platform device, auto-loading can easily be done by having a respective
module alias in place.

Signed-off-by: Jan Beulich &lt;jbeulich@suse.com&gt;
Cc: Aaron Sierra &lt;asierra@xes-inc.com&gt;
Acked-by: Guenter Roeck &lt;linux@roeck-us.net&gt;
Cc: Samuel Ortiz &lt;sameo@linux.intel.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>Viresh has moved</title>
<updated>2012-06-20T21:39:36Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.linux@gmail.com</email>
</author>
<published>2012-06-20T19:53:02Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=10d8935f46e5028847b179757ecbf9238b13d129'/>
<id>urn:sha1:10d8935f46e5028847b179757ecbf9238b13d129</id>
<content type='text'>
viresh.kumar@st.com email-id doesn't exist anymore as I have left the
company.  Replace ST's id with viresh.linux@gmail.com.

It also updates .mailmap file to fix address for 'git shortlog'

Signed-off-by: Viresh Kumar &lt;viresh.linux@gmail.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>watchdog: iTCO_wdt.c: fix printk format warnings</title>
<updated>2012-05-30T06:16:48Z</updated>
<author>
<name>Randy Dunlap</name>
<email>rdunlap@xenotime.net</email>
</author>
<published>2012-05-14T20:15:20Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=4b98b32aefb2c143a40cdaa254eb905f1bb06b5d'/>
<id>urn:sha1:4b98b32aefb2c143a40cdaa254eb905f1bb06b5d</id>
<content type='text'>
Fix printk format warnings:

drivers/watchdog/iTCO_wdt.c:577:3: warning: format '%04llx' expects type 'long long unsigned int', but argument 2 has type 'resource_size_t'
drivers/watchdog/iTCO_wdt.c:594:3: warning: format '%04llx' expects type 'long long unsigned int', but argument 2 has type 'resource_size_t'
drivers/watchdog/iTCO_wdt.c:600:2: warning: format '%04llx' expects type 'long long unsigned int', but argument 4 has type 'resource_size_t'

Signed-off-by: Randy Dunlap &lt;rdunlap@xenotime.net&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: sp805_wdt: Add clk_{un}prepare support</title>
<updated>2012-05-30T05:56:44Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.kumar@st.com</email>
</author>
<published>2012-03-12T04:22:17Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=d9df0ef1eb9e1048f0ab4a6e93c85f60c5594c2f'/>
<id>urn:sha1:d9df0ef1eb9e1048f0ab4a6e93c85f60c5594c2f</id>
<content type='text'>
clk_{un}prepare() routines are required for required on some platforms to run
part of clk enable/disable() routines from contexts that can schedule.

This patch adds support for these routines in sp805 driver.

Signed-off-by: Viresh Kumar &lt;viresh.kumar@st.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: sp805_wdt: convert to watchdog core</title>
<updated>2012-05-30T05:56:41Z</updated>
<author>
<name>Viresh Kumar</name>
<email>viresh.kumar@st.com</email>
</author>
<published>2012-03-12T04:22:16Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=4a516539faba13deca2399cff8faaa84d251a4ea'/>
<id>urn:sha1:4a516539faba13deca2399cff8faaa84d251a4ea</id>
<content type='text'>
This patch converts existing sp805 watchdog driver to use already in place
common infrastructure present in watchdog core. With this lot of code goes away.

Signed-off-by: Viresh Kumar &lt;viresh.kumar@st.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>Watchdog: DA9052/53 PMIC watchdog support</title>
<updated>2012-05-30T05:56:12Z</updated>
<author>
<name>Ashish Jangam</name>
<email>ashish.jangam@kpitcummins.com</email>
</author>
<published>2012-05-24T13:01:14Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=664a0d7862a6b10c709d4b4a3655fe2c59a20064'/>
<id>urn:sha1:664a0d7862a6b10c709d4b4a3655fe2c59a20064</id>
<content type='text'>
This driver adds support for the watchdog functionality provided by
the Dialog Semiconductor DA9052 PMIC chip.

Tested on samsung smdkv6410 and i.mx53 QS boards.

Signed-off-by: Anthony Olech &lt;Anthony.Olech@diasemi.com&gt;
Signed-off-by: Ashish Jangam &lt;ashish.jangam@kpitcummins.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: Add support for dynamically allocated watchdog_device structs</title>
<updated>2012-05-30T05:55:31Z</updated>
<author>
<name>Hans de Goede</name>
<email>hdegoede@redhat.com</email>
</author>
<published>2012-05-22T09:40:26Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=e907df32725204d6d2cb79b872529911c8eadcdf'/>
<id>urn:sha1:e907df32725204d6d2cb79b872529911c8eadcdf</id>
<content type='text'>
If a driver's watchdog_device struct is part of a dynamically allocated
struct (which it often will be), merely locking the module is not enough,
even with a drivers module locked, the driver can be unbound from the device,
examples:
1) The root user can unbind it through sysfd
2) The i2c bus master driver being unloaded for an i2c watchdog

I will gladly admit that these are corner cases, but we still need to handle
them correctly.

The fix for this consists of 2 parts:
1) Add ref / unref operations, so that the driver can refcount the struct
   holding the watchdog_device struct and delay freeing it until any
   open filehandles referring to it are closed
2) Most driver operations will do IO on the device and the driver should not
   do any IO on the device after it has been unbound. Rather then letting each
   driver deal with this internally, it is better to ensure at the watchdog
   core level that no operations (other then unref) will get called after
   the driver has called watchdog_unregister_device(). This actually is the
   bulk of this patch.

Signed-off-by: Hans de Goede &lt;hdegoede@redhat.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
<entry>
<title>watchdog: Add Locking support</title>
<updated>2012-05-30T05:55:23Z</updated>
<author>
<name>Hans de Goede</name>
<email>hdegoede@redhat.com</email>
</author>
<published>2012-05-22T09:40:26Z</published>
<link rel='alternate' type='text/html' href='https://git.kobert.dev/pm24.git/commit/?id=f4e9c82f64b524314a390b13d3ba7d483f09258f'/>
<id>urn:sha1:f4e9c82f64b524314a390b13d3ba7d483f09258f</id>
<content type='text'>
This patch fixes some potential multithreading issues, despite only
allowing one process to open the /dev/watchdog device, we can still get
called multiple times at the same time, since a program could be using thread,
or could share the fd after a fork.

This causes 2 potential problems:
1) watchdog_start / open do an unlocked test_n_set / test_n_clear,
   if these 2 race, the watchdog could be stopped while the active
   bit indicates it is running or visa versa.

2) Most watchdog_dev drivers probably assume that only one
   watchdog-op will get called at a time, this is not necessary
   true atm.

Signed-off-by: Hans de Goede &lt;hdegoede@redhat.com&gt;
Signed-off-by: Wim Van Sebroeck &lt;wim@iguana.be&gt;
</content>
</entry>
</feed>
