Sender: Guenter Roeck <groeck7@gmail.com>
Message-ID: <b7da8789-6c55-cf41-b95a-e87f9341c438@roeck-us.net>
Date:   Mon, 6 Feb 2023 05:55:40 -0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH 0/5] diag288 watchdog fixes and improvements
Content-Language: en-US
To:     Heiko Carstens <hca@linux.ibm.com>,
        Wim Van Sebroeck <wim@linux-watchdog.org>
Cc:     linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org,
        Alexander Egorenkov <egorenar@linux.ibm.com>
References: <20230203073958.1585738-1-egorenar@linux.ibm.com>
 <Y+DPnGN9l7Ix+TX7@osiris>
From:   Guenter Roeck <linux@roeck-us.net>
In-Reply-To: <Y+DPnGN9l7Ix+TX7@osiris>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

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