Re: [PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-09 Thread Mark Brown
On Sat, Dec 08, 2012 at 09:56:48AM +0800, Axel Lin wrote: > There is only one user calling _rdev_to_offset() function. > Remove _rdev_to_offset() makes the code simpler. Applied, thanks. Milo, you should probably add a MAINTAINERS entry for this one too. -- To unsubscribe from this list: send

RE: [PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-09 Thread Kim, Milo
> There is only one user calling _rdev_to_offset() function. > Remove _rdev_to_offset() makes the code simpler. > > Signed-off-by: Axel Lin It looks good to me, thanks! Acked-by: Milo(Woogyom) Kim

RE: [PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-09 Thread Kim, Milo
There is only one user calling _rdev_to_offset() function. Remove _rdev_to_offset() makes the code simpler. Signed-off-by: Axel Lin axel@ingics.com It looks good to me, thanks! Acked-by: Milo(Woogyom) Kim milo@ti.com

Re: [PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-09 Thread Mark Brown
On Sat, Dec 08, 2012 at 09:56:48AM +0800, Axel Lin wrote: There is only one user calling _rdev_to_offset() function. Remove _rdev_to_offset() makes the code simpler. Applied, thanks. Milo, you should probably add a MAINTAINERS entry for this one too. -- To unsubscribe from this list: send the

[PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-07 Thread Axel Lin
There is only one user calling _rdev_to_offset() function. Remove _rdev_to_offset() makes the code simpler. Signed-off-by: Axel Lin --- drivers/regulator/lp872x.c | 36 +++- 1 file changed, 7 insertions(+), 29 deletions(-) diff --git

[PATCH] regulator: lp872x: Kill _rdev_to_offset() function

2012-12-07 Thread Axel Lin
There is only one user calling _rdev_to_offset() function. Remove _rdev_to_offset() makes the code simpler. Signed-off-by: Axel Lin axel@ingics.com --- drivers/regulator/lp872x.c | 36 +++- 1 file changed, 7 insertions(+), 29 deletions(-) diff --git