[Buildroot] [PATCH v3] i7z: new package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Apr 27 20:14:51 UTC 2016


Hello,

On Wed, 27 Apr 2016 15:12:44 +0200, Vincent Stehlé wrote:
> "A better i7 (and now i3, i5) reporting tool for Linux."
> 
> i7z can print out the C-states and temperature for i3, i5 and i7 based Core
> processors from Intel (including Nehalems, Sandy Bridge and Ivy Bridge).
> 
> https://github.com/ajaiantilal/i7z
> 
> Signed-off-by: Vincent Stehlé <vincent.stehle at intel.com>
> Cc: Bernd Kuhls <bernd.kuhls at t-online.de>
> ---

Applied to master, thanks.

However, I don't know if you had a look at the source code itself. But
it is really truly horrible. I think I had never seen a function taking
34 arguments, so much code WRITTEN ALMOST ENTIRELY IN CAPS and
generally such an atrocity. And I'm used to looking at vendor code in
Linux BSPs, so I'm somewhat used to all sort of atrocities.

  https://github.com/ajaiantilal/i7z/blob/master/i7z_Dual_Socket.c

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list