From: Jan Weitzel <j.weitzel@phytec.de>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH 2/2] ARM: omap: Add bootsource serial to xload
Date: Thu, 13 Nov 2014 13:34:07 +0100 [thread overview]
Message-ID: <5464A53F.1030505@phytec.de> (raw)
In-Reply-To: <20141113073732.GN30369@pengutronix.de>
Am 13.11.2014 um 08:37 schrieb Sascha Hauer:
> On Wed, Nov 12, 2014 at 10:24:48AM +0100, Jan Weitzel wrote:
>> Am 12.11.2014 um 09:22 schrieb Sascha Hauer:
>>> Hi Jan,
>>>
>>> On Wed, Nov 12, 2014 at 08:41:54AM +0100, Jan Weitzel wrote:
>>>> If booted from serial via xmodem, also get barebox.bin per xmodem
>>>> For first stage you need the .pblx file instead of MLO.
>>>>
>>>> Signed-off-by: Jan Weitzel <j.weitzel@phytec.de>
>>>> ---
>>>> arch/arm/mach-omap/Kconfig | 7 +++++++
>>>> arch/arm/mach-omap/xload.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++
>>>> 2 files changed, 53 insertions(+)
>>>>
>>>> diff --git a/arch/arm/mach-omap/Kconfig b/arch/arm/mach-omap/Kconfig
>>>> index f9b5ec3..953dd8f 100644
>>>> --- a/arch/arm/mach-omap/Kconfig
>>>> +++ b/arch/arm/mach-omap/Kconfig
>>>> @@ -118,6 +118,13 @@ config OMAP4_USBBOOT
>>>> You need the utility program omap4_usbboot to boot from USB.
>>>> Please read omap4_usb_booting.txt for more information.
>>>> +config OMAP_SERIALBOOT
>>>> + bool
>>>> + default y
>>> No default y please. Better keep it disabled by default because the
>>> OMAP4 xload configs are quiet tight I think.
>> Is adding it to am335x_mlo_defconfig ok? Because of the size I added the
>> ARCH_AM33XX dependency.
> I overlooked this dependency. So the dependency is only because of the
> binary size and it's otherwise usable on Omap aswell? In this case I
> suggest to drop the dependency. And yes, ok to add it to the
> am335x_mlo_defconfig.
Only AM335x set BOOTSOURCE_SERIAL by now
Jan
>
> Sascha
>
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2014-11-13 12:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-12 7:41 [PATCH 1/2] xymodem: select CRC16 from lib Jan Weitzel
2014-11-12 7:41 ` [PATCH 2/2] ARM: omap: Add bootsource serial to xload Jan Weitzel
2014-11-12 8:22 ` Sascha Hauer
2014-11-12 9:24 ` Jan Weitzel
2014-11-13 7:37 ` Sascha Hauer
2014-11-13 12:31 ` [PATCH v2] " Jan Weitzel
2014-11-17 7:34 ` Sascha Hauer
2014-11-13 12:34 ` Jan Weitzel [this message]
2014-11-12 8:23 ` [PATCH 1/2] xymodem: select CRC16 from lib Sascha Hauer
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=5464A53F.1030505@phytec.de \
--to=j.weitzel@phytec.de \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.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 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.