diag288 watchdog fixes and improvements

[PATCH 0/5] diag288 watchdog fixes and improvements

Posted by Alexander Egorenkov 2 years, 7 months ago

Minor code refactoring to improve readability of the driver,
reduce code duplication and remove dead code.

Alexander Egorenkov (5):
  watchdog: diag288_wdt: get rid of register asm
  watchdog: diag288_wdt: remove power management
  watchdog: diag288_wdt: unify command buffer handling for diag288 zvm
  watchdog: diag288_wdt: de-duplicate diag_stat_inc() calls
  watchdog: diag288_wdt: unify lpar and zvm diag288 helpers

 drivers/watchdog/diag288_wdt.c | 162 ++++++++-------------------------
 1 file changed, 37 insertions(+), 125 deletions(-)

-- 
2.37.2

Re: [PATCH 0/5] diag288 watchdog fixes and improvements

Posted by Heiko Carstens 2 years, 7 months ago

On Fri, Feb 03, 2023 at 08:39:53AM +0100, Alexander Egorenkov wrote:
> Minor code refactoring to improve readability of the driver,
> reduce code duplication and remove dead code.
> 
> Alexander Egorenkov (5):
>   watchdog: diag288_wdt: get rid of register asm
>   watchdog: diag288_wdt: remove power management
>   watchdog: diag288_wdt: unify command buffer handling for diag288 zvm
>   watchdog: diag288_wdt: de-duplicate diag_stat_inc() calls
>   watchdog: diag288_wdt: unify lpar and zvm diag288 helpers
> 
>  drivers/watchdog/diag288_wdt.c | 162 ++++++++-------------------------
>  1 file changed, 37 insertions(+), 125 deletions(-)

Guenter, Wim, how should this go upstream?

I can easily pick this up via the s390 tree for the next merge window.
Please let me know.

Thanks,
Heiko

Re: [PATCH 0/5] diag288 watchdog fixes and improvements

Posted by Guenter Roeck 2 years, 7 months ago

On 2/6/23 01:59, Heiko Carstens wrote:
> On Fri, Feb 03, 2023 at 08:39:53AM +0100, Alexander Egorenkov wrote:
>> Minor code refactoring to improve readability of the driver,
>> reduce code duplication and remove dead code.
>>
>> Alexander Egorenkov (5):
>>    watchdog: diag288_wdt: get rid of register asm
>>    watchdog: diag288_wdt: remove power management
>>    watchdog: diag288_wdt: unify command buffer handling for diag288 zvm
>>    watchdog: diag288_wdt: de-duplicate diag_stat_inc() calls
>>    watchdog: diag288_wdt: unify lpar and zvm diag288 helpers
>>
>>   drivers/watchdog/diag288_wdt.c | 162 ++++++++-------------------------
>>   1 file changed, 37 insertions(+), 125 deletions(-)
> 
> Guenter, Wim, how should this go upstream?
> 
> I can easily pick this up via the s390 tree for the next merge window.
> Please let me know.
> 

I have it currently in my watchdog-next tree, but that is not in linux-next.
Fine with me to go through the s390 tree.

Guenter

Re: [PATCH 0/5] diag288 watchdog fixes and improvements

Posted by Heiko Carstens 2 years, 7 months ago

On Mon, Feb 06, 2023 at 05:55:40AM -0800, Guenter Roeck wrote:
> On 2/6/23 01:59, Heiko Carstens wrote:
> > On Fri, Feb 03, 2023 at 08:39:53AM +0100, Alexander Egorenkov wrote:
> > > Minor code refactoring to improve readability of the driver,
> > > reduce code duplication and remove dead code.
> > > 
> > > Alexander Egorenkov (5):
> > >    watchdog: diag288_wdt: get rid of register asm
> > >    watchdog: diag288_wdt: remove power management
> > >    watchdog: diag288_wdt: unify command buffer handling for diag288 zvm
> > >    watchdog: diag288_wdt: de-duplicate diag_stat_inc() calls
> > >    watchdog: diag288_wdt: unify lpar and zvm diag288 helpers
> > > 
> > >   drivers/watchdog/diag288_wdt.c | 162 ++++++++-------------------------
> > >   1 file changed, 37 insertions(+), 125 deletions(-)
> > 
> > Guenter, Wim, how should this go upstream?
> > 
> > I can easily pick this up via the s390 tree for the next merge window.
> > Please let me know.
> 
> I have it currently in my watchdog-next tree, but that is not in linux-next.
> Fine with me to go through the s390 tree.

Applied to s390 tree - should be in next linux-next release.

Thank you!