Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2217495/?format=api
{ "id": 2217495, "url": "http://patchwork.ozlabs.org/api/patches/2217495/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260329-rust_leds-v13-3-21a599c5b2d1@posteo.de/", "project": { "id": 28, "url": "http://patchwork.ozlabs.org/api/projects/28/?format=api", "name": "Linux PCI development", "link_name": "linux-pci", "list_id": "linux-pci.vger.kernel.org", "list_email": "linux-pci@vger.kernel.org", "web_url": null, "scm_url": null, "webscm_url": null, "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260329-rust_leds-v13-3-21a599c5b2d1@posteo.de>", "list_archive_url": null, "date": "2026-03-29T17:22:36", "name": "[v13,3/3] rust: leds: add multicolor classdev abstractions", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "b3a6de1d3aa7ceb2847f515b3fd909c524a4bf21", "submitter": { "id": 91819, "url": "http://patchwork.ozlabs.org/api/people/91819/?format=api", "name": "Markus Probst", "email": "markus.probst@posteo.de" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/linux-pci/patch/20260329-rust_leds-v13-3-21a599c5b2d1@posteo.de/mbox/", "series": [ { "id": 497934, "url": "http://patchwork.ozlabs.org/api/series/497934/?format=api", "web_url": "http://patchwork.ozlabs.org/project/linux-pci/list/?series=497934", "date": "2026-03-29T17:22:33", "name": "rust: leds: add led classdev abstractions", "version": 13, "mbox": "http://patchwork.ozlabs.org/series/497934/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2217495/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2217495/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "\n <linux-pci+bounces-51389-incoming=patchwork.ozlabs.org@vger.kernel.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "linux-pci@vger.kernel.org" ], "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n secure) header.d=posteo.de header.i=@posteo.de header.a=rsa-sha256\n header.s=2017 header.b=VxCRHg7h;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c0a:e001:db::12fc:5321; helo=sea.lore.kernel.org;\n envelope-from=linux-pci+bounces-51389-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=posteo.de header.i=@posteo.de\n header.b=\"VxCRHg7h\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=185.67.36.66", "smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=posteo.de", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=posteo.de" ], "Received": [ "from sea.lore.kernel.org (sea.lore.kernel.org\n [IPv6:2600:3c0a:e001:db::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fkLsD1QN4z1y1P\n\tfor <incoming@patchwork.ozlabs.org>; Mon, 30 Mar 2026 04:26:40 +1100 (AEDT)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 6CA073034DDC\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 29 Mar 2026 17:22:47 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id B67D2385520;\n\tSun, 29 Mar 2026 17:22:45 +0000 (UTC)", "from mout02.posteo.de (mout02.posteo.de [185.67.36.66])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 14B283859C5\n\tfor <linux-pci@vger.kernel.org>; Sun, 29 Mar 2026 17:22:43 +0000 (UTC)", "from submission (posteo.de [185.67.36.169])\n\tby mout02.posteo.de (Postfix) with ESMTPS id F066E240103\n\tfor <linux-pci@vger.kernel.org>; Sun, 29 Mar 2026 19:22:37 +0200 (CEST)", "from customer (localhost [127.0.0.1])\n\tby submission (posteo.de) with ESMTPSA id 4fkLmW17Nyz9rxB;\n\tSun, 29 Mar 2026 19:22:35 +0200 (CEST)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1774804965; cv=none;\n b=aqqUbdNN3VAwhq+C9rkcLYMz+DZRLWTVxw6CaODxuz8Ylg2Xt00sTIcxhZUKQpokW+kZZNp1bpGq6WpRnhM2HMfaNLBgHo1liCRh1VcW/toz7ww9SM3yA8y2PKQRQ4PA0Wp312PPWAAcg21YIABvgRkLZMgrzLNy28VfZFvelUk=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1774804965; c=relaxed/simple;\n\tbh=Vd4ztAIIe3dZLs6GaRmT/tNaiOnzNJCI9BZUDIR6Eu4=;\n\th=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References:\n\t In-Reply-To:To:Cc;\n b=c+w4W2V/i45XIfjcnUaPzO487Skt7F/ibXe7gLf1Q/pumviE7yuvXziwm6pQfWop7H9WxbNTklyFrLgUf/NkpnwxLOisWTyFO/YJv1F7+8x55bjOaU7lLXNy2IerRNB0+uzDgd6qqZxvZXxTwXMfxiw+obXdTMuwtrxmaW2CH+I=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=posteo.de;\n spf=pass smtp.mailfrom=posteo.de;\n dkim=pass (2048-bit key) header.d=posteo.de header.i=@posteo.de\n header.b=VxCRHg7h; arc=none smtp.client-ip=185.67.36.66", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=posteo.de; s=2017;\n\tt=1774804957; bh=joSMi7pDlFO/HtroYqlGztfxeKGo9pyV8e9cWGgHrlI=;\n\th=From:Date:Subject:MIME-Version:Content-Type:\n\t Content-Transfer-Encoding:Message-Id:To:Cc:Autocrypt:OpenPGP:From;\n\tb=VxCRHg7h4At7/qLaBdQ08SaFeU+NY0+7V/5AFnSsc8PJMxN2JNADv9BGrBTxqsf9p\n\t 6RII0mCb9rHhTYEPQ4WmY0e5/+6pTs8A7gIK5evlucMOaY5S6m6MZQWfKr3TXkowoe\n\t fyTqKqefwZzo+qaiEMNpPlMHwyroPp9d7n9bFHr9ltLSk0FlXkZA7tEDgOo7N0goQE\n\t Zdd6jyF6oTqbjLFUwAewjzygnHBTGjzFhW8ppmjM/7zdlNewQ+i/rXhLewtlUvOgIB\n\t 8kYgeBRfmzGRMu6HzF9M98i1i4wTkIHq19X3dLYC0Leh6kQwgROZ2/0zgEp6vc/jTx\n\t VkVouJk+jG7Eg==", "From": "Markus Probst <markus.probst@posteo.de>", "Date": "Sun, 29 Mar 2026 17:22:36 +0000", "Subject": "[PATCH v13 3/3] rust: leds: add multicolor classdev abstractions", "Precedence": "bulk", "X-Mailing-List": "linux-pci@vger.kernel.org", "List-Id": "<linux-pci.vger.kernel.org>", "List-Subscribe": "<mailto:linux-pci+subscribe@vger.kernel.org>", "List-Unsubscribe": "<mailto:linux-pci+unsubscribe@vger.kernel.org>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "Message-Id": "<20260329-rust_leds-v13-3-21a599c5b2d1@posteo.de>", "References": "<20260329-rust_leds-v13-0-21a599c5b2d1@posteo.de>", "In-Reply-To": "<20260329-rust_leds-v13-0-21a599c5b2d1@posteo.de>", "To": "Lee Jones <lee@kernel.org>, Pavel Machek <pavel@kernel.org>,\n Greg Kroah-Hartman <gregkh@linuxfoundation.org>,\n Dave Ertman <david.m.ertman@intel.com>, Ira Weiny <ira.weiny@intel.com>,\n Leon Romanovsky <leon@kernel.org>, Miguel Ojeda <ojeda@kernel.org>,\n Alex Gaynor <alex.gaynor@gmail.com>, Gary Guo <gary@garyguo.net>,\n\t=?utf-8?q?Bj=C3=B6rn_Roy_Baron?= <bjorn3_gh@protonmail.com>,\n Benno Lossin <lossin@kernel.org>, Andreas Hindborg <a.hindborg@kernel.org>,\n Alice Ryhl <aliceryhl@google.com>, Trevor Gross <tmgross@umich.edu>,\n Danilo Krummrich <dakr@kernel.org>, \"Rafael J. Wysocki\" <rafael@kernel.org>,\n Bjorn Helgaas <bhelgaas@google.com>,\n =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= <kwilczynski@kernel.org>,\n Boqun Feng <boqun@kernel.org>, Boqun Feng <boqun@kernel.org>", "Cc": "rust-for-linux@vger.kernel.org, linux-leds@vger.kernel.org,\n linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,\n Markus Probst <markus.probst@posteo.de>", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=20776;\n i=markus.probst@posteo.de; h=from:subject:message-id;\n bh=Vd4ztAIIe3dZLs6GaRmT/tNaiOnzNJCI9BZUDIR6Eu4=;\n b=owEBiQJ2/ZANAwAIATR2H/jnrUPSAcsmYgBpyV/TbqzDU7TwT7Kny9Lg4V01vlCuoOpnL8gV5\n M5NEVd/J+iJAk8EAAEIADkWIQSCdBjE9KxY53IwxHM0dh/4561D0gUCaclf0xsUgAAAAAAEAA5t\n YW51MiwyLjUrMS4xMSwyLDIACgkQNHYf+OetQ9Ko1w//ayEXm+UHfjX29jskh7k90p/11PUHdpA\n tRemzRphpfmOIo4YUR9wSkV6mi9EHck4qAv8Ee8RFfaUAsiOCWDGqOWsJsDx/NFWmLC1DciXn8s\n N2v1EhRGO0WT7nI76i673K31Y2rGLKp90yHL+DZI/XPcJbF2uvOVxwMKJZAhgsUYTObBFK/vQqY\n pmA6Q4gMOOVSnEllmCuHZT7Clq81MAcmua4XSDY5Oh9OxRPkly4yX2t5Fj7qJxLBob29nX5pv4g\n hKkNggjaBkRzBDkW57t8Evo1rEGKbtgplbj6fQB4VK5OVj/I47MUaGW+9Heaqijg82jkcITsFFw\n /3IHw+4GkN3tVcEf0eTkjvwXCtF0Rg8C0PgUAu2mPHbQ/dFd4m7alU4Un2/ybYeaKZk6JUrp4rG\n kTUpxpV5Mg11K8n2SvX4ZuGsz63ToTEOq22X6D22F6NLDcymbOLkr91IXHdG8zVSWWJMS46NKW/\n N9W+Rw5m6GPUBxZFlt6BpI0pxbBqtCG2KTrcUATtRAPG2fv9nVSLb0k1YVqoRsd8hTvxj+SP/DW\n 3NNUh5Nhqb2b+kdVUGit/3lg0kql0UGDhdGlGTYJQ3HyVTKOnhRNT0O8w00VILXuFbctxiCgzQc\n D4dt0QIgRzpWR02+47spNaeSzRTNE4+5tOQA1e3apl0llLrk0JHE=", "X-Developer-Key": "i=markus.probst@posteo.de; a=openpgp;\n fpr=827418C4F4AC58E77230C47334761FF8E7AD43D2", "Autocrypt": "addr=markus.probst@posteo.de; prefer-encrypt=mutual;\n keydata=xsFNBGiDvXgBEADAXUceKafpl46S35UmDh2wRvvx+UfZbcTjeQOlSwKP7YVJ4JOZrVs93qReNLkO\n WguIqPBxR9blQ4nyYrqSCV+MMw/3ifyXIm6Pw2YRUDg+WTEOjTixRCoWDgUj1nOsvJ9tVAm76Ww+\n /pAnepVRafMID0rqEfD9oGv1YrfpeFJhyE2zUw3SyyNLIKWD6QeLRhKQRbSnsXhGLFBXCqt9k5JA\n RhgQof9zvztcCVlT5KVvuyfC4H+HzeGmu9201BVyihJwKdcKPq+n/aY5FUVxNTgtI9f8wIbmfAja\n oT1pjXSp+dszakA98fhONM98pOq723o/1ZGMZukyXFfsDGtA3BB79HoopHKujLGWAGskzClwTjRQ\n xBqxh/U/lL1pc+0xPWikTNCmtziCOvv0KA0arDOMQlyFvImzX6oGVgE4ksKQYbMZ3Ikw6L1Rv1J+\n FvN0aNwOKgL2ztBRYscUGcQvA0Zo1fGCAn/BLEJvQYShWKeKqjyncVGoXFsz2AcuFKe1pwETSsN6\n OZncjy32e4ktgs07cWBfx0v62b8md36jau+B6RVnnodaA8++oXl3FRwiEW8XfXWIjy4umIv93tb8\n 8ekYsfOfWkTSewZYXGoqe4RtK80ulMHb/dh2FZQIFyRdN4HOmB4FYO5sEYFr9YjHLmDkrUgNodJC\n XCeMe4BO4iaxUQARAQABzRdtYXJrdXMucHJvYnN0QHBvc3Rlby5kZcLBkQQTAQgAOxYhBIJ0GMT0\n rFjncjDEczR2H/jnrUPSBQJog714AhsDBQsJCAcCAiICBhUKCQgLAgQWAgMBAh4HAheAAAoJEDR2\n H/jnrUPSgdkQAISaTk2D345ehXEkn5z2yUEjaVjHIE7ziqRaOgn/QanCgeTUinIv6L6QXUFvvIfH\n 1OLPwQ1hfvEg9NnNLyFezWSy6jvoVBTIPqicD/r3FkithnQ1IDkdSjrarPMxJkvuh3l7XZHo49GV\n HQ8i5zh5w4YISrcEtE99lJisvni2Jqx7we5tey9voQFDyM8jxlSWv3pmoUTCtBkX/eKHJXosgsuS\n B4TGDCVPOjla/emI5c9MhMG7O4WEEmoSdPbmraPw66YZD6uLyhV4DPHbiDWRzXWnClHSyjB9rky9\n lausFxogvu4l9H+KDsXIadNDWdLdu1/enS/wDd9zh5S78rY2jeXaG4mnf4seEKamZ7KQ6FIHrcyP\n ezdDzssPQcTQcGRMQzCn6wP3tlGk7rsfmyHMlFqdRoNNv+ZER/OkmZFPW655zRfbMi0vtrqK2Awm\n 9ggobb1oktfd9PPNXMUY+DNVlgR2G7jLnenSoQausLUm0pHoNE8TWFv851Y6SOYnvn488sP1Tki5\n F3rKwclawQFHUXTCQw+QSh9ay8xgnNZfH+u9NY7w3gPoeKBOAFcBc2BtzcgekeWS8qgEmm2/oNFV\n G0ivPQbRx8FjRKbuF7g3YhgNZZ0ac8FneuUtJ2PkSIFTZhaAiC0utvxk0ndmWFiW4acEkMZGrLaM\n L2zWNjrqwsD2zsFNBGiDvXgBEADCXQy1n7wjRxG12DOVADawjghKcG+5LtEf31WftHKLFbp/HArj\n BhkT6mj+CCI1ClqY+FYU5CK/s0ScMfLxRGLZ0Ktzawb78vOgBVFT3yB1yWBTewsAXdqNqRooaUNo\n 8cG/NNJLjhccH/7PO/FWX5qftOVUJ/AIsAhKQJ18Tc8Ik73v427EDxuKb9mTAnYQFA3Ev3hAiVbO\n 6Rv39amVOfJ8sqwiSUGidj2Fctg2aB5JbeMln0KCUbTD1LhEFepeKypfofAXQbGwaCjAhmkWy/q3\n IT1mUrPxOngbxdRoOx1tGUC0HCMUW1sFaJgQPMmDcR0JGPOpgsKnitsSnN7ShcCr1buel7vLnUMD\n +TAZ5opdoF6HjAvAnBQaijtK6minkrM0seNXnCg0KkV8xhMNa6zCs1rq4GgjNLJue2EmuyHooHA4\n 7JMoLVHcxVeuNTp6K2+XRx0Pk4e2Lj8IVy9yEYyrywEOC5XRW37KJjsiOAsumi1rkvM7QREWgUDe\n Xs0+RpxI3QrrANh71fLMRo7LKRF3Gvw13NVCCC9ea20P4PwhgWKStkwO2NO+YJsAoS1QycMi/vKu\n 0EHhknYXamaSV50oZzHKmX56vEeJHTcngrM8R1SwJCYopCx9gkz90bTVYlitJa5hloWTYeMD7FNj\n Y6jfVSzgM/K4gMgUNDW/PPGeMwARAQABwsF2BBgBCAAgFiEEgnQYxPSsWOdyMMRzNHYf+OetQ9IF\n AmiDvXgCGwwACgkQNHYf+OetQ9LHDBAAhk+ab8+WrbS/b1/gYW3q1KDiXU719nCtfkUVXKidW5Ec\n Idlr5HGt8ilLoxSWT2Zi368iHCXS0WenGgPwlv8ifvB7TOZiiTDZROZkXjEBmU4nYjJ7GymawpWv\n oQwjMsPuq6ysbzWtOZ7eILx7cI0FjQeJ/Q2baRJub0uAZNwBOxCkAS6lpk5Fntd2u8CWmDQo4SYp\n xeuQ+pwkp0yEP30RhN2BO2DXiBEGSZSYh+ioGbCHQPIV3iVj0h6lcCPOqopZqyeCfigeacBI0nvN\n jHWz/spzF3+4OS+3RJvoHtAQmProxyGib8iVsTxgZO3UUi4TSODeEt0i0kHSPY4sCciOyXfAyYoD\n DFqhRjOEwBBxhr+scU4C1T2AflozvDwq3VSONjrKJUkhd8+WsdXxMdPFgBQuiKKwUy11mz6KQfcR\n wmDehF3UaUoxa+YIhWPbKmycxuX/D8SvnqavzAeAL1OcRbEI/HsoroVlEFbBRNBZLJUlnTPs8ZcU\n 4+8rq5YX1GUrJL3jf6SAfSgO7UdkEET3PdcKFYtS+ruV1Cp5V0q4kCfI5jk25iiz8grM2wOzVSsc\n l1mEkhiEPH87HP0whhb544iioSnumd3HJKL7dzhRegsMizatupp8D65A2JziW0WKopa1iw9fti3A\n aBeNN4ijKZchBXHPgVx+YtWRHfcm4l8=", "OpenPGP": "url=https://posteo.de/keys/markus.probst@posteo.de.asc;\n preference=encrypt" }, "content": "Implement the abstractions needed for multicolor led class devices,\nincluding:\n\n* `led::MultiColor` - the led mode implementation\n\n* `MultiColorSubLed` - a safe wrapper arround `mc_subled`\n\n* `led::MultiColorDevice` - a safe wrapper around `led_classdev_mc`\n\n* `led::DeviceBuilder::build_multicolor` - a function to register a new\n multicolor led class device\n\nSigned-off-by: Markus Probst <markus.probst@posteo.de>\n---\n rust/bindings/bindings_helper.h | 1 +\n rust/kernel/led.rs | 30 +++-\n rust/kernel/led/multicolor.rs | 387 ++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 417 insertions(+), 1 deletion(-)", "diff": "diff --git a/rust/bindings/bindings_helper.h b/rust/bindings/bindings_helper.h\nindex 083cc44aa952..3171e3e6351c 100644\n--- a/rust/bindings/bindings_helper.h\n+++ b/rust/bindings/bindings_helper.h\n@@ -63,6 +63,7 @@\n #include <linux/ioport.h>\n #include <linux/jiffies.h>\n #include <linux/jump_label.h>\n+#include <linux/led-class-multicolor.h>\n #include <linux/mdio.h>\n #include <linux/mm.h>\n #include <linux/miscdevice.h>\ndiff --git a/rust/kernel/led.rs b/rust/kernel/led.rs\nindex 5035563d68a3..a862d88cab29 100644\n--- a/rust/kernel/led.rs\n+++ b/rust/kernel/led.rs\n@@ -33,8 +33,12 @@\n }, //\n };\n \n+#[cfg(CONFIG_LEDS_CLASS_MULTICOLOR)]\n+mod multicolor;\n mod normal;\n \n+#[cfg(CONFIG_LEDS_CLASS_MULTICOLOR)]\n+pub use multicolor::{MultiColor, MultiColorDevice, MultiColorSubLed};\n pub use normal::{Device, Normal};\n \n /// The name of the led is determined by the driver.\n@@ -279,7 +283,24 @@ pub enum Color {\n Violet = bindings::LED_COLOR_ID_VIOLET,\n Yellow = bindings::LED_COLOR_ID_YELLOW,\n Ir = bindings::LED_COLOR_ID_IR,\n+ #[cfg_attr(\n+ CONFIG_LEDS_CLASS_MULTICOLOR,\n+ doc = \"Use this color for a [`MultiColor`] led.\"\n+ )]\n+ #[cfg_attr(\n+ not(CONFIG_LEDS_CLASS_MULTICOLOR),\n+ doc = \"Use this color for a `MultiColor` led.\"\n+ )]\n+ /// If the led supports RGB, use [`Color::Rgb`] instead.\n Multi = bindings::LED_COLOR_ID_MULTI,\n+ #[cfg_attr(\n+ CONFIG_LEDS_CLASS_MULTICOLOR,\n+ doc = \"Use this color for a [`MultiColor`] led with rgb support.\"\n+ )]\n+ #[cfg_attr(\n+ not(CONFIG_LEDS_CLASS_MULTICOLOR),\n+ doc = \"Use this color for a `MultiColor` led with rgb support.\"\n+ )]\n Rgb = bindings::LED_COLOR_ID_RGB,\n Purple = bindings::LED_COLOR_ID_PURPLE,\n Orange = bindings::LED_COLOR_ID_ORANGE,\n@@ -319,7 +340,14 @@ fn try_from(value: u32) -> core::result::Result<Self, Self::Error> {\n ///\n /// Each led mode has its own led class device type with different capabilities.\n ///\n-/// See [`Normal`].\n+#[cfg_attr(\n+ CONFIG_LEDS_CLASS_MULTICOLOR,\n+ doc = \"See [`Normal`] and [`MultiColor`].\"\n+)]\n+#[cfg_attr(\n+ not(CONFIG_LEDS_CLASS_MULTICOLOR),\n+ doc = \"See [`Normal`] and `MultiColor`.\"\n+)]\n pub trait Mode: private::Sealed {\n /// The class device for the led mode.\n type Device<T: LedOps<Mode = Self>>;\ndiff --git a/rust/kernel/led/multicolor.rs b/rust/kernel/led/multicolor.rs\nnew file mode 100644\nindex 000000000000..726fdaf068cb\n--- /dev/null\n+++ b/rust/kernel/led/multicolor.rs\n@@ -0,0 +1,387 @@\n+// SPDX-License-Identifier: GPL-2.0\n+\n+//! Led mode for the `struct led_classdev_mc`.\n+//!\n+//! C header: [`include/linux/led-class-multicolor.h`](srctree/include/linux/led-class-multicolor.h)\n+\n+use crate::alloc::KVec;\n+\n+use super::*;\n+\n+/// The led mode for the `struct led_classdev_mc`. Leds with this mode can have multiple colors.\n+pub enum MultiColor {}\n+impl Mode for MultiColor {\n+ type Device<T: LedOps<Mode = Self>> = MultiColorDevice<T>;\n+}\n+impl private::Sealed for MultiColor {}\n+\n+/// The multicolor sub led info representation.\n+///\n+/// This structure represents the Rust abstraction for a C `struct mc_subled`.\n+#[repr(C)]\n+#[derive(Copy, Clone, Debug)]\n+#[non_exhaustive]\n+pub struct MultiColorSubLed {\n+ /// the color of the sub led\n+ pub color: Color,\n+ /// the brightness of the sub led.\n+ ///\n+ /// The value will be automatically calculated.\n+ /// See `MultiColor::pre_brightness_set`.\n+ pub brightness: u32,\n+ /// the intensity of the sub led.\n+ pub intensity: u32,\n+ /// arbitrary data for the driver to store.\n+ pub channel: u32,\n+}\n+\n+// We directly pass a reference to the `subled_info` field in `led_classdev_mc` to the driver via\n+// `Device::subleds()`.\n+// We need safeguards to ensure `MultiColorSubLed` and `mc_subled` stay identical.\n+const _: () = {\n+ use core::mem::offset_of;\n+\n+ const fn assert_same_type<T>(_: &T, _: &T) {}\n+\n+ let rust_zeroed = MultiColorSubLed {\n+ color: Color::White,\n+ brightness: 0,\n+ intensity: 0,\n+ channel: 0,\n+ };\n+ let c_zeroed = bindings::mc_subled {\n+ color_index: 0,\n+ brightness: 0,\n+ intensity: 0,\n+ channel: 0,\n+ };\n+\n+ assert!(offset_of!(MultiColorSubLed, color) == offset_of!(bindings::mc_subled, color_index));\n+ assert_same_type(&0u32, &c_zeroed.color_index);\n+\n+ assert!(\n+ offset_of!(MultiColorSubLed, brightness) == offset_of!(bindings::mc_subled, brightness)\n+ );\n+ assert_same_type(&rust_zeroed.brightness, &c_zeroed.brightness);\n+\n+ assert!(offset_of!(MultiColorSubLed, intensity) == offset_of!(bindings::mc_subled, intensity));\n+ assert_same_type(&rust_zeroed.intensity, &c_zeroed.intensity);\n+\n+ assert!(offset_of!(MultiColorSubLed, channel) == offset_of!(bindings::mc_subled, channel));\n+ assert_same_type(&rust_zeroed.channel, &c_zeroed.channel);\n+\n+ assert!(size_of::<MultiColorSubLed>() == size_of::<bindings::mc_subled>());\n+};\n+\n+impl MultiColorSubLed {\n+ /// Create a new multicolor sub led info.\n+ pub const fn new(color: Color) -> Self {\n+ Self {\n+ color,\n+ brightness: 0,\n+ intensity: 0,\n+ channel: 0,\n+ }\n+ }\n+\n+ /// Set arbitrary data for the driver.\n+ pub const fn channel(mut self, channel: u32) -> Self {\n+ self.channel = channel;\n+ self\n+ }\n+\n+ /// Set the initial intensity of the subled.\n+ pub const fn initial_intensity(mut self, intensity: u32) -> Self {\n+ self.intensity = intensity;\n+ self\n+ }\n+}\n+\n+/// The multicolor led class device representation.\n+///\n+/// This structure represents the Rust abstraction for a multicolor led class device.\n+#[pin_data(PinnedDrop)]\n+pub struct MultiColorDevice<T: LedOps<Mode = MultiColor>> {\n+ #[pin]\n+ ops: T,\n+ #[pin]\n+ classdev: Opaque<bindings::led_classdev_mc>,\n+}\n+\n+impl<'a, S: DeviceBuilderState> DeviceBuilder<'a, S> {\n+ /// Registers a new [`MulticolorDevice`].\n+ pub fn build_multicolor<T: LedOps<Mode = MultiColor>>(\n+ self,\n+ parent: &'a T::Bus,\n+ ops: impl PinInit<T, Error> + 'a,\n+ subleds: &'a [MultiColorSubLed],\n+ ) -> impl PinInit<Devres<MultiColorDevice<T>>, Error> + 'a {\n+ Devres::new(\n+ parent.as_ref(),\n+ try_pin_init!(MultiColorDevice {\n+ ops <- ops,\n+ classdev <- Opaque::try_ffi_init(|ptr: *mut bindings::led_classdev_mc| {\n+ let mut used = 0;\n+ if subleds.iter().any(|subled| {\n+ let bit = 1 << (subled.color as u32);\n+ if (used & bit) != 0 {\n+ true\n+ } else {\n+ used |= bit;\n+ false\n+ }\n+ }) {\n+ dev_err!(parent.as_ref(), \"duplicate color in multicolor led\\n\");\n+ return Err(EINVAL);\n+ }\n+ let mut subleds_vec = KVec::new();\n+ subleds_vec.extend_from_slice(subleds, GFP_KERNEL)?;\n+ let (subled_info, num_colors, capacity) = subleds_vec.into_raw_parts();\n+ debug_assert_eq!(num_colors, capacity);\n+\n+ // SAFETY: `try_ffi_init` guarantees that `ptr` is valid for write.\n+ // `led_classdev_mc` gets fully initialized in-place by\n+ // `led_classdev_multicolor_register_ext` including `mutex` and `list_head`.\n+ unsafe {\n+ ptr.write(bindings::led_classdev_mc {\n+ led_cdev: bindings::led_classdev {\n+ brightness_set: (!T::BLOCKING)\n+ .then_some(Adapter::<T>::brightness_set_callback),\n+ brightness_set_blocking: T::BLOCKING\n+ .then_some(Adapter::<T>::brightness_set_blocking_callback),\n+ brightness_get: T::HAS_BRIGHTNESS_GET\n+ .then_some(Adapter::<T>::brightness_get_callback),\n+ blink_set: T::HAS_BLINK_SET\n+ .then_some(Adapter::<T>::blink_set_callback),\n+ max_brightness: T::MAX_BRIGHTNESS,\n+ brightness: self.initial_brightness,\n+ default_trigger: self\n+ .default_trigger\n+ .map_or(core::ptr::null(), CStrExt::as_char_ptr),\n+ color: self.color as u32,\n+ name: self.name.map_or(core::ptr::null(), CStrExt::as_char_ptr),\n+ ..bindings::led_classdev::default()\n+ },\n+ num_colors: u32::try_from(num_colors)?,\n+ // CAST: The safeguards in the const block ensure that\n+ // `MultiColorSubLed` has an identical layout to `mc_subled`.\n+ subled_info: subled_info.cast::<bindings::mc_subled>(),\n+ })\n+ };\n+\n+ let mut init_data = bindings::led_init_data {\n+ fwnode: self\n+ .fwnode\n+ .as_ref()\n+ .map_or(core::ptr::null_mut(), |fwnode| fwnode.as_raw()),\n+ default_label: core::ptr::null(),\n+ devicename: self\n+ .devicename\n+ .map_or(core::ptr::null(), CStrExt::as_char_ptr),\n+ devname_mandatory: self.devname_mandatory,\n+ };\n+\n+ // SAFETY:\n+ // - `parent.as_ref().as_raw()` is guaranteed to be a pointer to a valid\n+ // `device`.\n+ // - `ptr` is guaranteed to be a pointer to an initialized `led_classdev_mc`.\n+ to_result(unsafe {\n+ bindings::led_classdev_multicolor_register_ext(\n+ parent.as_ref().as_raw(),\n+ ptr,\n+ if self.name.is_none() {\n+ &raw mut init_data\n+ } else {\n+ core::ptr::null_mut()\n+ },\n+ )\n+ })\n+ .inspect_err(|_err| {\n+ // SAFETY: `subled_info` is guaranteed to be a valid array pointer to\n+ // `mc_subled` with the length and capacity of `num_colors`.\n+ drop(unsafe { KVec::from_raw_parts(subled_info, num_colors, num_colors) });\n+ })?;\n+\n+ core::mem::forget(self.fwnode); // keep the reference count incremented\n+\n+ Ok::<_, Error>(())\n+ }),\n+ }),\n+ )\n+ }\n+}\n+\n+impl<T: LedOps<Mode = MultiColor>> MultiColorDevice<T> {\n+ /// # Safety\n+ /// `led_cdev` must be a valid pointer to a `led_classdev` embedded within a\n+ /// `led::MultiColorDevice`.\n+ unsafe fn from_raw<'a>(led_cdev: *mut bindings::led_classdev) -> &'a Self {\n+ // SAFETY: The function's contract guarantees that `led_cdev` points to a `led_classdev`\n+ // field embedded within a valid `led::MultiColorDevice`. `container_of!` can therefore\n+ // safely calculate the address of the containing struct.\n+ let led_mc_cdev = unsafe { container_of!(led_cdev, bindings::led_classdev_mc, led_cdev) };\n+\n+ // SAFETY: It is guaranteed that `led_mc_cdev` points to a `led_classdev_mc`\n+ // field embedded within a valid `led::MultiColorDevice`. `container_of!` can therefore\n+ // safely calculate the address of the containing struct.\n+ unsafe { &*container_of!(Opaque::cast_from(led_mc_cdev), Self, classdev) }\n+ }\n+\n+ fn parent(&self) -> &device::Device<Bound> {\n+ // SAFETY: `self.classdev.get()` is guaranteed to be a valid pointer to `led_classdev_mc`.\n+ unsafe { device::Device::from_raw((*(*self.classdev.get()).led_cdev.dev).parent) }\n+ }\n+\n+ /// Returns the subleds passed to [`Device::new_multicolor`].\n+ pub fn subleds(&self) -> &[MultiColorSubLed] {\n+ // SAFETY: The existence of `self` guarantees that `self.classdev.get()` is a pointer to a\n+ // valid `led_classdev_mc`.\n+ let raw = unsafe { &*self.classdev.get() };\n+ // SAFETY: `raw.subled_info` is a valid pointer to `mc_subled[num_colors]`.\n+ // CAST: The safeguards in the const block ensure that `MultiColorSubLed` has an identical\n+ // layout to `mc_subled`.\n+ unsafe {\n+ core::slice::from_raw_parts(\n+ raw.subled_info.cast::<MultiColorSubLed>(),\n+ raw.num_colors as usize,\n+ )\n+ }\n+ }\n+}\n+\n+// SAFETY: A `led::MultiColorDevice` can be unregistered from any thread.\n+unsafe impl<T: LedOps<Mode = MultiColor> + Send> Send for MultiColorDevice<T> {}\n+\n+// SAFETY: `led::MultiColorDevice` can be shared among threads because all methods of `led::Device`\n+// are thread safe.\n+unsafe impl<T: LedOps<Mode = MultiColor> + Sync> Sync for MultiColorDevice<T> {}\n+\n+struct Adapter<T: LedOps<Mode = MultiColor>> {\n+ _p: PhantomData<T>,\n+}\n+\n+impl<T: LedOps<Mode = MultiColor>> Adapter<T> {\n+ /// # Safety\n+ /// `led_cdev` must be a valid pointer to a `led_classdev` embedded within a\n+ /// `led::MultiColorDevice`.\n+ /// This function is called on setting the brightness of a led.\n+ unsafe extern \"C\" fn brightness_set_callback(\n+ led_cdev: *mut bindings::led_classdev,\n+ brightness: u32,\n+ ) {\n+ // SAFETY: The function's contract guarantees that `led_cdev` is a valid pointer to a\n+ // `led_classdev` embedded within a `led::MultiColorDevice`.\n+ let classdev = unsafe { MultiColorDevice::<T>::from_raw(led_cdev) };\n+ // SAFETY: `classdev.parent()` is guaranteed to be contained in `T::Bus`.\n+ let parent = unsafe { T::Bus::from_device(classdev.parent()) };\n+\n+ // SAFETY: `classdev.classdev.get()` is guaranteed to be a pointer to a valid\n+ // `led_classdev_mc`.\n+ unsafe { bindings::led_mc_calc_color_components(classdev.classdev.get(), brightness) };\n+\n+ let _ = classdev.ops.brightness_set(parent, classdev, brightness);\n+ }\n+\n+ /// # Safety\n+ /// `led_cdev` must be a valid pointer to a `led_classdev` embedded within a\n+ /// `led::MultiColorDevice`.\n+ /// This function is called on setting the brightness of a led immediately.\n+ unsafe extern \"C\" fn brightness_set_blocking_callback(\n+ led_cdev: *mut bindings::led_classdev,\n+ brightness: u32,\n+ ) -> i32 {\n+ from_result(|| {\n+ // SAFETY: The function's contract guarantees that `led_cdev` is a valid pointer to a\n+ // `led_classdev` embedded within a `led::MultiColorDevice`.\n+ let classdev = unsafe { MultiColorDevice::<T>::from_raw(led_cdev) };\n+ // SAFETY: `classdev.parent()` is guaranteed to be contained in `T::Bus`.\n+ let parent = unsafe { T::Bus::from_device(classdev.parent()) };\n+\n+ // SAFETY: `classdev.classdev.get()` is guaranteed to be a pointer to a valid\n+ // `led_classdev_mc`.\n+ unsafe { bindings::led_mc_calc_color_components(classdev.classdev.get(), brightness) };\n+\n+ classdev.ops.brightness_set(parent, classdev, brightness)?;\n+ Ok(0)\n+ })\n+ }\n+\n+ /// # Safety\n+ /// `led_cdev` must be a valid pointer to a `led_classdev` embedded within a\n+ /// `led::MultiColorDevice`.\n+ /// This function is called on getting the brightness of a led.\n+ unsafe extern \"C\" fn brightness_get_callback(led_cdev: *mut bindings::led_classdev) -> u32 {\n+ // SAFETY: The function's contract guarantees that `led_cdev` is a valid pointer to a\n+ // `led_classdev` embedded within a `led::MultiColorDevice`.\n+ let classdev = unsafe { MultiColorDevice::<T>::from_raw(led_cdev) };\n+ // SAFETY: `classdev.parent()` is guaranteed to be contained in `T::Bus`.\n+ let parent = unsafe { T::Bus::from_device(classdev.parent()) };\n+\n+ classdev.ops.brightness_get(parent, classdev)\n+ }\n+\n+ /// # Safety\n+ /// `led_cdev` must be a valid pointer to a `led_classdev` embedded within a\n+ /// `led::MultiColorDevice`.\n+ /// `delay_on` and `delay_off` must be valid pointers to `usize` and have\n+ /// exclusive access for the period of this function.\n+ /// This function is called on enabling hardware accelerated blinking.\n+ unsafe extern \"C\" fn blink_set_callback(\n+ led_cdev: *mut bindings::led_classdev,\n+ delay_on: *mut usize,\n+ delay_off: *mut usize,\n+ ) -> i32 {\n+ from_result(|| {\n+ // SAFETY: The function's contract guarantees that `led_cdev` is a valid pointer to a\n+ // `led_classdev` embedded within a `led::MultiColorDevice`.\n+ let classdev = unsafe { MultiColorDevice::<T>::from_raw(led_cdev) };\n+ // SAFETY: `classdev.parent()` is guaranteed to be contained in `T::Bus`.\n+ let parent = unsafe { T::Bus::from_device(classdev.parent()) };\n+\n+ classdev.ops.blink_set(\n+ parent,\n+ classdev,\n+ // SAFETY: The function's contract guarantees that `delay_on` points to a `usize`\n+ // and is exclusive for the period of this function.\n+ unsafe { &mut *delay_on },\n+ // SAFETY: The function's contract guarantees that `delay_off` points to a `usize`\n+ // and is exclusive for the period of this function.\n+ unsafe { &mut *delay_off },\n+ )?;\n+ Ok(0)\n+ })\n+ }\n+}\n+\n+#[pinned_drop]\n+impl<T: LedOps<Mode = MultiColor>> PinnedDrop for MultiColorDevice<T> {\n+ fn drop(self: Pin<&mut Self>) {\n+ let raw = self.classdev.get();\n+ // SAFETY: The existence of `self` guarantees that `self.classdev.get()` is a pointer to a\n+ // valid `led_classdev_mc`.\n+ let dev: &device::Device = unsafe { device::Device::from_raw((*raw).led_cdev.dev) };\n+\n+ let _fwnode = dev\n+ .fwnode()\n+ // SAFETY: the reference count of `fwnode` has previously been\n+ // incremented in `led::Device::new`.\n+ .map(|fwnode| unsafe { ARef::from_raw(NonNull::from(fwnode)) });\n+\n+ // SAFETY: The existence of `self` guarantees that `self.classdev` has previously been\n+ // successfully registered with `led_classdev_multicolor_register_ext`.\n+ unsafe { bindings::led_classdev_multicolor_unregister(raw) };\n+\n+ // SAFETY: `raw` is guaranteed to be a valid pointer to `led_classdev_mc`.\n+ let led_cdev = unsafe { &*raw };\n+\n+ // SAFETY: `subled_info` is guaranteed to be a valid array pointer to `mc_subled` with the\n+ // length and capacity of `led_cdev.num_colors`. See `led::MulticolorDevice::new`.\n+ drop(unsafe {\n+ KVec::from_raw_parts(\n+ led_cdev.subled_info,\n+ led_cdev.num_colors as usize,\n+ led_cdev.num_colors as usize,\n+ )\n+ });\n+ }\n+}\n", "prefixes": [ "v13", "3/3" ] }