On Tue, Aug 26, 2014 at 11:04:56AM -0600, Stephen Warren wrote: > On 08/26/2014 09:33 AM, Thierry Reding wrote: > >From: Thierry Reding <tred...@nvidia.com> > > > >Provide a new modifier to vsprintf() to print phys_addr_t variables to > >avoid having to cast or #ifdef when printing them out. The %pa modifier > >is used for this purpose, so phys_addr_t variables need to be passed by > >reference, like so: > > > > phys_addr_t start = 0; > > > > printf("start: %pa\n", &start); > > > >Depending on the size of phys_addr_t this will print out the address > >with 8 or 16 hexadecimal digits following a 0x prefix. > > The series, > > Tested-by: Stephen Warren <swar...@nvidia.com> > > Note that I did see the following printed a couple of times when I executed > "run bootcmd_pxe": > > pci_hose_bus_to_phys: invalid physical address > > ... but everything worked perfectly, so I guess we can track that down > later.
Yes, it should definitely be tracked down. I don't see that message on my setup. I've seen it for example when noncached_alloc() fails and returns 0, but in that case everything shouldn't be working perfectly. It would be helpful if that message showed what physical address was considered invalid. Thierry
pgp2jHHgKZFHP.pgp
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot