[PATCH] cptm1217: check if interrupts are masked at probe

devendra.aaru devendra.aaru at gmail.com
Mon Jan 5 06:34:32 UTC 2015


>>> you are addressing the memory leak which is not mentioned in change-log.
>>
>> i am pasting part of commit log here:
>>
>> commit log: "added the check and the failure path".
>>
>> Is this not enough to understand ?
>
> you should have mentioned that memory is never released in previous
> section of code hence freeing memory or you can also include "Fix
> memory Leak" to make it clear what has been handled in the proposed
> patch.IMO this should be the way and lets wait for Greg & other
> opinion too.

Which previous section of code ? I did add the check for the interrupt
mask. This will be my last reply to you. sorry.


More information about the devel mailing list