[Intel-gfx] [PATCH 00/12] Add more DMT and common modes
Dave Airlie
airlied at gmail.com
Fri Apr 20 14:05:48 CEST 2012
On Thu, Apr 19, 2012 at 3:58 PM, Takashi Iwai <tiwai at suse.de> wrote:
> At Thu, 19 Apr 2012 14:03:58 +0200,
> Takashi Iwai 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 ;)
I've pulled the series into -next along with the NULL check fix.
the only outstanding bit is the hack.
Thanks,
Dave.
More information about the Intel-gfx
mailing list