From nobody Mon Feb 9 12:29:05 2026 Received: from srv01.abscue.de (abscue.de [89.58.28.240]) (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 0E5C321859A; Tue, 7 Oct 2025 18:20:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=89.58.28.240 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861251; cv=none; b=jcrqMEBdeFp5Y2zo49JXaOlF+8wam5GXXgBY2NeG5/lwUDmE4LkE9XkkQAu9XREZm6sHFtOmCRcU9iScEz4YYs3iCwF5PxG1UkxC5K94RXGoKxTqkdmKjde9lMb9OaYg+oqyEhlB7uHQ/8YUd/ElDGi34ua8ISucCN6BEV2pUoY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861251; c=relaxed/simple; bh=PDVGCyOxhSCPu4V5/+2P720vq13mPR+UE3R0ZxJcAEs=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=fWqJ4Cw/QkjNq+plsULjH73JWzzEQRB6f9jIakWW/Ypun5+WNkXRiASMiKsSBGtYccOjsYICR080wc3vGYzzDcVKRUpqfXRoB0EFUDRXJKbOFkfSqXV8ccja8Dhgp/fZ1Sd1LikdS16E1rt7fXqJXgFy3d2j3RBf16yBAsyDuDo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de; spf=pass smtp.mailfrom=abscue.de; arc=none smtp.client-ip=89.58.28.240 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=abscue.de Received: from srv01.abscue.de (localhost [127.0.0.1]) by spamfilter.srv.local (Postfix) with ESMTP id 9963C1C3BB4; Tue, 07 Oct 2025 20:15:18 +0200 (CEST) X-Spam-Level: Received: from fluffy-mammal.metal.fwg-cag.de (unknown [IPv6:2001:9e8:cdfe:cd00:889e:d706:979d:b1a]) by srv01.abscue.de (Postfix) with ESMTPSA id E43691C3B04; Tue, 07 Oct 2025 20:15:17 +0200 (CEST) From: =?utf-8?q?Otto_Pfl=C3=BCger?= Date: Tue, 07 Oct 2025 20:14:19 +0200 Subject: [PATCH 1/3] mfd: sprd-sc27xx: Integrate power off and reboot support Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251007-sc27xx-mfd-poweroff-v1-1-89a2f919b731@abscue.de> References: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> In-Reply-To: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> To: Lee Jones , Orson Zhai , Baolin Wang , Chunyan Zhang , Mark Brown , Sebastian Reichel Cc: Rob Herring , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-pm@vger.kernel.org, =?utf-8?q?Otto_Pfl=C3=BCger?= X-Mailer: b4 0.14.2 The SC27xx PMICs allow restarting and powering off the device. Since this functionality is rather simple and not configurable in any way, make it part of the main PMIC driver. Signed-off-by: Otto Pfl=C3=BCger --- drivers/mfd/sprd-sc27xx-spi.c | 152 ++++++++++++++++++++++++++++++++++++++= ++++ 1 file changed, 152 insertions(+) diff --git a/drivers/mfd/sprd-sc27xx-spi.c b/drivers/mfd/sprd-sc27xx-spi.c index d6b4350779e6aecfa19d9fa21b9174447d589e33..28dc82002e3c8eb7748c45f9f1b= dadea4feb8206 100644 --- a/drivers/mfd/sprd-sc27xx-spi.c +++ b/drivers/mfd/sprd-sc27xx-spi.c @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -21,10 +22,48 @@ #define SPRD_SC2730_IRQ_BASE 0x80 #define SPRD_SC2730_IRQ_NUMS 10 #define SPRD_SC2730_CHG_DET 0x1b9c + +#define SPRD_SC2730_PWR_PD_HW 0x1820 +#define SPRD_SC2730_SOFT_RST_HW 0x1824 +#define SPRD_SC2730_SLP_CTRL 0x1a48 +#define SPRD_SC2730_RST_STATUS 0x1bac +#define SPRD_SC2730_SWRST_CTRL0 0x1bf8 + #define SPRD_SC2731_IRQ_BASE 0x140 #define SPRD_SC2731_IRQ_NUMS 16 #define SPRD_SC2731_CHG_DET 0xedc =20 +#define SPRD_SC2731_PWR_PD_HW 0xc2c +#define SPRD_SC2731_SLP_CTRL 0xdf0 +#define SPRD_SC2731_RST_STATUS 0xee8 + +/* PMIC power off and reset definition */ +#define SPRD_SC2730_LDO_XTL_EN BIT(2) +#define SPRD_SC2730_SLP_LDO_PD_EN BIT(0) + +#define SPRD_SC2731_LDO_XTL_EN BIT(3) +#define SPRD_SC2731_SLP_LDO_PD_EN BIT(0) + +#define SPRD_PMIC_PWR_OFF BIT(0) +#define SPRD_PMIC_RESET BIT(0) +#define SPRD_PMIC_SOFT_RST_EN BIT(4) + +#define HWRST_STATUS_SECURITY 0x02 +#define HWRST_STATUS_RECOVERY 0x20 +#define HWRST_STATUS_NORMAL 0x40 +#define HWRST_STATUS_ALARM 0x50 +#define HWRST_STATUS_SLEEP 0x60 +#define HWRST_STATUS_FASTBOOT 0x30 +#define HWRST_STATUS_SPECIAL 0x70 +#define HWRST_STATUS_PANIC 0x80 +#define HWRST_STATUS_CFTREBOOT 0x90 +#define HWRST_STATUS_AUTODLOADER 0xa0 +#define HWRST_STATUS_IQMODE 0xb0 +#define HWRST_STATUS_SPRDISK 0xc0 +#define HWRST_STATUS_FACTORYTEST 0xe0 +#define HWRST_STATUS_WATCHDOG 0xf0 +#define HWRST_STATUS_MASK 0xff + /* PMIC charger detection definition */ #define SPRD_PMIC_CHG_DET_DELAY_US 200000 #define SPRD_PMIC_CHG_DET_TIMEOUT 2000000 @@ -48,6 +87,14 @@ struct sprd_pmic_data { u32 irq_base; u32 num_irqs; u32 charger_det; + + u32 poweroff_reg; + u32 slp_ctrl_reg; + u32 slp_ctrl_mask; + + u32 reset_reg; + u32 rst_sts_reg; + u32 swrst_ctrl_reg; }; =20 /* @@ -59,12 +106,26 @@ static const struct sprd_pmic_data sc2730_data =3D { .irq_base =3D SPRD_SC2730_IRQ_BASE, .num_irqs =3D SPRD_SC2730_IRQ_NUMS, .charger_det =3D SPRD_SC2730_CHG_DET, + + .poweroff_reg =3D SPRD_SC2730_PWR_PD_HW, + .slp_ctrl_reg =3D SPRD_SC2730_SLP_CTRL, + .slp_ctrl_mask =3D SPRD_SC2730_LDO_XTL_EN | SPRD_SC2730_SLP_LDO_PD_EN, + + .reset_reg =3D SPRD_SC2730_SOFT_RST_HW, + .rst_sts_reg =3D SPRD_SC2730_RST_STATUS, + .swrst_ctrl_reg =3D SPRD_SC2730_SWRST_CTRL0, }; =20 static const struct sprd_pmic_data sc2731_data =3D { .irq_base =3D SPRD_SC2731_IRQ_BASE, .num_irqs =3D SPRD_SC2731_IRQ_NUMS, .charger_det =3D SPRD_SC2731_CHG_DET, + + .poweroff_reg =3D SPRD_SC2731_PWR_PD_HW, + .slp_ctrl_reg =3D SPRD_SC2731_SLP_CTRL, + .slp_ctrl_mask =3D SPRD_SC2731_LDO_XTL_EN | SPRD_SC2731_SLP_LDO_PD_EN, + + .rst_sts_reg =3D SPRD_SC2731_RST_STATUS, }; =20 enum usb_charger_type sprd_pmic_detect_charger_type(struct device *dev) @@ -149,6 +210,79 @@ static const struct regmap_config sprd_pmic_config =3D= { .max_register =3D 0xffff, }; =20 +static int sprd_pmic_poweroff(struct sys_off_data *off_data) +{ + struct sprd_pmic *ddata =3D off_data->cb_data; + const struct sprd_pmic_data *pdata =3D ddata->pdata; + + regmap_clear_bits(ddata->regmap, pdata->slp_ctrl_reg, + pdata->slp_ctrl_mask); + + regmap_write(ddata->regmap, pdata->poweroff_reg, SPRD_PMIC_PWR_OFF); + + mdelay(1000); + + pr_emerg("Unable to poweroff system\n"); + + return NOTIFY_DONE; +} + +static int sprd_pmic_restart(struct sys_off_data *off_data) +{ + struct sprd_pmic *ddata =3D off_data->cb_data; + const struct sprd_pmic_data *pdata =3D ddata->pdata; + u32 reboot_mode; + + if (!off_data->cmd) + reboot_mode =3D HWRST_STATUS_NORMAL; + else if (!strcmp(off_data->cmd, "recovery")) + reboot_mode =3D HWRST_STATUS_RECOVERY; + else if (!strcmp(off_data->cmd, "alarm")) + reboot_mode =3D HWRST_STATUS_ALARM; + else if (!strcmp(off_data->cmd, "fastsleep")) + reboot_mode =3D HWRST_STATUS_SLEEP; + else if (!strcmp(off_data->cmd, "bootloader")) + reboot_mode =3D HWRST_STATUS_FASTBOOT; + else if (!strcmp(off_data->cmd, "panic")) + reboot_mode =3D HWRST_STATUS_PANIC; + else if (!strcmp(off_data->cmd, "special")) + reboot_mode =3D HWRST_STATUS_SPECIAL; + else if (!strcmp(off_data->cmd, "cftreboot")) + reboot_mode =3D HWRST_STATUS_CFTREBOOT; + else if (!strcmp(off_data->cmd, "autodloader")) + reboot_mode =3D HWRST_STATUS_AUTODLOADER; + else if (!strcmp(off_data->cmd, "iqmode")) + reboot_mode =3D HWRST_STATUS_IQMODE; + else if (!strcmp(off_data->cmd, "sprdisk")) + reboot_mode =3D HWRST_STATUS_SPRDISK; + else if (!strcmp(off_data->cmd, "tospanic")) + reboot_mode =3D HWRST_STATUS_SECURITY; + else if (!strcmp(off_data->cmd, "factorytest")) + reboot_mode =3D HWRST_STATUS_FACTORYTEST; + else + reboot_mode =3D HWRST_STATUS_NORMAL; + + regmap_update_bits(ddata->regmap, pdata->rst_sts_reg, + HWRST_STATUS_MASK, reboot_mode); + + /* + * On SC2731, this part is skipped because there is no reset register + * and the restart must be performed using the watchdog. + */ + if (pdata->reset_reg) { + regmap_set_bits(ddata->regmap, pdata->swrst_ctrl_reg, + SPRD_PMIC_SOFT_RST_EN); + + regmap_write(ddata->regmap, pdata->reset_reg, SPRD_PMIC_RESET); + + mdelay(1000); + + pr_emerg("Unable to restart system\n"); + } + + return NOTIFY_DONE; +} + static int sprd_pmic_probe(struct spi_device *spi) { struct sprd_pmic *ddata; @@ -204,6 +338,24 @@ static int sprd_pmic_probe(struct spi_device *spi) return ret; } =20 + ret =3D devm_register_sys_off_handler(&spi->dev, SYS_OFF_MODE_RESTART, + 192, sprd_pmic_restart, ddata); + if (ret) { + dev_err(&spi->dev, "Failed to register restart handler: %d\n", + ret); + return ret; + } + + ret =3D devm_register_sys_off_handler(&spi->dev, + SYS_OFF_MODE_POWER_OFF, + SYS_OFF_PRIO_DEFAULT, + sprd_pmic_poweroff, ddata); + if (ret) { + dev_err(&spi->dev, "Failed to register poweroff handler: %d\n", + ret); + return ret; + } + ret =3D devm_of_platform_populate(&spi->dev); if (ret) { dev_err(&spi->dev, "Failed to populate sub-devices %d\n", ret); --=20 2.50.0 From nobody Mon Feb 9 12:29:05 2026 Received: from srv01.abscue.de (abscue.de [89.58.28.240]) (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 0E63321D3E8; Tue, 7 Oct 2025 18:20:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=89.58.28.240 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861256; cv=none; b=GdYXipFesO5Wupd8mBmZQ6UgllIx/qYnnREvE6Z1/Dxem3wbusKElL5wpZ8OkAZTVG4Lt2SaFfJXB7m+pZE5s2Mmg77youxT6OsKdQ9k32WnDinayx1mIlgmG0WsrA1RkTC/FAJFtt1PV/Db/BX50eENrJxlbOjkv8/NFZ3paiU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861256; c=relaxed/simple; bh=ncPpQZbFWw8NMSWBPoo/kgtWe5usLDXcsXQuGtximK4=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=bpil9E2wOYpUvGrlyNVG/b+WdMFmJ3CtkkF2WKDJQiTc9+7nSZ4QfDDyuLECFNLLJjdPTDQ+Pb2j7VgzT2XHuvWTzQMVd33UeOieJJqKxutz0fo7TSSCZ7tJFnBW2vqHT3EzahCaVvY+LNUS2/ajDnBCBlmMsq2/O2NbzRWNWu0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de; spf=pass smtp.mailfrom=abscue.de; arc=none smtp.client-ip=89.58.28.240 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=abscue.de Received: from srv01.abscue.de (localhost [127.0.0.1]) by spamfilter.srv.local (Postfix) with ESMTP id 02DAB1C40CA; Tue, 07 Oct 2025 20:15:18 +0200 (CEST) X-Spam-Level: Received: from fluffy-mammal.metal.fwg-cag.de (unknown [IPv6:2001:9e8:cdfe:cd00:889e:d706:979d:b1a]) by srv01.abscue.de (Postfix) with ESMTPSA id 5DF991C3B06; Tue, 07 Oct 2025 20:15:18 +0200 (CEST) From: =?utf-8?q?Otto_Pfl=C3=BCger?= Date: Tue, 07 Oct 2025 20:14:20 +0200 Subject: [PATCH 2/3] spi: sprd-adi: Remove code for storing the reboot mode Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251007-sc27xx-mfd-poweroff-v1-2-89a2f919b731@abscue.de> References: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> In-Reply-To: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> To: Lee Jones , Orson Zhai , Baolin Wang , Chunyan Zhang , Mark Brown , Sebastian Reichel Cc: Rob Herring , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-pm@vger.kernel.org, =?utf-8?q?Otto_Pfl=C3=BCger?= X-Mailer: b4 0.14.2 This functionality is now provided by the PMIC driver. Ideally, the entire reboot code should be removed since it does not belong in the bus driver, but it is kept for now since there is no driver for the PMIC watchdog on SC2731 yet. Signed-off-by: Otto Pfl=C3=BCger Acked-by: Mark Brown --- drivers/spi/spi-sprd-adi.c | 73 +-----------------------------------------= ---- 1 file changed, 1 insertion(+), 72 deletions(-) diff --git a/drivers/spi/spi-sprd-adi.c b/drivers/spi/spi-sprd-adi.c index 262c11d977ea350c3b8a861064d9f109cc57a45e..47e6fbbf9991dbe77ad22e03c97= 09e6ee3a1b01e 100644 --- a/drivers/spi/spi-sprd-adi.c +++ b/drivers/spi/spi-sprd-adi.c @@ -101,27 +101,10 @@ #define BIT_WDG_EN BIT(2) =20 /* Registers definitions for PMIC */ -#define PMIC_RST_STATUS 0xee8 #define PMIC_MODULE_EN 0xc08 #define PMIC_CLK_EN 0xc18 #define PMIC_WDG_BASE 0x80 =20 -/* Definition of PMIC reset status register */ -#define HWRST_STATUS_SECURITY 0x02 -#define HWRST_STATUS_RECOVERY 0x20 -#define HWRST_STATUS_NORMAL 0x40 -#define HWRST_STATUS_ALARM 0x50 -#define HWRST_STATUS_SLEEP 0x60 -#define HWRST_STATUS_FASTBOOT 0x30 -#define HWRST_STATUS_SPECIAL 0x70 -#define HWRST_STATUS_PANIC 0x80 -#define HWRST_STATUS_CFTREBOOT 0x90 -#define HWRST_STATUS_AUTODLOADER 0xa0 -#define HWRST_STATUS_IQMODE 0xb0 -#define HWRST_STATUS_SPRDISK 0xc0 -#define HWRST_STATUS_FACTORYTEST 0xe0 -#define HWRST_STATUS_WATCHDOG 0xf0 - /* Use default timeout 50 ms that converts to watchdog values */ #define WDG_LOAD_VAL ((50 * 32768) / 1000) #define WDG_LOAD_MASK GENMASK(15, 0) @@ -139,7 +122,6 @@ struct sprd_adi_data { u32 slave_addr_size; int (*read_check)(u32 val, u32 reg); int (*restart)(struct sys_off_data *data); - void (*wdg_rst)(void *p); }; =20 struct sprd_adi { @@ -355,58 +337,10 @@ static int sprd_adi_transfer_one(struct spi_controlle= r *ctlr, return ret; } =20 -static void sprd_adi_set_wdt_rst_mode(void *p) -{ -#if IS_ENABLED(CONFIG_SPRD_WATCHDOG) - u32 val; - struct sprd_adi *sadi =3D (struct sprd_adi *)p; - - /* Init watchdog reset mode */ - sprd_adi_read(sadi, PMIC_RST_STATUS, &val); - val |=3D HWRST_STATUS_WATCHDOG; - sprd_adi_write(sadi, PMIC_RST_STATUS, val); -#endif -} - static int sprd_adi_restart(struct sprd_adi *sadi, unsigned long mode, const char *cmd, struct sprd_adi_wdg *wdg) { - u32 val, reboot_mode =3D 0; - - if (!cmd) - reboot_mode =3D HWRST_STATUS_NORMAL; - else if (!strncmp(cmd, "recovery", 8)) - reboot_mode =3D HWRST_STATUS_RECOVERY; - else if (!strncmp(cmd, "alarm", 5)) - reboot_mode =3D HWRST_STATUS_ALARM; - else if (!strncmp(cmd, "fastsleep", 9)) - reboot_mode =3D HWRST_STATUS_SLEEP; - else if (!strncmp(cmd, "bootloader", 10)) - reboot_mode =3D HWRST_STATUS_FASTBOOT; - else if (!strncmp(cmd, "panic", 5)) - reboot_mode =3D HWRST_STATUS_PANIC; - else if (!strncmp(cmd, "special", 7)) - reboot_mode =3D HWRST_STATUS_SPECIAL; - else if (!strncmp(cmd, "cftreboot", 9)) - reboot_mode =3D HWRST_STATUS_CFTREBOOT; - else if (!strncmp(cmd, "autodloader", 11)) - reboot_mode =3D HWRST_STATUS_AUTODLOADER; - else if (!strncmp(cmd, "iqmode", 6)) - reboot_mode =3D HWRST_STATUS_IQMODE; - else if (!strncmp(cmd, "sprdisk", 7)) - reboot_mode =3D HWRST_STATUS_SPRDISK; - else if (!strncmp(cmd, "tospanic", 8)) - reboot_mode =3D HWRST_STATUS_SECURITY; - else if (!strncmp(cmd, "factorytest", 11)) - reboot_mode =3D HWRST_STATUS_FACTORYTEST; - else - reboot_mode =3D HWRST_STATUS_NORMAL; - - /* Record the reboot mode */ - sprd_adi_read(sadi, wdg->rst_sts, &val); - val &=3D ~HWRST_STATUS_WATCHDOG; - val |=3D reboot_mode; - sprd_adi_write(sadi, wdg->rst_sts, val); + u32 val; =20 /* Enable the interface clock of the watchdog */ sprd_adi_read(sadi, wdg->wdg_en, &val); @@ -448,7 +382,6 @@ static int sprd_adi_restart_sc9860(struct sys_off_data = *data) { struct sprd_adi_wdg wdg =3D { .base =3D PMIC_WDG_BASE, - .rst_sts =3D PMIC_RST_STATUS, .wdg_en =3D PMIC_MODULE_EN, .wdg_clk =3D PMIC_CLK_EN, }; @@ -568,9 +501,6 @@ static int sprd_adi_probe(struct platform_device *pdev) =20 sprd_adi_hw_init(sadi); =20 - if (sadi->data->wdg_rst) - sadi->data->wdg_rst(sadi); - ctlr->dev.of_node =3D pdev->dev.of_node; ctlr->bus_num =3D pdev->id; ctlr->num_chipselect =3D num_chipselect; @@ -606,7 +536,6 @@ static struct sprd_adi_data sc9860_data =3D { .slave_addr_size =3D ADI_10BIT_SLAVE_ADDR_SIZE, .read_check =3D sprd_adi_read_check_r2, .restart =3D sprd_adi_restart_sc9860, - .wdg_rst =3D sprd_adi_set_wdt_rst_mode, }; =20 static struct sprd_adi_data sc9863_data =3D { --=20 2.50.0 From nobody Mon Feb 9 12:29:05 2026 Received: from srv01.abscue.de (abscue.de [89.58.28.240]) (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 0E6912206A7; Tue, 7 Oct 2025 18:20:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=89.58.28.240 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861251; cv=none; b=CC2bYpPMvE94vPg4zOycJolqnBm0eQsiuwXDXkpj8lwPryWvrroHAqSkLiqRM9qh2loeweWG2TsnzYEbcYROwA5ZbwD0VharXrWOh5LiQBwT1haY1b6uewts3si51GHyX27FZvhUgyz6zKDZut7NovP2abSsEViK8BqZysspL5g= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1759861251; c=relaxed/simple; bh=8/b7jwWBDbOGO7fjgsRdi05ElyTw54npNSsGTIgmHMk=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=FVuJc+RUDCZMDVImeF8T9t59EcJ8Li9DwIQJ9D5GL0kcFNj57UG4MQAZoAlbQ5GVEk7Y6IadAx50NIKrL4GR0rOzgDb6WTioyp4QeCEe62IXqO3qkl+RF0raxguYkVnoYj95ftoUunbXTFyhpG1wIoS9R4qKThlrJhXSq2bI4pk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de; spf=pass smtp.mailfrom=abscue.de; arc=none smtp.client-ip=89.58.28.240 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=abscue.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=abscue.de Received: from srv01.abscue.de (localhost [127.0.0.1]) by spamfilter.srv.local (Postfix) with ESMTP id DE8A01C3AC5; Tue, 07 Oct 2025 20:15:21 +0200 (CEST) X-Spam-Level: Received: from fluffy-mammal.metal.fwg-cag.de (unknown [IPv6:2001:9e8:cdfe:cd00:889e:d706:979d:b1a]) by srv01.abscue.de (Postfix) with ESMTPSA id CA4E61C40C9; Tue, 07 Oct 2025 20:15:18 +0200 (CEST) From: =?utf-8?q?Otto_Pfl=C3=BCger?= Date: Tue, 07 Oct 2025 20:14:21 +0200 Subject: [PATCH 3/3] power: reset: sc27xx: Drop unused driver Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20251007-sc27xx-mfd-poweroff-v1-3-89a2f919b731@abscue.de> References: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> In-Reply-To: <20251007-sc27xx-mfd-poweroff-v1-0-89a2f919b731@abscue.de> To: Lee Jones , Orson Zhai , Baolin Wang , Chunyan Zhang , Mark Brown , Sebastian Reichel Cc: Rob Herring , linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, linux-pm@vger.kernel.org, =?utf-8?q?Otto_Pfl=C3=BCger?= X-Mailer: b4 0.14.2 This driver was never actually probed because it was missing an OF match table and was not integrated into the MFD driver. Remove it now that the power off functionality is handled directly in the MFD driver. Signed-off-by: Otto Pfl=C3=BCger Acked-by: Sebastian Reichel --- drivers/power/reset/Kconfig | 9 ---- drivers/power/reset/Makefile | 1 - drivers/power/reset/sc27xx-poweroff.c | 79 -------------------------------= ---- 3 files changed, 89 deletions(-) diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig index 8248895ca90389c1db33c7b09e5f5925a9034cee..c4a28f6f04f6be13d20ce2b0842= 7fd1679b1df5a 100644 --- a/drivers/power/reset/Kconfig +++ b/drivers/power/reset/Kconfig @@ -320,15 +320,6 @@ config SYSCON_REBOOT_MODE register, then the bootloader can read it to take different action according to the mode. =20 -config POWER_RESET_SC27XX - tristate "Spreadtrum SC27xx PMIC power-off driver" - depends on MFD_SC27XX_PMIC || COMPILE_TEST - help - This driver supports powering off a system through - Spreadtrum SC27xx series PMICs. The SC27xx series - PMICs includes the SC2720, SC2721, SC2723, SC2730 - and SC2731 chips. - config NVMEM_REBOOT_MODE tristate "Generic NVMEM reboot mode driver" depends on OF diff --git a/drivers/power/reset/Makefile b/drivers/power/reset/Makefile index 51da87e05ce76bc608d961485063555c3ba5d96c..cabaa0de2de68794bea5f952385= 5bb9ef0083ef0 100644 --- a/drivers/power/reset/Makefile +++ b/drivers/power/reset/Makefile @@ -37,6 +37,5 @@ obj-$(CONFIG_POWER_RESET_SYSCON_POWEROFF) +=3D syscon-pow= eroff.o obj-$(CONFIG_POWER_RESET_RMOBILE) +=3D rmobile-reset.o obj-$(CONFIG_REBOOT_MODE) +=3D reboot-mode.o obj-$(CONFIG_SYSCON_REBOOT_MODE) +=3D syscon-reboot-mode.o -obj-$(CONFIG_POWER_RESET_SC27XX) +=3D sc27xx-poweroff.o obj-$(CONFIG_NVMEM_REBOOT_MODE) +=3D nvmem-reboot-mode.o obj-$(CONFIG_POWER_MLXBF) +=3D pwr-mlxbf.o diff --git a/drivers/power/reset/sc27xx-poweroff.c b/drivers/power/reset/sc= 27xx-poweroff.c deleted file mode 100644 index 90287c31992c4889f9241e82a21a1949ecca7702..000000000000000000000000000= 0000000000000 --- a/drivers/power/reset/sc27xx-poweroff.c +++ /dev/null @@ -1,79 +0,0 @@ -// SPDX-License-Identifier: GPL-2.0 -/* - * Copyright (C) 2018 Spreadtrum Communications Inc. - * Copyright (C) 2018 Linaro Ltd. - */ - -#include -#include -#include -#include -#include -#include -#include - -#define SC27XX_PWR_PD_HW 0xc2c -#define SC27XX_PWR_OFF_EN BIT(0) -#define SC27XX_SLP_CTRL 0xdf0 -#define SC27XX_LDO_XTL_EN BIT(3) - -static struct regmap *regmap; - -/* - * On Spreadtrum platform, we need power off system through external SC27xx - * series PMICs, and it is one similar SPI bus mapped by regmap to access = PMIC, - * which is not fast io access. - * - * So before stopping other cores, we need release other cores' resource by - * taking cpus down to avoid racing regmap or spi mutex lock when poweroff - * system through PMIC. - */ -static void sc27xx_poweroff_shutdown(void) -{ -#ifdef CONFIG_HOTPLUG_CPU - int cpu; - - for_each_online_cpu(cpu) { - if (cpu !=3D smp_processor_id()) - remove_cpu(cpu); - } -#endif -} - -static struct syscore_ops poweroff_syscore_ops =3D { - .shutdown =3D sc27xx_poweroff_shutdown, -}; - -static void sc27xx_poweroff_do_poweroff(void) -{ - /* Disable the external subsys connection's power firstly */ - regmap_write(regmap, SC27XX_SLP_CTRL, SC27XX_LDO_XTL_EN); - - regmap_write(regmap, SC27XX_PWR_PD_HW, SC27XX_PWR_OFF_EN); -} - -static int sc27xx_poweroff_probe(struct platform_device *pdev) -{ - if (regmap) - return -EINVAL; - - regmap =3D dev_get_regmap(pdev->dev.parent, NULL); - if (!regmap) - return -ENODEV; - - pm_power_off =3D sc27xx_poweroff_do_poweroff; - register_syscore_ops(&poweroff_syscore_ops); - return 0; -} - -static struct platform_driver sc27xx_poweroff_driver =3D { - .probe =3D sc27xx_poweroff_probe, - .driver =3D { - .name =3D "sc27xx-poweroff", - }, -}; -module_platform_driver(sc27xx_poweroff_driver); - -MODULE_DESCRIPTION("Power off driver for SC27XX PMIC Device"); -MODULE_AUTHOR("Baolin Wang "); -MODULE_LICENSE("GPL v2"); --=20 2.50.0