lkml.org 
[lkml]   [2020]   [Mar]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [patch 02/22] x86/cpu: Add conistent CPU match macros
Date
Andy Shevchenko <andy.shevchenko@gmail.com> writes:
>
>> Also a add a few model constants for Centaur CPUs and QUARK.
>
> I would perhaps made this as a separate change(s).

Can do.

>> +#define X86_MATCH_VENDOR_FAM_MODEL_FEATURE(_vendor, _family, _model, \
>> + _feature, _data) { \
>
> I would leave it on one line despite the length, but it's up to you.
>
>> + .vendor = X86_VENDOR_##_vendor, \
>> + .family = _family, \
>> + .model = _model, \
>> + .feature = _feature, \
>
>> + .driver_data = (unsigned long) _data \
>
> For sake of consistency shouldn't be this kernel_ulong_t ?

I can change that though in kernel space this does not matter.

> Or we are going to get rid of that type?

No.

Thanks,

tglx

\
 
 \ /
  Last update: 2020-03-20 21:28    [W:0.084 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site