[wiz at NetBSD.org: [PATCH:xf86-video-glint] Fix DDC2 on PGX32/Raptor 8P]

[Matt Turner]

On Thu, Feb 19, 2015 at 6:28 AM, Thomas Klausner <wiz at netbsd.org> wrote:
> On Wed, Feb 18, 2015 at 10:49:35PM -0800, Matt Turner wrote:
>> On Fri, Jul 12, 2013 at 3:30 AM, Thomas Klausner <wiz at netbsd.org> wrote:
>> > Hi Alan!
>> >
>> > Thanks for the patch. I can't test myself, but have forwarded your
>> > patch to Michael Lorenz, who came up with the original patch. I hope
>> > he'll find time to test it soon.
>> >  Thomas
>> >
>> > On Thu, Jul 11, 2013 at 11:55:00AM +0100, Alan wrote:
>> >> I took a quick look and it seems that this bit doesn't exist on PM3,
>> >> only PM2.
>> >>
>> >> Can you test this alternative patch ?
>> >>
>> >> Thanks,
>> >>
>> >> Alan.
>>
>> I just noticed this when going through some old email and checked my
>> Permedia docs.
>>
>> My Permedia3 and 4 docs says bit 9 "UseMonitorID" exists, same
>> behavior as Permedia 2.
>
> Thanks for checking up on this. Does this mean we can commit the patch
> Michael suggested (attached again)?
>  Thomas

No, that patch looks like a hack. It's attempting to do the same as
Alan's patch, but Alan's patch, minus the incorrect comment, is better
believable. That is, of course, assuming someone has tested it and
confirmed that it fixes the problem.