Re: [U-Boot] [PATCH v3 01/30] vsprintf: Add modifier for phys_addr_t

2014-11-17 Thread Thierry Reding
Hi Simon, Thanks for taking this over. Besides the three small nitpicks (which aren't really blockers, so feel free to ignore) this looks like what I would've done had I respun the series. Thierry pgp5d5C1G4rUn.pgp Description: PGP signature ___

Re: [U-Boot] [PATCH v3 01/30] vsprintf: Add modifier for phys_addr_t

2014-11-17 Thread Simon Glass
On 17 November 2014 10:18, Thierry Reding tred...@nvidia.com wrote: Hi Simon, Thanks for taking this over. Besides the three small nitpicks (which aren't really blockers, so feel free to ignore) this looks like what I would've done had I respun the series. Great - thanks for going through

[U-Boot] [PATCH v3 01/30] vsprintf: Add modifier for phys_addr_t

2014-11-12 Thread Simon Glass
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