From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Laxman Dewangan <ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH 1/5] ARM: DT: tegra114: add APB DMA controller DT entry
Date: Fri, 08 Mar 2013 10:51:08 -0700 [thread overview]
Message-ID: <513A250C.1000202@wwwdotorg.org> (raw)
In-Reply-To: <1362750782-15174-2-git-send-email-ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
On 03/08/2013 06:52 AM, Laxman Dewangan wrote:
> NVIDIA's Tegra114 has 32 channels APB DMA controller. Add DT entry for
> APB DMA controllers and make it compatible with "nvidia,tegra114-apbdma".
> diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
> + apbdma: dma {
> + compatible = "nvidia,tegra114-apbdma";
So I know that the Tegra114 HW has a new channel-pause feature, which
the driver /can/ use. However, if the driver didn't know about that
feature, and continued to use the global-pause feature, would it still
work fine?
In other words, is the Tegra114 HW 100% backwards-compatible with the
Tegra30 HW, it's just that there are new features that SW could
optionally use?
If that is true, then we should also include "nvidia,tegra30-apbdma" in
the compatible value.
WARNING: multiple messages have this Message-ID (diff)
From: swarren@wwwdotorg.org (Stephen Warren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/5] ARM: DT: tegra114: add APB DMA controller DT entry
Date: Fri, 08 Mar 2013 10:51:08 -0700 [thread overview]
Message-ID: <513A250C.1000202@wwwdotorg.org> (raw)
In-Reply-To: <1362750782-15174-2-git-send-email-ldewangan@nvidia.com>
On 03/08/2013 06:52 AM, Laxman Dewangan wrote:
> NVIDIA's Tegra114 has 32 channels APB DMA controller. Add DT entry for
> APB DMA controllers and make it compatible with "nvidia,tegra114-apbdma".
> diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
> + apbdma: dma {
> + compatible = "nvidia,tegra114-apbdma";
So I know that the Tegra114 HW has a new channel-pause feature, which
the driver /can/ use. However, if the driver didn't know about that
feature, and continued to use the global-pause feature, would it still
work fine?
In other words, is the Tegra114 HW 100% backwards-compatible with the
Tegra30 HW, it's just that there are new features that SW could
optionally use?
If that is true, then we should also include "nvidia,tegra30-apbdma" in
the compatible value.
WARNING: multiple messages have this Message-ID (diff)
From: Stephen Warren <swarren@wwwdotorg.org>
To: Laxman Dewangan <ldewangan@nvidia.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org,
pdeschrijver@nvidia.com
Subject: Re: [PATCH 1/5] ARM: DT: tegra114: add APB DMA controller DT entry
Date: Fri, 08 Mar 2013 10:51:08 -0700 [thread overview]
Message-ID: <513A250C.1000202@wwwdotorg.org> (raw)
In-Reply-To: <1362750782-15174-2-git-send-email-ldewangan@nvidia.com>
On 03/08/2013 06:52 AM, Laxman Dewangan wrote:
> NVIDIA's Tegra114 has 32 channels APB DMA controller. Add DT entry for
> APB DMA controllers and make it compatible with "nvidia,tegra114-apbdma".
> diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi
> + apbdma: dma {
> + compatible = "nvidia,tegra114-apbdma";
So I know that the Tegra114 HW has a new channel-pause feature, which
the driver /can/ use. However, if the driver didn't know about that
feature, and continued to use the global-pause feature, would it still
work fine?
In other words, is the Tegra114 HW 100% backwards-compatible with the
Tegra30 HW, it's just that there are new features that SW could
optionally use?
If that is true, then we should also include "nvidia,tegra30-apbdma" in
the compatible value.
next prev parent reply other threads:[~2013-03-08 17:51 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-08 13:52 [PATCH 0/5] ARM: DT: tegra114: Add DT entry for different controller Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
2013-03-08 13:52 ` [PATCH 1/5] ARM: DT: tegra114: add APB DMA controller DT entry Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
[not found] ` <1362750782-15174-2-git-send-email-ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 17:51 ` Stephen Warren [this message]
2013-03-08 17:51 ` Stephen Warren
2013-03-08 17:51 ` Stephen Warren
2013-03-08 18:06 ` Laxman Dewangan
2013-03-08 18:06 ` Laxman Dewangan
[not found] ` <513A2888.5020402-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 18:41 ` Stephen Warren
2013-03-08 18:41 ` Stephen Warren
2013-03-08 18:41 ` Stephen Warren
[not found] ` <513A30D3.4020700-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-03-08 18:46 ` Laxman Dewangan
2013-03-08 18:46 ` Laxman Dewangan
2013-03-08 18:46 ` Laxman Dewangan
2013-03-08 13:52 ` [PATCH 2/5] ARM: DT: tegra114: Add i2c " Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
2013-03-08 13:52 ` Laxman Dewangan
2013-03-08 13:53 ` [PATCH 3/5] ARM: DT: tegra114:add aliases and DMA requestor for serial controller Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
[not found] ` <1362750782-15174-4-git-send-email-ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 17:54 ` Stephen Warren
2013-03-08 17:54 ` Stephen Warren
2013-03-08 17:54 ` Stephen Warren
[not found] ` <513A25E4.8040504-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-03-08 18:04 ` Laxman Dewangan
2013-03-08 18:04 ` Laxman Dewangan
2013-03-08 18:04 ` Laxman Dewangan
[not found] ` <513A281B.9010702-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 18:40 ` Stephen Warren
2013-03-08 18:40 ` Stephen Warren
2013-03-08 18:40 ` Stephen Warren
[not found] ` <1362750782-15174-1-git-send-email-ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 13:53 ` [PATCH 4/5] ARM: DT: tegra114: add KBC controller DT entry Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
[not found] ` <1362750782-15174-5-git-send-email-ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 18:04 ` Stephen Warren
2013-03-08 18:04 ` Stephen Warren
2013-03-08 18:04 ` Stephen Warren
2013-03-08 18:14 ` Laxman Dewangan
2013-03-08 18:14 ` Laxman Dewangan
[not found] ` <513A2A6D.7080604-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-03-08 18:42 ` Stephen Warren
2013-03-08 18:42 ` Stephen Warren
2013-03-08 18:42 ` Stephen Warren
2013-03-08 13:53 ` [PATCH 5/5] ARM: DT: tegra114: Add spi " Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
2013-03-08 13:53 ` Laxman Dewangan
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=513A250C.1000202@wwwdotorg.org \
--to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
--cc=ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.