Hi Sergei, On Thu, Jul 3, 2014 at 3:50 PM, Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> wrote: >> --- a/arch/mips/pci/pci-alchemy.c >> +++ b/arch/mips/pci/pci-alchemy.c >> @@ -394,11 +396,24 @@ static int alchemy_pci_probe(struct platform_device >> *pdev) >> + ret = clk_prepare_enable(c); >> + if (ret) { >> + dev_err(&pdev->dev, "cannot enable PCI clock\n"); >> + clk_put(c); >> + goto out2; > > > Isn't it simpler to add one more label before clk_put() at end of > function? Yes, I have changed it locally. Thank you! Manuel