Re: [PATCH v11 0/2] Update ASUS WMI supported boards

From: Guenter Roeck
Date: Mon Nov 15 2021 - 20:53:15 EST


On 11/15/21 2:01 PM, Andy Shevchenko wrote:
On Mon, Nov 15, 2021 at 11:09 PM Denis Pauk <pauk.denis@xxxxxxxxx> 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. Looking into the commit log,
I do see:

Signed-off-by: Denis Pauk <pauk.denis@xxxxxxxxx>
Co-developed-by: Eugene Shalygin <eugene.shalygin@xxxxxxxxx>
Signed-off-by: Eugene Shalygin <eugene.shalygin@xxxxxxxxx>
Co-developed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

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.
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.

Note that v11 of this patch series is missing from
https://patchwork.kernel.org/project/linux-hwmon/list/
for some reason.

Guenter