Hi Gilad, On Tue, May 15, 2018 at 2:29 PM, Gilad Ben-Yossef <gilad@xxxxxxxxxxxxx> wrote: > This patch adds the clock used by the CryptoCell 630p instance in the SoC. > > Signed-off-by: Gilad Ben-Yossef <gilad@xxxxxxxxxxxxx> Thanks for your patch! > --- a/drivers/clk/renesas/r8a7795-cpg-mssr.c > +++ b/drivers/clk/renesas/r8a7795-cpg-mssr.c > @@ -132,6 +132,7 @@ static struct mssr_mod_clk r8a7795_mod_clks[] __initdata = { > DEF_MOD("sys-dmac2", 217, R8A7795_CLK_S0D3), > DEF_MOD("sys-dmac1", 218, R8A7795_CLK_S0D3), > DEF_MOD("sys-dmac0", 219, R8A7795_CLK_S0D3), > + DEF_MOD("ccree", 229, R8A7795_CLK_S3D2), I don't know if "ccree" is the proper name for this clock, as there may be multiple instances. I also can't verify the parent clock. > DEF_MOD("cmt3", 300, R8A7795_CLK_R), > DEF_MOD("cmt2", 301, R8A7795_CLK_R), > DEF_MOD("cmt1", 302, R8A7795_CLK_R), Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds