linux-tegra.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Joerg Roedel <joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
Cc: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	Alexandre Courbot
	<gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Hiroshi Doyu <hdoyu-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH 1/4] iommu/tegra-smmu: Add debugfs support
Date: Fri, 27 Mar 2015 11:15:41 +0100	[thread overview]
Message-ID: <20150327101539.GA17976@ulmo.nvidia.com> (raw)
In-Reply-To: <1427450847-17719-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 1686 bytes --]

On Fri, Mar 27, 2015 at 11:07:24AM +0100, Thierry Reding wrote:
> From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> Provide clients and swgroups files in debugfs. These files show for
> which clients IOMMU translation is enabled and which ASID is associated
> with each SWGROUP.
> 
> Cc: Hiroshi Doyu <hdoyu-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  drivers/iommu/tegra-smmu.c | 109 +++++++++++++++++++++++++++++++++++++++++++++
>  include/soc/tegra/mc.h     |   5 +++
>  2 files changed, 114 insertions(+)

Hi Joerg,

I forgot to mention that this patch depends on another patch that I plan
to take through the Tegra tree. That patch adds the group names that are
used by the debugfs support introduced here.

There are a couple of ways to solve this: omit this patch for v4.1 and
merge it for v4.2, at which point the dependency will have been merged.
That's obviously the simplest one and given that this is debugfs support
it isn't very critical, just nice to have.

Another possibility would be for you to ack this patch so that I can
take it into the same branch as the dependency. The remainder of this
series doesn't depend on either of the two patches, so they should be
fine for you to take into your tree.

Yet another alternative would be for me to provide a stable branch that
can be merged into the Tegra and IOMMU trees, but quite frankly I don't
think it's worth jumping through hoops like that for this particular
patch. I'd prefer either the first or second alternative, whichever
suits you best.

Thanks,
Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2015-03-27 10:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-27 10:07 [PATCH 1/4] iommu/tegra-smmu: Add debugfs support Thierry Reding
     [not found] ` <1427450847-17719-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-03-27 10:07   ` [PATCH 2/4] iommu/tegra: Setup aperture Thierry Reding
     [not found]     ` <1427450847-17719-2-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-03-27 10:19       ` Thierry Reding
     [not found]         ` <20150327101935.GB17976-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-31 13:35           ` Joerg Roedel
2015-03-27 10:07   ` [PATCH 3/4] iommu/tegra: gart: Set aperture at domain initialization time Thierry Reding
2015-03-27 10:07   ` [PATCH 4/4] iommu/tegra: smmu: Compute PFN mask at runtime Thierry Reding
2015-03-27 10:15   ` Thierry Reding [this message]
     [not found]     ` <20150327101539.GA17976-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-31 15:02       ` [PATCH 1/4] iommu/tegra-smmu: Add debugfs support Joerg Roedel
     [not found]         ` <20150331150214.GE22683-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-04-14 11:07           ` 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=20150327101539.GA17976@ulmo.nvidia.com \
    --to=thierry.reding-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=hdoyu-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
    --cc=joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@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 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).