On Thu, Apr 19, 2012 at 1:03 PM, Takashi Iwai <tiwai at suse.de> 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.