* [PATCH] soc/tegra: pmc: update Tegra234 reset sources
@ 2022-03-30 6:36 Sandipan Patra
2022-03-30 9:16 ` Andy Shevchenko
0 siblings, 1 reply; 3+ messages in thread
From: Sandipan Patra @ 2022-03-30 6:36 UTC (permalink / raw)
To: treding, jonathanh, digetx, ulf.hansson, andriy.shevchenko, cai.huoqing
Cc: bbasu, linux-tegra, linux-kernel, Sandipan Patra
Reset_sources list is updated to add all reset sources
and removing ones that do not actually exist.
Signed-off-by: Sandipan Patra <spatra@nvidia.com>
---
drivers/soc/tegra/pmc.c | 33 +++++++++++++++++++++++++--------
1 file changed, 25 insertions(+), 8 deletions(-)
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index fdf508e03400..d9129b940e3b 100644
--- a/drivers/soc/tegra/pmc.c
+++ b/drivers/soc/tegra/pmc.c
@@ -3766,7 +3766,7 @@ static const struct tegra_pmc_regs tegra234_pmc_regs = {
};
static const char * const tegra234_reset_sources[] = {
- "SYS_RESET_N",
+ "SYS_RESET_N", /*0*/
"AOWDT",
"BCCPLEXWDT",
"BPMPWDT",
@@ -3775,18 +3775,35 @@ static const char * const tegra234_reset_sources[] = {
"APEWDT",
"LCCPLEXWDT",
"SENSOR",
- "AOTAG",
- "VFSENSOR",
+ NULL,
+ NULL, /*10*/
"MAINSWRST",
"SC7",
"HSM",
- "CSITE",
+ NULL,
"RCEWDT",
- "PVA0WDT",
- "PVA1WDT",
- "L1A_ASYNC",
+ NULL,
+ NULL,
+ NULL,
"BPMPBOOT",
- "FUSECRC",
+ "FUSECRC", /*20*/
+ "DCEWDT",
+ "PSCWDT",
+ "PSC",
+ "CSITE_SW",
+ "POD",
+ "SCPM",
+ "VREFRO_POWERBAD",
+ "VMON",
+ "FMON",
+ "FSI_R5WDT", /*30*/
+ "FSI_THERM",
+ "FSI_R52C0WDT",
+ "FSI_R52C1WDT",
+ "FSI_R52C2WDT",
+ "FSI_R52C3WDT",
+ "FSI_FMON",
+ "FSI_VMON" /*37*/
};
static const struct tegra_wake_event tegra234_wake_events[] = {
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] soc/tegra: pmc: update Tegra234 reset sources
2022-03-30 6:36 [PATCH] soc/tegra: pmc: update Tegra234 reset sources Sandipan Patra
@ 2022-03-30 9:16 ` Andy Shevchenko
2022-04-01 10:12 ` Sandipan Patra
0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2022-03-30 9:16 UTC (permalink / raw)
To: Sandipan Patra
Cc: treding, jonathanh, digetx, ulf.hansson, cai.huoqing, bbasu,
linux-tegra, linux-kernel
On Wed, Mar 30, 2022 at 12:06:35PM +0530, Sandipan Patra wrote:
> Reset_sources list is updated to add all reset sources
> and removing ones that do not actually exist.
...
> + "SYS_RESET_N", /*0*/
Missed spaces in the comment here and everywhere else.
Not sure about indices. If it's going to be a part of the hardware programming
interface it may make sense to use hexadecimal rather than decimal.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH] soc/tegra: pmc: update Tegra234 reset sources
2022-03-30 9:16 ` Andy Shevchenko
@ 2022-04-01 10:12 ` Sandipan Patra
0 siblings, 0 replies; 3+ messages in thread
From: Sandipan Patra @ 2022-04-01 10:12 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Thierry Reding, Jonathan Hunter, digetx, ulf.hansson,
cai.huoqing, Bibek Basu, linux-tegra, linux-kernel
Thanks Andy for reviewing the patch.
> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Sent: Wednesday, March 30, 2022 2:46 PM
> To: Sandipan Patra <spatra@nvidia.com>
> Cc: Thierry Reding <treding@nvidia.com>; Jonathan Hunter
> <jonathanh@nvidia.com>; digetx@gmail.com; ulf.hansson@linaro.org;
> cai.huoqing@linux.dev; Bibek Basu <bbasu@nvidia.com>; linux-
> tegra@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] soc/tegra: pmc: update Tegra234 reset sources
>
> External email: Use caution opening links or attachments
>
>
> On Wed, Mar 30, 2022 at 12:06:35PM +0530, Sandipan Patra wrote:
> > Reset_sources list is updated to add all reset sources and removing
> > ones that do not actually exist.
>
> ...
>
> > + "SYS_RESET_N", /*0*/
>
> Missed spaces in the comment here and everywhere else.
ACK
>
> Not sure about indices. If it's going to be a part of the hardware
> programming interface it may make sense to use hexadecimal rather than
> decimal.
Agree. That's a better suggestion for readability too, I will modify it to have hexadecimal in next patch.
Thanks & Regards,
Sandipan
>
> --
> With Best Regards,
> Andy Shevchenko
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-04-01 10:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-30 6:36 [PATCH] soc/tegra: pmc: update Tegra234 reset sources Sandipan Patra
2022-03-30 9:16 ` Andy Shevchenko
2022-04-01 10:12 ` Sandipan Patra
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).