linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: markus.mayer@linaro.org (Markus Mayer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/2] watchdog: bcm281xx: Watchdog Driver
Date: Tue, 19 Nov 2013 11:07:52 -0800	[thread overview]
Message-ID: <1384888074-19696-1-git-send-email-markus.mayer@linaro.org> (raw)

This is version 4 of the watchdog driver for the BCM281xx family of mobile
SoCs.

Changes since version 3:
- Removed #include <linux/miscdevice.h>
- Added Reviewed-by:/Acked-by: Guenter Roeck

Changes since version 2:

- BCM_KONA_WDT_NAME: "bcm-kona-wdt" -> "bcm_kona_wdt"
- Don't initialize static variable
- Removed '{'/'}' from single-line statements
- busy_count retains the maximum loop count of the longest loop rather
  than an over-all maximum
- Use seq_puts() in lieu of seq_printf() in 2 locations
- Removed error message after devm_kzalloc()
- Added MODULE_ALIAS

Changes since version 1:

- Added module name to "help" section in Kconfig A few cosmetic code
- simplifications and fixes Removed most dev_info() calls and changed the
- remaining ones to dev_dbg() Renamed SECWDOG_WD_LOAD_FLAG_MASK to
- SECWDOG_WD_LOAD_FLAG Added some comments to secure_register_read() and
- struct bcm_kona_wdt Added delay to secure_register_read() Reduced maximum
- retry loop from 10000 to 1000 Introduced "busy_count" variable to count
- how often secure_register_read() gets stalled; this is available through
- debugfs Simplified secure_register_read() to return -ETIMEDOUT rather than
- using a variable parameter to indicate a timeout error Got rid of all uses
- of -EAGAIN Fixed return value check for debugfs_create_dir() Simplified
- bcm_kona_wdt_debugfs_init() by getting rid of goto Created new generic
- function bcm_kona_wdt_ctrl_reg_modify() The following functions now use
- bcm_kona_wdt_ctrl_reg_modify():
    - bcm_kona_wdt_set_resolution_reg() bcm_kona_wdt_set_timeout_reg()
    - bcm_kona_wdt_stop()
- Made bcm_kona_wdt_set_timeout_reg() more generic, so bcm_kona_wdt_start()
  can use it Removed MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR)

Markus Mayer (2):
  watchdog: bcm281xx: Watchdog Driver
  ARM: bcm281xx: watchdog configuration

 arch/arm/configs/bcm_defconfig  |    3 +
 drivers/watchdog/Kconfig        |   22 +++
 drivers/watchdog/Makefile       |    1 +
 drivers/watchdog/bcm_kona_wdt.c |  365 +++++++++++++++++++++++++++++++++++++++
 4 files changed, 391 insertions(+)
 create mode 100644 drivers/watchdog/bcm_kona_wdt.c

-- 
1.7.9.5

             reply	other threads:[~2013-11-19 19:07 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-19 19:07 Markus Mayer [this message]
2013-11-19 19:07 ` [PATCH v4 1/2] watchdog: bcm281xx: Watchdog Driver Markus Mayer
2013-11-20 16:12   ` Markus Mayer
2013-11-20 16:27     ` Guenter Roeck
2013-11-20 18:24       ` Markus Mayer
2013-11-20 18:40         ` Guenter Roeck
2013-11-20 18:42           ` Markus Mayer
2013-11-19 19:07 ` [PATCH v4 2/2] ARM: bcm281xx: watchdog configuration Markus Mayer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1384888074-19696-1-git-send-email-markus.mayer@linaro.org \
    --to=markus.mayer@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).