Re: [PATCH v6] devres: combine function devm_ioremap*

2018-02-12 Thread Greg KH
On Mon, Feb 12, 2018 at 07:07:34PM +0800, Yisheng Xie wrote: > Hi Greg, Christophe, > > Any comment about this version? And sorry to disturb :) Please give me a chance to dig out from the merge window patch debt. I'll get to this in a week or so... But at first glance, looks fine to me. greg

Re: [PATCH v6] devres: combine function devm_ioremap*

2018-02-12 Thread Yisheng Xie
Hi Christophe, On 2018/2/12 19:21, Christophe LEROY wrote: > Hi Yisheng, > > I don't have any comment. > > Reviewed-by: Thanks Yisheng > > Christophe > > Le 12/02/2018 à 12:07, Yisheng Xie a écrit : >> Hi Greg, Christophe, >> >> Any comment about this version? And sorry to disturb :) >> >>

Re: [PATCH v6] devres: combine function devm_ioremap*

2018-02-12 Thread Christophe LEROY
Hi Yishen, I don't have any comment. Reviewed-by: Christophe Le 12/02/2018 à 12:07, Yisheng Xie a écrit : Hi Greg, Christophe, Any comment about this version? And sorry to disturb :) Thanks Yisheng On 2018/1/29 19:48, Yisheng Xie wrote: When I tried to use devm_ioremap function and rev

Re: [PATCH v6] devres: combine function devm_ioremap*

2018-02-12 Thread Yisheng Xie
Hi Greg, Christophe, Any comment about this version? And sorry to disturb :) Thanks Yisheng On 2018/1/29 19:48, Yisheng Xie wrote: > When I tried to use devm_ioremap function and review related > code, I found devm_ioremap_* almost have the similar realize > with each other, which can be combi

[PATCH v6] devres: combine function devm_ioremap*

2018-01-29 Thread Yisheng Xie
When I tried to use devm_ioremap function and review related code, I found devm_ioremap_* almost have the similar realize with each other, which can be combined. In the former version, I have tried to kill ioremap_cache to reduce the size of devres, which can not work for ioremap is not the same a