Re: [PATCH v3 0/6] Add support for PDC Global on SDM845 SoCs

From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Sibi Sankar <sibis@codeaurora.org>, p.zabel@pengutronix.de
Cc: mka@chromium.org, robh+dt@kernel.org,
	linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	mark.rutland@arm.com, linux-arm-msm@vger.kernel.org,
	tsoni@codeaurora.org
Subject: Re: [PATCH v3 0/6] Add support for PDC Global on SDM845 SoCs
Date: Mon, 3 Sep 2018 12:36:47 -0700	[thread overview]
Message-ID: <20180903193647.GJ3456@tuxbook-pro> (raw)
In-Reply-To: <20180829191215.10071-1-sibis@codeaurora.org>

On Wed 29 Aug 12:12 PDT 2018, Sibi Sankar wrote:

> This patch series add support for PDC Global (Power Domain Controller)
> on SDM845 SoCs and adds pdc reset lines assert/deassert to remoteproc
> Q6v5 modem-pil. The first two patches adds PDC Global reset driver to
> control reset signals of Modem, Compute, Display, GPU, Debug, AOP,
> Sensors, Audio, SP and APPS. The last four patches (cleans up)/adds pdc
> reset lines to q6v5 bindings and asserts/deasserts in modem start/stop
> path.
> 
> V3:
>   refactored pdc reset driver to remove unused layer of indirection
>   as suggested by Matthias
>   Other minor fixes suggested by Matthias/Bjorn
> 
> V2:
>   Incorporated Philipp/Bjorn/Rob suggestions
>   Renamed reset binding to pdc-global.txt
>   replaced offset with #define of register name
>   replaced with devm_reset_control_get_exclusive()
>   Separted dt binding from the drivers
> 
> Sibi Sankar (6):
>   dt-bindings: reset: Add PDC Global binding for SDM845 SoCs
>   reset: qcom: PDC Global (Power Domain Controller) reset controller

Philipp, there's no compile time dependencies between the PDC and
remoteproc patches in this series. Will you take these two through your
tree and I'll take the remaining four through the remoteproc tree?

Regards,
Bjorn

>   dt-bindings: remoteproc: qcom: Remove additional definition tag
>   dt-bindings: remoteproc: Add PDC reset binding for Q6V5 PIL
>   remoteproc: qcom: q6v5-pil: Explicitly get mss_restart line
>   remoteproc: qcom: q6v5-pil: Add PDC reset for modem on SDM845 SoCs
> 
>  .../bindings/remoteproc/qcom,q6v5.txt         |   8 +-
>  .../bindings/reset/qcom,pdc-global.txt        |  52 ++++++++
>  drivers/remoteproc/qcom_q6v5_pil.c            |  31 ++++-
>  drivers/reset/Kconfig                         |   9 ++
>  drivers/reset/Makefile                        |   1 +
>  drivers/reset/reset-qcom-pdc.c                | 124 ++++++++++++++++++
>  include/dt-bindings/reset/qcom,sdm845-pdc.h   |  20 +++
>  7 files changed, 237 insertions(+), 8 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/reset/qcom,pdc-global.txt
>  create mode 100644 drivers/reset/reset-qcom-pdc.c
>  create mode 100644 include/dt-bindings/reset/qcom,sdm845-pdc.h
> 
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
> 