On 08/24/2012 09:15 PM, Tom Rini wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it this way. I'll send a patch to
change this shortly (if everything works
Hi Tom,
On 08/23/2012 11:39 PM, Tom Rini wrote:
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_ARM
gd = gdata;
+#endif
So, here's what I don't understand. On ARM, in
Hi Tom,
On 08/23/2012 11:52 PM, Tom Rini wrote:
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_PPC
+static void __noreturn jump_to_image_linux(void *arg)
+{
+debug(Entering
Hi Tom,
On 08/23/2012 09:31 PM, Tom Rini wrote:
@@ -89,7 +106,11 @@ void spl_parse_image_header(const struct image_header
*header)
spl_image.size = __be32_to_cpu(header-ih_size) + header_size;
spl_image.entry_point = __be32_to_cpu(header-ih_load);
/* Load
Hello Stefan
On 24.08.2012 10:17, Stefan Roese wrote:
Hi Tom,
On 08/23/2012 09:31 PM, Tom Rini wrote:
@@ -89,7 +106,11 @@ void spl_parse_image_header(const struct image_header
*header)
spl_image.size = __be32_to_cpu(header-ih_size) + header_size;
Hi Heiko,
On 08/24/2012 12:17 PM, Heiko Schocher wrote:
BTW: There also seems to be a bug in some of the SPL loaders:
For example in drivers/mtd/nand/nand_spl_load.c:
...
if (header-ih_os == IH_OS_LINUX) {
/* happy - was a linux */
On 08/24/2012 12:56 PM, Stefan Roese wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it this way. I'll send a patch to
change this shortly (if everything
Hi Stefan,
2012/8/24 Stefan Roese s...@denx.de:
On 08/24/2012 12:56 PM, Stefan Roese wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it this way. I'll send a
Hi Daniel,
On 08/24/2012 01:49 PM, Daniel Schwierzeck wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it this way. I'll send a patch to
change this shortly
Hi Stefan,
2012/8/24 Stefan Roese s...@denx.de:
Hi Daniel,
On 08/24/2012 01:49 PM, Daniel Schwierzeck wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it
On 08/24/2012 12:01 AM, Stefan Roese wrote:
Hi Tom,
On 08/23/2012 11:39 PM, Tom Rini wrote:
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_ARM
gd = gdata;
+#endif
So, here's
Hi Daniel,
On 08/24/2012 04:11 PM, Stefan Roese wrote:
The u-boot image is merged with SPL
image without any padding or
fixed flash offsets to achieve a maximum reduction of flash footprint.
Interesting. I'm still padding to the fixed offset. Let me look into
squeezing those two images
On 08/24/2012 05:55 PM, Tom Rini wrote:
So, here's what I don't understand. On ARM, in general, we can't rely
on the global data pointer register (r8) to be set to a useful value, so
we do the above to ensure it points to something useful. Are you always
able to rely on r2 it looks like
On 08/24/2012 09:07 AM, Stefan Roese wrote:
On 08/24/2012 05:55 PM, Tom Rini wrote:
So, here's what I don't understand. On ARM, in general, we can't rely
on the global data pointer register (r8) to be set to a useful value, so
we do the above to ensure it points to something useful. Are you
Hi Stefan,
2012/8/24 Stefan Roese stefan.ro...@gmail.com:
Hi Daniel,
On 08/24/2012 04:11 PM, Stefan Roese wrote:
The u-boot image is merged with SPL
image without any padding or
fixed flash offsets to achieve a maximum reduction of flash footprint.
Interesting. I'm still padding to the
On 08/24/2012 06:19 PM, Tom Rini wrote:
What do you think? Can you move this gd init stuff into such a common
ARM spl file in the next patchset version?
Yes. We should probably say it's the job of board_init_f. I know full
U-Boot does it at the start of board_init_r but I think for SPL it
On 08/24/2012 06:42 PM, Daniel Schwierzeck wrote:
The u-boot image is merged with SPL
image without any padding or
fixed flash offsets to achieve a maximum reduction of flash footprint.
Interesting. I'm still padding to the fixed offset. Let me look into
squeezing those two images together
On 08/24/2012 04:13 AM, Stefan Roese wrote:
On 08/24/2012 12:56 PM, Stefan Roese wrote:
I still would like to move to my suggestion to not copy the header and
use the mkimage header values ih_load and ih_ep directly. Right now I
don't see any showstopper for doing it this way. I'll send a
2012/8/24 Stefan Roese stefan.ro...@gmail.com:
On 08/24/2012 06:42 PM, Daniel Schwierzeck wrote:
The u-boot image is merged with SPL
image without any padding or
fixed flash offsets to achieve a maximum reduction of flash footprint.
Interesting. I'm still padding to the fixed offset. Let me
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_ARM
/* Define global data structure pointer to it*/
gd_t gdata __attribute__ ((section(.data)));
+#endif
So you handle cleaning up the
On 08/23/2012 07:10 PM, Tom Rini wrote:
+#ifdef CONFIG_ARM
/* Define global data structure pointer to it*/
gd_t gdata __attribute__ ((section(.data)));
+#endif
So you handle cleaning up the BSS differently, interesting. I'm going
to see if that would work for ARM too..
Yes. Might be
On 08/23/2012 11:16 AM, Stefan Roese wrote:
On 08/23/2012 07:10 PM, Tom Rini wrote:
+#ifdef CONFIG_ARM
/* Define global data structure pointer to it*/
gd_t gdata __attribute__ ((section(.data)));
+#endif
So you handle cleaning up the BSS differently, interesting. I'm going
to see if
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_ARM
gd = gdata;
+#endif
So, here's what I don't understand. On ARM, in general, we can't rely
on the global data pointer register
On 08/23/2012 01:12 AM, Stefan Roese wrote:
This patch enables the SPL framework to be used on powerpc platforms
and not only ARM.
[snip]
+#ifdef CONFIG_PPC
+static void __noreturn jump_to_image_linux(void *arg)
+{
+ debug(Entering kernel arg pointer: 0x%p\n, arg);
+ typedef void
24 matches
Mail list logo