Re: [REGRESSION] hp-wmi fails to probe on HP OMEN (board 8D41)

[Krishna Chomal <krishna.chomal108@gmail.com>]

On Thu, May 07, 2026 at 12:59:41PM +0200, Thorsten Leemhuis wrote:
>On 5/7/26 11:55, Ilpo Järvinen wrote:
>> On Thu, 7 May 2026, Thorsten Leemhuis wrote:
>>> On 5/4/26 09:22, Thorsten Leemhuis wrote:
>>>> On 5/3/26 09:26, Krishna Chomal wrote:
>>>>> On Thu, Apr 30, 2026 at 12:59:58PM +0800, yue li wrote:
>>>>>> [...]
>>>>>> Regression:
>>>>>> The hp-wmi driver works correctly on kernel 6.18.6-zen1-1-zen,
>>>>>> but fails to probe on 7.0.2-zen1-1-zen with error -22.
>>>>>>
>>>>>> Issue:
>>>>>> On the affected kernel, loading the module fails:
>>>>> [...]
>>>>> Thanks for reporting. This is a known issue [1]. There are 2 failures in
>>>>> your dmesg output:
>>>>> 1. Unknown EC layout: The fix for this has been sent recently.
>>>>>    Currently, it is in the review-ilpo-fixes branch [2].
>>>
>>> It[1] afaics is sitting there for a week already afaics.
>>
>> I'm sorry to be a disappointment here, but I don't find a week a
>> particularly long time.
>
>For a normal patch, I fully agree. But for a regression fix, it is a
>long time[1]. And to me it looked like the patch "platform/x86: hp-wmi:
>silence unknown board warning for 8D41"[2] was a the second and
>necessary part of a fix for the regression Yue Li reported at the start
>of this thread[3] -- without making it obvious in the patch description
>that it was one, so you couldn't know, which is why I CCed you here.
>
>But from looking at Krishna's earlier reply in this thread again, it now
>seems to me like I misinterpreted the "applying the above mentioned 2
>patch" statement, as 9d317a54e46d3b ("platform/x86: hp-wmi: fix fan
>table parsing") [already in v7.1-rc1 and hopefully backported soon]
>seems to be the main fix and the other one ("platform/x86: hp-wmi:
>silence unknown board warning for 8D41"[2] just silences a warning that
>is not causing trouble.
>

Yes that's right. The fan table fix allows the driver to be loaded and
fixes the regression, while the follow-up patch is only to silence the
dmesg warning. I will be more explicit from next time, sorry for any
confusion that I might have created.

>Sorry, my deepest apologies for causing trouble.
>
>Ciao, Thorsten
>
>[1]https://www.kernel.org/doc/html/latest/process/handling-regressions.html#on-how-quickly-regressions-should-be-fixed
>[2]https://lore.kernel.org/all/20260429180953.129885-1-krishna.chomal108@gmail.com/
>
>[3]https://lore.kernel.org/all/CABd86uaQY9-ZSU-_PHKtZZyWnDFHLEsuaM+NBV5nOLWA+YSX7A@mail.gmail.com/