* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-16 12:41 ` Banajit Goswami
0 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-16 12:41 UTC (permalink / raw)
To: linux-samsung-soc; +Cc: kgene.kim, ben-linux, linux-arm-kernel, Banajit Goswami
Remove un-used backlight code for SMDK6410 board
Signed-off-by: Banajit Goswami <banajit.g@samsung.com>
---
arch/arm/mach-s3c64xx/mach-smdk6410.c | 39 ---------------------------------
1 files changed, 0 insertions(+), 39 deletions(-)
diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-s3c64xx/mach-smdk6410.c
index ecbea92..a9f3183 100644
--- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
+++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
@@ -262,45 +262,6 @@ static struct samsung_keypad_platdata smdk6410_keypad_data __initdata = {
.cols = 8,
};
-static int smdk6410_backlight_init(struct device *dev)
-{
- int ret;
-
- ret = gpio_request(S3C64XX_GPF(15), "Backlight");
- if (ret) {
- printk(KERN_ERR "failed to request GPF for PWM-OUT1\n");
- return ret;
- }
-
- /* Configure GPIO pin with S3C64XX_GPF15_PWM_TOUT1 */
- s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_SFN(2));
-
- return 0;
-}
-
-static void smdk6410_backlight_exit(struct device *dev)
-{
- s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_OUTPUT);
- gpio_free(S3C64XX_GPF(15));
-}
-
-static struct platform_pwm_backlight_data smdk6410_backlight_data = {
- .pwm_id = 1,
- .max_brightness = 255,
- .dft_brightness = 255,
- .pwm_period_ns = 78770,
- .init = smdk6410_backlight_init,
- .exit = smdk6410_backlight_exit,
-};
-
-static struct platform_device smdk6410_backlight_device = {
- .name = "pwm-backlight",
- .dev = {
- .parent = &s3c_device_timer[1].dev,
- .platform_data = &smdk6410_backlight_data,
- },
-};
-
static struct map_desc smdk6410_iodesc[] = {};
static struct platform_device *smdk6410_devices[] __initdata = {
--
1.7.2.3
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-16 12:41 ` Banajit Goswami
0 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-16 12:41 UTC (permalink / raw)
To: linux-arm-kernel
Remove un-used backlight code for SMDK6410 board
Signed-off-by: Banajit Goswami <banajit.g@samsung.com>
---
arch/arm/mach-s3c64xx/mach-smdk6410.c | 39 ---------------------------------
1 files changed, 0 insertions(+), 39 deletions(-)
diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-s3c64xx/mach-smdk6410.c
index ecbea92..a9f3183 100644
--- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
+++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
@@ -262,45 +262,6 @@ static struct samsung_keypad_platdata smdk6410_keypad_data __initdata = {
.cols = 8,
};
-static int smdk6410_backlight_init(struct device *dev)
-{
- int ret;
-
- ret = gpio_request(S3C64XX_GPF(15), "Backlight");
- if (ret) {
- printk(KERN_ERR "failed to request GPF for PWM-OUT1\n");
- return ret;
- }
-
- /* Configure GPIO pin with S3C64XX_GPF15_PWM_TOUT1 */
- s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_SFN(2));
-
- return 0;
-}
-
-static void smdk6410_backlight_exit(struct device *dev)
-{
- s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_OUTPUT);
- gpio_free(S3C64XX_GPF(15));
-}
-
-static struct platform_pwm_backlight_data smdk6410_backlight_data = {
- .pwm_id = 1,
- .max_brightness = 255,
- .dft_brightness = 255,
- .pwm_period_ns = 78770,
- .init = smdk6410_backlight_init,
- .exit = smdk6410_backlight_exit,
-};
-
-static struct platform_device smdk6410_backlight_device = {
- .name = "pwm-backlight",
- .dev = {
- .parent = &s3c_device_timer[1].dev,
- .platform_data = &smdk6410_backlight_data,
- },
-};
-
static struct map_desc smdk6410_iodesc[] = {};
static struct platform_device *smdk6410_devices[] __initdata = {
--
1.7.2.3
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-16 12:41 ` Banajit Goswami
@ 2011-08-19 1:59 ` Mark Brown
-1 siblings, 0 replies; 16+ messages in thread
From: Mark Brown @ 2011-08-19 1:59 UTC (permalink / raw)
To: Banajit Goswami; +Cc: linux-samsung-soc, kgene.kim, ben-linux, linux-arm-kernel
On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> Remove un-used backlight code for SMDK6410 board
Is the data actually wrong? If not it'd seem better to finish hooking
it up properly than to remove it.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 1:59 ` Mark Brown
0 siblings, 0 replies; 16+ messages in thread
From: Mark Brown @ 2011-08-19 1:59 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> Remove un-used backlight code for SMDK6410 board
Is the data actually wrong? If not it'd seem better to finish hooking
it up properly than to remove it.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-19 1:59 ` Mark Brown
@ 2011-08-19 4:10 ` Banajit Goswami
-1 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 4:10 UTC (permalink / raw)
To: Mark Brown
Cc: Banajit Goswami, linux-samsung-soc, kgene.kim, ben-linux,
linux-arm-kernel
Hi Mark,
On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> Remove un-used backlight code for SMDK6410 board
>
> Is the data actually wrong? If not it'd seem better to finish hooking
> it up properly than to remove it.
No, this code is no longer needed, as we are having a common backlight
infrastructure , which serves the same purpose.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 4:10 ` Banajit Goswami
0 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 4:10 UTC (permalink / raw)
To: linux-arm-kernel
Hi Mark,
On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> Remove un-used backlight code for SMDK6410 board
>
> Is the data actually wrong? ?If not it'd seem better to finish hooking
> it up properly than to remove it.
No, this code is no longer needed, as we are having a common backlight
infrastructure , which serves the same purpose.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-19 4:10 ` Banajit Goswami
@ 2011-08-19 4:53 ` Mark Brown
-1 siblings, 0 replies; 16+ messages in thread
From: Mark Brown @ 2011-08-19 4:53 UTC (permalink / raw)
To: Banajit Goswami
Cc: Banajit Goswami, linux-samsung-soc, kgene.kim, ben-linux,
linux-arm-kernel
On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
> On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
> <broonie@opensource.wolfsonmicro.com> wrote:
> > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> >> Remove un-used backlight code for SMDK6410 board
> > Is the data actually wrong? If not it'd seem better to finish hooking
> > it up properly than to remove it.
> No, this code is no longer needed, as we are having a common backlight
> infrastructure , which serves the same purpose.
Ideally your commit log should make this clear - it looks like you're
just deleting unreferenced code.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 4:53 ` Mark Brown
0 siblings, 0 replies; 16+ messages in thread
From: Mark Brown @ 2011-08-19 4:53 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
> On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
> <broonie@opensource.wolfsonmicro.com> wrote:
> > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> >> Remove un-used backlight code for SMDK6410 board
> > Is the data actually wrong? ?If not it'd seem better to finish hooking
> > it up properly than to remove it.
> No, this code is no longer needed, as we are having a common backlight
> infrastructure , which serves the same purpose.
Ideally your commit log should make this clear - it looks like you're
just deleting unreferenced code.
^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-16 12:41 ` Banajit Goswami
@ 2011-08-19 5:24 ` Kukjin Kim
-1 siblings, 0 replies; 16+ messages in thread
From: Kukjin Kim @ 2011-08-19 5:24 UTC (permalink / raw)
To: 'Banajit Goswami', linux-samsung-soc; +Cc: ben-linux, linux-arm-kernel
Banajit Goswami wrote:
>
> Remove un-used backlight code for SMDK6410 board
>
> Signed-off-by: Banajit Goswami <banajit.g@samsung.com>
> ---
> arch/arm/mach-s3c64xx/mach-smdk6410.c | 39
---------------------------------
> 1 files changed, 0 insertions(+), 39 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-
> s3c64xx/mach-smdk6410.c
> index ecbea92..a9f3183 100644
> --- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
> +++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> @@ -262,45 +262,6 @@ static struct samsung_keypad_platdata
> smdk6410_keypad_data __initdata = {
> .cols = 8,
> };
>
> -static int smdk6410_backlight_init(struct device *dev)
> -{
> - int ret;
> -
> - ret = gpio_request(S3C64XX_GPF(15), "Backlight");
> - if (ret) {
> - printk(KERN_ERR "failed to request GPF for PWM-OUT1\n");
> - return ret;
> - }
> -
> - /* Configure GPIO pin with S3C64XX_GPF15_PWM_TOUT1 */
> - s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_SFN(2));
> -
> - return 0;
> -}
> -
> -static void smdk6410_backlight_exit(struct device *dev)
> -{
> - s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_OUTPUT);
> - gpio_free(S3C64XX_GPF(15));
> -}
> -
> -static struct platform_pwm_backlight_data smdk6410_backlight_data = {
> - .pwm_id = 1,
> - .max_brightness = 255,
> - .dft_brightness = 255,
> - .pwm_period_ns = 78770,
> - .init = smdk6410_backlight_init,
> - .exit = smdk6410_backlight_exit,
> -};
> -
> -static struct platform_device smdk6410_backlight_device = {
> - .name = "pwm-backlight",
> - .dev = {
> - .parent = &s3c_device_timer[1].dev,
> - .platform_data = &smdk6410_backlight_data,
> - },
> -};
> -
> static struct map_desc smdk6410_iodesc[] = {};
>
> static struct platform_device *smdk6410_devices[] __initdata = {
> --
> 1.7.2.3
OK, applied.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 5:24 ` Kukjin Kim
0 siblings, 0 replies; 16+ messages in thread
From: Kukjin Kim @ 2011-08-19 5:24 UTC (permalink / raw)
To: linux-arm-kernel
Banajit Goswami wrote:
>
> Remove un-used backlight code for SMDK6410 board
>
> Signed-off-by: Banajit Goswami <banajit.g@samsung.com>
> ---
> arch/arm/mach-s3c64xx/mach-smdk6410.c | 39
---------------------------------
> 1 files changed, 0 insertions(+), 39 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/mach-smdk6410.c b/arch/arm/mach-
> s3c64xx/mach-smdk6410.c
> index ecbea92..a9f3183 100644
> --- a/arch/arm/mach-s3c64xx/mach-smdk6410.c
> +++ b/arch/arm/mach-s3c64xx/mach-smdk6410.c
> @@ -262,45 +262,6 @@ static struct samsung_keypad_platdata
> smdk6410_keypad_data __initdata = {
> .cols = 8,
> };
>
> -static int smdk6410_backlight_init(struct device *dev)
> -{
> - int ret;
> -
> - ret = gpio_request(S3C64XX_GPF(15), "Backlight");
> - if (ret) {
> - printk(KERN_ERR "failed to request GPF for PWM-OUT1\n");
> - return ret;
> - }
> -
> - /* Configure GPIO pin with S3C64XX_GPF15_PWM_TOUT1 */
> - s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_SFN(2));
> -
> - return 0;
> -}
> -
> -static void smdk6410_backlight_exit(struct device *dev)
> -{
> - s3c_gpio_cfgpin(S3C64XX_GPF(15), S3C_GPIO_OUTPUT);
> - gpio_free(S3C64XX_GPF(15));
> -}
> -
> -static struct platform_pwm_backlight_data smdk6410_backlight_data = {
> - .pwm_id = 1,
> - .max_brightness = 255,
> - .dft_brightness = 255,
> - .pwm_period_ns = 78770,
> - .init = smdk6410_backlight_init,
> - .exit = smdk6410_backlight_exit,
> -};
> -
> -static struct platform_device smdk6410_backlight_device = {
> - .name = "pwm-backlight",
> - .dev = {
> - .parent = &s3c_device_timer[1].dev,
> - .platform_data = &smdk6410_backlight_data,
> - },
> -};
> -
> static struct map_desc smdk6410_iodesc[] = {};
>
> static struct platform_device *smdk6410_devices[] __initdata = {
> --
> 1.7.2.3
OK, applied.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-19 4:53 ` Mark Brown
@ 2011-08-19 5:29 ` Kukjin Kim
-1 siblings, 0 replies; 16+ messages in thread
From: Kukjin Kim @ 2011-08-19 5:29 UTC (permalink / raw)
To: 'Mark Brown', 'Banajit Goswami'
Cc: 'Banajit Goswami',
linux-samsung-soc, ben-linux, linux-arm-kernel
Mark Brown wrote:
>
> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
> > On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
> > <broonie@opensource.wolfsonmicro.com> wrote:
> > > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> > >> Remove un-used backlight code for SMDK6410 board
>
> > > Is the data actually wrong? If not it'd seem better to finish hooking
> > > it up properly than to remove it.
>
> > No, this code is no longer needed, as we are having a common backlight
> > infrastructure , which serves the same purpose.
>
> Ideally your commit log should make this clear - it looks like you're
> just deleting unreferenced code.
Yes, I agree with your opinion...It was difficult to know why it is un-used
code when I applied that.
OK, I will amend its commit log in my tree to clear.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 5:29 ` Kukjin Kim
0 siblings, 0 replies; 16+ messages in thread
From: Kukjin Kim @ 2011-08-19 5:29 UTC (permalink / raw)
To: linux-arm-kernel
Mark Brown wrote:
>
> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
> > On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
> > <broonie@opensource.wolfsonmicro.com> wrote:
> > > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
> > >> Remove un-used backlight code for SMDK6410 board
>
> > > Is the data actually wrong? ?If not it'd seem better to finish hooking
> > > it up properly than to remove it.
>
> > No, this code is no longer needed, as we are having a common backlight
> > infrastructure , which serves the same purpose.
>
> Ideally your commit log should make this clear - it looks like you're
> just deleting unreferenced code.
Yes, I agree with your opinion...It was difficult to know why it is un-used
code when I applied that.
OK, I will amend its commit log in my tree to clear.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-19 5:29 ` Kukjin Kim
@ 2011-08-19 5:39 ` Banajit Goswami
-1 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 5:39 UTC (permalink / raw)
To: Kukjin Kim
Cc: Mark Brown, Banajit Goswami, linux-samsung-soc, ben-linux,
linux-arm-kernel
Hi Mr. Kim,
On Fri, Aug 19, 2011 at 10:59 AM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Mark Brown wrote:
>>
>> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
>> > On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
>> > <broonie@opensource.wolfsonmicro.com> wrote:
>> > > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> > >> Remove un-used backlight code for SMDK6410 board
>>
>> > > Is the data actually wrong? If not it'd seem better to finish hooking
>> > > it up properly than to remove it.
>>
>> > No, this code is no longer needed, as we are having a common backlight
>> > infrastructure , which serves the same purpose.
>>
>> Ideally your commit log should make this clear - it looks like you're
>> just deleting unreferenced code.
>
> Yes, I agree with your opinion...It was difficult to know why it is un-used
> code when I applied that.
>
Yes, I will explain more in comments in future!
> OK, I will amend its commit log in my tree to clear.
Thank you!
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 5:39 ` Banajit Goswami
0 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 5:39 UTC (permalink / raw)
To: linux-arm-kernel
Hi Mr. Kim,
On Fri, Aug 19, 2011 at 10:59 AM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> Mark Brown wrote:
>>
>> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
>> > On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
>> > <broonie@opensource.wolfsonmicro.com> wrote:
>> > > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> > >> Remove un-used backlight code for SMDK6410 board
>>
>> > > Is the data actually wrong? ?If not it'd seem better to finish hooking
>> > > it up properly than to remove it.
>>
>> > No, this code is no longer needed, as we are having a common backlight
>> > infrastructure , which serves the same purpose.
>>
>> Ideally your commit log should make this clear - it looks like you're
>> just deleting unreferenced code.
>
> Yes, I agree with your opinion...It was difficult to know why it is un-used
> code when I applied that.
>
Yes, I will explain more in comments in future!
> OK, I will amend its commit log in my tree to clear.
Thank you!
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: S3C64XX: Remove un-used code
2011-08-19 4:53 ` Mark Brown
@ 2011-08-19 6:59 ` Banajit Goswami
-1 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 6:59 UTC (permalink / raw)
To: Mark Brown
Cc: Banajit Goswami, linux-samsung-soc, kgene.kim, ben-linux,
linux-arm-kernel
Hi Mark,
On Fri, Aug 19, 2011 at 10:23 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
>> On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
>> <broonie@opensource.wolfsonmicro.com> wrote:
>> > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> >> Remove un-used backlight code for SMDK6410 board
>
>> > Is the data actually wrong? If not it'd seem better to finish hooking
>> > it up properly than to remove it.
>
>> No, this code is no longer needed, as we are having a common backlight
>> infrastructure , which serves the same purpose.
>
> Ideally your commit log should make this clear - it looks like you're
> just deleting unreferenced code.
Thanks.
Will add more description now on.
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: S3C64XX: Remove un-used code
@ 2011-08-19 6:59 ` Banajit Goswami
0 siblings, 0 replies; 16+ messages in thread
From: Banajit Goswami @ 2011-08-19 6:59 UTC (permalink / raw)
To: linux-arm-kernel
Hi Mark,
On Fri, Aug 19, 2011 at 10:23 AM, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Fri, Aug 19, 2011 at 09:40:29AM +0530, Banajit Goswami wrote:
>> On Fri, Aug 19, 2011 at 7:29 AM, Mark Brown
>> <broonie@opensource.wolfsonmicro.com> wrote:
>> > On Tue, Aug 16, 2011 at 06:11:28PM +0530, Banajit Goswami wrote:
>> >> Remove un-used backlight code for SMDK6410 board
>
>> > Is the data actually wrong? ?If not it'd seem better to finish hooking
>> > it up properly than to remove it.
>
>> No, this code is no longer needed, as we are having a common backlight
>> infrastructure , which serves the same purpose.
>
> Ideally your commit log should make this clear - it looks like you're
> just deleting unreferenced code.
Thanks.
Will add more description now on.
Thanks and Regards,
Banajit Goswami
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2011-08-19 6:59 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-16 12:41 [PATCH] ARM: S3C64XX: Remove un-used code Banajit Goswami
2011-08-16 12:41 ` Banajit Goswami
2011-08-19 1:59 ` Mark Brown
2011-08-19 1:59 ` Mark Brown
2011-08-19 4:10 ` Banajit Goswami
2011-08-19 4:10 ` Banajit Goswami
2011-08-19 4:53 ` Mark Brown
2011-08-19 4:53 ` Mark Brown
2011-08-19 5:29 ` Kukjin Kim
2011-08-19 5:29 ` Kukjin Kim
2011-08-19 5:39 ` Banajit Goswami
2011-08-19 5:39 ` Banajit Goswami
2011-08-19 6:59 ` Banajit Goswami
2011-08-19 6:59 ` Banajit Goswami
2011-08-19 5:24 ` Kukjin Kim
2011-08-19 5:24 ` Kukjin Kim
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.