subject:"\[PATCH 1\/2\] powerpc\: don't pass unused regs around in head

Re: [PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

2008-10-14 Thread Sebastian Andrzej Siewior

* Benjamin Herrenschmidt | 2008-10-14 10:25:12 [+1100]:

On Sun, 2008-10-12 at 16:08 +0200, Sebastian Andrzej Siewior wrote:
 From: Sebastian Andrzej Siewior [EMAIL PROTECTED]
 
 This looks like a relict from arch/ppc. machine_init() is accepting
 only two parameters (dtb, phys) and is using only the first one.

This isn't 100% correct actually...

First, the base head_32.S (could be called head_6xx.S I suppose)
supports a few more calling conventions such as the real OF one, with
added support for initrd and cmdline in registers , and the BootX one.
Yep, those extra register are only used in prom_init() or bootx_init().
In case we called one of those two we end up again in __start(dtb, phys, 0)

Then, calling convention for the other cases is slightly better defined
than just having r3 contain a device-tree pointer. The physical address
is an important part of it, the fact that r5 is NULL to differenciate
from an OF entry too, and we're moving toward the full ePAPR definition.
Okay. Power_ePAPR_APPROVED_v1.0.pdf says we have r4, r5, r8, r9 = 0, r6 =
magic and r7 size of the initial mapped area. I don't see the physical
address here.

Do you want me to update the comment / patch description or leave
everything as it? I guess we don't have to pass unused register to
machine_init(), do we?

Cheers,
Ben.

Sebastian
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

2008-10-13 Thread Benjamin Herrenschmidt

On Sun, 2008-10-12 at 16:08 +0200, Sebastian Andrzej Siewior wrote:
 From: Sebastian Andrzej Siewior [EMAIL PROTECTED]
 
 This looks like a relict from arch/ppc. machine_init() is accepting
 only two parameters (dtb, phys) and is using only the first one.

This isn't 100% correct actually...

First, the base head_32.S (could be called head_6xx.S I suppose)
supports a few more calling conventions such as the real OF one, with
added support for initrd and cmdline in registers , and the BootX one.

Then, calling convention for the other cases is slightly better defined
than just having r3 contain a device-tree pointer. The physical address
is an important part of it, the fact that r5 is NULL to differenciate
from an OF entry too, and we're moving toward the full ePAPR definition.

Cheers,
Ben.


___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

[PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

2008-10-12 Thread Sebastian Andrzej Siewior

From: Sebastian Andrzej Siewior [EMAIL PROTECTED]

This looks like a relict from arch/ppc. machine_init() is accepting
only two parameters (dtb, phys) and is using only the first one.

Signed-off-by: Sebastian Andrzej Siewior [EMAIL PROTECTED]
---
 arch/powerpc/kernel/head_32.S|8 +---
 arch/powerpc/kernel/head_40x.S   |   14 +-
 arch/powerpc/kernel/head_44x.S   |   14 +-
 arch/powerpc/kernel/head_8xx.S   |   14 +-
 arch/powerpc/kernel/head_fsl_booke.S |   14 +-
 5 files changed, 5 insertions(+), 59 deletions(-)

diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S
index a6de6db..2f5c4fc 100644
--- a/arch/powerpc/kernel/head_32.S
+++ b/arch/powerpc/kernel/head_32.S
@@ -89,11 +89,7 @@ _ENTRY(_start);
  * This is jumped to on prep systems right after the kernel is relocated
  * to its proper place in memory by the boot loader.  The expected layout
  * of the regs is:
- *   r3: ptr to residual data
- *   r4: initrd_start or if no initrd then 0
- *   r5: initrd_end - unused if r4 is 0
- *   r6: Start of command line string
- *   r7: End of command line string
+ *   r3 - Device tree pointer (DRAM, frequency, MAC address, etc.)
  *
  * This just gets a minimal mmu environment setup so we can call
  * start_here() to do the real work.
@@ -128,7 +124,6 @@ __start:
 #endif /* CONFIG_PPC_PMAC */
 
 1: mr  r31,r3  /* save parameters */
-   mr  r30,r4
li  r24,0   /* cpu # */
 
 /*
@@ -1017,7 +1012,6 @@ start_here:
  * and set up the MMU.
  */
mr  r3,r31
-   mr  r4,r30
bl  machine_init
bl  __save_cpu_setup
bl  MMU_init
diff --git a/arch/powerpc/kernel/head_40x.S b/arch/powerpc/kernel/head_40x.S
index 56d8e5d..1cdd4ce 100644
--- a/arch/powerpc/kernel/head_40x.S
+++ b/arch/powerpc/kernel/head_40x.S
@@ -44,11 +44,7 @@
  * execution begins here, the following registers contain valid, yet
  * optional, information:
  *
- *   r3 - Board info structure pointer (DRAM, frequency, MAC address, etc.)
- *   r4 - Starting address of the init RAM disk
- *   r5 - Ending address of the init RAM disk
- *   r6 - Start of kernel command line string (e.g. mem=96m)
- *   r7 - End of kernel command line string
+ *   r3 - Device tree pointer (DRAM, frequency, MAC address, etc.)
  *
  * This is all going to change RSN when we add bi_recs...  -- Dan
  */
@@ -59,10 +55,6 @@ _ENTRY(_start);
/* Save parameters we are passed.
*/
mr  r31,r3
-   mr  r30,r4
-   mr  r29,r5
-   mr  r28,r6
-   mr  r27,r7
 
/* We have to turn on the MMU right away so we get cache modes
 * set correctly.
@@ -848,10 +840,6 @@ start_here:
  * Decide what sort of machine this is and initialize the MMU.
  */
mr  r3,r31
-   mr  r4,r30
-   mr  r5,r29
-   mr  r6,r28
-   mr  r7,r27
bl  machine_init
bl  MMU_init
 
diff --git a/arch/powerpc/kernel/head_44x.S b/arch/powerpc/kernel/head_44x.S
index f3a1ea9..8098b59 100644
--- a/arch/powerpc/kernel/head_44x.S
+++ b/arch/powerpc/kernel/head_44x.S
@@ -43,11 +43,7 @@
  * execution begins here, the following registers contain valid, yet
  * optional, information:
  *
- *   r3 - Board info structure pointer (DRAM, frequency, MAC address, etc.)
- *   r4 - Starting address of the init RAM disk
- *   r5 - Ending address of the init RAM disk
- *   r6 - Start of kernel command line string (e.g. mem=128)
- *   r7 - End of kernel command line string
+ *   r3 - Device tree pointer (DRAM, frequency, MAC address, etc.)
  *
  */
.section.text.head, ax
@@ -62,10 +58,6 @@ _ENTRY(_start);
  * Save parameters we are passed
  */
mr  r31,r3
-   mr  r30,r4
-   mr  r29,r5
-   mr  r28,r6
-   mr  r27,r7
li  r24,0   /* CPU number */
 
 /*
@@ -241,10 +233,6 @@ skpinv:addir4,r4,1 /* 
Increment */
  * Decide what sort of machine this is and initialize the MMU.
  */
mr  r3,r31
-   mr  r4,r30
-   mr  r5,r29
-   mr  r6,r28
-   mr  r7,r27
bl  machine_init
bl  MMU_init
 
diff --git a/arch/powerpc/kernel/head_8xx.S b/arch/powerpc/kernel/head_8xx.S
index 3c9452d..a61d563 100644
--- a/arch/powerpc/kernel/head_8xx.S
+++ b/arch/powerpc/kernel/head_8xx.S
@@ -46,11 +46,7 @@ _ENTRY(_start);
  * This port was done on an MBX board with an 860.  Right now I only
  * support an ELF compressed (zImage) boot from EPPC-Bug because the
  * code there loads up some registers before calling us:
- *   r3: ptr to board info data
- *   r4: initrd_start or if no initrd then 0
- *   r5: initrd_end - unused if r4 is 0
- *   r6: Start of command line string
- *   r7: End of command line string
+ *   r3 - Device tree pointer (DRAM, frequency, MAC address, etc.)

Re: [PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

Re: [PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

[PATCH 1/2] powerpc: don't pass unused regs around in head_.*.S

3 matches

Site Navigation

Mail list logo

Footer information