Florian Fainelli writes: > On 04/18/2017 04:38 PM, Eric Anholt wrote: >> For the Raspberry Pi's bindings, the power domain also implicitly >> turns on the clock and deasserts reset, but for the new Cygnus port we >> start representing the clock in the devicetree. >> >> v2: Document the clock-names property, check for -ENOENT for no clock >> in DT. >> >> Signed-off-by: Eric Anholt >> --- > >> + if (v3d->clk) >> + clk_disable_unprepare(v3d->clk); > > The clock API allows you to pass a NULL clk and do nothing in these > cases which is what you seem to have done a few lines below, you could > simplify these checks? Sounds good.