Hi Tom, On 05/11/2017 04:20 AM, Tom Rini wrote: > In some places we check if part_config is set to MMCPART_NOAVAILABLE > (0xff). With part_config being a char this is always false. We should > be using a u8 to store this value instead, after a quick consultation > with the Linux Kernel. Reported by clang-3.8. > > Cc: Jaehoon Chung <jh80.ch...@samsung.com> > Signed-off-by: Tom Rini <tr...@konsulko.com>
Applied to u-boot-mmc. Thanks! > --- > include/mmc.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/mmc.h b/include/mmc.h > index fad12d608cef..00576fa3d0a3 100644 > --- a/include/mmc.h > +++ b/include/mmc.h > @@ -430,7 +430,7 @@ struct mmc { > u8 part_support; > u8 part_attr; > u8 wr_rel_set; > - char part_config; > + u8 part_config; > uint tran_speed; > uint read_bl_len; > uint write_bl_len; > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot