[PATCH] drm: Pretty print out the reason for rejecting the mode

Alex Deucher alexdeucher at gmail.com
Wed Apr 6 07:32:36 PDT 2011


On Tue, Apr 5, 2011 at 5:41 PM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> On Tue, 5 Apr 2011 23:39:19 +0200, Julien Cristau <jcristau at debian.org> wrote:
>> On Tue, Apr  5, 2011 at 21:50:15 +0100, Chris Wilson wrote:
>>
>> > +   if ((unsigned)status > ARRAY_SIZE(strings))
>>
>> off by one?
>
> And didn't check for holes.
>
> /me wishes he never sent this momentarily useful bit of code. ;-)

I think it's still useful if we can get it fixed up.  I've recently
run into some cases on radeon where this would be helpful.

Alex

> -Chris
>
> --
> Chris Wilson, Intel Open Source Technology Centre
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>


More information about the dri-devel mailing list