Signed-off-by: Alexander Shiyan <shc_w...@mail.ru>
---
 drivers/w1/masters/mxc_w1.c | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/drivers/w1/masters/mxc_w1.c b/drivers/w1/masters/mxc_w1.c
index 953168c..1e5d94c 100644
--- a/drivers/w1/masters/mxc_w1.c
+++ b/drivers/w1/masters/mxc_w1.c
@@ -108,7 +108,7 @@ static int mxc_w1_probe(struct platform_device *pdev)
        unsigned long clkrate;
        struct resource *res;
        unsigned int clkdiv;
-       int err = 0;
+       int err;
 
        mdev = devm_kzalloc(&pdev->dev, sizeof(struct mxc_w1_device),
                            GFP_KERNEL);
@@ -135,20 +135,23 @@ static int mxc_w1_probe(struct platform_device *pdev)
        if (IS_ERR(mdev->regs))
                return PTR_ERR(mdev->regs);
 
-       clk_prepare_enable(mdev->clk);
+       err = clk_prepare_enable(mdev->clk);
+       if (err)
+               return err;
+
        __raw_writeb(clkdiv - 1, mdev->regs + MXC_W1_TIME_DIVIDER);
 
        mdev->bus_master.data = mdev;
        mdev->bus_master.reset_bus = mxc_w1_ds2_reset_bus;
        mdev->bus_master.touch_bit = mxc_w1_ds2_touch_bit;
 
-       err = w1_add_master_device(&mdev->bus_master);
+       platform_set_drvdata(pdev, mdev);
 
+       err = w1_add_master_device(&mdev->bus_master);
        if (err)
-               return err;
+               clk_disable_unprepare(mdev->clk);
 
-       platform_set_drvdata(pdev, mdev);
-       return 0;
+       return err;
 }
 
 /*
-- 
1.8.3.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to