From nobody Sat Feb 7 06:35:34 2026 Received: from sender4-pp-f112.zoho.com (sender4-pp-f112.zoho.com [136.143.188.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 1DEB22E5418; Sat, 24 Jan 2026 12:02:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=pass smtp.client-ip=136.143.188.112 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769256144; cv=pass; b=GObwo77OHpfhoQjTnvPaIKYAhTTvgqZQZY89qJ8ls8gYj/jDkQkiE3Iq913TjDknOc/bRGVJgKP8mBV4QCtqq3USX2sBK7gKNMpdfG94rAs6kAgGuTyr+CLRypAnh+t+0FqVO0wg9lrIvHPAZ3gdDiU5pXZ/hsRQ4EP22KAYEX0= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1769256144; c=relaxed/simple; bh=/Z/DE6ZocI0XfXPVllp5nVbpyPqQzUJMrQT8bH8A+U8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=Zla1Iomv3v4XkYYEmRHQXtcCN5fH1jKjeHR7RQp1DpulMcdhovO7uPubwTUFKtYb3xZ/D1/EsY7A2gFQfa6OAdkD1ylQ/nS55zv0MZsPvB2LopsDOiMZCUitAXA+yPsNRiUykcdvzhWgsA96sGmZqCxv+GsGPKDyMIxyoRoHuxk= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b=PaLBEFlJ; arc=pass smtp.client-ip=136.143.188.112 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=collabora.com header.i=nicolas.frattaroli@collabora.com header.b="PaLBEFlJ" ARC-Seal: i=1; a=rsa-sha256; t=1769256117; cv=none; d=zohomail.com; s=zohoarc; b=jMCvXFkSrNIDzVIQlam3/MNoVaeG2J89k+lLga8DcrOwngDc7ypJEy9r6RW7FXQamk84Q8UCqZwlmhQWeuCvn4dtrkMz0dqgjbOKDuVANvWnoRpz2JnB7aFaltpHIbZAMIRN9lXxcpFWMja3ZeZSz3D+tcOws30vuMJXq8r4uCk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1769256117; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:MIME-Version:Message-ID:References:Subject:Subject:To:To:Message-Id:Reply-To; bh=mTFzbSzc3bOGqqfS2NiRWM2+n01ZbPaNCoS8FITbGLg=; b=LPOKssy20hava3s0l6BWsM8gwGcjOwnLhd7n8VEpX8fTq/BN4pKGWhPmTrIkNdxfZPUmxdSe9Sb0LwF/nls4QjDkQeKM+sUQyZjDbibom5RgmkYrdgU/G0oR84Q822fZoFIl+wAWFvTx1LJpoTjzGlNbx7gRtMdmTNcG65BwOO0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass header.i=collabora.com; spf=pass smtp.mailfrom=nicolas.frattaroli@collabora.com; dmarc=pass header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1769256116; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:Date:Date:Subject:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding:Message-Id:Message-Id:References:In-Reply-To:To:To:Cc:Cc:Reply-To; bh=mTFzbSzc3bOGqqfS2NiRWM2+n01ZbPaNCoS8FITbGLg=; b=PaLBEFlJlZg8rMfUrBtQF7PxaA9oQYebfk0tN2kTNQjsskj9ERh4SoltDhDGIxZ0 kMdrt1dTBlF450D5lHnj82lZxZcachJMIVWvZSwLIknRK8jIsjAezpFsoJ8Y9WpOISA qQlljZ//G3EAy2hLZ0FSymPphRbg26C3sc+MzQNA= Received: by mx.zohomail.com with SMTPS id 1769256114626347.97515531123327; Sat, 24 Jan 2026 04:01:54 -0800 (PST) From: Nicolas Frattaroli Date: Sat, 24 Jan 2026 13:00:53 +0100 Subject: [PATCH v6 07/24] scsi: ufs: mediatek: Rework 0.9V regulator 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: <20260124-mt8196-ufs-v6-7-e7c005b60028@collabora.com> References: <20260124-mt8196-ufs-v6-0-e7c005b60028@collabora.com> In-Reply-To: <20260124-mt8196-ufs-v6-0-e7c005b60028@collabora.com> To: Alim Akhtar , Avri Altman , Bart Van Assche , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Matthias Brugger , AngeloGioacchino Del Regno , Chunfeng Yun , Vinod Koul , Kishon Vijay Abraham I , Peter Wang , Stanley Jhu , "James E.J. Bottomley" , "Martin K. Petersen" , Philipp Zabel , Liam Girdwood , Mark Brown , Chaotian Jing , Neil Armstrong Cc: Louis-Alexis Eyraud , kernel@collabora.com, linux-scsi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-phy@lists.infradead.org, Nicolas Frattaroli X-Mailer: b4 0.14.3 The mediatek UFS host driver does some pretty bad stuff with regards to the 0.9V regulator. Instead of just checking for the presence of the regulator, it adds a cap if it's there, and then checks for the cap. It also sleeps to stabilise the supply after enabling the regulator, which is something that should be done by the regulator framework with the appropriate delay properties in the DTS instead of random sleeps in the driver code. Rework this code and rename it to the avdd09 name I've chosen in the binding for this supply name, instead of the downstream "va09" name that isn't used by the datasheets for any of these chips. Reviewed-by: Peter Wang Reviewed-by: AngeloGioacchino Del Regno Signed-off-by: Nicolas Frattaroli --- drivers/ufs/host/ufs-mediatek.c | 153 ++++++++++++++++++++++++++----------= ---- drivers/ufs/host/ufs-mediatek.h | 3 +- 2 files changed, 101 insertions(+), 55 deletions(-) diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-mediate= k.c index 5cf5f4c94b8f..7fcf4ceeb56e 100644 --- a/drivers/ufs/host/ufs-mediatek.c +++ b/drivers/ufs/host/ufs-mediatek.c @@ -38,6 +38,10 @@ static void _ufs_mtk_clk_scale(struct ufs_hba *hba, bool= scale_up); #define MAX_SUPP_MAC 64 #define MCQ_QUEUE_OFFSET(c) ((((c) >> 16) & 0xFF) * 0x200) =20 +struct ufs_mtk_soc_data { + bool has_avdd09; +}; + static const struct ufs_dev_quirk ufs_mtk_dev_fixups[] =3D { { .wmanufacturerid =3D UFS_ANY_VENDOR, .model =3D UFS_ANY_MODEL, @@ -48,13 +52,6 @@ static const struct ufs_dev_quirk ufs_mtk_dev_fixups[] = =3D { {} }; =20 -static const struct of_device_id ufs_mtk_of_match[] =3D { - { .compatible =3D "mediatek,mt8183-ufshci" }, - { .compatible =3D "mediatek,mt8195-ufshci" }, - {}, -}; -MODULE_DEVICE_TABLE(of, ufs_mtk_of_match); - /* * Details of UIC Errors */ @@ -106,13 +103,6 @@ static bool ufs_mtk_is_boost_crypt_enabled(struct ufs_= hba *hba) return host->caps & UFS_MTK_CAP_BOOST_CRYPT_ENGINE; } =20 -static bool ufs_mtk_is_va09_supported(struct ufs_hba *hba) -{ - struct ufs_mtk_host *host =3D ufshcd_get_variant(hba); - - return host->caps & UFS_MTK_CAP_VA09_PWR_CTRL; -} - static bool ufs_mtk_is_broken_vcc(struct ufs_hba *hba) { struct ufs_mtk_host *host =3D ufshcd_get_variant(hba); @@ -506,44 +496,70 @@ static int ufs_mtk_wait_link_state(struct ufs_hba *hb= a, u32 state, return -ETIMEDOUT; } =20 +static int ufs_mtk_09v_off(struct ufs_mtk_host *host) +{ + struct arm_smccc_res res; + int ret; + + if (!host->reg_avdd09) + return 0; + + ufs_mtk_va09_pwr_ctrl(res, 0); + ret =3D regulator_disable(host->reg_avdd09); + if (ret) { + dev_err(host->hba->dev, "Failed to disable avdd09-supply: %pe\n", + ERR_PTR(ret)); + ufs_mtk_va09_pwr_ctrl(res, 1); + return ret; + } + + return 0; +} + +static int ufs_mtk_09v_on(struct ufs_mtk_host *host) +{ + struct arm_smccc_res res; + int ret; + + if (!host->reg_avdd09) + return 0; + + ret =3D regulator_enable(host->reg_avdd09); + if (ret) { + dev_err(host->hba->dev, "Failed to enable avdd09-supply: %pe\n", + ERR_PTR(ret)); + return ret; + } + + ufs_mtk_va09_pwr_ctrl(res, 1); + + return 0; +} + static int ufs_mtk_mphy_power_on(struct ufs_hba *hba, bool on) { struct ufs_mtk_host *host =3D ufshcd_get_variant(hba); struct phy *mphy =3D host->mphy; - struct arm_smccc_res res; - int ret =3D 0; + int ret; =20 - if (!mphy || !(on ^ host->mphy_powered_on)) + if (!mphy || on =3D=3D host->mphy_powered_on) return 0; =20 if (on) { - if (ufs_mtk_is_va09_supported(hba)) { - ret =3D regulator_enable(host->reg_va09); - if (ret < 0) - goto out; - /* wait 200 us to stablize VA09 */ - usleep_range(200, 210); - ufs_mtk_va09_pwr_ctrl(res, 1); - } + ret =3D ufs_mtk_09v_on(host); + if (ret) + return ret; phy_power_on(mphy); } else { phy_power_off(mphy); - if (ufs_mtk_is_va09_supported(hba)) { - ufs_mtk_va09_pwr_ctrl(res, 0); - ret =3D regulator_disable(host->reg_va09); - } - } -out: - if (ret) { - dev_info(hba->dev, - "failed to %s va09: %d\n", - on ? "enable" : "disable", - ret); - } else { - host->mphy_powered_on =3D on; + ret =3D ufs_mtk_09v_off(host); + if (ret) + return ret; } =20 - return ret; + host->mphy_powered_on =3D on; + + return 0; } =20 static int ufs_mtk_get_host_clk(struct device *dev, const char *name, @@ -678,17 +694,6 @@ static void ufs_mtk_init_boost_crypt(struct ufs_hba *h= ba) return; } =20 -static void ufs_mtk_init_va09_pwr_ctrl(struct ufs_hba *hba) -{ - struct ufs_mtk_host *host =3D ufshcd_get_variant(hba); - - host->reg_va09 =3D regulator_get(hba->dev, "va09"); - if (IS_ERR(host->reg_va09)) - dev_info(hba->dev, "failed to get va09"); - else - host->caps |=3D UFS_MTK_CAP_VA09_PWR_CTRL; -} - static void ufs_mtk_init_host_caps(struct ufs_hba *hba) { struct ufs_mtk_host *host =3D ufshcd_get_variant(hba); @@ -697,9 +702,6 @@ static void ufs_mtk_init_host_caps(struct ufs_hba *hba) if (of_property_read_bool(np, "mediatek,ufs-boost-crypt")) ufs_mtk_init_boost_crypt(hba); =20 - if (of_property_read_bool(np, "mediatek,ufs-support-va09")) - ufs_mtk_init_va09_pwr_ctrl(hba); - if (of_property_read_bool(np, "mediatek,ufs-disable-ah8")) host->caps |=3D UFS_MTK_CAP_DISABLE_AH8; =20 @@ -1205,6 +1207,35 @@ static void ufs_mtk_init_mcq_irq(struct ufs_hba *hba) host->mcq_nr_intr =3D 0; } =20 +/** + * ufs_mtk_get_supplies - acquire variant-specific supplies + * @host: pointer to driver's private &struct ufs_mtk_host instance + * + * Returns 0 on success, negative errno on error. + */ +static int ufs_mtk_get_supplies(struct ufs_mtk_host *host) +{ + struct device *dev =3D host->hba->dev; + const struct ufs_mtk_soc_data *data =3D of_device_get_match_data(dev); + + if (!data || !data->has_avdd09) + return 0; + + host->reg_avdd09 =3D devm_regulator_get_optional(dev, "avdd09"); + if (IS_ERR(host->reg_avdd09)) { + if (PTR_ERR(host->reg_avdd09) =3D=3D -ENODEV) { + host->reg_avdd09 =3D NULL; + return 0; + } + + dev_err(dev, "Failed to get avdd09 regulator: %pe\n", + host->reg_avdd09); + return PTR_ERR(host->reg_avdd09); + } + + return 0; +} + /** * ufs_mtk_init - find other essential mmio bases * @hba: host controller instance @@ -1288,6 +1319,10 @@ static int ufs_mtk_init(struct ufs_hba *hba) =20 ufs_mtk_init_clocks(hba); =20 + err =3D ufs_mtk_get_supplies(host); + if (err) + goto out_variant_clear; + /* * ufshcd_vops_init() is invoked after * ufshcd_setup_clock(true) in ufshcd_hba_init() thus @@ -2336,6 +2371,18 @@ static const struct ufs_hba_variant_ops ufs_hba_mtk_= vops =3D { .config_scsi_dev =3D ufs_mtk_config_scsi_dev, }; =20 +static const struct ufs_mtk_soc_data mt8183_data =3D { + .has_avdd09 =3D true, +}; + +static const struct of_device_id ufs_mtk_of_match[] =3D { + { .compatible =3D "mediatek,mt8183-ufshci", .data =3D &mt8183_data }, + { .compatible =3D "mediatek,mt8192-ufshci" }, + { .compatible =3D "mediatek,mt8195-ufshci" }, + {}, +}; +MODULE_DEVICE_TABLE(of, ufs_mtk_of_match); + /** * ufs_mtk_probe - probe routine of the driver * @pdev: pointer to Platform device handle diff --git a/drivers/ufs/host/ufs-mediatek.h b/drivers/ufs/host/ufs-mediate= k.h index 4fce29d131d1..24c8941f6b86 100644 --- a/drivers/ufs/host/ufs-mediatek.h +++ b/drivers/ufs/host/ufs-mediatek.h @@ -125,7 +125,6 @@ enum { */ enum ufs_mtk_host_caps { UFS_MTK_CAP_BOOST_CRYPT_ENGINE =3D 1 << 0, - UFS_MTK_CAP_VA09_PWR_CTRL =3D 1 << 1, UFS_MTK_CAP_DISABLE_AH8 =3D 1 << 2, UFS_MTK_CAP_BROKEN_VCC =3D 1 << 3, =20 @@ -176,7 +175,7 @@ struct ufs_mtk_mcq_intr_info { =20 struct ufs_mtk_host { struct phy *mphy; - struct regulator *reg_va09; + struct regulator *reg_avdd09; struct reset_control_bulk_data resets[MTK_UFS_NUM_RESETS]; struct ufs_hba *hba; struct ufs_mtk_crypt_cfg *crypt; --=20 2.52.0