On Mon, 5 May 2014 08:24:22 -0600 Simon Glass <s...@chromium.org> wrote:
> > Hi Lukasz, > > On 4 May 2014 23:20, Lukasz Majewski <l.majew...@samsung.com> wrote: > > Hi Simon, > > > >> On 30 April 2014 03:39, Lukasz Majewski <l.majew...@samsung.com> > >> wrote: > >> > Code responsible for handling situation when ext4 has block size > >> > of 1024B can be ordered to take less space. > >> > > >> > This patch does that for ext4 common and write files. > >> > > >> > Signed-off-by: Lukasz Majewski <l.majew...@samsung.com> > >> > >> Reviewed-by: Simon Glass <s...@chromium.org> > >> > >> > --- > >> > fs/ext4/ext4_common.c | 6 ++---- > >> > fs/ext4/ext4_write.c | 50 > >> > ++++++++++++++++--------------------------------- 2 files > >> > changed, 18 insertions(+), 38 deletions(-) > >> > > >> > diff --git a/fs/ext4/ext4_common.c b/fs/ext4/ext4_common.c > >> > index 02da75c..62e2e80 100644 > >> > --- a/fs/ext4/ext4_common.c > >> > +++ b/fs/ext4/ext4_common.c > >> [snip] > >> > >> > @@ -181,10 +179,8 @@ static void > >> > delete_double_indirect_block(struct ext2_inode *inode) break; > >> > > >> > debug("DICB releasing %u\n", *di_buffer); > >> > - if (fs->blksz != 1024) { > >> > - bg_idx = (*di_buffer) / > >> > blk_per_grp; > >> > - } else { > >> > - bg_idx = (*di_buffer) / > >> > blk_per_grp; > >> > + bg_idx = (*di_buffer) / blk_per_grp; > >> > >> You don't need the brackets here (or below). > > > > Maybe the GIT formatting is a bit misleading, but I've double > > checked and it seems that those parenthesis are necessary here. > > OK. What is di_buffer such that (*di_buffer) works but *di_buffer > doesn't? It is hard to admit :-), but I've misunderstood you. I thought that you are talking about the {} parentheses. I will check the code tomorrow and prepare proper patch. > > Regards, > Simon Best regards, Lukasz Majewski
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot