On Tue, 2010-09-28 at 19:31 +0200, Wolfgang Denk wrote: > > -int memory_post_test (int flags) > > +__attribute__((weak)) > > +int arch_memory_test_prepare(u32 *vstart, u32 *size, phys_addr_t > > *phys_offset) > > phys_offset is unused here. Drop it? >
The phys_offset is not used by _this_ weak function but it is used by another function implemented in the third patch. Is it OK to leave the unused phys_offset here? Regards, York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
