Re: [PATCH] staging iio: Replace kmalloc with local variable

From: Jonathan Cameron
Date: Thu Jun 09 2011 - 06:38:45 EST


On 06/09/11 09:48, Jonathan Cameron wrote:
> On 06/09/11 09:34, anish singh wrote:
>>
>>
>> On Thu, Jun 9, 2011 at 1:59 PM, Jean Delvare <khali@xxxxxxxxxxxx <mailto:khali@xxxxxxxxxxxx>> wrote:
>>
>> On Thu, 9 Jun 2011 13:53:09 +0530, anish singh wrote:
>> > On Wed, Jun 8, 2011 at 10:41 AM, anish singh <anish198519851985@xxxxxxxxx <mailto:anish198519851985@xxxxxxxxx>>wrote:
>> > > On Tue, Jun 7, 2011 at 7:32 PM, Jean Delvare <khali@xxxxxxxxxxxx <mailto:khali@xxxxxxxxxxxx>> wrote:
>> > >> To be clearer, it is down to the i2c BUS (adapter) driver, NOT the i2c
>> > >> device driver. So the patch is correct.
>> > >>
>> > > I think i can take Jean ack on this?If yes then Joanthan kindly apply
>> > > this patch and i think you didn't lead me in wrong way as whatever
>> > > said by you is corroborated by jean also i.e. it is I2C bus driver
>> > > responsiblity
>> > > to care about DMA.
>> > >
>> > Sorry to ping you again.Can i take your ack on this?
>>
>> Yes of course.
>>
>> Acked-by: Jean Delvare <khali@xxxxxxxxxxxx <mailto:khali@xxxxxxxxxxxx>>
>>
>> Thanks a ton.Jonathan kindly apply it now :)
> Greg, I'll send this one on to you with the set I currently have out for review.
>
Doh, after all this, I just tried to apply this to find the code in question has already
gone. Anish, what tree are you working against? Looks like I did an equivalent clean up
(with a load of others) back in May then forgot about it.

Sorry all. I should have actually have checked it was still relevant rather than reviewing
purely on basis of content of patch...

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