Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935797AbcJUXPj (ORCPT ); Fri, 21 Oct 2016 19:15:39 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:43489 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934873AbcJUXPh (ORCPT ); Fri, 21 Oct 2016 19:15:37 -0400 DMARC-Filter: OpenDMARC Filter v1.3.1 smtp.codeaurora.org 51271615B1 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=pass smtp.mailfrom=spjoshi@codeaurora.org Subject: Re: [PATCH] remoteproc: Add support for xo clock To: Stephen Boyd References: <1476989724-7464-1-git-send-email-spjoshi@codeaurora.org> <20161020235401.GG26139@codeaurora.org> Cc: Ohad Ben-Cohen , Bjorn Andersson , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Trilok Soni From: Sarangdhar Joshi Message-ID: <24769e8c-d935-6386-5204-fe50dc1fe8ea@codeaurora.org> Date: Fri, 21 Oct 2016 16:15:35 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <20161020235401.GG26139@codeaurora.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1238 Lines: 44 On 10/20/2016 04:54 PM, Stephen Boyd wrote: > On 10/20, Sarangdhar Joshi wrote: >> Add xo clock support required for Qualcomm ADSP >> Peripheral Image Loader. > > Yes but why is xo needed? It is required to boot up the ADSP processor. The remoteproc driver keeps the xo clock enabled until the driver receives an "handover" interrupt in order to allow remote processor to vote for xo clock with rpm. I will update the commit text. > >> @@ -223,6 +232,17 @@ static irqreturn_t adsp_stop_ack_interrupt(int irq, void *dev) >> return IRQ_HANDLED; >> } >> >> +static int adsp_init_clock(struct qcom_adsp *adsp) >> +{ >> + adsp->xo = devm_clk_get(adsp->dev, "xo"); >> + if (IS_ERR(adsp->xo)) { >> + dev_err(adsp->dev, "failed to get xo clock"); > > What if it's a probe defer error? Probably best to just be > silent/debug level, or we need a specific test for EPROBE_DEFER > and then silence in that case. Sure, I will add a check for EPROBE_DEFER. Prefer driver to return meaningful error message in case of failure. > >> + return PTR_ERR(adsp->xo); >> + } > Thanks for reviewing the patch. Regards, Sarang -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project