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 5166948EC9D; Fri, 16 Oct 2026 12:43:38 +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=1792154623; cv=none; b=FC4GVxZF/hP8YsMmPlOCefnj94sp/SO6p74TN5czMP75Nti+CsIuP82OdeiPGLS6te+k1BWwhAIIXYa3kyD7cx29maW45hDrX09ciPxUoIhnDlj+MWUvDgvs3uYby2ylUGBc5OoyW0AMAcAL/xMAVOF6UXXkBXo3BDwID7lZXpg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1792154623; c=relaxed/simple; bh=dls08gHxBUjwJU+/AZOM7H6ToUoEuSmrDhJ9sY9mINE=; h=Message-ID:From:Subject:To:Cc:In-Reply-To:References:Content-Type: Date; b=huF+XrObSrQQJB+2kCJEsTUKoIOIS0HACyWsf+FJiKixVtA36yJSTRfSItNyzctEMpoMhGQ+I3BMcp6DHgt+l1DlF2gNvMUQkX2SneU8l5Gl1UHgb6bTTgTVpKaLenVsHZ89V3x3tKoTwKJ1G7jViEB7ej5g4rAu58Mr9Z3DseY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=d1MTx9Bp; 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="d1MTx9Bp" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 366DE1F000C4; Fri, 16 Oct 2026 12:43:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1792154618; bh=sFVNTFoGHUrASjkmYtjqwEFnuAOqRGnu4QXjJP3mM8E=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=d1MTx9Bp17fkiU+2EgdscZLYd+23JjVfiT17kwNdnFAbA0rzBYYT44HI0f1POl0TS NWOMfHQAuBvTDuM9Kb7wibrYjEvC07nrwkeGBMqTe/LBEG6f0Q+bmR9KUv73a5WYoz 7wE3HQ0qhvRfPMasY/gwwnTNm6RMPBLtD9j1Bhd6Jj/Zy9kA6g9qNwADx66rfP60QS TGq8zk8Fcne5BuoT0uDvIbbBC0k3aTPo1SqCB3ne7iRN3SobYYQ7msNzYsQQVjCnwJ hW8OyjLK9BWM4BR4bMHO2JBMPknqUYl1tVWZxO8knk6t9pRgOmSvT4/5xlF/82sb0w e92tVO3s8U/YA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH v14 13/23] scsi: ufs: mediatek: Remove vendor kernel quirks cruft Reply-To: sashiko-reviews@lists.linux.dev To: "Louis-Alexis Eyraud" Cc: linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-scsi@vger.kernel.org, robh@kernel.org, conor+dt@kernel.org, vkoul@kernel.org, neil.armstrong@linaro.org, olteanv@gmail.com In-Reply-To: <20261016-mt8196-ufs-v14-13-e8d565e0cd45@collabora.com> References: <20261016-mt8196-ufs-v14-0-e8d565e0cd45@collabora.com> <20261016-mt8196-ufs-v14-13-e8d565e0cd45@collabora.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Fri, 16 Oct 2026 12:43:37 +0000 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: - [Low] scsi: ufs: mediatek: Orphaned SIP macro and enum definitions -- commit e7bac0e7488467d72dbbc1b3e957e1e9f9f5563a Author: Nicolas Frattaroli scsi: ufs: mediatek: Remove vendor kernel quirks cruft =20 Both ufs_mtk_vreg_fix_vcc and ufs_mtk_vreg_fix_vccqx look like they are vendor kernel hacks to work around existing downstream device trees. Mainline does not need or want them, so remove them. > diff --git a/drivers/ufs/host/ufs-mediatek.c b/drivers/ufs/host/ufs-media= tek.c > index 46c9153be6e22..41e2e9f867013 100644 > --- a/drivers/ufs/host/ufs-mediatek.c > +++ b/drivers/ufs/host/ufs-mediatek.c > @@ -1066,73 +1066,6 @@ static void ufs_mtk_init_clocks(struct ufs_hba *hb= a) > } > } > =20 > -#define MAX_VCC_NAME 30 > -static int ufs_mtk_vreg_fix_vcc(struct ufs_hba *hba) > -{ > - struct ufs_vreg_info *info =3D &hba->vreg_info; > - struct device_node *np =3D hba->dev->of_node; > - struct device *dev =3D hba->dev; > - char vcc_name[MAX_VCC_NAME]; > - struct arm_smccc_res res; > - int err, ver; > - > - if (info->vcc) > - return 0; > - > - if (of_property_read_bool(np, "mediatek,ufs-vcc-by-num")) { > - ufs_mtk_get_vcc_num(res); [Severity: Low] Removing ufs_mtk_vreg_fix_vcc() removes the only caller of ufs_mtk_get_vcc_num(). Does this leave the ufs_mtk_get_vcc_num() macro, the UFS_MTK_SIP_GET_VCC_NUM definition, and enum ufs_mtk_vcc_num completely unused in drivers/ufs/host/ufs-mediatek-sip.h? Could these definitions also be removed to avoid leaving dead code? > - if (res.a1 > UFS_VCC_NONE && res.a1 < UFS_VCC_MAX) > - snprintf(vcc_name, MAX_VCC_NAME, "vcc-opt%lu", res.a1); [ ... ] --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261016-mt8196-ufs= -v14-0-e8d565e0cd45@collabora.com?part=3D13 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 5B909CCA464 for ; Fri, 16 Oct 2026 12:43:41 +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:Date:References: In-Reply-To:Cc:To:Subject:From:Message-ID:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kzQhJbBkPoYnDeSdyzD72fKkJAN9ZBluIi/A2SdwzRE=; b=TSueoCfuaRkfej M3MXhVVQ0J0JW0hRf7T/zBHjf0k8xCh5sp7por5p2QgbHeK7PmpRJ8+NgfRCUelWagdgQMF8QxuuG mSfxfszn/XlazBe0yi4qC4v3e3dFYU+xGye2r2f2S25l2wBu5j3BZr6Je0FFPCVYqbcipG0oRS7ms CAo4XaPEtd5e3On435CrLtogUUrTwXJg5dkKN1EDaoNG96s78gYxpww6Ky5Eo3M0+lu4a9Jj3ocje jN5Z0tgyWVhb5mAQVKP3aEt8siPD4+AaMP4WgwgC5BFRRXAiFou8Exdw0eda7lGZoPmi4SecOtsRf wnip8sVL0qxO7twsbf6g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xHhHg-00000002wmP-3e2t; Fri, 16 Oct 2026 12:43:40 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xHhHf-00000002wmD-3Aq1 for linux-phy@lists.infradead.org; Fri, 16 Oct 2026 12:43:39 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id F157460018; Fri, 16 Oct 2026 12:43:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 366DE1F000C4; Fri, 16 Oct 2026 12:43:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1792154618; bh=sFVNTFoGHUrASjkmYtjqwEFnuAOqRGnu4QXjJP3mM8E=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=d1MTx9Bp17fkiU+2EgdscZLYd+23JjVfiT17kwNdnFAbA0rzBYYT44HI0f1POl0TS NWOMfHQAuBvTDuM9Kb7wibrYjEvC07nrwkeGBMqTe/LBEG6f0Q+bmR9KUv73a5WYoz 7wE3HQ0qhvRfPMasY/gwwnTNm6RMPBLtD9j1Bhd6Jj/Zy9kA6g9qNwADx66rfP60QS TGq8zk8Fcne5BuoT0uDvIbbBC0k3aTPo1SqCB3ne7iRN3SobYYQ7msNzYsQQVjCnwJ hW8OyjLK9BWM4BR4bMHO2JBMPknqUYl1tVWZxO8knk6t9pRgOmSvT4/5xlF/82sb0w e92tVO3s8U/YA== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH v14 13/23] scsi: ufs: mediatek: Remove vendor kernel quirks cruft To: "Louis-Alexis Eyraud" Cc: linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-scsi@vger.kernel.org, robh@kernel.org, conor+dt@kernel.org, vkoul@kernel.org, neil.armstrong@linaro.org, olteanv@gmail.com In-Reply-To: <20261016-mt8196-ufs-v14-13-e8d565e0cd45@collabora.com> References: <20261016-mt8196-ufs-v14-0-e8d565e0cd45@collabora.com> <20261016-mt8196-ufs-v14-13-e8d565e0cd45@collabora.com> Date: Fri, 16 Oct 2026 12:43:37 +0000 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 MSBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW0xvd10gc2NzaTogdWZzOiBtZWRp YXRlazogT3JwaGFuZWQgU0lQIG1hY3JvIGFuZCBlbnVtIGRlZmluaXRpb25zCi0tCgpjb21taXQg ZTdiYWMwZTc0ODg0NjdkNzJkYmJjMWIzZTk1N2UxZTlmOWY1NTYzYQpBdXRob3I6IE5pY29sYXMg RnJhdHRhcm9saSA8bmljb2xhcy5mcmF0dGFyb2xpQGNvbGxhYm9yYS5jb20+CgogICAgc2NzaTog dWZzOiBtZWRpYXRlazogUmVtb3ZlIHZlbmRvciBrZXJuZWwgcXVpcmtzIGNydWZ0CiAgICAKQm90 aCB1ZnNfbXRrX3ZyZWdfZml4X3ZjYyBhbmQgdWZzX210a192cmVnX2ZpeF92Y2NxeCBsb29rIGxp a2UgdGhleSBhcmUKdmVuZG9yIGtlcm5lbCBoYWNrcyB0byB3b3JrIGFyb3VuZCBleGlzdGluZyBk b3duc3RyZWFtIGRldmljZSB0cmVlcy4KTWFpbmxpbmUgZG9lcyBub3QgbmVlZCBvciB3YW50IHRo ZW0sIHNvIHJlbW92ZSB0aGVtLgoKPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy91ZnMvaG9zdC91ZnMt bWVkaWF0ZWsuYyBiL2RyaXZlcnMvdWZzL2hvc3QvdWZzLW1lZGlhdGVrLmMKPiBpbmRleCA0NmM5 MTUzYmU2ZTIyLi40MWUyZTlmODY3MDEzIDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvdWZzL2hvc3Qv dWZzLW1lZGlhdGVrLmMKPiArKysgYi9kcml2ZXJzL3Vmcy9ob3N0L3Vmcy1tZWRpYXRlay5jCj4g QEAgLTEwNjYsNzMgKzEwNjYsNiBAQCBzdGF0aWMgdm9pZCB1ZnNfbXRrX2luaXRfY2xvY2tzKHN0 cnVjdCB1ZnNfaGJhICpoYmEpCj4gIAl9Cj4gIH0KPiAgCj4gLSNkZWZpbmUgTUFYX1ZDQ19OQU1F IDMwCj4gLXN0YXRpYyBpbnQgdWZzX210a192cmVnX2ZpeF92Y2Moc3RydWN0IHVmc19oYmEgKmhi YSkKPiAtewo+IC0Jc3RydWN0IHVmc192cmVnX2luZm8gKmluZm8gPSAmaGJhLT52cmVnX2luZm87 Cj4gLQlzdHJ1Y3QgZGV2aWNlX25vZGUgKm5wID0gaGJhLT5kZXYtPm9mX25vZGU7Cj4gLQlzdHJ1 Y3QgZGV2aWNlICpkZXYgPSBoYmEtPmRldjsKPiAtCWNoYXIgdmNjX25hbWVbTUFYX1ZDQ19OQU1F XTsKPiAtCXN0cnVjdCBhcm1fc21jY2NfcmVzIHJlczsKPiAtCWludCBlcnIsIHZlcjsKPiAtCj4g LQlpZiAoaW5mby0+dmNjKQo+IC0JCXJldHVybiAwOwo+IC0KPiAtCWlmIChvZl9wcm9wZXJ0eV9y ZWFkX2Jvb2wobnAsICJtZWRpYXRlayx1ZnMtdmNjLWJ5LW51bSIpKSB7Cj4gLQkJdWZzX210a19n ZXRfdmNjX251bShyZXMpOwoKW1NldmVyaXR5OiBMb3ddClJlbW92aW5nIHVmc19tdGtfdnJlZ19m aXhfdmNjKCkgcmVtb3ZlcyB0aGUgb25seSBjYWxsZXIgb2YKdWZzX210a19nZXRfdmNjX251bSgp LiBEb2VzIHRoaXMgbGVhdmUgdGhlIHVmc19tdGtfZ2V0X3ZjY19udW0oKSBtYWNybywKdGhlIFVG U19NVEtfU0lQX0dFVF9WQ0NfTlVNIGRlZmluaXRpb24sIGFuZCBlbnVtIHVmc19tdGtfdmNjX251 bSBjb21wbGV0ZWx5CnVudXNlZCBpbiBkcml2ZXJzL3Vmcy9ob3N0L3Vmcy1tZWRpYXRlay1zaXAu aD8KCkNvdWxkIHRoZXNlIGRlZmluaXRpb25zIGFsc28gYmUgcmVtb3ZlZCB0byBhdm9pZCBsZWF2 aW5nIGRlYWQgY29kZT8KCj4gLQkJaWYgKHJlcy5hMSA+IFVGU19WQ0NfTk9ORSAmJiByZXMuYTEg PCBVRlNfVkNDX01BWCkKPiAtCQkJc25wcmludGYodmNjX25hbWUsIE1BWF9WQ0NfTkFNRSwgInZj Yy1vcHQlbHUiLCByZXMuYTEpOwoKWyAuLi4gXQoKLS0gClNhc2hpa28gQUkgcmV2aWV3IMK3IGh0 dHBzOi8vc2FzaGlrby5kZXYvIy9wYXRjaHNldC8yMDI2MTAxNi1tdDgxOTYtdWZzLXYxNC0wLWU4 ZDU2NWUwY2Q0NUBjb2xsYWJvcmEuY29tP3BhcnQ9MTMKCi0tIApsaW51eC1waHkgbWFpbGluZyBs aXN0CmxpbnV4LXBoeUBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHBzOi8vbGlzdHMuaW5mcmFkZWFk Lm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXBoeQo=