On Tue, Apr 30, 2019 at 9:28 AM Nick Desaulniers <ndesaulniers@xxxxxxxxxx> wrote: > > On Tue, Apr 30, 2019 at 7:58 AM Phong Tran <tranmanphong@xxxxxxxxx> wrote: > > > > Now, make the loop explicit to avoid clang warning. > > > > ./include/linux/of.h:238:37: warning: multiple unsequenced modifications > > to 'cell' [-Wunsequenced] > > r = (r << 32) | be32_to_cpu(*(cell++)); > > ^~ > > ./include/linux/byteorder/generic.h:95:21: note: expanded from macro > > 'be32_to_cpu' > > ^ > > ./include/uapi/linux/byteorder/little_endian.h:40:59: note: expanded > > from macro '__be32_to_cpu' > > ^ > > ./include/uapi/linux/swab.h:118:21: note: expanded from macro '__swab32' > > ___constant_swab32(x) : \ > > ^ > > ./include/uapi/linux/swab.h:18:12: note: expanded from macro > > '___constant_swab32' > > (((__u32)(x) & (__u32)0x000000ffUL) << 24) | \ > > ^ > > > > Signed-off-by: Phong Tran <tranmanphong@xxxxxxxxx> > > Thanks for the patch. > Reported-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > Link: https://github.com/ClangBuiltLinux/linux/issues/460 > Suggested-by: David Laight <David.Laight@xxxxxxxxxx> sent too soon... Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> -- Thanks, ~Nick Desaulniers
