On 09/14, arvind.yadav...@gmail.com wrote:
> @@ -337,6 +339,8 @@ static void __init st_of_flexgen_setup(struct device_node 
> *np)
>       return;
>  
>  err:
> +     if (reg)

This is always true? Why check?

> +             iounmap(reg);
>       if (clk_data)
>               kfree(clk_data->clks);
>       kfree(clk_data);
> -- 
> 1.7.9.5
> 

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Reply via email to