On Wed, Jun 01, 2016 at 05:11:57PM -0700, Dave Hansen wrote:
> +#define INTEL_FAM6_MODEL_CORE_YONAH          0x0E
> +#define INTEL_FAM6_MODEL_CORE2_MEROM         0x0F

That "MODEL_" part looks redundant too IMO - you could simply do

INTEL_FAM6_NEHALEM and
INTEL_FAM6_SKYLAKE_DESKTOP
...

and so on and it is still clear what it is.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Reply via email to