> + devm_clk_unregister(i2c_dev->dev, bus_clk); Is there a reason you call this and not rely on devm to unregister? Other than that, looks good from what I can tell.