Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-29 Thread Mark Brown
On Wed, Aug 28, 2013 at 08:07:15PM -0700, Anton Vorontsov wrote: > These changes not so much burden for me personally (it was actually easier > for me to just apply the patch without all the arguing), but for those who > actually do real bugfixes/features in the drivers: their local development >

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-29 Thread Mark Brown
On Wed, Aug 28, 2013 at 08:07:15PM -0700, Anton Vorontsov wrote: These changes not so much burden for me personally (it was actually easier for me to just apply the patch without all the arguing), but for those who actually do real bugfixes/features in the drivers: their local development

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 07:07:14PM -0700, 'Greg Kroah-Hartman' wrote: > On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: > > On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > > > He doesn't want to take the patch. He's the maintainer so it's his > > > choice.

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Thursday, August 29, 2013 11:07 AM, Greg Kroah-Hartman wrote: > On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: > > On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > > > He doesn't want to take the patch. He's the maintainer so it's his > > > choice. That's the

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Thursday, August 29, 2013 10:19 AM, Anton Vorontsov wrote: > On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > > He doesn't want to take the patch. He's the maintainer so it's his > > choice. That's the end of the story. > > Just to clarify: I don't want to take the patch for

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread 'Greg Kroah-Hartman'
On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: > On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > > He doesn't want to take the patch. He's the maintainer so it's his > > choice. That's the end of the story. > > Just to clarify: I don't want to take the patch

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: > He doesn't want to take the patch. He's the maintainer so it's his > choice. That's the end of the story. Just to clarify: I don't want to take the patch for a reason, not just because of my mood today. Once the patch comes in

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Wednesday, August 28, 2013 5:37 PM, Dan Carpenter wrote: > > He doesn't want to take the patch. He's the maintainer so it's his > choice. That's the end of the story. > I know that the maintainer's choice is important, because I am also a maintainer of other system or drivers. But, there

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Dan Carpenter
He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Dan Carpenter
He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. regards, dan carpenter -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Wednesday, August 28, 2013 5:37 PM, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. I know that the maintainer's choice is important, because I am also a maintainer of other system or drivers. But, there is

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. Just to clarify: I don't want to take the patch for a reason, not just because of my mood today. Once the patch comes in

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread 'Greg Kroah-Hartman'
On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. Just to clarify: I don't want to take the patch for a

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Thursday, August 29, 2013 10:19 AM, Anton Vorontsov wrote: On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of the story. Just to clarify: I don't want to take the patch for a

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Jingoo Han
On Thursday, August 29, 2013 11:07 AM, Greg Kroah-Hartman wrote: On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end of

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-28 Thread Anton Vorontsov
On Wed, Aug 28, 2013 at 07:07:14PM -0700, 'Greg Kroah-Hartman' wrote: On Wed, Aug 28, 2013 at 06:18:49PM -0700, Anton Vorontsov wrote: On Wed, Aug 28, 2013 at 11:36:30AM +0300, Dan Carpenter wrote: He doesn't want to take the patch. He's the maintainer so it's his choice. That's the end

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-27 Thread Jingoo Han
On Wednesday, August 28, 2013 10:59 AM, Anton Vorontsov wrote: > On Tue, Aug 13, 2013 at 12:00:39PM +0300, Dan Carpenter wrote: > > > > > Use the wrapper function for retrieving the platform data instead of > > > > > accessing dev->platform_data directly. > > > > > > > > Um.. what is the benefit

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-27 Thread Anton Vorontsov
On Tue, Aug 13, 2013 at 12:00:39PM +0300, Dan Carpenter wrote: > > > > Use the wrapper function for retrieving the platform data instead of > > > > accessing dev->platform_data directly. > > > > > > Um.. what is the benefit or rationale of this patch? > > > > CC'ed Joe Perches, Dan Carpenter > >

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-27 Thread Anton Vorontsov
On Tue, Aug 13, 2013 at 12:00:39PM +0300, Dan Carpenter wrote: Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly. Um.. what is the benefit or rationale of this patch? CC'ed Joe Perches, Dan Carpenter Hi Anton

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-27 Thread Jingoo Han
On Wednesday, August 28, 2013 10:59 AM, Anton Vorontsov wrote: On Tue, Aug 13, 2013 at 12:00:39PM +0300, Dan Carpenter wrote: Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly. Um.. what is the benefit or rationale of this

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-13 Thread Dan Carpenter
On Mon, Aug 12, 2013 at 09:22:42AM +0900, Jingoo Han wrote: > On Saturday, August 10, 2013 7:05 AM, Anton Vorontsov wrote: > > On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: > > > Use the wrapper function for retrieving the platform data instead of > > > accessing dev->platform_data

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-13 Thread Dan Carpenter
On Mon, Aug 12, 2013 at 09:22:42AM +0900, Jingoo Han wrote: On Saturday, August 10, 2013 7:05 AM, Anton Vorontsov wrote: On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly.

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-11 Thread Jingoo Han
On Saturday, August 10, 2013 7:05 AM, Anton Vorontsov wrote: > On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: > > Use the wrapper function for retrieving the platform data instead of > > accessing dev->platform_data directly. > > Um.. what is the benefit or rationale of this patch?

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-11 Thread Jingoo Han
On Saturday, August 10, 2013 7:05 AM, Anton Vorontsov wrote: On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly. Um.. what is the benefit or rationale of this patch? CC'ed

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-09 Thread Anton Vorontsov
On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: > Use the wrapper function for retrieving the platform data instead of > accessing dev->platform_data directly. Um.. what is the benefit or rationale of this patch? Thanks, Anton -- To unsubscribe from this list: send the line

Re: [PATCH 33/35] power: use dev_get_platdata()

2013-08-09 Thread Anton Vorontsov
On Tue, Jul 30, 2013 at 05:19:27PM +0900, Jingoo Han wrote: Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly. Um.. what is the benefit or rationale of this patch? Thanks, Anton -- To unsubscribe from this list: send the line unsubscribe

[PATCH 33/35] power: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/power/88pm860x_battery.c|2 +- drivers/power/ab8500_btemp.c|2 +- drivers/power/ab8500_charger.c |2 +-

[PATCH 33/35] power: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev-platform_data directly. Signed-off-by: Jingoo Han jg1@samsung.com --- drivers/power/88pm860x_battery.c|2 +- drivers/power/ab8500_btemp.c|2 +- drivers/power/ab8500_charger.c |2 +-