Hi, On 8/22/24 5:38 PM, Gergo Koteles wrote: > While booting, Lenovo 14ARB7 reports 'lenovo-ymc: Unknown key 0 pressed' > warning. Because lenovo_ymc_probe() calls lenovo_ymc_notify() and that > time the YMC part is not triggered yet (which is mandatory for this > model). At the end of lenovo_ymc_notify() YMC is triggered by the > ideapad-laptop driver and then lenovo_ymc_notify() will be called by WMI > to report the actual value. > > Add 0x0 YMC state to the sparse keymap to ignore. > > Signed-off-by: Gergo Koteles <soyer@xxxxxx> Thank you for your patch, I've improved the commit message to address Ilpo's review comments and applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/commit/?h=review-hans&id=a8a2253e84dc62b0d50943408c480188a0086263 Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > drivers/platform/x86/lenovo-ymc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/platform/x86/lenovo-ymc.c b/drivers/platform/x86/lenovo-ymc.c > index e0bbd6a14a89..bd9f95404c7c 100644 > --- a/drivers/platform/x86/lenovo-ymc.c > +++ b/drivers/platform/x86/lenovo-ymc.c > @@ -43,6 +43,8 @@ struct lenovo_ymc_private { > }; > > static const struct key_entry lenovo_ymc_keymap[] = { > + /* Ignore the uninitialized state */ > + { KE_IGNORE, 0x00 }, > /* Laptop */ > { KE_SW, 0x01, { .sw = { SW_TABLET_MODE, 0 } } }, > /* Tablet */
