Re: [PATCH v5 2/2] hwmon: (coretemp) Report power and energy consumption

From: Srinivas Pandruvada
Date: Mon Sep 23 2013 - 19:17:23 EST


On 09/23/2013 03:45 PM, Guenter Roeck wrote:
On Mon, Sep 23, 2013 at 03:31:33PM -0700, Srinivas Pandruvada wrote:
On 09/23/2013 01:34 PM, Guenter Roeck wrote:
Supported by SandyBridge and newer CPUs.

Cc: Fenghua Yu <fenghua.yu@xxxxxxxxx>
Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
---
v5: Merged to 3.12-rc1
v4: Replaced INIT_DELAYED_WORK with INIT_DEFERRABLE_WORK
v3: Added Cc:
v2: Fix missing symbol error on i386 builds
Dropped leftover pr_info from debugging

I am inclined to drop this patch set, as I received neither test nor code
review feedback even though the patch set has been out there for a long time.
Also, I am concerned that it overlaps or even conflicts with the proposed
Power Cap/RAPL driver (https://lkml.org/lkml/2013/9/19/260).
<We feel that the power capping framework is the right place to show
the RAPL data. This is not just monitoring but also control.
There are other drivers getting developed using this framework to
expose an uniform sysfs interface.
Thought so. Have you thought about providing a hwmon bridge ?
<Good idea. I think after base version is accepted, we can add this. >

Thanks,
Guenter

Thanks,
Srinivas

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/