Re: [Intel-gfx] [PATCH 04/12] drm/i915: Rename and comment all the RPS *stuff*

2014-03-20 Thread Chris Wilson
On Wed, Mar 19, 2014 at 06:31:11PM -0700, Ben Widawsky wrote: The names of the struct members for RPS are stupid. Every time I need to do anything in this code I have to spend a significant amount of time to remember what it all means. By renaming the variables (and adding the comments) I hope

[Intel-gfx] [PATCH 04/12] drm/i915: Rename and comment all the RPS *stuff*

2014-03-19 Thread Ben Widawsky
The names of the struct members for RPS are stupid. Every time I need to do anything in this code I have to spend a significant amount of time to remember what it all means. By renaming the variables (and adding the comments) I hope to clear up the situation. Indeed doing this make some upcoming