On 1/28/20 10:44 AM, Patrick Delaunay wrote: > Solve type issue in stm32mp1_lse_enable and stm32mp1_clktree. > > This patch solves the warnings when compiling with W=1 > on stm32mp1 board: > > clk_stm32mp1.c: In function ‘stm32mp1_lse_enable’: > clk_stm32mp1.c:1238:15: warning: comparison of integer expressions > of different signedness: ‘u32’ {aka ‘unsigned int’} and ‘int’ > [-Wsign-compare] > clk_stm32mp1.c:1239:13: warning: comparison of integer expressions > of different signedness: ‘u32’ {aka ‘unsigned int’} and ‘int’ > [-Wsign-compare] > > clk_stm32mp1.c: In function ‘stm32mp1_clktree’: > clk_stm32mp1.c:1814:17: warning: comparison of integer expressions > of different signedness: ‘int’ and ‘unsigned int’ > [-Wsign-compare] > > Signed-off-by: Patrick Delaunay <patrick.delau...@st.com> > --- > > drivers/clk/clk_stm32mp1.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) > > diff --git a/drivers/clk/clk_stm32mp1.c b/drivers/clk/clk_stm32mp1.c > index da66bde41b..fd8c821e48 100644 > --- a/drivers/clk/clk_stm32mp1.c > +++ b/drivers/clk/clk_stm32mp1.c > @@ -1218,7 +1218,7 @@ static int stm32mp1_osc_wait(int enable, fdt_addr_t > rcc, u32 offset, > } > > static void stm32mp1_lse_enable(fdt_addr_t rcc, int bypass, int digbyp, > - int lsedrv) > + u32 lsedrv) > { > u32 value; > > @@ -1651,8 +1651,8 @@ static int stm32mp1_clktree(struct udevice *dev) > unsigned int clkdiv[CLKDIV_NB]; > unsigned int pllcfg[_PLL_NB][PLLCFG_NB]; > ofnode plloff[_PLL_NB]; > - int ret; > - int i, len; > + int ret, len; > + uint i; > int lse_css = 0; > const u32 *pkcs_cell; > > @@ -1698,7 +1698,8 @@ static int stm32mp1_clktree(struct udevice *dev) > stm32mp1_lsi_set(rcc, 1); > > if (priv->osc[_LSE]) { > - int bypass, digbyp, lsedrv; > + int bypass, digbyp; > + u32 lsedrv; > struct udevice *dev = priv->osc_dev[_LSE]; > > bypass = dev_read_bool(dev, "st,bypass");
Reviewed-by: Patrice Chotard <patrice.chot...@st.com> Thanks