Messages in this thread | | | Date | Fri, 5 Apr 2013 14:57:20 -0700 | From | Jacob Pan <> | Subject | Re: [PATCH v2] Introduce Intel RAPL cooling device driver |
| |
On Fri, 05 Apr 2013 14:26:35 -0700 Joe Perches <joe@perches.com> wrote:
> > +/* in the order of enum rapl_primitives */ > > +static struct rapl_primitive_info rpi[] = { > > const? I do need to override one entry for a special case. The hardware uses a different bit location for the same lock functionality.
The other comments are well taken,
-- Thanks,
Jacob
| |