On 22 November 2013 14:56, Markus Mayer <markus.mayer@xxxxxxxxxx> wrote:
> This is version 5 of the watchdog driver for the BCM281xx family of mobile
> SoCs.
>
> Changes since version 4:
> - Fixed whitespace issue in Kconfig "help" section
> - Rebased onto 3.13-rc1 (due to merge conflict in Makefile)
>
> 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
Hi,
This series seems approved and ready to go. I just wanted to
double-check that my understanding is correct and that it will be
queued for 3.14. However, if there is anything you need from me,
please let me know.
Thanks,
-Markus
--
Markus Mayer
Broadcom Landing Team
--
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
