mbox series

[v3,0/2] rtc: abx80x: add basic watchdog support

Message ID 20180911172826.7195-1-jgebben@sweptlaser.com
Headers show
Series rtc: abx80x: add basic watchdog support | expand

Message

Jeremy Gebben Sept. 11, 2018, 5:28 p.m. UTC 
This patch series adds watchdog support for abracon RTC chips which
include basic watchdog functionality.

Thank you for reviewing.

Changes in v3 (all in patch 2):
* fix CONFIG_WATCHDOG dependency again (hopefully correctly this time)
* don't start hardware when set_timeout is called
* remove unneeded WDOG_HW_RUNNING bit changes
* fix abx80x_setup_watchdog()
* fix formatting of multiline splits
 
Changes in v2:
* split out priv structure into a separate patch
* remove new Kconfig option
* fix CONFIG_WATCHDOG dependency
* fix WDT interrupt message


Jeremy Gebben (2):
  rtc: abx80x: use a 'priv' struct for client data
  rtc: abx80x: add basic watchdog support

 drivers/rtc/Kconfig      |   1 +
 drivers/rtc/rtc-abx80x.c | 143 +++++++++++++++++++++++++++++++++++----
 2 files changed, 132 insertions(+), 12 deletions(-)

Comments

Alexandre Belloni Sept. 12, 2018, 11:18 a.m. UTC | #1 
On 11/09/2018 11:28:24-0600, Jeremy Gebben wrote:
> This patch series adds watchdog support for abracon RTC chips which
> include basic watchdog functionality.
> 
> Thank you for reviewing.
> 
> Changes in v3 (all in patch 2):
> * fix CONFIG_WATCHDOG dependency again (hopefully correctly this time)
> * don't start hardware when set_timeout is called
> * remove unneeded WDOG_HW_RUNNING bit changes
> * fix abx80x_setup_watchdog()
> * fix formatting of multiline splits
>  
> Changes in v2:
> * split out priv structure into a separate patch
> * remove new Kconfig option
> * fix CONFIG_WATCHDOG dependency
> * fix WDT interrupt message
> 
> 
> Jeremy Gebben (2):
>   rtc: abx80x: use a 'priv' struct for client data
>   rtc: abx80x: add basic watchdog support
> 
>  drivers/rtc/Kconfig      |   1 +
>  drivers/rtc/rtc-abx80x.c | 143 +++++++++++++++++++++++++++++++++++----
>  2 files changed, 132 insertions(+), 12 deletions(-)
> 

Applied, thanks.