= >dm644x_clear_wbl_overflow; >> + else >> + return -ENODEV; > >Do you need clean up procedure if you return error here? I mean - >calls to release_mem_region, release_mem_region, etc > Oops! I need to add that. Thanks. >> + spin_lock_init(&oper_cfg.vpss_lock); >> + dev_info(&pdev->dev, "%s vpss probe success\n", >oper_cfg.vpss_name); >> + return 0; >> +fail3: >> + release_mem_region(oper_cfg.r2->start, oper_cfg.len2); >> +fail2: >> + iounmap(oper_cfg.vpss_bl_regs_base); >> +fail1: >> + release_mem_region(oper_cfg.r1->start, oper_cfg.len1); >> + return status; >> +} > > >-- >Best regards, Klimov Alexey
_______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
