On Mon, Feb 04, 2019 at 10:34:16AM +0100, Jean-Jacques Hiblot wrote: > > On 01/02/2019 19:59, Tom Rini wrote: > >On Fri, Feb 01, 2019 at 07:40:21PM +0100, Jean-Jacques Hiblot wrote: > >>There is no need to modify the buffer passed to ext4fs_write_file(). > >>The memset() call is not required here and was likely copied from the > >>equivalent part of the ext4fs_read_file() function where we do need it. > >> > >>Signed-off-by: Jean-Jacques Hiblot <jjhib...@ti.com> > >> > >>--- > >> > >>Changes in v3: > >>- reworded commit log > >> > >>Changes in v2: None > >> > >> fs/ext4/ext4_common.c | 2 +- > >> fs/ext4/ext4_common.h | 2 +- > >> fs/ext4/ext4_write.c | 11 +++++------ > >> include/ext4fs.h | 2 +- > >> 4 files changed, 8 insertions(+), 9 deletions(-) > >> > >>diff --git a/fs/ext4/ext4_common.c b/fs/ext4/ext4_common.c > >>index 67e2471bd3..b881482c39 100644 > >>--- a/fs/ext4/ext4_common.c > >>+++ b/fs/ext4/ext4_common.c > >>@@ -190,7 +190,7 @@ uint32_t ext4fs_div_roundup(uint32_t size, uint32_t n) > >> return res; > >> } > >>-void put_ext4(uint64_t off, void *buf, uint32_t size) > >>+void put_ext4(u64 off, const void *buf, u32 size) > >You missed my comment here, sorry. The rest of the function (and file?) > >is uintXX_t not uXX so please just add const here. Thanks! > > Yes I forgot about it. I sent a v4 with the changes. I had initially changed > thisĀ because of checkpatch: > > CHECK: Prefer kernel type 'u64' over 'uint64_t' > #51: FILE: fs/ext4/ext4_common.h:75: > +void put_ext4(uint64_t off, const void *buf, uint32_t size);
Ah, OK. Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot