From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH v2] remoteproc: qcom: Add venus rproc support on msm8996 platform. Date: Mon, 28 Nov 2016 17:36:47 -0800 Message-ID: <20161129013647.GX6095@codeaurora.org> References: <1480015122-27717-1-git-send-email-akdwived@codeaurora.org> <1480015122-27717-2-git-send-email-akdwived@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:59124 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750754AbcK2Bgt (ORCPT ); Mon, 28 Nov 2016 20:36:49 -0500 Content-Disposition: inline In-Reply-To: <1480015122-27717-2-git-send-email-akdwived@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Avaneesh Kumar Dwivedi Cc: bjorn.andersson@linaro.org, agross@codeaurora.org, linux-arm-msm@vger.kernel.org On 11/25, Avaneesh Kumar Dwivedi wrote: > This patch is based on > https://patchwork.kernel.org/patch/9415627/ > https://patchwork.kernel.org/patch/9415651/ > > This patch add clock initialization, enable and disable support. > Required resource name string and rating are differentiated based > on compatible string. Also added documentation for venus pil on > msm8996. > > Signed-off-by: Avaneesh Kumar Dwivedi > --- Why isn't Stan Cced on this patch? > .../devicetree/bindings/remoteproc/qcom,venus.txt | 3 +- > drivers/remoteproc/qcom_venus_pil.c | 115 ++++++++++++++++++++- > 2 files changed, 116 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,venus.txt b/Documentation/devicetree/bindings/remoteproc/qcom,venus.txt > index 2d73ba1..c986f52 100644 > --- a/Documentation/devicetree/bindings/remoteproc/qcom,venus.txt > +++ b/Documentation/devicetree/bindings/remoteproc/qcom,venus.txt > @@ -6,7 +6,8 @@ on the Qualcomm Venus remote processor core. > - compatible: > Usage: required > Value type: > - Definition: must contain "qcom,venus-pil" > + Definition: must contain "qcom,venus-pil" or > + "qcom,venus-8996-pil" > > - memory-region: > Usage: required No addition of clocks or clock-names properties in this document? > diff --git a/drivers/remoteproc/qcom_venus_pil.c b/drivers/remoteproc/qcom_venus_pil.c > index 6d4e55b..23b7e99 100644 > --- a/drivers/remoteproc/qcom_venus_pil.c > +++ b/drivers/remoteproc/qcom_venus_pil.c > @@ -194,8 +296,19 @@ static int venus_remove(struct platform_device *pdev) > return 0; > } > > +static const struct venus_rproc_res venus_8996_res = { > + .venus_clks = (char*[]){"mx", "cx", "pll", NULL}, > + .venus_clk_rate = {19200000, 19200000, 19200000, 80000000}, I'm very much lost. I don't really understand why we're adding clock control here. Perhaps that is the responsibility of the video driver itself and isn't supposed to be in the remoteproc portion? Testing things standalone without the video driver seems like a unit test, which is not too useful of a test if it doesn't mirror real use cases. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project