Sender: Guenter Roeck <groeck7@gmail.com>
Message-ID: <970802b8-f99d-0d26-cfcd-5b2079ed4d6f@roeck-us.net>
Date:   Wed, 15 Mar 2023 19:11:33 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.1
Subject: Re: [PATCH 2/2] hwmon: (nct6775) update ASUS WMI monitoring list
 A520/B360/B460/B550...
Content-Language: en-US
To:     Denis Pauk <pauk.denis@gmail.com>
Cc:     jdelvare@suse.com, linux-hwmon@vger.kernel.org,
        linux-kernel@vger.kernel.org, mischief@offblast.org,
        de99like@mennucci.debian.net, holger.kiehl@dwd.de
References: <20230315210135.2155-1-pauk.denis@gmail.com>
 <20230315210135.2155-2-pauk.denis@gmail.com>
 <20230315233054.5ac21db0@gmail.com>
 <19097c39-9703-6b7f-6cc4-8a157b00f368@roeck-us.net>
 <20230316000453.553bf6a8@gmail.com> <20230316001749.44e51d08@gmail.com>
 <20230316003519.27224216@gmail.com>
From:   Guenter Roeck <linux@roeck-us.net>
In-Reply-To: <20230316003519.27224216@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 3/15/23 15:35, Denis Pauk wrote:
> On Thu, 16 Mar 2023 00:17:49 +0200
> Denis Pauk <pauk.denis@gmail.com> wrote:
> 
>> On Thu, 16 Mar 2023 00:04:53 +0200
>> Denis Pauk <pauk.denis@gmail.com> wrote:
>>
>>> On Wed, 15 Mar 2023 14:58:24 -0700
>>> Guenter Roeck <linux@roeck-us.net> wrote:
>>>    
>>>> On 3/15/23 14:30, Denis Pauk wrote:
>>>>> On Wed, 15 Mar 2023 23:01:35 +0200
>>>>> Denis Pauk <pauk.denis@gmail.com> wrote:
>>>>>
>>>>> Tested-by: Holger Kiehl <holger.kiehl@dwd.de>
>>>>>
>>>>> Pro A520M-C II/CSM is also tested by Holger Kiehl
>>>>> https://patchwork.kernel.org/project/linux-hwmon/patch/868bdc4f-9d45-475c-963e-f5232a8b95@praktifix.dwd.de/
>>>>>
>>>>> Could it be applied as single patch or need to rebase over "Pro A520M-C
>>>>> II" patch?
>>>>>        
>>>> Sorry, I don't understand what you are trying to say. I just applied all
>>>> patches in sequence as received, with no conflicts. Should I undo that ?
>>>>
>>>> Guenter
>>>>      
>>>
>>> No, Thank you!
>>>
>>> I just like to mention that Holger Kiehl sent separate patch with
>>> "Pro A520M-C II" support and it could create conflicts. I have found it only
>>> when I have sent my patches.
>>>    
>>
>> I have rechecked repo and "Pro A520M-C II" is added twice after apply both of
>> patches (my and from Holger Kiehl), could you please remove one of mention of
>> it?
> 
> I have resent updated version of this patch without duplication of adding "Pro
> A520M-C II",
> https://patchwork.kernel.org/project/linux-hwmon/patch/20230315222702.1803-1-pauk.denis@gmail.com/
> 
> you could revert this patch and add apply new one, or fix this one.
> 

Done. You could avoid this kind of problem by keeping board names
in alphabetic order.

Guenter