[Intel-gfx] [PATCH 2/3] drm/i915: Rename HDMI register field definitions

Paulo Zanoni przanoni at gmail.com
Mon Oct 17 21:04:29 CEST 2011


2011/10/14 Chris Wilson <chris at chris-wilson.co.uk>:
> I'm a little more dubious over the value of this patch. The extra
> information is nice, but the patch doesn't impact much upon the
> readibility of the code. It seems like this should have the potential to
> clarify quite a bit of the logic and make it more consistent, yet that
> is not addressed here. Do you have plans to make use of the cleanup?

Well, "readability" has a different definition for each different
reader. The patch was based on my own definition, but I'm open to
suggestions. If you think it's not worth, just drop it: no problem.

> Does it catch any bugs?

Not yet, but it would have helped catch the one from my previous patch...
-- 
Paulo Zanoni



More information about the Intel-gfx mailing list