Hi Wolfram, On Tuesday, January 17, 2017, Wolfram Sang wrote: > Hi Chris, > > > + struct clk *clk2; > > I'd vote for 'clk_cd' since we are explicitly requesting that clock for it. > > > + ret = clk_prepare_enable(priv->clk2); > > + if (ret < 0) > > + return ret; > > You need to disable clk when failing. > > Regards, > > Wolfram Thanks for the feedback. I will make those changes. Chris
