Wolfram Sang wrote:
> On Mon, Apr 04, 2011 at 11:48:57AM +0900, Jaehoon Chung wrote:
>> Sometimes we can't add the device,but we didn't check any error status.
>> Need to check error status for mmc_add_host.
> 
> Thanks, this is correct, but...
> 
>> -    mmc_add_host(mmc);
>> +    ret = mmc_add_host(mmc);
>> +    if (unlikely(ret))
>> +            goto err_free_mmc;
> 
> ... you are leaking the irq and the lcd_classdev here.

Thanks for your comment, i will resend the patch.

> 
> Regards,
> 
>    Wolfram
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to