Hi Geert Thank you for your reveiw > > +static void __init cpg_mssr_reserved_exit(struct cpg_mssr_priv *priv) > > +{ > > + kfree(priv->reserved_ids); > > +} > > This function is called only once, so you might want to inline it manually. I want to keep init()/exit() pair because it is easy to understand. I will consider other comment and post v5 patch next week Thank you for your help !! Best regards --- Renesas Electronics Ph.D. Kuninori Morimoto