[PATCH 0/5] MIPS: start*.S and barebox.lds.S fixups

[PATCH 0/5] MIPS: start*.S and barebox.lds.S fixups

[Antony Pavlov]

Antony Pavlov (5):
  MIPS: start*.S: drop redundant .text
  MIPS: barebox.lds.S: drop unneeded '_start = .'
  MIPS: barebox.lds.S: remove unused got
  MIPS: barebox.lds.S: put .text_entry contents after _stext and _text
  MIPS: start.S: put entry code into the ".text_entry" section

 arch/mips/boot/start-pbl.S  | 1 -
 arch/mips/boot/start.S      | 3 +--
 arch/mips/lib/barebox.lds.S | 6 +-----
 3 files changed, 2 insertions(+), 8 deletions(-)

-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 1/5] MIPS: start*.S: drop redundant .text

[Antony Pavlov]

`.text' is equivalent to the `.section .text' directive.
The `.text' directive just before `.section ".text_bare_init"'
or just before `.section ".text_head_entry"' is redundant
so drop it.

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/boot/start-pbl.S | 1 -
 arch/mips/boot/start.S     | 1 -
 2 files changed, 2 deletions(-)

diff --git a/arch/mips/boot/start-pbl.S b/arch/mips/boot/start-pbl.S
index b6d127a039..d6e8231a73 100644
--- a/arch/mips/boot/start-pbl.S
+++ b/arch/mips/boot/start-pbl.S
@@ -25,7 +25,6 @@
 #include <board/board_pbl_start.h>
 
 	.set noreorder
-	.text
 	.section ".text_head_entry"
 	.align 4
 
diff --git a/arch/mips/boot/start.S b/arch/mips/boot/start.S
index 1ddce9533e..0828ee4927 100644
--- a/arch/mips/boot/start.S
+++ b/arch/mips/boot/start.S
@@ -20,7 +20,6 @@
 #include <asm/pbl_macros.h>
 
 	.set noreorder
-	.text
 	.section ".text_bare_init"
 	.align 4
 
-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 2/5] MIPS: barebox.lds.S: drop unneeded '_start = .'

[Antony Pavlov]

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/lib/barebox.lds.S | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/mips/lib/barebox.lds.S b/arch/mips/lib/barebox.lds.S
index 8057634ddb..ffe846728f 100644
--- a/arch/mips/lib/barebox.lds.S
+++ b/arch/mips/lib/barebox.lds.S
@@ -27,7 +27,6 @@ SECTIONS
 	. = ALIGN(4);
 	.text      :
 	{
-		_start = .;
 		*(.text_entry*)
 		_stext = .;
 		_text = .;
-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 3/5] MIPS: barebox.lds.S: remove unused got

[Antony Pavlov]

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/lib/barebox.lds.S | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/arch/mips/lib/barebox.lds.S b/arch/mips/lib/barebox.lds.S
index ffe846728f..cc214920e3 100644
--- a/arch/mips/lib/barebox.lds.S
+++ b/arch/mips/lib/barebox.lds.S
@@ -50,9 +50,6 @@ SECTIONS
 
 	.barebox_imd : { BAREBOX_IMD }
 
-	. = ALIGN(4);
-	.got : { *(.got*) }
-
 	. = .;
 	__barebox_cmd_start = .;
 	.barebox_cmd : { BAREBOX_CMDS }
-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 4/5] MIPS: barebox.lds.S: put .text_entry contents after _stext and _text

[Antony Pavlov]

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/lib/barebox.lds.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/lib/barebox.lds.S b/arch/mips/lib/barebox.lds.S
index cc214920e3..899f62b968 100644
--- a/arch/mips/lib/barebox.lds.S
+++ b/arch/mips/lib/barebox.lds.S
@@ -27,9 +27,9 @@ SECTIONS
 	. = ALIGN(4);
 	.text      :
 	{
-		*(.text_entry*)
 		_stext = .;
 		_text = .;
+		*(.text_entry*)
 		__bare_init_start = .;
 		*(.text_bare_init*)
 		__bare_init_end = .;
-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 5/5] MIPS: start.S: put entry code into the ".text_entry" section

[Antony Pavlov]

The ".text_bare_init" section is not intended to be used
for storing entry code.

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/boot/start.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/boot/start.S b/arch/mips/boot/start.S
index 0828ee4927..e937e89af1 100644
--- a/arch/mips/boot/start.S
+++ b/arch/mips/boot/start.S
@@ -20,7 +20,7 @@
 #include <asm/pbl_macros.h>
 
 	.set noreorder
-	.section ".text_bare_init"
+	.section ".text_entry"
 	.align 4
 
 EXPORT(_start)
-- 
2.11.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 0/5] MIPS: start*.S and barebox.lds.S fixups

[Sascha Hauer]

On Fri, May 05, 2017 at 02:34:30PM +0300, Antony Pavlov wrote:
> Antony Pavlov (5):
>   MIPS: start*.S: drop redundant .text
>   MIPS: barebox.lds.S: drop unneeded '_start = .'
>   MIPS: barebox.lds.S: remove unused got
>   MIPS: barebox.lds.S: put .text_entry contents after _stext and _text
>   MIPS: start.S: put entry code into the ".text_entry" section

Applied, thanks

Sascha

> 
>  arch/mips/boot/start-pbl.S  | 1 -
>  arch/mips/boot/start.S      | 3 +--
>  arch/mips/lib/barebox.lds.S | 6 +-----
>  3 files changed, 2 insertions(+), 8 deletions(-)
> 
> -- 
> 2.11.0
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox