public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] disk: part_efi: fix **pgpt_pte == NULL
Date: Wed, 19 Oct 2011 16:29:57 -0400	[thread overview]
Message-ID: <201110191629.59336.vapier@gentoo.org> (raw)
In-Reply-To: <CAD=FV=XH9AjYc4MpdM_=f7vH=iGEvhSo6RT7HF4MravBZ1k_TQ@mail.gmail.com>

On Wednesday 19 October 2011 15:59:27 Doug Anderson wrote:
> On Wed, Oct 19, 2011 at 12:56 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > %#p gives you the prefix under u-boot, but gcc warns about it since it
> > assumes
> > that %p always includes the 0x prefix :).
> 
> Ah, makes sense.  I'm just going to leave it as %p without the 0x.  That
> sound OK?  I assume that the patch looks OK to you now?

yeah, that's fine.  i think we'll want to update u-boot's printf implementation 
to prefix 0x with %p to match what everyone else is doing.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111019/14ab6d1c/attachment.pgp 

  reply	other threads:[~2011-10-19 20:29 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-19 18:04 [U-Boot] [PATCH 1/2] cosmetic: Replace __FUNCTION__ with __func__ in part_efi.c Doug Anderson
2011-10-19 18:04 ` [U-Boot] [PATCH 2/2] disk: part_efi: fix **pgpt_pte == NULL Doug Anderson
2011-10-19 18:21   ` Mike Frysinger
2011-10-19 19:47     ` Doug Anderson
2011-10-19 19:56       ` Mike Frysinger
2011-10-19 19:59         ` Doug Anderson
2011-10-19 20:29           ` Mike Frysinger [this message]
2011-10-27 21:50       ` Wolfgang Denk
2011-10-19 19:33   ` Richard Retanubun
2011-10-19 18:19 ` [U-Boot] [PATCH 1/2] cosmetic: Replace __FUNCTION__ with __func__ in part_efi.c Mike Frysinger
2011-10-19 19:24 ` Richard Retanubun
2011-10-27 21:46 ` Wolfgang Denk

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=201110191629.59336.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=u-boot@lists.denx.de \
    /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