From: Dmitry Osipenko <digetx@gmail.com>
To: Thierry Reding <thierry.reding@gmail.com>
Cc: Jonathan Hunter <jonathanh@nvidia.com>,
Rob Herring <robh+dt@kernel.org>,
devicetree@vger.kernel.org, linux-tegra@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 11/15] memory: tegra: Add Tegra210 memory controller hot resets
Date: Sat, 28 Apr 2018 11:18:38 +0300 [thread overview]
Message-ID: <935ad30c-1ade-3380-ad4d-f5ffbbe63a66@gmail.com> (raw)
In-Reply-To: <20180427093927.GK30388@ulmo>
On 27.04.2018 12:39, Thierry Reding wrote:
> On Fri, Apr 13, 2018 at 02:33:50PM +0300, Dmitry Osipenko wrote:
>> From: Thierry Reding <treding@nvidia.com>
>>
>> Define the table of memory controller hot resets for Tegra210.
>>
>> Signed-off-by: Thierry Reding <treding@nvidia.com>
>> ---
>> drivers/memory/tegra/tegra210.c | 45 +++++++++++++++++++++++++++++++++
>> 1 file changed, 45 insertions(+)
>>
>> diff --git a/drivers/memory/tegra/tegra210.c b/drivers/memory/tegra/tegra210.c
>> index b729f49ffc8f..d00a77160407 100644
>> --- a/drivers/memory/tegra/tegra210.c
>> +++ b/drivers/memory/tegra/tegra210.c
>> @@ -1080,6 +1080,48 @@ static const struct tegra_smmu_soc tegra210_smmu_soc = {
>> .num_asids = 128,
>> };
>>
>> +#define TEGRA210_MC_RESET(_name, _control, _status, _bit) \
>> + { \
>> + .name = #_name, \
>> + .id = TEGRA210_MC_RESET_##_name, \
>> + .control = _control, \
>> + .status = _status, \
>> + .bit = _bit, \
>> + }
>> +
>> +static const struct tegra_mc_reset tegra210_mc_resets[] = {
>> + TEGRA210_MC_RESET(AFI, 0x200, 0x204, 0),
>> + TEGRA210_MC_RESET(AVPC, 0x200, 0x204, 1),
>> + TEGRA210_MC_RESET(DC, 0x200, 0x204, 2),
>> + TEGRA210_MC_RESET(DCB, 0x200, 0x204, 3),
>> + TEGRA210_MC_RESET(HC, 0x200, 0x204, 6),
>> + TEGRA210_MC_RESET(HDA, 0x200, 0x204, 7),
>> + TEGRA210_MC_RESET(ISP2, 0x200, 0x204, 8),
>> + TEGRA210_MC_RESET(MPCORE, 0x200, 0x204, 9),
>> + TEGRA210_MC_RESET(NVENC, 0x200, 0x204, 11),
>> + TEGRA210_MC_RESET(PPCS, 0x200, 0x204, 14),
>> + TEGRA210_MC_RESET(SATA, 0x200, 0x204, 15),
>> + TEGRA210_MC_RESET(VI, 0x200, 0x204, 17),
>> + TEGRA210_MC_RESET(VIC, 0x200, 0x204, 18),
>> + TEGRA210_MC_RESET(XUSB_HOST, 0x200, 0x204, 19),
>> + TEGRA210_MC_RESET(XUSB_DEV, 0x200, 0x204, 20),
>> + TEGRA210_MC_RESET(A9AVP, 0x200, 0x204, 21),
>> + TEGRA210_MC_RESET(TSEC, 0x200, 0x204, 22),
>> + TEGRA210_MC_RESET(SDMMC1, 0x200, 0x204, 29),
>> + TEGRA210_MC_RESET(SDMMC2, 0x200, 0x204, 30),
>> + TEGRA210_MC_RESET(SDMMC3, 0x200, 0x204, 31),
>> + TEGRA210_MC_RESET(SDMMC4, 0x970, 0x974, 0),
>> + TEGRA210_MC_RESET(ISP2B, 0x970, 0x974, 1),
>> + TEGRA210_MC_RESET(GPU, 0x970, 0x974, 2),
>> + TEGRA210_MC_RESET(NVDEC, 0x970, 0x974, 5),
>> + TEGRA210_MC_RESET(APE, 0x970, 0x974, 6),
>> + TEGRA210_MC_RESET(SE, 0x970, 0x974, 7),
>> + TEGRA210_MC_RESET(NVJPG, 0x970, 0x974, 8),
>> + TEGRA210_MC_RESET(AXIAP, 0x970, 0x974, 11),
>> + TEGRA210_MC_RESET(ETR, 0x970, 0x974, 12),
>> + TEGRA210_MC_RESET(TSECB, 0x970, 0x974, 13),
>> +};
>
> Isn't this missing an include for the definitions? There is an include
> for dt-bindings/memory/tegra20-mc.h for the Tegra20 driver, but none of
> the others have it.
Those drivers already have dt-bindings included.
> No need to respin, though, I can add that when applying.
next prev parent reply other threads:[~2018-04-28 8:18 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-09 19:28 [PATCH v4 00/15] Memory controller hot reset Dmitry Osipenko
2018-04-09 19:28 ` [PATCH v4 01/15] dt-bindings: arm: tegra: Remove duplicated Tegra30+ MC binding Dmitry Osipenko
2018-04-27 9:35 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 02/15] dt-bindings: memory: tegra: Document #reset-cells property of the Tegra30 MC Dmitry Osipenko
2018-04-27 9:35 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 03/15] dt-bindings: arm: tegra: Document #reset-cells property of the Tegra20 MC Dmitry Osipenko
2018-04-27 9:35 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 04/15] dt-bindings: memory: tegra: Add hot resets definitions Dmitry Osipenko
2018-04-27 9:36 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 05/15] memory: tegra: Do not handle spurious interrupts Dmitry Osipenko
2018-04-27 9:36 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 06/15] memory: tegra: Setup interrupts mask before requesting IRQ Dmitry Osipenko
2018-04-27 9:36 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 07/15] memory: tegra: Apply interrupts mask per SoC Dmitry Osipenko
2018-04-27 9:36 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 08/15] memory: tegra: Remove unused headers inclusions Dmitry Osipenko
2018-04-27 9:37 ` Thierry Reding
2018-04-09 19:28 ` [PATCH v4 09/15] memory: tegra: Squash tegra20-mc into common tegra-mc driver Dmitry Osipenko
2018-04-27 9:34 ` Thierry Reding
2018-04-27 10:13 ` Dmitry Osipenko
2018-04-27 10:24 ` Thierry Reding
2018-04-27 10:56 ` Dmitry Osipenko
2018-04-30 8:07 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 10/15] memory: tegra: Introduce memory client hot reset Dmitry Osipenko
2018-04-30 8:08 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 11/15] memory: tegra: Add Tegra210 memory controller hot resets Dmitry Osipenko
2018-04-27 9:39 ` Thierry Reding
2018-04-28 8:18 ` Dmitry Osipenko [this message]
2018-04-30 8:08 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 12/15] memory: tegra: Add Tegra124 " Dmitry Osipenko
2018-04-30 8:08 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 13/15] memory: tegra: Add Tegra114 " Dmitry Osipenko
2018-04-30 8:09 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 14/15] memory: tegra: Add Tegra30 " Dmitry Osipenko
2018-04-30 8:09 ` Thierry Reding
2018-04-13 11:33 ` [PATCH v4 15/15] memory: tegra: Add Tegra20 " Dmitry Osipenko
2018-04-30 8:09 ` Thierry Reding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=935ad30c-1ade-3380-ad4d-f5ffbbe63a66@gmail.com \
--to=digetx@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=jonathanh@nvidia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).