Re: [coreboot] What happened to the Option ROM options?
On 21.06.2018 23:09, taii...@gmx.com wrote: > On 06/21/2018 04:51 AM, Nico Huber wrote: >> On 21.06.2018 03:47, taii...@gmx.com wrote: >>> Ah oops overlooked this. >>> https://review.coreboot.org/#/c/coreboot/+/2531/ >>> >>> Paging ron minnich! It seems at the end of the comments various people >>> note the mistake however it never got fixed. >> >> There is no mistake here. The change was only a little ahead of its >> time. There was limited support for non-VGA option ROMs but that was >> not working well and SeaBIOS has better support for it. Since [1], >> coreboot only executes the option ROMs of VGA devices. The later >> changes you noticed only fix the Kconfig options to reflect that. > Ah thanks for the info. > > But what about initializing non-integrated PCI-e graphics devices on a > board that has native graphics init (where the option doesn't appear) This is usually not useful with modern operating systems. But if you have a use case, patches are welcome. > and being able to choose the various YABEL options? I don't see any option that is currently hidden when Yabel is built in. Ofc, we don't show these option when Yabel is not compiled at all. Nico -- coreboot mailing list: coreboot@coreboot.org https://mail.coreboot.org/mailman/listinfo/coreboot
Re: [coreboot] What happened to the Option ROM options?
On 06/21/2018 04:51 AM, Nico Huber wrote: > On 21.06.2018 03:47, taii...@gmx.com wrote: >> Ah oops overlooked this. >> https://review.coreboot.org/#/c/coreboot/+/2531/ >> >> Paging ron minnich! It seems at the end of the comments various people >> note the mistake however it never got fixed. > > There is no mistake here. The change was only a little ahead of its > time. There was limited support for non-VGA option ROMs but that was > not working well and SeaBIOS has better support for it. Since [1], > coreboot only executes the option ROMs of VGA devices. The later > changes you noticed only fix the Kconfig options to reflect that. Ah thanks for the info. But what about initializing non-integrated PCI-e graphics devices on a board that has native graphics init (where the option doesn't appear) and being able to choose the various YABEL options? > > So if you need non-VGA option ROMs, consider using SeaBIOS. > >> If this is a mistake I will require someone else to fix it as I am not >> allowed to contribute on gerrit. > > You are free to contribute. And if you had listened in the last discus- > sion about our sign-off procedure, you would know that even you can do > so without sacrifice. Could you please provide a link for that? I always read all the replies but I hadn't noticed anything about being able to sidestep the requirement to use your "real" name. -- coreboot mailing list: coreboot@coreboot.org https://mail.coreboot.org/mailman/listinfo/coreboot
Re: [coreboot] What happened to the Option ROM options?
On 21.06.2018 03:47, taii...@gmx.com wrote: > Ah oops overlooked this. > https://review.coreboot.org/#/c/coreboot/+/2531/ > > Paging ron minnich! It seems at the end of the comments various people > note the mistake however it never got fixed. There is no mistake here. The change was only a little ahead of its time. There was limited support for non-VGA option ROMs but that was not working well and SeaBIOS has better support for it. Since [1], coreboot only executes the option ROMs of VGA devices. The later changes you noticed only fix the Kconfig options to reflect that. So if you need non-VGA option ROMs, consider using SeaBIOS. > If this is a mistake I will require someone else to fix it as I am not > allowed to contribute on gerrit. You are free to contribute. And if you had listened in the last discus- sion about our sign-off procedure, you would know that even you can do so without sacrifice. Nico [1] https://review.coreboot.org/#/c/coreboot/+/4545/ -- coreboot mailing list: coreboot@coreboot.org https://mail.coreboot.org/mailman/listinfo/coreboot
Re: [coreboot] What happened to the Option ROM options?
Ah oops overlooked this. https://review.coreboot.org/#/c/coreboot/+/2531/ Paging ron minnich! It seems at the end of the comments various people note the mistake however it never got fixed. If this is a mistake I will require someone else to fix it as I am not allowed to contribute on gerrit. -- coreboot mailing list: coreboot@coreboot.org https://mail.coreboot.org/mailman/listinfo/coreboot