Linux IOMMU Development
 help / color / mirror / Atom feed
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

  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