From: Laxman Dewangan <ldewangan@nvidia.com>
To: Vinod Koul <vinod.koul@intel.com>
Cc: "djbw@fb.com" <djbw@fb.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>
Subject: Re: [PATCH] dma: tegra: add support for Tegra114 SoC
Date: Sun, 6 Jan 2013 21:27:11 +0530 [thread overview]
Message-ID: <50E99ED7.4050406@nvidia.com> (raw)
In-Reply-To: <20130106143738.GC29209@intel.com>
On Sunday 06 January 2013 08:07 PM, Vinod Koul wrote:
> On Sat, Jan 05, 2013 at 05:36:08PM +0530, Laxman Dewangan wrote:
>> NVIDIA's Tegra114 has APB DMA controller which supports channel wise
>> pause control. The global pause is used for clock gating and hence
>> DMA registers are not accessible if DMAs are globally disabled.
>>
>> Add support for use of channel wise pause feature for Tegra114 SOCs.
> Looks fine to me. Only issue being that changelog tells me you are adding
> support for 114, but patch seems more of adding pause support and while at it
> also add entries for 114.
> Care to update changelog & title...
The channel wise pause come along with T114 and that's why it is
together in this change.
Probably I need to split the change into two patches: One for adding
feature of channel wise pause and other for adding support of T114.
This way it will not mixup the stuff and will much clear about change.
next prev parent reply other threads:[~2013-01-06 15:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-05 12:06 [PATCH] dma: tegra: add support for Tegra114 SoC Laxman Dewangan
2013-01-06 14:37 ` Vinod Koul
2013-01-06 15:57 ` Laxman Dewangan [this message]
2013-01-07 17:03 ` Stephen Warren
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=50E99ED7.4050406@nvidia.com \
--to=ldewangan@nvidia.com \
--cc=djbw@fb.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=vinod.koul@intel.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