On Fri, Feb 01, 2013 at 04:41:14PM -0800, Ben Widawsky wrote: > The RC6 VIDS has a linear ramp starting at 250mv, which means any values > below 250 are invalid. The old buggy macros tried to adjust for this to > be more flexible, but there is no need. As Dan pointed out the ENCODE > only ever has one value. The only invalid value for decode is an input > of 0 which means something is really wonky, and the cases where DECODE > are used either don't matter (debug values), or would be implicitly > correct (the check for less than 450). > > This patch makes simpler, easier to read macros which are actually > correct. Maybe this patch can actually fix some bugs now. > > Thanks to Dan for catching this. /me hides > > Cc: sta...@kernel.org > Reported-by: Dan Carpenter <dan.carpen...@oracle.com> > Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
