From: Joerg Roedel <jroedel-l3A5Bk7waGM@public.gmane.org>
To: Emil Medve <Emilian.Medve-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org>
Cc: Sethi Varun-B16395
<Varun.Sethi-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org>,
iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org
Subject: Re: [PATCH 4/4] iommu/fsl: Use %pa to print phys_addr_t
Date: Tue, 27 Jan 2015 12:08:05 +0100 [thread overview]
Message-ID: <20150127110805.GC2312@suse.de> (raw)
In-Reply-To: <54C76B23.3010809-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org>
On Tue, Jan 27, 2015 at 04:40:35AM -0600, Emil Medve wrote:
> Hello Joerg,
>
>
> On 01/26/2015 05:50 PM, Joerg Roedel wrote:
> > From: Joerg Roedel <jroedel-l3A5Bk7waGM-XMD5yJDbdMReXY1tMh2IBg@public.gmane.org>
> >
> > Fix two compile warnings.
> >
> > Signed-off-by: Joerg Roedel <jroedel-l3A5Bk7waGM-XMD5yJDbdMReXY1tMh2IBg@public.gmane.org>
> > ---
> > drivers/iommu/fsl_pamu.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/iommu/fsl_pamu.c b/drivers/iommu/fsl_pamu.c
> > index 80ac68d..e63904b 100644
> > --- a/drivers/iommu/fsl_pamu.c
> > +++ b/drivers/iommu/fsl_pamu.c
> > @@ -352,7 +352,7 @@ int pamu_config_ppaace(int liodn, phys_addr_t win_addr, phys_addr_t win_size,
> > unsigned long fspi;
> >
> > if ((win_size & (win_size - 1)) || win_size < PAMU_PAGE_SIZE) {
> > - pr_debug("window size too small or not a power of two %llx\n", win_size);
> > + pr_debug("window size too small or not a power of two %pa\n", &win_size);
> > return -EINVAL;
> > }
>
> Somewhere around line 1124 there another opportunity for this fix:
Thanks, I updated the patch.
Joerg
next prev parent reply other threads:[~2015-01-27 11:08 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-26 23:50 [PATCH 0/4] iommu: Small fixes and cleanups Joerg Roedel
2015-01-26 23:50 ` [PATCH 1/4] iommu: Make more drivers depend on COMPILE_TEST Joerg Roedel
[not found] ` <1422316230-18893-1-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-01-26 23:50 ` [PATCH 2/4] iommu/exynos: Make driver depend on REGULATOR Joerg Roedel
[not found] ` <1422316230-18893-3-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-02-03 14:37 ` Marek Szyprowski
2015-02-04 15:25 ` Joerg Roedel
2015-01-26 23:50 ` [PATCH 3/4] iommu/omap: Print phys_addr_t using %pa Joerg Roedel
2015-01-26 23:50 ` [PATCH 4/4] iommu/fsl: Use %pa to print phys_addr_t Joerg Roedel
[not found] ` <1422316230-18893-5-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-01-27 10:40 ` Emil Medve
[not found] ` <54C76B23.3010809-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org>
2015-01-27 11:08 ` Joerg Roedel [this message]
2015-01-30 13:30 ` [PATCH 0/4] iommu: Small fixes and cleanups Joerg Roedel
2015-01-27 7:14 ` Laurent Pinchart
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=20150127110805.GC2312@suse.de \
--to=jroedel-l3a5bk7wagm@public.gmane.org \
--cc=Emilian.Medve-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org \
--cc=Varun.Sethi-eDlz3WWmN0ll57MIdRCFDg@public.gmane.org \
--cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@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