From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S943640AbdEZNUh (ORCPT ); Fri, 26 May 2017 09:20:37 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:48404 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933335AbdEZNUc (ORCPT ); Fri, 26 May 2017 09:20:32 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 783A66146D Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=akdwived@codeaurora.org Subject: Re: [RESEND: PATCH v4 4/4] remoteproc: qcom: Add support for mss boot on msm8996 To: Bjorn Andersson Cc: sboyd@codeaurora.org, agross@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-remoteproc@vger.kernel.org References: <1494957722-13264-1-git-send-email-akdwived@codeaurora.org> <1494957722-13264-5-git-send-email-akdwived@codeaurora.org> <20170526060928.GH12920@tuxbook> From: "Dwivedi, Avaneesh Kumar (avani)" Message-ID: Date: Fri, 26 May 2017 18:50:27 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 In-Reply-To: <20170526060928.GH12920@tuxbook> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/26/2017 11:39 AM, Bjorn Andersson wrote: > On Tue 16 May 11:02 PDT 2017, Avaneesh Kumar Dwivedi wrote: > >> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt b/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt >> index 92347fe..f9dfb6c 100644 >> --- a/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt >> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,q6v5.txt >> @@ -9,8 +9,8 @@ on the Qualcomm Hexagon core. >> Definition: must be one of: >> "qcom,q6v5-pil", >> "qcom,msm8916-mss-pil", >> - "qcom,msm8974-mss-pil" >> - >> + "qcom,msm8974-mss-pil", >> + "qcom,msm8996-mss-pil" > Indentation please. OK. > >> - reg: >> Usage: required >> Value type: >> diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c > [..] >> +/* QDSP6v56 parameters */ >> +#define QDSP6v56_LDO_BYP BIT(25) >> +#define QDSP6v56_BHS_ON BIT(24) >> +#define QDSP6v56_CLAMP_WL BIT(21) >> +#define QDSP6v56_CLAMP_QMC_MEM BIT(22) >> +#define HALT_CHECK_MAX_LOOPS 200 >> +#define QDSP6SS_XO_CBCR 0x0038 >> +#define QDSP6SS_ACC_OVERRIDE_VAL 0x20 > Please keep the blank line between the defines and the struct > definition. OK. > >> struct reg_info { >> struct regulator *reg; >> int uV; > [..] >> >> +static const struct rproc_hexagon_res msm8996_mss = { >> + .hexagon_mba_image = "mba.mbn", >> + .proxy_supply = (struct qcom_mss_reg_res[]) { >> + {} >> + }, >> + .active_supply = (struct qcom_mss_reg_res[]) { >> + {}, >> + {} >> + }, > It's possible to just leave .proxy_supply and .active_supply out (i.e. > leaving them as NULL), as I made q6v5_regulator_init() handle this > gracefully a while back. OK. > > Regards, > Bjorn -- Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.