Re: [coreboot] [PATCH] x86: add coreboot framebuffer support

[Vladimir 'φ-coder/phcoder' Serbinenko]

On 06.09.2014 00:18, ron minnich wrote:
> Vladimir can you point me to that patch? This sounds interesting.

https://lkml.org/lkml/2010/8/25/190
> ron

[PATCH] x86: add coreboot framebuffer support

[H. Peter Anvin]

On 09/05/2014 02:23 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 06.09.2014 00:18, ron minnich wrote:
> > Vladimir can you point me to that patch? This sounds
> > interesting.
>
> https://lkml.org/lkml/2010/8/25/190

I believe *most* of this patch has already gotten merged as we now use
simplefb on x86 as well.  So all we need is probably the ID.

-hpa

[PATCH] x86: add coreboot framebuffer support

[Vladimir 'φ-coder/phcoder' Serbinenko]

On 06.09.2014 00:31, H. Peter Anvin wrote:
> On 09/05/2014 02:23 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> > On 06.09.2014 00:18, ron minnich wrote:
> > > Vladimir can you point me to that patch? This sounds
> > > interesting.
> >
> > https://lkml.org/lkml/2010/8/25/190
>
> I believe *most* of this patch has already gotten merged as we now use
> simplefb on x86 as well.  So all we need is probably the ID.

Yes, efifb has the same semantics. We just need some ID with clear
documentation saying sth like "implies framebuffer without anything
else" so that noone will get an idea to plug e.g. vga hooks into it.
> -hpa

[PATCH] x86: add coreboot framebuffer support

[H. Peter Anvin]

On 09/05/2014 02:40 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 06.09.2014 00:31, H. Peter Anvin wrote:
> > On 09/05/2014 02:23 PM, Vladimir 'φ-coder/phcoder' Serbinenko
> > wrote:
> > > On 06.09.2014 00:18, ron minnich wrote:
> > > > Vladimir can you point me to that patch? This sounds 
> > > > interesting.
> > >
> > > https://lkml.org/lkml/2010/8/25/190
> >
> > I believe *most* of this patch has already gotten merged as we
> > now use simplefb on x86 as well.  So all we need is probably the
> > ID.
>
> Yes, efifb has the same semantics. We just need some ID with clear 
> documentation saying sth like "implies framebuffer without
> anything else" so that noone will get an idea to plug e.g. vga
> hooks into it.

Want to make a patch?

-hpa

[PATCH] x86: add coreboot framebuffer support

[Gerd Hoffmann]

Hi,
> > Yes, efifb has the same semantics. We just need some ID with clear 
> > documentation saying sth like "implies framebuffer without
> > anything else" so that noone will get an idea to plug e.g. vga
> > hooks into it.
>
> Want to make a patch?

Attached.  Can someone test with the patched coreboot trampoline please?

cheers,
  Gerd