>>> On 03.05.17 at 10:44, <yi.y....@linux.intel.com> wrote: > static int find_cos(const uint32_t val[], unsigned int array_len, > enum psr_feat_type feat_type, > const struct psr_socket_info *info) > { > + unsigned int cos, i; > + const unsigned int *ref = info->cos_ref; > + const struct feat_node *feat; > + unsigned int cos_max;
Please can you declare all the "unsigned int"-s on one line? (i, btw, should go into the for scope.) Looks good otherwise, but again earlier comments apply. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel