From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-106112.protonmail.ch (mail-106112.protonmail.ch [79.135.106.112]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4378E19644B; Wed, 10 Sep 2025 12:01:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=79.135.106.112 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757505688; cv=none; b=TTli9QqERyKp8hkDsEM1bAHyNVdricQn/V1yj/2ZGxr5nlaHmsA/Z9kJnWAzy6GlfI/hE+32axUloBFMm2e0FsxTghrjioKeagEuqf0C2VfiexLq9rN1hP3MsAxsYVWnn54XN7L0lfPGO8WJ/mIW3820Xvm3deQUXiZ5fGkM8KY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757505688; c=relaxed/simple; bh=pJ6tiNTrjS4YxOdr4O7jX1Q9jdumfA7pm9EcJFYC3WI=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=iWt7+OwEf8dYm4kWvBQfi4KVpk8XOumAolpzmvcT87/hPy0S67TYVAwxns0+c0UUJaD/JBurBUZFvswMxJ37yqWScrM9XHFJuE1FNQYK/DmrPkHGiupG/KqIcpvpOEh5EsiMrd9xPDQ3e5ScRO6kzVETIECL/nLGHHTMV65PL9s= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vinarskis.com; spf=pass smtp.mailfrom=vinarskis.com; dkim=pass (2048-bit key) header.d=vinarskis.com header.i=@vinarskis.com header.b=Vq7egJMy; arc=none smtp.client-ip=79.135.106.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vinarskis.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vinarskis.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=vinarskis.com header.i=@vinarskis.com header.b="Vq7egJMy" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vinarskis.com; s=protonmail; t=1757505683; x=1757764883; bh=6Zua8HZFQSMU7UHKMIvAOiAzH8T9S1RtjgX2NSNgKC0=; h=From:Subject:Date:Message-Id:To:Cc:From:To:Cc:Date:Subject: Reply-To:Feedback-ID:Message-ID:BIMI-Selector; b=Vq7egJMyLWieTgV2yprw2WHZaaJBLKZDszFKw/2b3ogNcc7vQvZ5wGlVKwZ3cA7dR b4hRcPILf0jmD7newWaGvzOjp+tkkVaiLxiZRvb0yNClj/ixBOecu6NKRtxSgFMoEQ OE0c2qwO8QaOj1DbvMvSiK+n2XG2lxOoK49/fntzkwo9OStrBFYBhS8L8T8EkPNTdl xKHxjF6D/YL1COSwCkQa3xvsoY2hHCQxLC47eWJNvxi+XZiSdhI5T44Po5WsPhY+Qo pMNGZRlEy3v0NyXTCt36RTIj0UUD5jNbhcDSopvOzEBdxYD9SLWatF2uMDQQ9ARvuV 8iLqTmwuXpy1w== X-Pm-Submission-Id: 4cMK68619cz2ScCs From: Aleksandrs Vinarskis Subject: [PATCH v5 0/4] leds: privacy-led support for devicetree Date: Wed, 10 Sep 2025 14:01:07 +0200 Message-Id: <20250910-leds-v5-0-bb90a0f897d5@vinarskis.com> Precedence: bulk X-Mailing-List: linux-leds@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAINowWgC/23Q32rCMBQG8FcpuV7Kyd9GEfE9xi6y5LQGtd0SL Rviu+9YOyjiTeAL5/ddfFdWMCcsbF1dWcYxlTT0FMxbxcLe9x3yFCkzCdLACiQ/Yiw8WBFc0/g YHTA6/crYpp+p5v2D8j6V85B/p9ZR3H+fCkbBgWuvBAppZGM/d2PqfS6HVOownNi9ZJRLaGYoC WJ0rbWtsdG7V1AtoZuhImhWWsegAax9CfUCCpihJijaoCRo56RSz/D2GCDj94XmOz9WYCcsxU/ zravNf6MA3YCsG3q54LRv6XYHzD0e6yF3W+q6/QF6JJJYkgEAAA== X-Change-ID: 20250902-leds-c61c877add80 To: Hans de Goede , Lee Jones , Pavel Machek , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bryan O'Donoghue , Jingoo Han , Mauro Carvalho Chehab , Jean-Jacques Hiblot , Jacopo Mondi , Sakari Ailus , Bjorn Andersson , Konrad Dybcio , Daniel Thompson Cc: linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org, threeway@gmail.com, Andy Shevchenko , Aleksandrs Vinarskis , Linus Walleij X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3394; i=alex@vinarskis.com; h=from:subject:message-id; bh=pJ6tiNTrjS4YxOdr4O7jX1Q9jdumfA7pm9EcJFYC3WI=; b=owGbwMvMwCX2dl3hIv4AZgHG02pJDBkHM/odMqxfSO9Z1qCefF78+YdT56dFuWimHO/fO0X19 RHnFyvCO0pZGMS4GGTFFFm6/3xN61o0dy3DdY1vMHNYmUCGMHBxCsBEbjxm+J9yItE/7W3Uah2J XT43dHQDD77PO5V65hO/6YW+K4HBbukM/zTr7BcZe0VNdKneIvCIpVLP/Ii2QFuc8fWdW0SZFE9 rsAAA X-Developer-Key: i=alex@vinarskis.com; a=openpgp; fpr=8E21FAE2D2967BB123303E8C684FD4BA28133815 Re-spin of RFC patch from ~2.5 years ago [1]. v4l2 controls for privacy LEDs has landed, but the DT part was left out. Introduce missing dt-bindings, and commonize 'leds' parameter. Finally, add a patch to enable privacy-led on Lenovo Thinkpad x13s. With recent inflow of arm64-power laptops (Snapdragon X1E/X1P) which mostly use MIPI cameras, this feature becomes more desired. Original rebased patch is still working as expected (with respective DT changes) on Dell XPS 9345. Changelog to original series: - Pick RFC patch, pick R-by, drop RFC-related commit message part - Add new DT binding to describe generic LED consumer properties - Rebase and test on X1E laptop [1] https://lore.kernel.org/all/20230120114524.408368-6-hdegoede@redhat.com/ Signed-off-by: Aleksandrs Vinarskis --- Changes in v5: - Fix last occurance of `privacy-led` to `privacy` - Link to v4: https://lore.kernel.org/r/20250910-leds-v4-0-1fc320488233@vinarskis.com Changes in v4: - Rename `privacy-led` to `privacy`, depend on x86/ACPI change - Simplify example in dt-binding for led-consumer.yaml - This series now depends on: https://lore.kernel.org/all/20250910104702.7470-1-hansg@kernel.org/ - Link to v3: https://lore.kernel.org/r/20250908-leds-v3-0-5944dc400668@vinarskis.com Changes in v3: - Drop unnecessary 'dev->of_node' check by Andy - Keep 'panic-indicator;' in DT by Konrad - Fix property/property-name order by Konrad - Add missing 'led-names' enum definition for video devices by Rob - Add oneOf to 'leds' to avoid conflict with nodes of same name by Rob - Link to v2: https://lore.kernel.org/r/20250905-leds-v2-0-ed8f66f56da8@vinarskis.com Changes in v2: - Fixed mailing issue - Fixed wrong R-by, add my missing sign-off - Elaborated cover letter/commits descriptions to better describe why this is needed, and why trigger-source could not be used instead - dt-bindings: expanded schema description, fixed s/phandle/phandle-array/, expanded the example - dt-bindings: added patch to commonize 'leds' from other schemas - leds: dropped wrapper, dropped exporting of private functions - dts: added patch to utilize privacy-led on Lenovo Thinkpad x13s - Link to v1: https://lore.kernel.org/all/010201990a1f5ad8-fc97fc84-9ef9-4a03-bf1c-2d54423c6497-000000@eu-west-1.amazonses.com/ --- Aleksandrs Vinarskis (3): dt-bindings: leds: add generic LED consumer documentation dt-bindings: leds: commonize leds property arm64: dts: qcom: sc8280xp-x13s: enable camera privacy indicator Hans de Goede (1): leds: led-class: Add devicetree support to led_get() .../bindings/leds/backlight/led-backlight.yaml | 6 +- .../devicetree/bindings/leds/leds-consumer.yaml | 67 ++++++++++++++++++++++ .../bindings/leds/leds-group-multicolor.yaml | 5 +- .../bindings/media/video-interface-devices.yaml | 8 +++ .../dts/qcom/sc8280xp-lenovo-thinkpad-x13s.dts | 8 +-- drivers/leds/led-class.c | 17 +++++- 6 files changed, 96 insertions(+), 15 deletions(-) --- base-commit: 3db46a82d467bd23d9ebc473d872a865785299d8 change-id: 20250902-leds-c61c877add80 prerequisite-message-id: <20250910104702.7470-1-hansg@kernel.org> prerequisite-patch-id: 429eb224c966424b458a7179e10e1f3172408eb8 Best regards, -- Aleksandrs Vinarskis