lkml.org 
[lkml]   [2021]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v11 0/2] Update ASUS WMI supported boards
From
Date
On 11/16/21 2:38 AM, Andy Shevchenko wrote:
> On Mon, Nov 15, 2021 at 03:15:39PM -0800, Guenter Roeck wrote:
>> On 11/15/21 2:01 PM, Andy Shevchenko wrote:
>>> On Mon, Nov 15, 2021 at 11:09 PM Denis Pauk <pauk.denis@gmail.com> wrote:
>>>>
>>>> Add support by WMI interface provided by Asus for B550/X570 boards:
>>>> * PRIME X570-PRO,
>>>> * ROG CROSSHAIR VIII HERO
>>>> * ROG CROSSHAIR VIII DARK HERO
>>>> * ROG CROSSHAIR VIII FORMULA
>>>> * ROG STRIX X570-E GAMING
>>>> * ROG STRIX B550-I GAMING
>>>> * ROG STRIX B550-E GAMING
>>>>
>>>> Add support by WMI interface provided by Asus for X370/X470/
>>>> B450/X399 boards:
>>>> * ROG CROSSHAIR VI HERO,
>>>> * PRIME X399-A,
>>>> * PRIME X470-PRO,
>>>> * ROG CROSSHAIR VI EXTREME,
>>>> * ROG CROSSHAIR VI HERO (WI-FI AC),
>>>> * ROG CROSSHAIR VII HERO,
>>>> * ROG CROSSHAIR VII HERO (WI-FI),
>>>> * ROG STRIX Z390-F GAMING
>>>> * ROG STRIX B450-E GAMING,
>>>> * ROG STRIX B450-F GAMING,
>>>> * ROG STRIX B450-I GAMING,
>>>> * ROG STRIX X399-E GAMING,
>>>> * ROG STRIX X470-F GAMING,
>>>> * ROG STRIX X470-I GAMING,
>>>> * ROG ZENITH EXTREME,
>>>> * ROG ZENITH EXTREME ALPHA.
>>>>
>>>> I have added "ROG STRIX Z390-F GAMING" to list of supported boards in
>>>> asus_wmi_sensors.
>>>
>>> Guenter, what is your plan about this patch series? It seems it
>>> missed, by unknown (?) reason, the v5.16-rc1 (I remember seeing it in
>>> some of your tree branches at some point).
>>>
>>
>> I don't see it in my record. Earlier I was simply waiting for some
>> Reviewed-by: tags, which I have never seen.
>
> Ah, understood. Thank you for clarifications.
>
>> Looking into the commit log,
>> I do see:
>>
>> Signed-off-by: Denis Pauk <pauk.denis@gmail.com>
>> Co-developed-by: Eugene Shalygin <eugene.shalygin@gmail.com>
>> Signed-off-by: Eugene Shalygin <eugene.shalygin@gmail.com>
>> Co-developed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>>
>> Did you and Eugene indeed sign this off, ie did you write it, and
>> Eugene and Denis signed it off ? If so, the tags are in the wrong order.
>
> I'm not sure I follow. I have helped Denis with the code and according to
> Submitting Patches documentation the order should be chronological with the
> main author to be first in the list. Here the committer (submitter) is the
> same as the original author IIUC. (I can't speak for Eugene, though)
>

I got the patch from Denis. At the very least, Denis' signature should be
last. It is first.

Guenter

>> On the other side, if the code is ultimately from Denis, with your input,
>> the tags should be either Acked-by: or Reviewed-by: for both Eugene
>> and yourself.
>
> I'm fine with either Co-developed-by+SoB or Reviewed-by.
>
>> Note that v11 of this patch series is missing from
>> https://patchwork.kernel.org/project/linux-hwmon/list/
>> for some reason.
>
> Hmm... Denis, please check locally how you prepare your patches.
>
>

\
 
 \ /
  Last update: 2021-11-16 15:33    [W:0.142 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site