* [Patch V2] soc/tegra: pmc: update Tegra234 reset sources
@ 2022-04-01 10:49 Sandipan Patra
2022-04-01 13:44 ` Andy Shevchenko
0 siblings, 1 reply; 3+ messages in thread
From: Sandipan Patra @ 2022-04-01 10:49 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>
---
Update on V2 patch:
space inside comment and
Changed decimal to hexadecimal notation in the comments.
drivers/soc/tegra/pmc.c | 31 ++++++++++++++++++++++++-------
1 file changed, 24 insertions(+), 7 deletions(-)
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index fdf508e03400..3799669813e2 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", /* 0x0 */
"AOWDT",
"BCCPLEXWDT",
"BPMPWDT",
@@ -3775,18 +3775,35 @@ static const char * const tegra234_reset_sources[] = {
"APEWDT",
"LCCPLEXWDT",
"SENSOR",
- "AOTAG",
- "VFSENSOR",
+ NULL,
+ NULL,
"MAINSWRST",
"SC7",
"HSM",
- "CSITE",
+ NULL,
"RCEWDT",
- "PVA0WDT",
- "PVA1WDT",
- "L1A_ASYNC",
+ NULL, /* 0x10 */
+ NULL,
+ NULL,
"BPMPBOOT",
"FUSECRC",
+ "DCEWDT",
+ "PSCWDT",
+ "PSC",
+ "CSITE_SW",
+ "POD",
+ "SCPM",
+ "VREFRO_POWERBAD",
+ "VMON",
+ "FMON",
+ "FSI_R5WDT",
+ "FSI_THERM",
+ "FSI_R52C0WDT", /* 0x20 */
+ "FSI_R52C1WDT",
+ "FSI_R52C2WDT",
+ "FSI_R52C3WDT",
+ "FSI_FMON",
+ "FSI_VMON" /* 0x25 */
};
static const struct tegra_wake_event tegra234_wake_events[] = {
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Patch V2] soc/tegra: pmc: update Tegra234 reset sources
2022-04-01 10:49 [Patch V2] soc/tegra: pmc: update Tegra234 reset sources Sandipan Patra
@ 2022-04-01 13:44 ` Andy Shevchenko
2022-04-01 14:33 ` Sandipan Patra
0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2022-04-01 13:44 UTC (permalink / raw)
To: Sandipan Patra
Cc: treding, jonathanh, digetx, ulf.hansson, cai.huoqing, bbasu,
linux-tegra, linux-kernel
On Fri, Apr 01, 2022 at 04:19:35PM +0530, Sandipan Patra wrote:
> Reset_sources list is updated to add all reset sources
> and removing ones that do not actually exist.
...
> + NULL, /* 0x10 */
I would suggest to add the comments more often, like every 8 entries.
...
> + "FSI_VMON" /* 0x25 */
It would be good to have comma here as if this ever getting expanded
the additional churn may be avoided.
> };
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [Patch V2] soc/tegra: pmc: update Tegra234 reset sources
2022-04-01 13:44 ` Andy Shevchenko
@ 2022-04-01 14:33 ` Sandipan Patra
0 siblings, 0 replies; 3+ messages in thread
From: Sandipan Patra @ 2022-04-01 14:33 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Thierry Reding, Jonathan Hunter, digetx, ulf.hansson,
cai.huoqing, Bibek Basu, linux-tegra, linux-kernel
Thank you Andy.
Pushed V3 patch addressing recent comments.
Please help to review.
Thanks & Regards,
Sandipan
> -----Original Message-----
> From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Sent: Friday, April 1, 2022 7:14 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 V2] soc/tegra: pmc: update Tegra234 reset sources
>
> External email: Use caution opening links or attachments
>
>
> On Fri, Apr 01, 2022 at 04:19:35PM +0530, Sandipan Patra wrote:
> > Reset_sources list is updated to add all reset sources and removing
> > ones that do not actually exist.
>
> ...
>
> > + NULL, /* 0x10 */
>
> I would suggest to add the comments more often, like every 8 entries.
>
> ...
>
> > + "FSI_VMON" /* 0x25 */
>
> It would be good to have comma here as if this ever getting expanded the
> additional churn may be avoided.
>
> > };
>
> --
> With Best Regards,
> Andy Shevchenko
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-04-01 14:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-01 10:49 [Patch V2] soc/tegra: pmc: update Tegra234 reset sources Sandipan Patra
2022-04-01 13:44 ` Andy Shevchenko
2022-04-01 14:33 ` 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).