Re: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-06-12 Thread Wolfram Sang
On Tue, May 28, 2013 at 02:42:22PM +0800, Libo Chen wrote: > thanks for your review. I will update later. I agree with the remarks from Sachin. signature.asc Description: Digital signature

Re: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-05-27 Thread Libo Chen
thanks for your review. I will update later. Libo On 2013/5/24 11:19, Sachin Kamat wrote: > On 23 May 2013 17:30, Libo Chen wrote: >> peripheral_request_list has got free if any one faild, so no need to free >> again in err case. >> aovid this, convert them to devm_* API >> >> Signed-off-by: Li

RE: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-05-23 Thread Zhang, Sonic
Acked-by: Sonic Zhang >-Original Message- >From: Libo Chen [mailto:libo.c...@huawei.com] >Sent: Thursday, May 23, 2013 8:00 PM >To: w...@the-dreams.de >Cc: guz.f...@cn.fujitsu.com; Zhang, Sonic; uclinux-dist- >de...@blackfin.uclinux.org; linux-...@vger.kernel.org; linux- >ker...@vger.ker

Re: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-05-23 Thread Sachin Kamat
On 23 May 2013 17:30, Libo Chen wrote: > peripheral_request_list has got free if any one faild, so no need to free > again in err case. > aovid this, convert them to devm_* API > > Signed-off-by: Libo Chen It is a good practice to include changelog while submitting revised versions of the patch

Re: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-05-23 Thread Libo Chen
On 2013/5/24 10:49, Zhang, Sonic wrote: >> >- iface = kzalloc(sizeof(struct bfin_twi_iface), GFP_KERNEL); >> >+ iface = devm_kzalloc(&pdev->dev, sizeof(struct bfin_twi_iface), >> >+ GFP_KERNEL); >> > if (!iface) { >> > dev_err(&pdev->dev, "Cannot a

RE: [PATCH RFC v3 1/3] i2c: i2c-bfin-twi: convert to devm_* API

2013-05-23 Thread Zhang, Sonic
Hi Libo, >-Original Message- >From: Libo Chen [mailto:libo.c...@huawei.com] >Sent: Thursday, May 23, 2013 8:00 PM >To: w...@the-dreams.de >Cc: guz.f...@cn.fujitsu.com; Zhang, Sonic; uclinux-dist- >de...@blackfin.uclinux.org; linux-...@vger.kernel.org; linux- >ker...@vger.kernel.org; lize..