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 C1714E6FE24 for ; Tue, 23 Dec 2025 16:14:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=MVFVIKq0sBuyCffyYJ/1sD+iO89pe4U1D3i3N6wh6Y4=; b=pMnJYffRF6rUGYrLQ8UBQ/dJwH 50vvSrDmCEEBHVd8BzHilTmbjX5v1bXc3+2c2+ac3sRsbY7Wfu6/wAZbz9zXkoHNiNHB2CyosVOad 8fsO9RoEl9NzYnJLzGoDokDHAOwBU6q2Di8GjXhtQ5lTmCTDYkATjVKE+WsrcimSWlh24F40IH1Wu pQAPcBsfpKOGSyn73ayrUMogD2FhvMgflyyCTAMUnXypF3fD0MJ1bVc0zh9QbaRIgrKq70i4gPuTr ZhShaoL2ghzlZoUG1uTw2N6a8aDtLzAd+bDYeOf58wTXOg+jSt4Tzhsb+MVyPRe3H7MUFqmwC/e9T OYDzFOdw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1vY51T-0000000FkI4-0THO; Tue, 23 Dec 2025 16:14:07 +0000 Received: from sender4-pp-f112.zoho.com ([136.143.188.112]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1vY51P-0000000FkH3-2FsA; Tue, 23 Dec 2025 16:14:04 +0000 ARC-Seal: i=1; a=rsa-sha256; t=1766506430; cv=none; d=zohomail.com; s=zohoarc; b=GcLwAerCHEy/SR9JcYm3k6axVMsajNFFuPap2PgzrSCXjh4rWjBHBR68qGTzFmEtN3w7Ltea08lsuyXM5TqbBZ/wEy4Lt0BA0OELULdM6rGXc6Zpm/lcq+V2kll+jr5bGo9G+2hiQRG68jUw/5rriZEB9x1FO9zW043d4D4Ja4w= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1766506430; 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=MVFVIKq0sBuyCffyYJ/1sD+iO89pe4U1D3i3N6wh6Y4=; b=hgp16a5Gt/bQR/q68E5ve1yIbch7gWUfpYZwrYxqEPYRA7AjCYQWufj7UkH2WtntiJJcClOSpAjT8qbQ6hzQpa922gcQfGHjT2HYoNkxzsljhiHj3NZ1a+yqkgkpT8oA9vDxj8bmrSmjcajiXIH/eZp5StcOj4YLKIkQ6YTq43Y= 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=1766506430; s=zohomail; d=collabora.com; i=nicolas.frattaroli@collabora.com; h=From:From:To:To:Cc:Cc:Subject:Subject:Date:Date:Message-ID:In-Reply-To:References:MIME-Version:Content-Transfer-Encoding:Content-Type:Message-Id:Reply-To; bh=MVFVIKq0sBuyCffyYJ/1sD+iO89pe4U1D3i3N6wh6Y4=; b=lU9ueyei4bCn1/G3l4CDl0V7jCJjtHAIWGiYxDcQVWZ69Z0gd87bNml6FjOrSs7q Yr35Hpe+PYjszET5s4SQgWJvKVUq7y1hrqqsKtgZ0Dq8O+HDeUXJ73zJ+et/nhTZ5bH ZLnMXLNksOm4agx1xpj7JKjvKjzSQjIwePAVYmn4= Received: by mx.zohomail.com with SMTPS id 1766506427805485.7983551832393; Tue, 23 Dec 2025 08:13:47 -0800 (PST) From: Nicolas Frattaroli To: "chu.stanley@gmail.com" , "robh@kernel.org" , Chunfeng Yun =?UTF-8?B?KOS6keaYpeWzsCk=?= , "kishon@kernel.org" , "James.Bottomley@HansenPartnership.com" , "bvanassche@acm.org" , AngeloGioacchino Del Regno , "neil.armstrong@linaro.org" , "conor+dt@kernel.org" , Chaotian Jing =?UTF-8?B?KOS6leacneWkqSk=?= , "lgirdwood@gmail.com" , "vkoul@kernel.org" , "krzk+dt@kernel.org" , "p.zabel@pengutronix.de" , "alim.akhtar@samsung.com" , "matthias.bgg@gmail.com" , "avri.altman@wdc.com" , "martin.petersen@oracle.com" , "broonie@kernel.org" , Peter Wang =?UTF-8?B?KOeOi+S/oeWPiyk=?= Cc: "linux-scsi@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-phy@lists.infradead.org" , "linux-mediatek@lists.infradead.org" , Louis-Alexis Eyraud , "kernel@collabora.com" Subject: Re: [PATCH v4 07/25] scsi: ufs: mediatek: Rework 0.9V regulator Date: Tue, 23 Dec 2025 17:13:39 +0100 Message-ID: <14003986.uLZWGnKmhe@workhorse> In-Reply-To: <8206d9e715f7ef987b5369d0bda68cce13528112.camel@mediatek.com> References: <20251218-mt8196-ufs-v4-0-ddec7a369dd2@collabora.com> <20251218-mt8196-ufs-v4-7-ddec7a369dd2@collabora.com> <8206d9e715f7ef987b5369d0bda68cce13528112.camel@mediatek.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20251223_081403_624988_7088A35F X-CRM114-Status: GOOD ( 15.68 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Tuesday, 23 December 2025 10:35:39 Central European Standard Time Peter = Wang (=E7=8E=8B=E4=BF=A1=E5=8F=8B) wrote: > On Thu, 2025-12-18 at 13:54 +0100, Nicolas Frattaroli wrote: > >=20 > > +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; > >=20 >=20 >=20 > Hi Nicolas, >=20 > It seems that has_avdd09 is not necessary, because if the=20 > platform does not support avdd09, it will return an error > (-ENODEV) and bypass the avdd09 flow, right? >=20 > Thanks > Peter >=20 >=20 >=20 >=20 Hi, that would allow compatibles that are not allowed to have an avdd09 regulator by the binding (because the SoC doesn't have that pin) to have one specified in the device tree get picked up. This would then cause those devices with such a device tree to enter ufs_mtk_va09_pwr_ctrl(), which is not what we want. While we could blame this on the DT being wrong in that case, I think it's better if we avoid such a situation entirely. Kind regards, Nicolas Frattaroli