From: Jun Yan <jerrysteve1101@gmail•com>
To: linusw@kernel•org, dmitry.baryshkov@oss•qualcomm.com,
Lee Jones <lee@kernel•org>, Pavel Machek <pavel@kernel•org>,
Rob Herring <robh@kernel•org>,
Krzysztof Kozlowski <krzk+dt@kernel•org>,
Conor Dooley <conor+dt@kernel•org>,
Bjorn Andersson <andersson@kernel•org>,
Konrad Dybcio <konradybcio@kernel•org>,
Andrew Lunn <andrew@lunn•ch>,
Gregory Clement <gregory.clement@bootlin•com>,
Sebastian Hesselbarth <sebastian.hesselbarth@gmail•com>,
Bartosz Golaszewski <brgl@kernel•org>,
Vincent Knecht <vincent.knecht@mailoo•org>,
Grant Feng <von81@163•com>,
Andre Przywara <andre.przywara@arm•com>,
Claudiu Beznea <claudiu.beznea@tuxon•dev>,
Wei Xu <xuwei5@hisilicon•com>,
Sudeep Holla <sudeep.holla@kernel•org>,
Tony Lindgren <tony@atomide•com>,
Robert Marko <robert.marko@sartura•hr>
Cc: Jun Yan <jerrysteve1101@gmail•com>, Pavel Machek <pavel@ucw•cz>,
Krzysztof Kozlowski <krzk@kernel•org>,
Patrice Chotard <patrice.chotard@foss•st.com>,
Kunihiko Hayashi <hayashi.kunihiko@socionext•com>,
Michal Simek <michal.simek@amd•com>,
linux-leds@vger•kernel.org, devicetree@vger•kernel.org,
linux-kernel@vger•kernel.org, linux-arm-msm@vger•kernel.org,
linux-arm-kernel@lists•infradead.org, linux-gpio@vger•kernel.org
Subject: [PATCH v2 1/6] dt-bindings: leds: issi,is31fl319x: Update description for the shutdown-gpios property
Date: Mon, 25 May 2026 22:46:07 +0800 [thread overview]
Message-ID: <20260525144629.498630-2-jerrysteve1101@gmail.com> (raw)
In-Reply-To: <20260525144629.498630-1-jerrysteve1101@gmail.com>
The IS31FL319X series features an SDB shutdown pin.
Driving it low (active low) places the chip into hardware shutdown
mode for power saving, while all register contents are preserved
and registers are not reset.
Update description for the shutdown down (SDB) pin and fix the example
device tree binding.
Fixes: dbc801b472c1 ("dt-bindings: leds: Convert is31fl319x to dtschema")
Signed-off-by: Jun Yan <jerrysteve1101@gmail•com>
Reviewed-by: Rob Herring (Arm) <robh@kernel•org>
---
.../devicetree/bindings/leds/issi,is31fl319x.yaml | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml b/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
index 906735acfbaf..ebf2bed51e7f 100644
--- a/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
+++ b/Documentation/devicetree/bindings/leds/issi,is31fl319x.yaml
@@ -39,7 +39,12 @@ properties:
shutdown-gpios:
maxItems: 1
- description: GPIO attached to the SDB pin.
+ description: |
+ GPIO attached to the chip's SDB pin.
+ Driving this GPIO low places the chip into hardware shutdown mode
+ for power saving. All register contents are preserved and registers
+ are not reset during shutdown. The chip exits hardware shutdown mode
+ when the SDB pin is pulled high.
audio-gain-db:
default: 0
@@ -174,7 +179,7 @@ examples:
#address-cells = <1>;
#size-cells = <0>;
- shutdown-gpios = <&gpio0 11 GPIO_ACTIVE_HIGH>;
+ shutdown-gpios = <&gpio0 11 GPIO_ACTIVE_LOW>;
led@1 {
reg = <1>;
--
2.54.0
next prev parent reply other threads:[~2026-05-25 14:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-25 14:46 [PATCH v2 0/6] leds: is31fl319x: Fix shutdown GPIO and update DT bindings Jun Yan
2026-05-25 14:46 ` Jun Yan [this message]
2026-05-25 14:46 ` [PATCH v2 2/6] leds: is31fl319x: Fix shutdown GPIO initial state and remove redundant startup pulse Jun Yan
2026-06-01 8:20 ` Krzysztof Kozlowski
2026-05-25 14:46 ` [PATCH v2 3/6] gpiolib: of: add quirk for IS31FL319X shutdown line Jun Yan
2026-05-26 8:40 ` Bartosz Golaszewski
2026-05-29 20:10 ` Linus Walleij
2026-05-25 14:46 ` [PATCH v2 4/6] arm64: dts: qcom: msm8916-alcatel-idol347: Fix sn3190 shutdown GPIO polarity Jun Yan
2026-05-25 14:46 ` [PATCH v2 5/6] ARM: dts: qcom: msm8974-oneplus-bacon: Fix sn3193 " Jun Yan
2026-05-25 14:46 ` [PATCH v2 6/6] arm64: dts: marvell: armada-7040-mochabin: Fix is31fl3199 " Jun Yan
2026-06-01 6:58 ` Gregory CLEMENT
2026-06-01 8:21 ` Krzysztof Kozlowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260525144629.498630-2-jerrysteve1101@gmail.com \
--to=jerrysteve1101@gmail$(echo .)com \
--cc=andersson@kernel$(echo .)org \
--cc=andre.przywara@arm$(echo .)com \
--cc=andrew@lunn$(echo .)ch \
--cc=brgl@kernel$(echo .)org \
--cc=claudiu.beznea@tuxon$(echo .)dev \
--cc=conor+dt@kernel$(echo .)org \
--cc=devicetree@vger$(echo .)kernel.org \
--cc=dmitry.baryshkov@oss$(echo .)qualcomm.com \
--cc=gregory.clement@bootlin$(echo .)com \
--cc=hayashi.kunihiko@socionext$(echo .)com \
--cc=konradybcio@kernel$(echo .)org \
--cc=krzk+dt@kernel$(echo .)org \
--cc=krzk@kernel$(echo .)org \
--cc=lee@kernel$(echo .)org \
--cc=linusw@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
--cc=linux-arm-msm@vger$(echo .)kernel.org \
--cc=linux-gpio@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-leds@vger$(echo .)kernel.org \
--cc=michal.simek@amd$(echo .)com \
--cc=patrice.chotard@foss$(echo .)st.com \
--cc=pavel@kernel$(echo .)org \
--cc=pavel@ucw$(echo .)cz \
--cc=robert.marko@sartura$(echo .)hr \
--cc=robh@kernel$(echo .)org \
--cc=sebastian.hesselbarth@gmail$(echo .)com \
--cc=sudeep.holla@kernel$(echo .)org \
--cc=tony@atomide$(echo .)com \
--cc=vincent.knecht@mailoo$(echo .)org \
--cc=von81@163$(echo .)com \
--cc=xuwei5@hisilicon$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox