Messages in this thread |  | | Date | Tue, 21 Aug 2018 15:17:40 -0700 | From | Bjorn Andersson <> | Subject | Re: [PATCH 2/4] reset: qcom: PDC (Power Domain Controller) reset controller |
| |
On Fri 27 Jul 08:28 PDT 2018, Sibi Sankar wrote:
> Add reset controller for SDM845 SoC to control reset signals > provided by PDC for Modem, Compute, Display, GPU, Debug, AOP, > Sensors, Audio, SP and APPS > > Signed-off-by: Sibi Sankar <sibis@codeaurora.org> > --- > drivers/reset/Kconfig | 9 +++ > drivers/reset/Makefile | 1 + > drivers/reset/reset-qcom-pdc.c | 139 +++++++++++++++++++++++++++++++++ > 3 files changed, 149 insertions(+) > create mode 100644 drivers/reset/reset-qcom-pdc.c > > diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig > index 13d28fdbdbb5..5344e202a630 100644 > --- a/drivers/reset/Kconfig > +++ b/drivers/reset/Kconfig > @@ -98,6 +98,15 @@ config RESET_QCOM_AOSS > reset signals provided by AOSS for Modem, Venus, ADSP, > GPU, Camera, Wireless, Display subsystem. Otherwise, say N. > > +config RESET_QCOM_PDC > + bool "Qcom PDC Reset Driver"
"Qualcomm"
And I do not see a strong reason for this not being tristate, the consumers we've talked about so far can all be modules.
> + depends on ARCH_QCOM || COMPILE_TEST > + help > + This enables the PDC (Power Domain Controller) reset driver > + for Qualcomm Technologies Inc SDM845 SoCs. Say Y if you want > + to control reset signals provided by PDC for Modem, Compute, > + Display, GPU, Debug, AOP, Sensors, Audio, SP and APPS. > +
Implementation looks fine.
Regards, Bjorn
|  |