Hi Alex, On 15 June 2018 at 06:42, Alexander Graf <ag...@suse.de> wrote: > The fs_read() and fs_write() functions are internal interfaces that > naturally want to get pointers as arguments. Most users so far even > have pointers and explicitly cast them into integers just to be able > to pass them into the function. > > Convert them over to instead take a pointer argument for the buffer. > That way any sandbox mapping gets greatly simplified and users of > the API intuitively know what to do. > > Signed-off-by: Alexander Graf <ag...@suse.de> > --- > board/BuR/common/common.c | 2 +- > board/gdsys/p1022/controlcenterd-id.c | 10 +++++----- > cmd/mvebu/bubt.c | 4 ++-- > common/splash_source.c | 4 +++- > drivers/bootcount/bootcount_ext.c | 12 ++++++------ > drivers/fpga/zynqpl.c | 8 +++++--- > fs/fs.c | 20 ++++++++++---------- > include/fs.h | 12 ++++++------ > lib/efi_loader/efi_file.c | 6 ++---- > 9 files changed, 40 insertions(+), 38 deletions(-)
U-Boot uses addresses for loading and managing images. I don't see a good reason to change that. We expect all logging to emit an address rather than a pointer, for example. See for example all the FIT and legacy image stuff, bootm, all the commands, etc. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
