Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1506350

Re: [PATCH] remoteproc: Add support for xo clock

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] remoteproc: Add support for xo clock Sarangdhar Joshi <spjoshi@codeaurora.org> - 2016-10-20 21:00 +0200
  Re: [PATCH] remoteproc: Add support for xo clock Stephen Boyd <sboyd@codeaurora.org> - 2016-10-21 02:00 +0200
    Re: [PATCH] remoteproc: Add support for xo clock Sarangdhar Joshi <spjoshi@codeaurora.org> - 2016-10-22 01:20 +0200

csiph-web