public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Igor Grinberg <grinberg@compulab.co.il>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/3] arm: omap3: Add SPL support to cm_t35
Date: Mon, 17 Jun 2013 16:38:57 +0300	[thread overview]
Message-ID: <51BF1171.5010400@compulab.co.il> (raw)
In-Reply-To: <20130617123850.GA8619@bill-the-cat>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 06/17/13 15:38, Tom Rini wrote:
> On Mon, Jun 17, 2013 at 02:53:27PM +0300, Igor Grinberg wrote:
>> Hi Stefan,
>>
>> On 06/14/13 11:55, Stefan Roese wrote:
>>> Add SPL U-Boot support to replace x-loader on the Compulab cm_t35
>>> board. Currently only the 256MiB SDRAM board versions are supported.
>>>
>>> Tested by booting via MMC and NAND.
>>>
>>> Signed-off-by: Stefan Roese <sr@denx.de>
>>> Cc: Tom Rini <trini@ti.com>
>>> Cc: Igor Grinberg <grinberg@compulab.co.il>
> [snip]
>>> +/*
>>> + * 1MB into the SDRAM to allow for SPL's bss at the beginning of SDRAM
>>> + * 64 bytes before this address should be set aside for u-boot.img's
>>> + * header. That is 0x800FFFC0--0x80100000 should not be used for any
>>> + * other needs.
>>> + */
>>> +#define CONFIG_SYS_TEXT_BASE		0x80100000
>>
>> Now this is a problem.
>> This breaks the backward compatibility with our X-Loader and we
>> cannot just switch to 80100000...
> 
> And thinking back to when I was doing more OMAP3 conversions, there's no
> requirement to break compatibility with x-loader either.  You just need
> to take care where you place things, see doc/SPL/README.omap3 for the
> SPL and X-Loader compatible setup.

Actually, I was thinking about adding a target in boards.cfg, but
if we can make both (X-Loader and SPL) live in piece, then IMO,
we should go for it.

Thanks for the pointer!


- -- 
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)

iQIcBAEBAgAGBQJRvxFwAAoJEBDE8YO64Efa0DsP/R0053VMz10juUsP38dFb7lb
xcKQFmvJe0W1skIJZmFAa+1gUNYf1EMcHGx7JqIwtsVDBsHxK5fzAbj1mU4RbQUx
zY5nmGtKlfiV1xPMQSrwo6rDxX3pPRId5rfC9sxpCwdliCFzvypfP6mPTcGm10db
2MygHvm+cXU1hNi7JPF3eRCVuqDhO/mrnES0MD2tkxfF88sJOYvvCBSLndJy+cF2
8FrnAC1w+4xnUqT33VTAf/P1+/2qzFcnrVXRNBn/fRfvahT91EuGwWrzqRrvvwNd
lMTwSreO4RvvG6pa0RGgu6I6/6Ao5CYbjkznDzeyMl+001a2hxIlwuXb+yJaNneN
lISSWSAnTT6AcoIRDGVOmFkWOzfmgZoqsikeBKA22jyzZymoXWwhJ5RM5/l0/OKu
V7/X2vWuzTeAzbOlRMof25DtLk/vXdYA4vt6b9iyIffVPSq9WWGGJhPhgSfQ5rW1
S8k1JXbIn99TY6igyaK+7rf42FxOMZLyPdKH53qCM2G80XGy+Df7y2ZHyv8ssQRY
f5w2CYSDo4ph01tGo3hI5bvz671yAoBUs6+/7ERofJ9gx2R0XOKUhfY6znIXeDMl
o8AqyYpXTboDG9FRH0cE6Lud4CUt2sY2NQ/CkfdXyYIfeOGVqWUbLefc9cZNBGDe
4AoUHehr5tgIFNiT5yP+
=X70e
-----END PGP SIGNATURE-----

  reply	other threads:[~2013-06-17 13:38 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-14  8:54 [U-Boot] [PATCH 1/3] arm: spl: Fix SPL booting for OMAP3 Stefan Roese
2013-06-14  8:55 ` [U-Boot] [PATCH 2/3] arm: omap3: spl: Fix problem with 8bit NAND devices Stefan Roese
2013-07-30 13:25   ` [U-Boot] [U-Boot, " Tom Rini
2013-06-14  8:55 ` [U-Boot] [PATCH 3/3] arm: omap3: Add SPL support to cm_t35 Stefan Roese
2013-06-17 11:53   ` Igor Grinberg
2013-06-17 12:38     ` Tom Rini
2013-06-17 13:38       ` Igor Grinberg [this message]
2013-06-17 13:52         ` Stefan Roese
2013-06-17 14:03   ` [U-Boot] [PATCH v2 " Stefan Roese
2013-06-18  6:14     ` Nikita Kiryanov
2013-07-30 10:52   ` [U-Boot] [PATCH v3 " Stefan Roese
2013-07-30 12:10     ` Albert ARIBAUD
2013-07-30 12:14       ` Stefan Roese
2013-11-15  7:51   ` [U-Boot] [PATCH v4] " Stefan Roese
     [not found]     ` <5288BBAC.2020307@compulab.co.il>
     [not found]       ` <5295BF6C.20902@compulab.co.il>
     [not found]         ` <5295C3F8.50101@denx.de>
     [not found]           ` <529E01B7.4070402@compulab.co.il>
2013-12-04 11:38             ` [U-Boot] Fwd: " Stefan Roese
2013-12-04 11:57               ` Tom Rini
2013-12-04 12:02                 ` Stefan Roese
2013-12-04 12:15               ` Gupta, Pekon
2013-12-04 12:38                 ` Stefan Roese
2013-06-20 16:42 ` [U-Boot] [PATCH 1/3] arm: spl: Fix SPL booting for OMAP3 Albert ARIBAUD
2013-06-20 17:01   ` Stefan Roese
2013-06-20 17:51     ` Albert ARIBAUD
2013-06-20 18:28       ` Stefan Roese
2013-06-20 19:18         ` Albert ARIBAUD
2013-06-21  2:13 ` [U-Boot] [PATCH v2 " Stefan Roese
2013-06-21  8:57   ` Albert ARIBAUD
2013-06-21  9:10 ` [U-Boot] [PATCH v3 " Stefan Roese
2013-06-21 10:30   ` Albert ARIBAUD
2013-06-21 10:39     ` Stefan Roese
2013-06-21 10:42 ` [U-Boot] [PATCH v4 " Stefan Roese
2013-06-21 11:02   ` Albert ARIBAUD
2013-06-25  7:14 ` [U-Boot] [PATCH v5 " Stefan Roese
2013-06-27  8:27   ` Albert ARIBAUD
2013-07-03 19:47     ` Tom Rini
2013-07-04 11:58       ` Albert ARIBAUD
2013-07-15 14:33 ` [U-Boot] [PATCH " Tom Rini
2013-07-16  6:24   ` Stefan Roese
2013-07-16 14:36     ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51BF1171.5010400@compulab.co.il \
    --to=grinberg@compulab.co.il \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox