From nobody Mon Feb 9 21:22:12 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 53DD8451052; Thu, 8 Jan 2026 10:52:28 +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=1767869560; cv=pass; b=psoH0PGiOeI8ouATBxFEnPylq13iAUgwn4JTot/KeFiVWFPHfwGu59qQzGavMRTG18W2J3QQJDHGyppReLKPLI9Tim5EALFKuSdlwczJYkqOxbzM3+WN0S3KqVMitu8TMm9m1JeWbp6qZ4Sn5j4FyE95t7KEo1cheQVpFA3hCUw= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767869560; c=relaxed/simple; bh=kqIyQt7NEkQSlEqUmR6LqlqdjqrxdPqyl07sIWuuIqM=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=BI+/vYFTnYX5NfsJ3vlbGLW1CZTQ4fDY9OTa8jmr4AQZbyZgOrvbFRuTpzlZRhC8BRpdb824Zw87h16nQh8tO01GeppH3tSFfktI8k9PQPwvbpzBiB87mYgukflIlchckt8R7B8C+O6bo5Y5VlFUz2r1cbL1Bds6SYSU5MK48ak= 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=Wklxt5oS; 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="Wklxt5oS" ARC-Seal: i=1; a=rsa-sha256; t=1767869505; cv=none; d=zohomail.com; s=zohoarc; b=Y04e26l8tv6btNlc4maXiEVtZktoqYLvRsejC2ErC/WtYhrR9U1L74L2NXK2DpAQ49HJswaRJCudvqZ/NHC6spgaSPA+ws0z+/UBwYCLSb/jq8lYjZAP8YGKOMHUAVx0f37ssnbgX7zFsski3hW/yIDiyns5kA1zvgXHFjXN3dg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1767869505; 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=0zzshRGMTJOiW70/VRTeC7VSzPEBqleDsHFXVaWhQdo=; b=AvN/k4+pkszcTXW8VNo6qX0HiP12vLl9RFCT3UFuxOE5KbXIdpkMbyK6WhlW36wG9nDiNZxZdoe9FwrUYiLidqWAbPIqxLiY5dSbP7Q/jYbG7wNoEGdsJkkFUANt6Co89ps6cKlWj7o4W0pX/WmEWpOPhkP2h5op8k8RpkRioOQ= 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=1767869505; 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=0zzshRGMTJOiW70/VRTeC7VSzPEBqleDsHFXVaWhQdo=; b=Wklxt5oS7il6GjuOF4HvtBA5kILVWY/DGiVgDsfsv9tbZieF720Fo1LOjSdFHdy9 yX22r/Ov9zMCwzPsdCOZYmEBAeDLOcVXlzJc2N/JHLSKBMaZ9B2HyitR4C1rx75EiF9 JU1p82Wx9YRRsrSrO6ygzQxrXQKyQGvj6pSWOfvs= Received: by mx.zohomail.com with SMTPS id 1767869504677534.810494104054; Thu, 8 Jan 2026 02:51:44 -0800 (PST) From: Nicolas Frattaroli Date: Thu, 08 Jan 2026 11:49:37 +0100 Subject: [PATCH v5 18/24] scsi: ufs: mediatek: Clean up logging prints 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: <20260108-mt8196-ufs-v5-18-49215157ec41@collabora.com> References: <20260108-mt8196-ufs-v5-0-49215157ec41@collabora.com> In-Reply-To: <20260108-mt8196-ufs-v5-0-49215157ec41@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 Linux kernel's log buffer provides many levels of verbosity, associated with different semantic meanings. Care should be taken to only log useful information to the info level, and log errors to the error level. The MediaTek UFS driver does not do this. It freely logs verbose debug information to the info level, errors to the info level, and sometimes errors to the warning level. Adjust all the wrapped kprintf invocations to rectify this situation. Use user-friendly %pe format codes for printing errors where possible. Reviewed-by: AngeloGioacchino Del Regno Signed-off-by: Nicolas Frattaroli --- drivers/ufs/host/ufs-mediatek.c | 99 ++++++++++++++++++-------------------= ---- 1 file changed, 43 insertions(+), 56 deletions(-) diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-mediate= k.c index 2b7def2cde54..566d61ff9faf 100644 --- a/drivers/ufs/host/ufs-mediatek.c +++ b/drivers/ufs/host/ufs-mediatek.c @@ -192,8 +192,8 @@ static void ufs_mtk_crypto_enable(struct ufs_hba *hba) =20 ufs_mtk_crypto_ctrl(res, 1); if (res.a0) { - dev_info(hba->dev, "%s: crypto enable failed, err: %lu\n", - __func__, res.a0); + dev_err(hba->dev, "%s: crypto enable failed with error %lu, disabling\n", + __func__, res.a0); hba->caps &=3D ~UFSHCD_CAP_CRYPTO; } } @@ -542,40 +542,38 @@ static void ufs_mtk_boost_crypt(struct ufs_hba *hba, = bool boost) =20 ret =3D clk_prepare_enable(cfg->clk_crypt_mux); if (ret) { - dev_info(hba->dev, "clk_prepare_enable(): %d\n", - ret); + dev_err(hba->dev, "%s: Failed to enable clk_crypt_mux: %pe\n", + __func__, ERR_PTR(ret)); return; } =20 if (boost) { ret =3D regulator_set_voltage(reg, volt, INT_MAX); if (ret) { - dev_info(hba->dev, - "failed to set vcore to %d\n", volt); + dev_err(hba->dev, "%s: Failed to set vcore to %d: %pe\n", + __func__, volt, ERR_PTR(ret)); goto out; } =20 - ret =3D clk_set_parent(cfg->clk_crypt_mux, - cfg->clk_crypt_perf); + ret =3D clk_set_parent(cfg->clk_crypt_mux, cfg->clk_crypt_perf); if (ret) { - dev_info(hba->dev, - "failed to set clk_crypt_perf\n"); + dev_err(hba->dev, "%s: Failed to reparent clk_crypt_perf: %pe\n", + __func__, ERR_PTR(ret)); regulator_set_voltage(reg, 0, INT_MAX); goto out; } } else { - ret =3D clk_set_parent(cfg->clk_crypt_mux, - cfg->clk_crypt_lp); + ret =3D clk_set_parent(cfg->clk_crypt_mux, cfg->clk_crypt_lp); if (ret) { - dev_info(hba->dev, - "failed to set clk_crypt_lp\n"); + dev_err(hba->dev, "%s: Failed to reparent clk_crypt_lp: %pe\n", + __func__, ERR_PTR(ret)); goto out; } =20 ret =3D regulator_set_voltage(reg, 0, INT_MAX); if (ret) { - dev_info(hba->dev, - "failed to set vcore to MIN\n"); + dev_err(hba->dev, "%s: Failed to set vcore to minimum: %pe\n", + __func__, ERR_PTR(ret)); } } out: @@ -763,10 +761,8 @@ static int ufs_mtk_setup_clocks(struct ufs_hba *hba, b= ool on, if (clk_pwr_off) { ufs_mtk_pwr_ctrl(hba, false); } else { - dev_warn(hba->dev, "Clock is not turned off, hba->ahit =3D 0x%x, AHIT = =3D 0x%x\n", - hba->ahit, - ufshcd_readl(hba, - REG_AUTO_HIBERNATE_IDLE_TIMER)); + dev_warn(hba->dev, "Clock isn't off, hba->ahit =3D 0x%x, AHIT =3D 0x%x\= n", + hba->ahit, ufshcd_readl(hba, REG_AUTO_HIBERNATE_IDLE_TIMER)); } ufs_mtk_mcq_disable_irq(hba); } else if (on && status =3D=3D POST_CHANGE) { @@ -810,11 +806,11 @@ static void ufs_mtk_mcq_set_irq_affinity(struct ufs_h= ba *hba, unsigned int cpu) _cpu =3D (cpu =3D=3D 0) ? 3 : cpu; ret =3D irq_set_affinity(irq, cpumask_of(_cpu)); if (ret) { - dev_err(hba->dev, "set irq %d affinity to CPU %d failed\n", + dev_err(hba->dev, "setting irq %d affinity to CPU %d failed\n", irq, _cpu); return; } - dev_info(hba->dev, "set irq %d affinity to CPU: %d\n", irq, _cpu); + dev_dbg(hba->dev, "set irq %d affinity to CPU %d\n", irq, _cpu); } =20 static bool ufs_mtk_is_legacy_chipset(struct ufs_hba *hba, u32 hw_ip_ver) @@ -830,7 +826,8 @@ static bool ufs_mtk_is_legacy_chipset(struct ufs_hba *h= ba, u32 hw_ip_ver) default: break; } - dev_info(hba->dev, "legacy IP version - 0x%x, is legacy : %d", hw_ip_ver,= is_legacy); + dev_dbg(hba->dev, "IP version 0x%x, legacy =3D %s", hw_ip_ver, + str_true_false(is_legacy)); =20 return is_legacy; } @@ -935,15 +932,12 @@ static void ufs_mtk_init_clocks(struct ufs_hba *hba) } } =20 - list_for_each_entry(clki, head, list) { - dev_info(hba->dev, "clk \"%s\" present", clki->name); - } + list_for_each_entry(clki, head, list) + dev_dbg(hba->dev, "clk \"%s\" present", clki->name); =20 if (!ufs_mtk_is_clk_scale_ready(hba)) { hba->caps &=3D ~UFSHCD_CAP_CLK_SCALING; - dev_info(hba->dev, - "%s: Clk-scaling not ready. Feature disabled.", - __func__); + dev_info(hba->dev, "%s: Clock scaling unavailable", __func__); return; } =20 @@ -953,8 +947,8 @@ static void ufs_mtk_init_clocks(struct ufs_hba *hba) */ reg =3D devm_regulator_get_optional(dev, "dvfsrc-vcore"); if (IS_ERR(reg)) { - dev_info(dev, "failed to get dvfsrc-vcore: %ld", - PTR_ERR(reg)); + if (PTR_ERR(reg) !=3D -ENODEV) + dev_err(dev, "Failed to get dvfsrc-vcore: %pe\n", reg); return; } =20 @@ -968,12 +962,9 @@ static void ufs_mtk_init_clocks(struct ufs_hba *hba) host->mclk.vcore_volt =3D volt; =20 /* If default boot is max gear, request vcore */ - if (reg && volt && host->clk_scale_up) { - if (regulator_set_voltage(reg, volt, INT_MAX)) { - dev_info(hba->dev, - "Failed to set vcore to %d\n", volt); - } - } + if (reg && volt && host->clk_scale_up) + if (regulator_set_voltage(reg, volt, INT_MAX)) + dev_err(hba->dev, "Failed to set vcore to %d\n", volt); } =20 static void ufs_mtk_setup_clk_gating(struct ufs_hba *hba) @@ -1060,7 +1051,7 @@ static void ufs_mtk_init_mcq_irq(struct ufs_hba *hba) } host->mcq_intr_info[i].hba =3D hba; host->mcq_intr_info[i].irq =3D irq; - dev_info(hba->dev, "get platform mcq irq: %d, %d\n", i, irq); + dev_dbg(hba->dev, "get platform mcq irq: %d, %d\n", i, irq); } =20 return; @@ -1307,10 +1298,8 @@ static int ufs_mtk_pre_pwr_change(struct ufs_hba *hb= a, host_params.desired_working_mode =3D UFS_PWM_MODE; =20 ret =3D ufshcd_negotiate_pwr_params(&host_params, dev_max_params, dev_req= _params); - if (ret) { - pr_info("%s: failed to determine capabilities\n", - __func__); - } + if (ret) + dev_warn(hba->dev, "%s: failed to determine capabilities\n", __func__); =20 if (ufs_mtk_pmc_via_fastauto(hba, dev_req_params)) { ufs_mtk_adjust_sync_length(hba); @@ -1356,10 +1345,9 @@ static int ufs_mtk_pre_pwr_change(struct ufs_hba *hb= a, ret =3D ufshcd_uic_change_pwr_mode(hba, FASTAUTO_MODE << 4 | FASTAUTO_MODE); =20 - if (ret) { - dev_err(hba->dev, "%s: HSG1B FASTAUTO failed ret=3D%d\n", - __func__, ret); - } + if (ret) + dev_err(hba->dev, "%s: HSG1B FASTAUTO failed: %pe\n", + __func__, ERR_PTR(ret)); } =20 /* if already configured to the requested pwr_mode, skip adapt */ @@ -1409,7 +1397,7 @@ static int ufs_mtk_auto_hibern8_disable(struct ufs_hb= a *hba) =20 out: if (ret) { - dev_warn(hba->dev, "exit h8 state fail, ret=3D%d\n", ret); + dev_err(hba->dev, "Failed to exit h8 state: %pe\n", ERR_PTR(ret)); =20 ufshcd_force_error_recovery(hba); =20 @@ -1571,7 +1559,7 @@ static int ufs_mtk_device_reset(struct ufs_hba *hba) /* Some devices may need time to respond to rst_n */ usleep_range(10000, 15000); =20 - dev_info(hba->dev, "device reset done\n"); + dev_dbg(hba->dev, "device reset done\n"); =20 return 0; } @@ -1607,12 +1595,12 @@ static int ufs_mtk_link_set_hpm(struct ufs_hba *hba) /* Check link state to make sure exit h8 success */ err =3D ufs_mtk_wait_idle_state(hba, 5); if (err) { - dev_warn(hba->dev, "wait idle fail, err=3D%d\n", err); + dev_err(hba->dev, "Failed to wait for idle: %pe\n", ERR_PTR(err)); return err; } err =3D ufs_mtk_wait_link_state(hba, VS_LINK_UP, 100); if (err) { - dev_warn(hba->dev, "exit h8 state fail, err=3D%d\n", err); + dev_err(hba->dev, "Failed to wait for link to be up: %pe\n", ERR_PTR(err= )); return err; } ufshcd_set_link_active(hba); @@ -1905,20 +1893,19 @@ static void ufs_mtk_event_notify(struct ufs_hba *hb= a, =20 /* Print details of UIC Errors */ if (evt <=3D UFS_EVT_DME_ERR) { - dev_info(hba->dev, - "Host UIC Error Code (%s): %08x\n", - ufs_uic_err_str[evt], val); + dev_err(hba->dev, "Host UIC Error Code (%s): %08x\n", + ufs_uic_err_str[evt], val); reg =3D val; } =20 if (evt =3D=3D UFS_EVT_PA_ERR) { for_each_set_bit(bit, ®, ARRAY_SIZE(ufs_uic_pa_err_str)) - dev_info(hba->dev, "%s\n", ufs_uic_pa_err_str[bit]); + dev_err(hba->dev, "%s\n", ufs_uic_pa_err_str[bit]); } =20 if (evt =3D=3D UFS_EVT_DL_ERR) { for_each_set_bit(bit, ®, ARRAY_SIZE(ufs_uic_dl_err_str)) - dev_info(hba->dev, "%s\n", ufs_uic_dl_err_str[bit]); + dev_err(hba->dev, "%s\n", ufs_uic_dl_err_str[bit]); } } =20 @@ -2123,7 +2110,7 @@ static int ufs_mtk_mcq_config_resource(struct ufs_hba= *hba) =20 /* fail mcq initialization if interrupt is not filled properly */ if (!host->mcq_nr_intr) { - dev_info(hba->dev, "IRQs not ready. MCQ disabled."); + dev_err(hba->dev, "IRQs not ready. MCQ disabled."); return -EINVAL; } =20 --=20 2.52.0