Hi, On 16 July 2015 at 01:01, Albert ARIBAUD <[email protected]> wrote: > Hello York, > > On Wed, 15 Jul 2015 13:26:46 -0700, York Sun <[email protected]> > wrote: >> fdt_addr_t and fdt_size_t can be either 64-bit or 32-bit, depending >> on the architecture. Change the type to phys_addr_t and phys_size_t. > > If the only reason for changing the type is to avoid warnings in > debug() then I don't think this is the right approach (see my message > re PRIu64). > > If, on the other hand, there is reason to believe the fdt_addr_t and > fdt_size_t are wrongly defined and should be typedef'd or ~define's as > phys_*_t, then the rationale for this should be exposed. > > Amicalement, > -- > Albert.
For LBA we do this in ide.h: #ifdef CONFIG_SYS_64BIT_LBA typedef uint64_t lbaint_t; #define LBAF "%llx" #define LBAFU "%llu" #else typedef ulong lbaint_t; #define LBAF "%lx" #define LBAFU "%lu" #endif How about something similar for this case? It would be nice to avoid promoting everything to 64-bit given that very few platforms are actually 64-bit. Regards, Simon _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
