On Wed, 21 Mar 2012 18:30:11 -0700
Ben Widawsky <[email protected]> wrote:

> On Wed, 21 Mar 2012 12:48:25 -0700
> Jesse Barnes <[email protected]> wrote:
> 
> > From: Rohit Jain <[email protected]>
> > 
> > Added support for parsing the OEM Customizable Modes Block (#20)
> > in the VBIOS table.
> > 
> > Signed-off-by: Rohit Jain <[email protected]>
> > Reviewed-by: Shobhit Kumar <[email protected]>
> > Reviewed-by: Vijay A. Purushothaman <[email protected]>
> > Acked-by: Jesse Barnes <[email protected]>
> > Reviewed-by: Eugeni Dodonov <[email protected]>
> > Signed-off-by: Artem Bityutskiy <[email protected]>
> > Signed-off-by: Jesse Barnes <[email protected]>
> > ---
> 
> I'll be honest, I haven't read through the whole series yet, but I
> really don't see the point of this patch, at least in the next few
> patches. Could we modify the commit to describe why we need this (or
> drop it)?

This one is unrelated to VLV, it's just extra VBT gravy.  Obviously it
needs to stuff the mode info somewhere though...

-- 
Jesse Barnes, Intel Open Source Technology Center

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to