Re: [PATCH V4 0/2] watchdog: Sysfs status read support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/05/2015 11:43 PM, Pratyush Anand wrote:
Hi Wim,

I did not see it in v4.3-rc3 Fixes.
This has already been reviewed by Guenter.
Please let me know if I need to do anything more around it.

It is too late for 4.3, so I guess it will have to wait for 4.4.

Guenter

~Pratyush

On 07/09/2015:09:54:50 AM, Pratyush Anand wrote:
These patches provide support to read different watchdog device status
through sysfs interface.

Changes since V3:
* Added Reviewed by tag
* Corrected a checkpatch warning

Changes since V2:
* Used static struct class watchdog_class in stead of pointer. It helped in
  keep using device_create().
* Above logic was moved to a separate patch.
* Changed subject line of other patch to look more relevant

Changes since V1(RFC):
* Removed keepalive and start ABI
* timeout is read only now
* state returns text
* only supported ABI visible
* ABI contact changed to MAINTAINER
* unnecessary mutex removed
* aligned continuation with '('
* unnecessary initialization of status (= 0) corrected
* unnecessary else removed
* used __ATTRIBUTE_GROUPS
* removed watchdog_device_create and added functionality in
* watchdog_dev_register.
* optimized nowayout_show
* Now no -EOPNOTSUPP return for timeout read in case of wdd->timeout = 0.

Pratyush Anand (2):
   watchdog: Use static struct class watchdog_class in stead of pointer
   watchdog: Read device status through sysfs attributes

  Documentation/ABI/testing/sysfs-class-watchdog |  51 ++++++++++
  drivers/watchdog/watchdog_core.c               |  17 +---
  drivers/watchdog/watchdog_core.h               |   2 +-
  drivers/watchdog/watchdog_dev.c                | 136 ++++++++++++++++++++++++-
  4 files changed, 187 insertions(+), 19 deletions(-)
  create mode 100644 Documentation/ABI/testing/sysfs-class-watchdog

--
2.5.0


--
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



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux