Re: [Intel-gfx] [PATCH 00/12] Add more DMT and common modes

[Dave Airlie <airlied@xxxxxxxxx>]

On Thu, Apr 19, 2012 at 1:03 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> At Tue, 17 Apr 2012 17:26:26 +0200,
> Takashi Iwai wrote:
>>
>> At Fri, 13 Apr 2012 16:56:26 -0400,
>> Adam Jackson wrote:
>> >
>> > On 4/13/12 4:33 PM, Adam Jackson wrote:
>> > > Incorporates some feedback from Rodrigo and Takashi.  Major themes of the
>> > > series:
>> > >
>> > > - Fix the DMT list to include reduced-blanking modes
>> > > - Add modes from DMT for any range descriptor type
>> > > - Add an extra modes list for things not in DMT
>> > > - For ranges that specify a formula, generate timings from the extra modes
>> > >
>> > > This still doesn't address the driver policy decision of "I know I have
>> > > a scaler, add modes as if there were a range descriptor even if there's
>> > > not one".  But it should at least make clear what such a helper function
>> > > should do.
>> >
>> > One minor buglet in this series that's not obvious from inspection (and
>> > that I didn't realize until just now) is that putting 1366x768 in the
>> > minimode list won't do what you want, since the mode you get back will
>> > be 1368x768.  Probably we should move the manual-underscan hack into the
>> > timing generators themselves.
>>
>> Sounds like a good compromise.  We have already hacks in drm_edid.c
>> for HDMI TV, so one exception more isn't that bad ;)
>
> FYI, I tried the hack below and it seems working.
>
Is this hack going to be a real patch? ajax care to review?

Dave.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel