All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm: vmlinux.lds.S: drop redundant .comment
@ 2014-02-12 11:13 Mark Rutland
  0 siblings, 0 replies; 3+ messages in thread
From: Mark Rutland @ 2014-02-12 11:13 UTC (permalink / raw)
  To: linux-arm-kernel

Commit 78d7530ac3 ("ARM: Clean up linker script using new linker script
macros.") modified the arm kernel linker script to use the STABS_DEBUG
macro, but left a .comment section definition. As STABS_DEBUG defines
the .comment section in an identical way, the second section definition
is redundant and can be removed.

This patch removes the redundant .comment section definition.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
---
 arch/arm/kernel/vmlinux.lds.S | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/kernel/vmlinux.lds.S b/arch/arm/kernel/vmlinux.lds.S
index 7bcee5c..6f57cb9 100644
--- a/arch/arm/kernel/vmlinux.lds.S
+++ b/arch/arm/kernel/vmlinux.lds.S
@@ -318,7 +318,6 @@ SECTIONS
 	_end = .;
 
 	STABS_DEBUG
-	.comment 0 : { *(.comment) }
 }
 
 /*
-- 
1.8.1.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH] arm: vmlinux.lds.S: drop redundant .comment
  2014-06-24 17:29 Mark Rutland
@ 2014-06-30 14:45 ` Mark Rutland
  0 siblings, 0 replies; 3+ messages in thread
From: Mark Rutland @ 2014-06-30 14:45 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On Tue, Jun 24, 2014 at 06:29:08PM +0100, Mark Rutland wrote:
> Commit 78d7530ac3 ("ARM: Clean up linker script using new linker script
> macros.") modified the arm kernel linker script to use the STABS_DEBUG
> macro, but left a .comment section definition. As STABS_DEBUG defines
> the .comment section in an identical way, the second section definition
> is redundant and can be removed.
> 
> This patch removes the redundant .comment section definition.
> 
> Signed-off-by: Mark Rutland <mark.rutland@arm.com>
> ---
>  arch/arm/kernel/vmlinux.lds.S | 1 -
>  1 file changed, 1 deletion(-)

On the assumption that the below patch is fine I've dropped it into the
patch system as 8088/1.

Thanks,
Mark.

> 
> diff --git a/arch/arm/kernel/vmlinux.lds.S b/arch/arm/kernel/vmlinux.lds.S
> index 7bcee5c..6f57cb9 100644
> --- a/arch/arm/kernel/vmlinux.lds.S
> +++ b/arch/arm/kernel/vmlinux.lds.S
> @@ -318,7 +318,6 @@ SECTIONS
>  	_end = .;
>  
>  	STABS_DEBUG
> -	.comment 0 : { *(.comment) }
>  }
>  
>  /*
> -- 
> 1.9.1
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] arm: vmlinux.lds.S: drop redundant .comment
@ 2014-06-24 17:29 Mark Rutland
  2014-06-30 14:45 ` Mark Rutland
  0 siblings, 1 reply; 3+ messages in thread
From: Mark Rutland @ 2014-06-24 17:29 UTC (permalink / raw)
  To: linux-arm-kernel

Commit 78d7530ac3 ("ARM: Clean up linker script using new linker script
macros.") modified the arm kernel linker script to use the STABS_DEBUG
macro, but left a .comment section definition. As STABS_DEBUG defines
the .comment section in an identical way, the second section definition
is redundant and can be removed.

This patch removes the redundant .comment section definition.

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
---
 arch/arm/kernel/vmlinux.lds.S | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/kernel/vmlinux.lds.S b/arch/arm/kernel/vmlinux.lds.S
index 7bcee5c..6f57cb9 100644
--- a/arch/arm/kernel/vmlinux.lds.S
+++ b/arch/arm/kernel/vmlinux.lds.S
@@ -318,7 +318,6 @@ SECTIONS
 	_end = .;
 
 	STABS_DEBUG
-	.comment 0 : { *(.comment) }
 }
 
 /*
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-06-30 14:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-12 11:13 [PATCH] arm: vmlinux.lds.S: drop redundant .comment Mark Rutland
2014-06-24 17:29 Mark Rutland
2014-06-30 14:45 ` Mark Rutland

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.