From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f49.google.com (mail-qv1-f49.google.com [209.85.219.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 882D9241686 for ; Mon, 8 Jun 2026 19:05:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780945516; cv=none; b=mKaKGS+uZ0ZrgoAyVEgx5W548vcRAwKFe/NnEUogOC/Ik0mMAfMEwR/qFtubDKzanoHVKxF7Z4q6HsLZR6+0LsrbWAc/ICUwwkgPsTVUaPXP7D+vZLNFSk/PVTZ3HGQbYgw40IKzBAGhFcaegM2xpPRBMpP/NT9yqwdbYUQwj5g= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780945516; c=relaxed/simple; bh=zEQw9EzLJfdU0N6hQoXrThJLRbefFVlSvhYTIjcEwWI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=nEv0Vi8lsIeUsTlwpJtzNCor3VNSMLrws8CDdS1IOP+U9P3yq9jtde0Bris368QHrF7b49GuzmKA+yHiXdH53Qy4lVa8YYICfzivyj1MC1XZfYnkCSJ2tzDZ39s0cb7XIByQoUUvaJqZj6oEjC46ksPni/AWoQDYBJsFe1Y1Lzw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=GP2emADW; arc=none smtp.client-ip=209.85.219.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="GP2emADW" Received: by mail-qv1-f49.google.com with SMTP id 6a1803df08f44-8ccf01ba514so39577566d6.0 for ; Mon, 08 Jun 2026 12:05:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780945514; x=1781550314; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=MHfJg8I40AyrFWL2HTXVtd3AE1IeaVRWavtCFPKxV9Y=; b=GP2emADWJmk4QH3D0vRVdwcxcfa2Z43DiMAKcSbzRPC3MIi7B2mStf7PoEgSvy0ZMD hViY+anxcZ3cSz1yLR92jZ8utCmHGq4GC1Oz5hB9z61nCkISVntnTo0jpiZiww8kQPwz eeuoOQ9wWUMx1ZyEzyRGSyv9i6efVYO83OA+O/bQXPZsfPw8NHnFKE0w9MY+F636XmfV YMgFyam8ueJuu3GHD3HTCGglOZVoleAPXxOMhzlELKfpaJklwFvGmm65DWxa8KoWoGDg QovHLlXiimJSN0/kn5J+F1WPWmfPzYZe/Ufzp1clAKtuBdSS8L/sGMoEMVjpEaSi4X0T zQmw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780945514; x=1781550314; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=MHfJg8I40AyrFWL2HTXVtd3AE1IeaVRWavtCFPKxV9Y=; b=WAst8+dbScFsLJuIhXPij9DiqF8ncsrScCsV5dP68yhwInkV7zWCtusmjbCALssoLY i/xJ+0lMy3gejY5QLH95G0weKmVWCqPkWeXsuejeo4sGaXi3QvStwELxKJt3xr3Yr3v+ mZqSwydKc1H8m/uQikpuReTwsd0ogLwj3j8N/tYTOkPvgsUGe9CXym79DwDDoA8qBnVQ j2X3BXyDYN/fctQS7l6NNKGbluS5eiNJDNw14hDvyJlR7GsZIv6ptT9k7OzcyWG0g9mL 46OLnt8tu41hj0uO7iL/j1LAhhyJC+2NYBGq2bgjv5EoT1t6sGtri4Wb2F5xgeEBZFqA Hdvg== X-Forwarded-Encrypted: i=1; AFNElJ/L0Nz5cys3tZ1xSCiNhnrTgM6l2Bt2Yc0QDnfnkC9pruLUuDKwH6JaZHTkbVx17/RHcBZa7sHNgimzrHE=@vger.kernel.org X-Gm-Message-State: AOJu0YwyMC6Da/77Z9eBiDGZl2asm6Kby+bEat+jFQUkAyqbvRSW22z0 4qazkPOa73U1k1m8GGH5evPAtR3KWDCx/kwhgFgu8q4AeSeWDbliMQXT X-Gm-Gg: Acq92OE+p8pkTxdyExbgZVxh9PE0SI5/YyfWcWUc/wihJkyMOOVLEH88SJuPcJhlAWG w40I9qmtn4WKAePeR4jk61cp3xTR4uj/dycTkBP0wFCh5uylSWHHhUDUSJ3J/W5QYqeA0jA4cLb s0q/DWmPgs5aP/KrqrR6uGf5YZxuy+AzGmzxBqvoPXhK9FBgXYJ+ZkVLgvGTRye7AUMQzPg1+4w 6cicBF8GXk2Lhn++NZGwO/6y+XM3x5rTZJp/c2zxVG8442MyBLkg6YHM12f1mstjfHt8dutXVE4 Ap7cM5U4wGXUg2DHegm9bg9rpiArANUfHX9hTsNcTyH0o6Eq/c92kjCLGdvMShGISMzfDrQlK7L mns+q5R1Gl5Trqft+rLdlEtPjLO4nrY5dwb+i5DVymYrOimQAbZUVoBIXuyDHV4cIhopNBTaMDX TB/NMJJz98ijsrDaYj9PDgBROQIpiwlaG734pPHvmlbvVEACT8TLuktPtbFTgRaYo6kflvFLFUz EiAt1aS9BMiBOVu2w== X-Received: by 2002:a05:622a:1187:b0:517:c65c:488b with SMTP id d75a77b69052e-517c65c771emr13993181cf.0.1780945514369; Mon, 08 Jun 2026 12:05:14 -0700 (PDT) Received: from fedora (pool-100-11-178-145.phlapa.fios.verizon.net. [100.11.178.145]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-51775df4dd6sm160482541cf.27.2026.06.08.12.05.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 08 Jun 2026 12:05:13 -0700 (PDT) From: Dave Carey To: johannes.goede@oss.qualcomm.com Cc: platform-driver-x86@vger.kernel.org, pithenrich2d@googlemail.com, mpearson-lenovo@squebb.ca, hdegoede@redhat.com, ilpo.jarvinen@linux.intel.com, armin.wolf@outlook.de, linux-kernel@vger.kernel.org, Dave Carey Subject: Re: [PATCH v5] platform/x86/lenovo: Add Yoga Book 9 keyboard dock detection driver Date: Mon, 8 Jun 2026 15:05:07 -0400 Message-ID: <20260608190507.77836-1-carvsdriver@gmail.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: References: <20260521144034.282419-1-carvsdriver@gmail.com> <20260527122701.242907-1-carvsdriver@gmail.com> <78833ba9-7ea0-4c7f-8d41-ac0fd64ba15c@oss.qualcomm.com> <20260608125211.11957-1-carvsdriver@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Hi Hans, Confirmed: on the YB9, both lenovo-ymc and yb9-kbdock register input nodes with SW_TABLET_MODE in their capability bitmap (SW=2 in /proc/bus/input/devices). You are right that this is a problem regardless of which one fires events -- GNOME reads the capability at startup. The recommended fix would be: add a DMI blocklist to lenovo_ymc_probe() to return early on the Yoga Book 9, matching on: DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"), DMI_MATCH(DMI_PRODUCT_NAME, "83KJ"), That leaves yb9-kbdock as the sole SW_TABLET_MODE source on this hardware. Two questions on how you'd like to proceed: 1. Should the lenovo-ymc fix come as a standalone patch, or would you prefer a v6 2-patch series with the ymc change as patch 1/2 and the kbdock driver as patch 2/2? 2. For the early return in probe(): you suggested "return 0" to avoid WMI framework error messages. That requires a NULL guard in lenovo_ymc_notify() since priv is never set in that path. Is that the right approach, or would you prefer a different pattern? Thanks, Dave