Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp517233ybk; Wed, 20 May 2020 05:35:36 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzbZaEwR1oJKw89QVpzCrzCAVSAkf6XXN3dNqquFrYbiDUKa33Z8Hku68LpCaw1luhRayRr X-Received: by 2002:a17:906:d148:: with SMTP id br8mr3408263ejb.356.1589978135847; Wed, 20 May 2020 05:35:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589978135; cv=none; d=google.com; s=arc-20160816; b=leWfP5zffzVTlDey2MZScrm7NXwjOvtyQtcYBABlislRDtgoy4FZLm2+1+6wqdetya mwHxrr8S72GVMOSisaoCxRTPbZJUxxK9lEDDirELUyLyARTE7QLKAZicUkogARSipT0B pxvd4UfyoyNgHQvxYBgB8JHPO/zQZ+DMhAWo0qe72u43K5tQwGz0TaV4Ho8OWvQaeIWQ iUjJ8ipSK6oZMWhkkqnTvEtZrGFLLSfFDaBTvi7M8Jj/C9Y7D5WMhu0r+5OjOHZyqNVz Ft77qYctvDRU9ONigP+ClLfuHe7dXGb0TEnVQPl23BSdo0neSPxlFsjPG82Z23MxDQor BbJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=+QDCNbCKRNnmN59N9fyjzwoXpesFZYqir7SoPkMED5U=; b=hB5CYjoY0Nlwzlsnpdrj+dh3vZ7UMirbI0CMriFej7W0UuL6pH0bxT5M7TupKbX/Dl 3wV0RHpt4lmfTFmgnf401trHXJnfAmJIOHt17NPpGHqmc9oLkHovO+09njNM/A8OWYCS UIIDwVWuoWXzZtlVRiscls2nQckWDjCqlX4PNdc0voClo8reAYbzaDk9t49bEUVrJ6Qk SHh0aeJgvdeX8opkXnpj+OvL85t1yQxZIMhrU4TqBks5FzN0DDCuT40COtBITTVtXbp2 zPGuD9WX2IVStje3InyMPDDfAJWS3O+AZZSRBUm59pxouvTLU/UxibterIncP7MTUkU7 16og== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=LoxEWlrz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g14si1440480edl.432.2020.05.20.05.35.12; Wed, 20 May 2020 05:35:35 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=LoxEWlrz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726566AbgETMbi (ORCPT + 99 others); Wed, 20 May 2020 08:31:38 -0400 Received: from mail.kernel.org ([198.145.29.99]:33840 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726439AbgETMbi (ORCPT ); Wed, 20 May 2020 08:31:38 -0400 Received: from localhost (fw-tnat.cambridge.arm.com [217.140.96.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 191D320756; Wed, 20 May 2020 12:31:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1589977897; bh=rL8WkukpYlQE2FvyEL1Ca998vF4yjcYjdKtcskbOga4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=LoxEWlrzY++Y0BFWiabgMwjNGX/nvadbakyFaRZq89IjX/nhET64+AAldhU56uv4y /N2F8Ge6zCrPiaYs24N4AeJZ8C+ZXyDCJWLD1XynAuUuEF6n7NdU/UlXGv8qQP01VQ yNqJhM8BUg7QoFPleCnp2BaLZ0B5zDCzVNny15Vs= Date: Wed, 20 May 2020 13:31:35 +0100 From: Mark Brown To: "Andrew-sh.Cheng" Cc: MyungJoo Ham , Kyungmin Park , Chanwoo Choi , Rob Herring , Mark Rutland , Matthias Brugger , "Rafael J . Wysocki" , Viresh Kumar , Nishanth Menon , Stephen Boyd , Liam Girdwood , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, srv_heupstream@mediatek.com Subject: Re: [PATCH 09/12] devfreq: add mediatek cci devfreq Message-ID: <20200520123135.GD4823@sirena.org.uk> References: <20200520034307.20435-1-andrew-sh.cheng@mediatek.com> <20200520034307.20435-10-andrew-sh.cheng@mediatek.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="2iBwrppp/7QCDedR" Content-Disposition: inline In-Reply-To: <20200520034307.20435-10-andrew-sh.cheng@mediatek.com> X-Cookie: You can't get there from here. User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --2iBwrppp/7QCDedR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, May 20, 2020 at 11:43:04AM +0800, Andrew-sh.Cheng wrote: > + cci_df->proc_reg = devm_regulator_get_optional(cci_dev, "proc"); > + ret = PTR_ERR_OR_ZERO(cci_df->proc_reg); > + if (ret) { > + if (ret != -EPROBE_DEFER) > + dev_err(cci_dev, "failed to get regulator for CCI: %d\n", > + ret); > + return ret; > + } > + ret = regulator_enable(cci_df->proc_reg); The code appears to require a regulator (and I'm guessing the device needs power) so why is this using regulator_get_optional()? --2iBwrppp/7QCDedR Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl7FIyYACgkQJNaLcl1U h9D0kAf/f2yyGL/KhJ2Y+J/Xvk72EFz2UdjxlDjJo9wXYf3J4HWB9sgTXydoZoRj bPlZzQtBeUPNmvX60Nczzt1V0KYNXns6mOfBOTueIAlSLeOnWBLV2+IUedCAVOJw NX2i3pZdZ2p7yOO9QGU2s8hxc3uwCHKETMa5U/B6ENwkQkkOlJCNkjVYYiuoAJn+ 9mIq1Ko2yfPHqJPyh/wP/CTC7AqXyPnQmO+rAlua+v8ua7RXRqAlkI/LpWhhpe1U PAdhYsvKa9TiyafrrUPW1TPbTBRnWpHd1s2gjhpiGuuDhJyEaGjxsm0dz82gs5ml aut3SlENmvhhRkfxSKh8LBYpjvJ0hA== =vHQ8 -----END PGP SIGNATURE----- --2iBwrppp/7QCDedR--