mbox series

[v5,0/7] RZG2L_WDT Fixes and Improvements

Message ID 20220225175320.11041-1-biju.das.jz@bp.renesas.com
Headers show
Series RZG2L_WDT Fixes and Improvements | expand

Message

Biju Das Feb. 25, 2022, 5:53 p.m. UTC 
The first 4 patch in this series fixes the below issues
1) 32 bit overflow issue
2) Runtime PM usage count issue
3) BUG: Invalid context during reset.
4) Reset control imbalance

The later 3 patches are enhancements to the WDT driver.
1) Adding error check for reset_control_deassert() and fixing reset_control imbalance.
2) Generate Parity error for WDT reset
3) Add support for set_timeout callback

v4->v5:
 * Updated commit description of patch#4
 * Added Rb tag from Geert.
 * Separated reset control imbalance from patch#4

Biju Das (7):
  watchdog: rzg2l_wdt: Fix 32bit overflow issue
  watchdog: rzg2l_wdt: Fix Runtime PM usage
  watchdog: rzg2l_wdt: Fix 'BUG: Invalid wait context'
  watchdog: rzg2l_wdt: Fix reset control imbalance
  watchdog: rzg2l_wdt: Add error check for reset_control_deassert
  watchdog: rzg2l_wdt: Use force reset for WDT reset
  watchdog: rzg2l_wdt: Add set_timeout callback

 drivers/watchdog/rzg2l_wdt.c | 83 ++++++++++++++++++++----------------
 1 file changed, 47 insertions(+), 36 deletions(-)

Comments

Biju Das March 7, 2022, 8:52 a.m. UTC | #1 
Hi All,

Gentle ping. Are we happy with this patch series?

Please let me know.

Cheers,
Biju

> Subject: [PATCH v5 0/7] RZG2L_WDT Fixes and Improvements
> 
> The first 4 patch in this series fixes the below issues
> 1) 32 bit overflow issue
> 2) Runtime PM usage count issue
> 3) BUG: Invalid context during reset.
> 4) Reset control imbalance
> 
> The later 3 patches are enhancements to the WDT driver.
> 1) Adding error check for reset_control_deassert() and fixing
> reset_control imbalance.
> 2) Generate Parity error for WDT reset
> 3) Add support for set_timeout callback
> 
> v4->v5:
>  * Updated commit description of patch#4
>  * Added Rb tag from Geert.
>  * Separated reset control imbalance from patch#4
> 
> Biju Das (7):
>   watchdog: rzg2l_wdt: Fix 32bit overflow issue
>   watchdog: rzg2l_wdt: Fix Runtime PM usage
>   watchdog: rzg2l_wdt: Fix 'BUG: Invalid wait context'
>   watchdog: rzg2l_wdt: Fix reset control imbalance
>   watchdog: rzg2l_wdt: Add error check for reset_control_deassert
>   watchdog: rzg2l_wdt: Use force reset for WDT reset
>   watchdog: rzg2l_wdt: Add set_timeout callback
> 
>  drivers/watchdog/rzg2l_wdt.c | 83 ++++++++++++++++++++----------------
>  1 file changed, 47 insertions(+), 36 deletions(-)
> 
> --
> 2.17.1
Guenter Roeck March 9, 2022, 11:42 p.m. UTC | #2 
On 3/7/22 00:52, Biju Das wrote:
> Hi All,
> 
> Gentle ping. Are we happy with this patch series?
> 
> Please let me know.
> 

Should be good to go. I added the series to my watchdog-next branch.
Usually Wim picks it up from there.

Guenter

> Cheers,
> Biju
> 
>> Subject: [PATCH v5 0/7] RZG2L_WDT Fixes and Improvements
>>
>> The first 4 patch in this series fixes the below issues
>> 1) 32 bit overflow issue
>> 2) Runtime PM usage count issue
>> 3) BUG: Invalid context during reset.
>> 4) Reset control imbalance
>>
>> The later 3 patches are enhancements to the WDT driver.
>> 1) Adding error check for reset_control_deassert() and fixing
>> reset_control imbalance.
>> 2) Generate Parity error for WDT reset
>> 3) Add support for set_timeout callback
>>
>> v4->v5:
>>   * Updated commit description of patch#4
>>   * Added Rb tag from Geert.
>>   * Separated reset control imbalance from patch#4
>>
>> Biju Das (7):
>>    watchdog: rzg2l_wdt: Fix 32bit overflow issue
>>    watchdog: rzg2l_wdt: Fix Runtime PM usage
>>    watchdog: rzg2l_wdt: Fix 'BUG: Invalid wait context'
>>    watchdog: rzg2l_wdt: Fix reset control imbalance
>>    watchdog: rzg2l_wdt: Add error check for reset_control_deassert
>>    watchdog: rzg2l_wdt: Use force reset for WDT reset
>>    watchdog: rzg2l_wdt: Add set_timeout callback
>>
>>   drivers/watchdog/rzg2l_wdt.c | 83 ++++++++++++++++++++----------------
>>   1 file changed, 47 insertions(+), 36 deletions(-)
>>
>> --
>> 2.17.1
>
Biju Das March 10, 2022, 6:53 a.m. UTC | #3 
Hi Guenter,

> Subject: Re: [PATCH v5 0/7] RZG2L_WDT Fixes and Improvements
> 
> On 3/7/22 00:52, Biju Das wrote:
> > Hi All,
> >
> > Gentle ping. Are we happy with this patch series?
> >
> > Please let me know.
> >
> 
> Should be good to go. I added the series to my watchdog-next branch.
> Usually Wim picks it up from there.

Thanks,
Biju

> >> Subject: [PATCH v5 0/7] RZG2L_WDT Fixes and Improvements
> >>
> >> The first 4 patch in this series fixes the below issues
> >> 1) 32 bit overflow issue
> >> 2) Runtime PM usage count issue
> >> 3) BUG: Invalid context during reset.
> >> 4) Reset control imbalance
> >>
> >> The later 3 patches are enhancements to the WDT driver.
> >> 1) Adding error check for reset_control_deassert() and fixing
> >> reset_control imbalance.
> >> 2) Generate Parity error for WDT reset
> >> 3) Add support for set_timeout callback
> >>
> >> v4->v5:
> >>   * Updated commit description of patch#4
> >>   * Added Rb tag from Geert.
> >>   * Separated reset control imbalance from patch#4
> >>
> >> Biju Das (7):
> >>    watchdog: rzg2l_wdt: Fix 32bit overflow issue
> >>    watchdog: rzg2l_wdt: Fix Runtime PM usage
> >>    watchdog: rzg2l_wdt: Fix 'BUG: Invalid wait context'
> >>    watchdog: rzg2l_wdt: Fix reset control imbalance
> >>    watchdog: rzg2l_wdt: Add error check for reset_control_deassert
> >>    watchdog: rzg2l_wdt: Use force reset for WDT reset
> >>    watchdog: rzg2l_wdt: Add set_timeout callback
> >>
> >>   drivers/watchdog/rzg2l_wdt.c | 83 ++++++++++++++++++++---------------
> -
> >>   1 file changed, 47 insertions(+), 36 deletions(-)
> >>
> >> --
> >> 2.17.1
> >