Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 9/9] boot/syslinux: add an option to install an MBR blob
Date: Mon, 28 Apr 2014 22:57:42 +0200	[thread overview]
Message-ID: <535EC0C6.9050905@mind.be> (raw)
In-Reply-To: <20140428195241.GG3501@free.fr>

On 28/04/14 21:52, Yann E. MORIN wrote:
> Arnout, All,
> 
> On 2014-04-28 18:23 +0200, Arnout Vandecappelle spake thusly:
>               ^^^^^
> 
> I don't know why, but I only received it now (delivered to my server
> at 21:03:30 +0200) ? ... :-/

 That's because I read and reply to my buildroot mail on the train,
offline. It then sits on my mail server until I turn on my laptop again
in the evening (or sometimes the following day...).

> 
>> On 25/04/14 00:30, Yann E. MORIN wrote:
>>> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>>>
>>> This MBR blob will look for an active partition, and boot the bootcode
>>> present in that partition. This can be used to boot an extlinux-prepared
>>> partition.
>>>
>>> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
>>> Cc: Thomas Lundquist <lists@zelow.no>
>>> Cc: Frank Hunleth <fhunleth@troodon-software.com>
>>> ---
>>>  boot/syslinux/Config.in   | 13 ++++++++++++-
>>>  boot/syslinux/syslinux.mk |  1 +
>>>  2 files changed, 13 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/boot/syslinux/Config.in b/boot/syslinux/Config.in
>>> index d556c55..7d4dd08 100644
>>> --- a/boot/syslinux/Config.in
>>> +++ b/boot/syslinux/Config.in
>>> @@ -15,7 +15,7 @@ choice
>>>  
>>>  config BR2_TARGET_SYSLINUX_LEGACY_BIOS
>>>  	bool "bios (legacy)"
>>> -	select BR2_TARGET_SYSLINUX_ISOLINUX if !BR2_TARGET_SYSLINUX_PXELINUX
>>> +	select BR2_TARGET_SYSLINUX_ISOLINUX if !BR2_TARGET_SYSLINUX_HAS_BIOS_IMG
>>
>>  Hm, I don't really like this very much. Adding complexity like this for
>> behaviour which is not even really perfect...
> 
> Since you replied to my other reply (he!), I think switching all of this
> to a all-choice makes this comment moot, right? :-)

 Indeed, as I said in that reply :-)



 Regards,
 Arnout

> 
>>> +config BR2_TARGET_SYSLINUX_MBR
>>> +	bool "mbr"
>>> +	select BR2_TARGET_SYSLINUX_HAS_BIOS_IMG
>>> +	help
>>> +	  Install the legacy BIOS 'mbr' image, to boot off a
>>
>>  See my comment in 4/9 about "legacy".
> 
> Ditto, already covered.
> 
>>  BTW, while you're at it, perhaps you can add options for syslinux (=fat)
>> and extlinux?
> 
> For extlinux, please see my previous answer.
> 
> I'll have a look at syslinux, but I believe it is the same as extlinux.
> 
> Regards,
> Yann E. MORIN.
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

  parent reply	other threads:[~2014-04-28 20:57 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-24 22:30 [Buildroot] [PATCH 0/9] Bump and improve syslinux (branch yem/adopted) Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 1/9] boot/syslinux: needs an ia32-capable compiler Yann E. MORIN
2014-04-28  5:55   ` Arnout Vandecappelle
2014-04-28 17:51     ` Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 2/9] boot/syslinux: prepare to install non-core images Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 3/9] boot/syslinux: remove 'default y' in sub-options Yann E. MORIN
2014-04-24 22:55   ` Arnout Vandecappelle
2014-04-25 14:54     ` Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 4/9] boot/syslinux: rewrite options prompts Yann E. MORIN
2014-04-28  5:57   ` Arnout Vandecappelle
2014-04-28 17:36     ` Yann E. MORIN
2014-04-28 19:17       ` Arnout Vandecappelle
2014-04-24 22:30 ` [Buildroot] [PATCH 5/9] package/ucl: new package Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 6/9] package/upx: " Yann E. MORIN
2014-04-28  6:03   ` Arnout Vandecappelle
2014-04-28 17:43     ` Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 7/9] boot/syslinux: bump version Yann E. MORIN
2014-04-28  6:14   ` Arnout Vandecappelle
2014-04-28 17:26     ` Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 8/9] boot/syslinux: add option to install the EFI image Yann E. MORIN
2014-04-28  6:18   ` Arnout Vandecappelle
2014-04-28 17:15     ` Yann E. MORIN
2014-04-28 19:16       ` Arnout Vandecappelle
2014-04-28 19:45         ` Yann E. MORIN
2014-04-28 19:51           ` Arnout Vandecappelle
2014-04-24 22:30 ` [Buildroot] [PATCH 9/9] boot/syslinux: add an option to install an MBR blob Yann E. MORIN
2014-04-28 16:23   ` Arnout Vandecappelle
2014-04-28 19:52     ` Yann E. MORIN
2014-04-28 19:58       ` Frank Hunleth
2014-04-28 20:39         ` Yann E. MORIN
2014-04-28 20:45           ` Frank Hunleth
2014-04-28 20:57       ` Arnout Vandecappelle [this message]
  -- strict thread matches above, loose matches on Subject: below --
2014-04-29 16:49 [Buildroot] [PATCH 0/9 v2] Bump and improve syslinux (branch yem/syslinux) Yann E. MORIN
2014-04-29 16:49 ` [Buildroot] [PATCH 9/9] boot/syslinux: add an option to install an MBR blob Yann E. MORIN

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=535EC0C6.9050905@mind.be \
    --to=arnout@mind.be \
    --cc=buildroot@busybox.net \
    /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