From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E7566CA5FA5 for ; Thu, 1 Oct 2026 08:32:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: References:In-Reply-To:Cc:To:Subject:From:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ImNO3Y6veJJ4GdqZvJceva41kbecj1bALqJnpZ17/Dk=; b=MsAgWbO8h5Tqbe VTAucM+oH+ZFxuPNWP+ScrK7xaSXT9pNBPV71Fj9/pyd8756GcP/fG0qUepLpgw5Z7H5Dj5zdZVKX FZoRZ/o8BUEgBOTqEjHIKXGtA9CHV1cUqpdSxdu/qWvezNZmttljQBbou10NRB7uqTSnOwcWREE3h RCjnQ0IIqUTY+1XJ4adeI+R2DeQS8d0SHq+hXg+N7tlmH1GB/tM7NnmgQrX2m6VtiZTEKNlQImqg/ ey9jmfrJURfWPiIlCfPbbmg0mwdDi2B9n4SmbL6f1oLsX70Nz3CIkvYTVK+v7YTBZPhtBzHPHAHIA a2Y3cPSSRPf1nEc/29vg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xCCD4-00000008B4K-26eM; Thu, 01 Oct 2026 08:32:10 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xCCD3-00000008B48-21f0 for linux-phy@lists.infradead.org; Thu, 01 Oct 2026 08:32:09 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by sea.source.kernel.org (Postfix) with ESMTP id 386C941521; Thu, 1 Oct 2026 08:32:09 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9DE081F00898; Thu, 1 Oct 2026 08:32:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790843529; bh=mpY7MydxNt5SFaoXiVAk9+VF27dF7m0gGzMCvNKGgWQ=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=I+2CrSH/LpqRidQAyz2KJdfq89yXzRyEQ6Fpoy8+LI6Xh1ueYLo7zJqhOuc4caY3J 9P27DtK2kMTnLBHFL4qJaLTmcfAsxTIOiGKzRyb/hIkMi1cxFosJramru+TOKXU791 IVL9FCUC8tCXQnKHMJfulDPq2O9YBRFGq5EWSrRKehVjrR2KjTPbrEJTHBB1ACtydF Y7WR3vwJrOSyJajiBaZ/PrTWZatIMZRKuhPOWAjsCrCChQio5MXQ/GOgGfuNhGdnPo ztdzmr9g8FiitxfZvv2T0jTyW79bExhGZexr5Xt4/FIz9Dk1yz7LuDOypKClqMMJKM vtnQlzuavbC6g== From: sashiko-bot@kernel.org Subject: Re: [PATCH v13 17/24] scsi: ufs: mediatek: Clean up logging prints To: "Louis-Alexis Eyraud" Cc: vkoul@kernel.org, olteanv@gmail.com, linux-scsi@vger.kernel.org, linux-phy@lists.infradead.org, conor+dt@kernel.org, neil.armstrong@linaro.org, devicetree@vger.kernel.org, robh@kernel.org In-Reply-To: <20261001-mt8196-ufs-v13-17-60854adab6d9@collabora.com> References: <20261001-mt8196-ufs-v13-0-60854adab6d9@collabora.com> <20261001-mt8196-ufs-v13-17-60854adab6d9@collabora.com> Date: Thu, 01 Oct 2026 08:32:08 +0000 Message-Id: <20261001083208.9DE081F00898@smtp.kernel.org> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg MSBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW01lZGl1bV0gdWZzOiAlcGUgb2Jm dXNjYXRlcyBwb3NpdGl2ZSBlcnJvciBmcm9tIHVmc2hjZF91aWNfY2hhbmdlX3B3cl9tb2RlKCkK LS0KCmNvbW1pdCAzYjhiYzcyMTZkY2UzZmEyYjQ1OGM0MDk1YjQzMmRlOTE0NDNmZDJjCkF1dGhv cjogTmljb2xhcyBGcmF0dGFyb2xpIDxuaWNvbGFzLmZyYXR0YXJvbGlAY29sbGFib3JhLmNvbT4K CnNjc2k6IHVmczogbWVkaWF0ZWs6IENsZWFuIHVwIGxvZ2dpbmcgcHJpbnRzCgpUaGlzIHBhdGNo IGNsZWFucyB1cCBsb2dnaW5nIHByaW50cyBpbiB0aGUgTWVkaWFUZWsgVUZTIGRyaXZlciBieQph ZGp1c3RpbmcgdmVyYm9zaXR5IGxldmVscyBhbmQgdXNpbmcgdGhlICVwZSBmb3JtYXQgc3BlY2lm aWVyIGZvcgpwcmludGluZyBlcnJvciBwb2ludGVycyBpbnN0ZWFkIG9mIGdlbmVyaWMgaW50ZWdl ciBsb2dnaW5nLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy91ZnMvaG9zdC91ZnMtbWVkaWF0ZWsu YyBiL2RyaXZlcnMvdWZzL2hvc3QvdWZzLW1lZGlhdGVrLmMKPiBpbmRleCAyZWRmYThiZDg3N2Fm Li44ZTVhMGEzOWNlNmNmIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvdWZzL2hvc3QvdWZzLW1lZGlh dGVrLmMKPiArKysgYi9kcml2ZXJzL3Vmcy9ob3N0L3Vmcy1tZWRpYXRlay5jClsgLi4uIF0KPiBA QCAtMTQxMSwxMCArMTQwMyw5IEBAIHN0YXRpYyBpbnQgdWZzX210a19wcmVfcHdyX2NoYW5nZShz dHJ1Y3QgdWZzX2hiYSAqaGJhLAo+ICAJCXJldCA9IHVmc2hjZF91aWNfY2hhbmdlX3B3cl9tb2Rl KGhiYSwKPiAgCQkJCQlGQVNUQVVUT19NT0RFIDw8IDQgfCBGQVNUQVVUT19NT0RFKTsKPiAgCj4g LQkJaWYgKHJldCkgewo+IC0JCQlkZXZfZXJyKGhiYS0+ZGV2LCAiJXM6IEhTRzFCIEZBU1RBVVRP IGZhaWxlZCByZXQ9JWRcbiIsCj4gLQkJCQlfX2Z1bmNfXywgcmV0KTsKPiAtCQl9Cj4gKwkJaWYg KHJldCkKPiArCQkJZGV2X2VycihoYmEtPmRldiwgIiVzOiBIU0cxQiBGQVNUQVVUTyBmYWlsZWQ6 ICVwZVxuIiwKPiArCQkJCV9fZnVuY19fLCBFUlJfUFRSKHJldCkpOwoKW1NldmVyaXR5OiBNZWRp dW1dCldoZW4gdWZzaGNkX3VpY19jaGFuZ2VfcHdyX21vZGUoKSBmYWlscyBkdWUgdG8gYSBwb3Np dGl2ZSBVUE1DUlMgc3RhdHVzCmNvZGUgKGUuZy4sIFBXUl9SRU1PVEUgb3IgUFdSX0JVU1kpLCB0 aGUgcmV0dXJuZWQgcG9zaXRpdmUgdmFsdWUgaXMKZXZhbHVhdGVkIGJ5IGlmIChyZXQpIGFuZCBw YXNzZWQgdG8gRVJSX1BUUigpLgoKQmVjYXVzZSB0aGUgJXBlIGZvcm1hdCBzcGVjaWZpZXIgY2hl Y2tzIElTX0VSUigpLCB3aGljaCBldmFsdWF0ZXMgdG8gZmFsc2UKZm9yIHNtYWxsIHBvc2l0aXZl IGludGVnZXJzLCBpdCB3aWxsIGZhbGwgYmFjayB0byBwcmludGluZyBhIGhhc2hlZCBwb2ludGVy Cmluc3RlYWQgb2YgdGhlIG51bWVyaWMgZXJyb3IgdmFsdWUuIAoKV2lsbCB0aGlzIG9iZnVzY2F0 ZSB0aGUgYWN0dWFsIGVycm9yIGNvZGUgZHVyaW5nIFVGUyBsaW5rIGluaXRpYWxpemF0aW9uIG9y CnBvd2VyIG1vZGUgY2hhbmdlIGZhaWx1cmVzPwoKPiAgCX0KPiAgCj4gIAkvKiBpZiBhbHJlYWR5 IGNvbmZpZ3VyZWQgdG8gdGhlIHJlcXVlc3RlZCBwd3JfbW9kZSwgc2tpcCBhZGFwdCAqLwoKLS0g ClNhc2hpa28gQUkgcmV2aWV3IMK3IGh0dHBzOi8vc2FzaGlrby5kZXYvIy9wYXRjaHNldC8yMDI2 MTAwMS1tdDgxOTYtdWZzLXYxMy0wLTYwODU0YWRhYjZkOUBjb2xsYWJvcmEuY29tP3BhcnQ9MTcK Ci0tIApsaW51eC1waHkgbWFpbGluZyBsaXN0CmxpbnV4LXBoeUBsaXN0cy5pbmZyYWRlYWQub3Jn Cmh0dHBzOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXBoeQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 A047131ED83; Thu, 1 Oct 2026 08:32:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790843530; cv=none; b=jqUhn51LvLcVo4pZZxDUuztJDdvfXnw8hNemiKVJ279gickeFuq/9HSzdY4rZ8b1O0/XxIK5ATgoXWQ8Exb9u1oJSdWEuBLUQ07I3kBSuwo0zqisTja/OxAPCsMXx0HNBTd1BJAniwTOMZ3Y94UxK1wTYOFsXAokPTo3IdrVBDg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790843530; c=relaxed/simple; bh=glhKukGiJzIZVH57m6PrhvO+x0h376PbA7JcNCP8iDA=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=oNiO7fcL383hqisS/pRsPVOGimFsPmK90z6ZGRI0jOX5k1jOVtRAr413Y0/oa8+SzMBxolyP4LM7IJ2Cn1jVztqrtSn1LUm0AZPfFKJWs5tDxz8yKNB4G1zgMfSU2dkfhbfUgvtPOA8kpwteRxDanH29ozNUFcbPXE9NfODG1zQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=I+2CrSH/; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="I+2CrSH/" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9DE081F00898; Thu, 1 Oct 2026 08:32:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790843529; bh=mpY7MydxNt5SFaoXiVAk9+VF27dF7m0gGzMCvNKGgWQ=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=I+2CrSH/LpqRidQAyz2KJdfq89yXzRyEQ6Fpoy8+LI6Xh1ueYLo7zJqhOuc4caY3J 9P27DtK2kMTnLBHFL4qJaLTmcfAsxTIOiGKzRyb/hIkMi1cxFosJramru+TOKXU791 IVL9FCUC8tCXQnKHMJfulDPq2O9YBRFGq5EWSrRKehVjrR2KjTPbrEJTHBB1ACtydF Y7WR3vwJrOSyJajiBaZ/PrTWZatIMZRKuhPOWAjsCrCChQio5MXQ/GOgGfuNhGdnPo ztdzmr9g8FiitxfZvv2T0jTyW79bExhGZexr5Xt4/FIz9Dk1yz7LuDOypKClqMMJKM vtnQlzuavbC6g== From: sashiko-bot@kernel.org Subject: Re: [PATCH v13 17/24] scsi: ufs: mediatek: Clean up logging prints Reply-To: sashiko-reviews@lists.linux.dev To: "Louis-Alexis Eyraud" Cc: vkoul@kernel.org, olteanv@gmail.com, linux-scsi@vger.kernel.org, linux-phy@lists.infradead.org, conor+dt@kernel.org, neil.armstrong@linaro.org, devicetree@vger.kernel.org, robh@kernel.org In-Reply-To: <20261001-mt8196-ufs-v13-17-60854adab6d9@collabora.com> References: <20261001-mt8196-ufs-v13-0-60854adab6d9@collabora.com> <20261001-mt8196-ufs-v13-17-60854adab6d9@collabora.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Thu, 01 Oct 2026 08:32:08 +0000 Message-Id: <20261001083208.9DE081F00898@smtp.kernel.org> Precedence: bulk X-Mailing-List: linux-scsi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 1 potential issue(= s) to consider: - [Medium] ufs: %pe obfuscates positive error from ufshcd_uic_change_pwr_mo= de() -- commit 3b8bc7216dce3fa2b458c4095b432de91443fd2c Author: Nicolas Frattaroli scsi: ufs: mediatek: Clean up logging prints This patch cleans up logging prints in the MediaTek UFS driver by adjusting verbosity levels and using the %pe format specifier for printing error pointers instead of generic integer logging. > diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-media= tek.c > index 2edfa8bd877af..8e5a0a39ce6cf 100644 > --- a/drivers/ufs/host/ufs-mediatek.c > +++ b/drivers/ufs/host/ufs-mediatek.c [ ... ] > @@ -1411,10 +1403,9 @@ static int ufs_mtk_pre_pwr_change(struct ufs_hba *= hba, > 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)); [Severity: Medium] When ufshcd_uic_change_pwr_mode() fails due to a positive UPMCRS status code (e.g., PWR_REMOTE or PWR_BUSY), the returned positive value is evaluated by if (ret) and passed to ERR_PTR(). Because the %pe format specifier checks IS_ERR(), which evaluates to false for small positive integers, it will fall back to printing a hashed pointer instead of the numeric error value.=20 Will this obfuscate the actual error code during UFS link initialization or power mode change failures? > } > =20 > /* if already configured to the requested pwr_mode, skip adapt */ --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261001-mt8196-ufs= -v13-0-60854adab6d9@collabora.com?part=3D17