Hi Geert, Thanks for your work. On 2019-12-06 14:32:54 +0100, Geert Uytterhoeven wrote: > All multipliers and dividers are small. > Storing them in u8 instead of unsigned int reduces kernel size for a > generic kernel by ca. 0.5 KiB. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- > To be queued in clk-renesas-for-v5.6. Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > > drivers/clk/renesas/rcar-gen2-cpg.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/clk/renesas/rcar-gen2-cpg.h b/drivers/clk/renesas/rcar-gen2-cpg.h > index db2f57ef2f9984e4..bdcd4a38d48d01bd 100644 > --- a/drivers/clk/renesas/rcar-gen2-cpg.h > +++ b/drivers/clk/renesas/rcar-gen2-cpg.h > @@ -24,10 +24,10 @@ enum rcar_gen2_clk_types { > }; > > struct rcar_gen2_cpg_pll_config { > - unsigned int extal_div; > - unsigned int pll1_mult; > - unsigned int pll3_mult; > - unsigned int pll0_mult; /* leave as zero if PLL0CR exists */ > + u8 extal_div; > + u8 pll1_mult; > + u8 pll3_mult; > + u8 pll0_mult; /* leave as zero if PLL0CR exists */ > }; > > struct clk *rcar_gen2_cpg_clk_register(struct device *dev, > -- > 2.17.1 > -- Regards, Niklas Söderlund