Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp2953133ybl; Mon, 20 Jan 2020 12:41:10 -0800 (PST) X-Google-Smtp-Source: APXvYqz7uuf0F/dcrNlQvllL79GENE27+hFUTxpMSxz3USQLCa95bA7YnIfW4rQWnUG9kTm7G+oi X-Received: by 2002:a9d:198b:: with SMTP id k11mr995415otk.295.1579552869856; Mon, 20 Jan 2020 12:41:09 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1579552869; cv=none; d=google.com; s=arc-20160816; b=ue7RB7A1PffDUCGbjW96j9OpkyvO14PsAqIBz9RL3crsKAKa8FfICy8CgYTcQHAEY6 KfJndopV7g1eB4/b8H26OsNXKR6NEVMRfEPL4Ol8R2ViBcY5bQAGSvwBod3wD7+Y4JE6 26WJgTUN5qV4kK0zzjp24pPtdxpY1StgR+JVfIoFnXM/t+PV7LfXxYBddPHBK0s/uRvT CDR7Eh+s8pOQrbHN0KHetHLwNtbqZFK4//10Nik4WkK3cGvAVxogBT4v34IoN2FJWpvo c87H8bKcsGRO8qwc5MwfTHmc04sPBIdfZ79/nrHiX1X6F3T3g9fVk72CiPvVfRrQznBY wUWQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dmarc-filter :dkim-signature; bh=hMMkrdpbQuWKKh6Fs/yw9vSMFlUgvTehkHZkaLEuz20=; b=jjo9jolZNAk4aDUhSp22XialFkOTOznYnJ5c3ypLFWf9acRx7D5T3CKKTy6rBWP0oH +wslPogQiQDUe3sUoHX2LUsV6KVgt1I9xriCwR6VDi2+K0YVR5HETm/dyEmVkKTZ34Db rcTY+T0pQUdFcMJL004m6LOVra8msiXBPk87kLbmyAOF6nOegD1vOSp8hIJUMnSfNyp8 rABHmm4Sb3U6eiug/rK5EG3oaJOKYHKjIWu+youN/F1sQzkb6DnKexgUaIaGDQX23Tvm 4ZqK7RdtlGscGrxoFO2zEgNTPhISfZy1zP6QJFVe8Qt1XB5gOHWD+FTbwZPP6j6c+TyZ kxmw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@mg.codeaurora.org header.s=smtp header.b=FLZ3fLYR; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e190si19237017oib.113.2020.01.20.12.40.57; Mon, 20 Jan 2020 12:41:09 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=fail header.i=@mg.codeaurora.org header.s=smtp header.b=FLZ3fLYR; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726894AbgATUkD (ORCPT + 99 others); Mon, 20 Jan 2020 15:40:03 -0500 Received: from mail25.static.mailgun.info ([104.130.122.25]:53888 "EHLO mail25.static.mailgun.info" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726752AbgATUkC (ORCPT ); Mon, 20 Jan 2020 15:40:02 -0500 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1579552802; h=Content-Transfer-Encoding: Content-Type: In-Reply-To: MIME-Version: Date: Message-ID: From: References: Cc: To: Subject: Sender; bh=hMMkrdpbQuWKKh6Fs/yw9vSMFlUgvTehkHZkaLEuz20=; b=FLZ3fLYRGCmkyJJ++zNVIhbkxwEnEYsyju0s3282FXJd2sgF4fnzmcU7kJH4hwUm2uxjaeVM fNjdaXmeMJFAnsNGyEERxcg40iH3MjMnxExKOQLXI5uJGu6zssznuZOaXyfJMd+LD0KZdFtM raOPrBTJ1/xcp7jjVqpnkOiL5cc= X-Mailgun-Sending-Ip: 104.130.122.25 X-Mailgun-Sid: WyI0MWYwYSIsICJsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnIiwgImJlOWU0YSJd Received: from smtp.codeaurora.org (ec2-35-166-182-171.us-west-2.compute.amazonaws.com [35.166.182.171]) by mxa.mailgun.org with ESMTP id 5e26101b.7fd2f4c67e68-smtp-out-n01; Mon, 20 Jan 2020 20:39:55 -0000 (UTC) Received: by smtp.codeaurora.org (Postfix, from userid 1001) id 4B889C433A2; Mon, 20 Jan 2020 20:39:55 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-caf-mail-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=2.0 tests=ALL_TRUSTED,SPF_NONE autolearn=unavailable autolearn_force=no version=3.4.0 Received: from [10.46.161.159] (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: asutoshd) by smtp.codeaurora.org (Postfix) with ESMTPSA id 68823C43383; Mon, 20 Jan 2020 20:39:52 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 68823C43383 Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; spf=none smtp.mailfrom=asutoshd@codeaurora.org Subject: Re: [PATCH v4 4/8] scsi: ufs: Move ufshcd_get_max_pwr_mode() to ufshcd_device_params_init() To: Bean Huo , alim.akhtar@samsung.com, avri.altman@wdc.com, jejb@linux.ibm.com, martin.petersen@oracle.com, stanley.chu@mediatek.com, beanhuo@micron.com, bvanassche@acm.org, tomas.winkler@intel.com, cang@codeaurora.org Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org References: <20200120130820.1737-1-huobean@gmail.com> <20200120130820.1737-5-huobean@gmail.com> From: "Asutosh Das (asd)" Message-ID: <87a60011-75da-f38c-07fe-105e97074f0c@codeaurora.org> Date: Mon, 20 Jan 2020 12:39:51 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 MIME-Version: 1.0 In-Reply-To: <20200120130820.1737-5-huobean@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/20/2020 5:08 AM, Bean Huo wrote: > From: Bean Huo > > ufshcd_get_max_pwr_mode() only need to be called once while booting, take > it out from ufshcd_probe_hba() and inline into ufshcd_device_params_init(). > > Reviewed-by: Stanley Chu > Signed-off-by: Bean Huo > --- Reviewed-by: Asutosh Das > drivers/scsi/ufs/ufshcd.c | 12 +++++++----- > 1 file changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c > index 935b50861864..5dfe760f2786 100644 > --- a/drivers/scsi/ufs/ufshcd.c > +++ b/drivers/scsi/ufs/ufshcd.c > @@ -6959,6 +6959,11 @@ static int ufshcd_device_params_init(struct ufs_hba *hba) > QUERY_FLAG_IDN_PWR_ON_WPE, &flag)) > hba->dev_info.f_power_on_wp_en = flag; > > + /* Probe maximum power mode co-supported by both UFS host and device */ > + if (ufshcd_get_max_pwr_mode(hba)) > + dev_err(hba->dev, > + "%s: Failed getting max supported power mode\n", > + __func__); > out: > return ret; > } > @@ -7057,11 +7062,8 @@ static int ufshcd_probe_hba(struct ufs_hba *hba, bool async) > ufshcd_force_reset_auto_bkops(hba); > hba->wlun_dev_clr_ua = true; > > - if (ufshcd_get_max_pwr_mode(hba)) { > - dev_err(hba->dev, > - "%s: Failed getting max supported power mode\n", > - __func__); > - } else { > + /* Gear up to HS gear if supported */ > + if (hba->max_pwr_info.is_valid) { > /* > * Set the right value to bRefClkFreq before attempting to > * switch to HS gears. > -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, Linux Foundation Collaborative Project