Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-10 Thread Peter Stuge
Timothy Pearson wrote: Maybe a BDF blacklist, perhaps stored in CBFS? >>> >>> I might implement something like that in the future if I have >>> time/inclination, but for now the on/off switch is sufficient. >> >> Sufficient sure, but it is certainly using a sledgehammer to pound a nail. >> >>

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-10 Thread Timothy Pearson
On 02/10/2015 08:02 PM, Peter Stuge wrote: Timothy Pearson wrote: Is there anything else that needs to be done before this can be merged? Have you considered creating a more fine-grained control knob than simply global on/off? Maybe a BDF blacklist, perhaps stored in CBFS? I might implement

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-10 Thread Peter Stuge
Timothy Pearson wrote: >>> Is there anything else that needs to be done before this can be merged? >> >> Have you considered creating a more fine-grained control knob than >> simply global on/off? >> >> Maybe a BDF blacklist, perhaps stored in CBFS? > > I might implement something like that in the

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-10 Thread Timothy Pearson
On 02/10/2015 06:45 PM, Peter Stuge wrote: Timothy Pearson wrote: Is there anything else that needs to be done before this can be merged? Have you considered creating a more fine-grained control knob than simply global on/off? Maybe a BDF blacklist, perhaps stored in CBFS? //Peter

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-10 Thread Peter Stuge
Timothy Pearson wrote: > Is there anything else that needs to be done before this can be merged? Have you considered creating a more fine-grained control knob than simply global on/off? Maybe a BDF blacklist, perhaps stored in CBFS? //Peter ___ SeaBI

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-09 Thread Paul Menzel
Am Montag, den 09.02.2015, 11:57 -0600 schrieb Timothy Pearson: > On 02/09/2015 03:28 AM, Paolo Bonzini wrote: > > > > On 06/02/2015 22:13, Timothy Pearson wrote: > >> +config CBFS_OPTIONROMS_ONLY > >> +depends on OPTIONROMS > > > > Probably "depends on OPTIONROMS&& COREBOOT_FLASH" is

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-09 Thread Timothy Pearson
On 02/09/2015 03:28 AM, Paolo Bonzini wrote: On 06/02/2015 22:13, Timothy Pearson wrote: +config CBFS_OPTIONROMS_ONLY +depends on OPTIONROMS Probably "depends on OPTIONROMS&& COREBOOT_FLASH" is better. Paolo +bool "Only execute option ROMs stored in CBFS" +def

Re: [SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-09 Thread Paolo Bonzini
On 06/02/2015 22:13, Timothy Pearson wrote: > +config CBFS_OPTIONROMS_ONLY > +depends on OPTIONROMS Probably "depends on OPTIONROMS && COREBOOT_FLASH" is better. Paolo > +bool "Only execute option ROMs stored in CBFS" > +default "n" > +help

[SeaBIOS] [PATCH] Add option to only execute CBFS option ROMs

2015-02-06 Thread Timothy Pearson
All, This is my first time submitting a patch to SeaBIOS for review/inclusion; I assume this list is the correct initial contact? I have run into several situations where it is neccessary to execute only the relevant option ROMs contained in CBFS while ignoring any option ROM binary code pre