From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oo1-f51.google.com (mail-oo1-f51.google.com [209.85.161.51]) (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 625C123394B for ; Wed, 20 May 2026 13:34:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779284057; cv=none; b=bfcZUPHYMiGpBxUaKIMoYVRZw6Ss1HbdeatrB7GLPk+PJCbyPYMmS889GFqOO0NuOyElYot+RuPfhMRY5McnxzCMJ1BmZR9PqhIq+6Qq9EVHxeZpLTUORGTK/Fm8CifeEYN8yphMJg1IoTFMacV48nZqCLx65w/VB3j+t8HVRow= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779284057; c=relaxed/simple; bh=Ad8yPDRxvnBjx/qpAX9cAhi7tpYDFaAo7GPdeLtpqzg=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=NlwhC/uJTO7T5LHra6JXg3k/tnQRUWLfP+klJixfbIT8TylRxOBrGPHBljXI4q8I4tVkrSEU5h81AYe9sb0fWNrSAlwBA/kbcJ49FH239bllYhXIF2OLR6PvFFVuA7+UYR9AoxMPmKwI+EAQrMxc5uP67MeCXToWVzRdSIOaMpE= 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=TkJ3nSos; arc=none smtp.client-ip=209.85.161.51 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="TkJ3nSos" Received: by mail-oo1-f51.google.com with SMTP id 006d021491bc7-68470763896so2539961eaf.1 for ; Wed, 20 May 2026 06:34:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779284055; x=1779888855; 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=xnWJZvYMA6qN0Zy8jazbsh97hf1g91RlijsDyV35hD8=; b=TkJ3nSosm6Kv4W+E595AYiW3Swq8Shawm6ZtaWLx2fbcCLEEeAgEjJ37QY+0agXVm8 6BtPn50XvIJN6q7eFyF6J4lTUecUMiGYt2lQ2dhbat1p8S7Ifb02mplQjwwQ9ImICKUS Wi1lpljRkFWcolCOcSZ5ohUO0LKe2LEem+zgLp2hX0Z+NjMwyQOIxSBbKfazqGf8xXxS 9/D4GYih3O7jPj1BzjdPxduCI/XUxp/ubOE7RBLUL2asjvEZYOlb/JK+CycjhfHOhNRT x9zrlNRAqYZk+PvcbQV1JqOP1HbxBJJ71VUCdmP2KiX8+3RsC6YGhdlZXmCiupbjdMbC FbSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779284055; x=1779888855; 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=xnWJZvYMA6qN0Zy8jazbsh97hf1g91RlijsDyV35hD8=; b=hr3kD8+1klmPm0as1MwW1u7p4J8CeKKayAIFnHxlPzA7eAuvLai7kSzapTodig57/5 9l1CjIFpB20pdiJHXCR9Yg9RoDE2/lK81NvhSCXsRZo1EhDQg0ScgK+I3up/g0LFbl6i 9DFrNNcb/47TTNrfkaJwEWuzO+CPH/5tqUjj4mdbQYzKBaKIWKASN56MbmhCjyZR3kJG 4m1MpuksdbK7dFiEb5K3XPM1BcqzHqyeEMC3o6tQ0hFUJKVHe9GtYPTEER3XTBHs+SK7 CN/iZ7AaG8Bux2uSyO1pth+nfhqgPY/wTuwhSsj/vWR7ydCexFPWXEeARG7WsjnWJk0J uCNw== X-Gm-Message-State: AOJu0YxFd6+sOmaMQE+n34b20qQxDneYtdlh8f2gXfzPdCZyOJDWkXct HLbpRxjsa6KzXOzeGa2Rd91XZk008POZyeRW4UOhxI7EOD+yEKmCbLWchnMsOITC X-Gm-Gg: Acq92OGMfgD0lgonKBUvTZrQVX6BaU+XYJM/SebP2vxGk50Gn6h/i13qjJI6qDwNL7x EdeX/RR8AKlTD2t1BK20MUbAmnYY68qnybT7qqAYfnrJHCOZljdlG/UOSMRzCvaaTzSIHRimL8C lAICjBQummMSawnwBLN24s71Tus9dSw40PaDi4/BTGD51wjwn5f7DTq2d6TVHufpGucrjFL85WD LQeoncWZ97oHJXb66UfR83s2aMXbQL2MVAerl7dOwXGud31rgRvNfWm4IqoiaXrNO/jLfO8/bw+ j3yA/+4wHqozNsCgniKO+zXt1rIFlFRGD2ashnbG5C9QeCQujOl7FunC+Il2bW2VsG1cI/Oyx0b sRBLR5tOxZ6KuLNOSLV8p3Y4vfRfmNT1AH0OEo+jWIaUxabAnCb4/l7RUCwyCSUkVktCsM8yvzC wSj2TpUVETs2qYVauXXnDDJd0/hLFeEDns17VlQm8= X-Received: by 2002:a05:6820:f005:b0:694:9fda:6366 with SMTP id 006d021491bc7-69c9429333emr13992078eaf.6.1779284055178; Wed, 20 May 2026 06:34:15 -0700 (PDT) Received: from arch.localdomain ([2409:8a28:a52:5b11:a58f:5208:2fa7:bba5]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-43a956fa075sm10253951fac.10.2026.05.20.06.34.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 May 2026 06:34:14 -0700 (PDT) From: Jun Yan To: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-leds@vger.kernel.org Cc: lee@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, luccafachinetti@gmail.com, pzalewski@thegoodpenguin.co.uk, daniel@zonque.org, Jun Yan Subject: [PATCH v6 4/6] leds: is31fl32xx: Add powerdown pin for hardware shutdown mode Date: Wed, 20 May 2026 21:33:41 +0800 Message-ID: <20260520133343.1423946-5-jerrysteve1101@gmail.com> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20260520133343.1423946-1-jerrysteve1101@gmail.com> References: <20260520133343.1423946-1-jerrysteve1101@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 The IS31FL32XX 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. Deassert/assert the pin accordingly during probe() and remove() to manage hardware shutdown mode. Signed-off-by: Jun Yan --- drivers/leds/leds-is31fl32xx.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/leds/leds-is31fl32xx.c b/drivers/leds/leds-is31fl32xx.c index fe07acbb103a..b5207ad6ae96 100644 --- a/drivers/leds/leds-is31fl32xx.c +++ b/drivers/leds/leds-is31fl32xx.c @@ -10,6 +10,7 @@ */ #include +#include #include #include #include @@ -60,6 +61,7 @@ struct is31fl32xx_led_data { struct is31fl32xx_priv { const struct is31fl32xx_chipdef *cdef; struct i2c_client *client; + struct gpio_desc *powerdown_gpio; unsigned int num_leds; struct is31fl32xx_led_data leds[]; }; @@ -421,6 +423,14 @@ static int is31fl32xx_parse_dt(struct device *dev, const struct is31fl32xx_chipdef *cdef = priv->cdef; int ret = 0; + /* Driving this GPIO line low takes the chip out of shutdown, + * as it is flagged as GPIO_ACTIVE_LOW in provider (such as the device tree). + */ + priv->powerdown_gpio = devm_gpiod_get_optional(dev, "powerdown", GPIOD_OUT_LOW); + if (IS_ERR(priv->powerdown_gpio)) + return dev_err_probe(dev, PTR_ERR(priv->powerdown_gpio), + "Failed to get powerdown gpio\n"); + if ((cdef->output_frequency_setting_reg != IS31FL32XX_REG_NONE) && of_property_read_bool(dev_of_node(dev), "issi,22khz-pwm")) { @@ -609,6 +619,9 @@ static void is31fl32xx_remove(struct i2c_client *client) if (ret) dev_err(&client->dev, "Failed to reset registers on removal (%pe)\n", ERR_PTR(ret)); + + if (priv->powerdown_gpio) + gpiod_set_value(priv->powerdown_gpio, 1); } /* @@ -643,4 +656,4 @@ module_i2c_driver(is31fl32xx_driver); MODULE_AUTHOR("David Rivshin "); MODULE_DESCRIPTION("ISSI IS31FL32xx LED driver"); -MODULE_LICENSE("GPL v2"); +MODULE_LICENSE("GPL v2"); \ No newline at end of file -- 2.54.0