Hi Stephen,

On 30/8/2018 5:59 PM, Stephen Hemminger wrote:

On Thu, 30 Aug 2018 11:54:17 +0100
David Hunt [off-list ref] wrote:

quoted

Signed-off-by: David Hunt <redacted>
---
  lib/librte_power/channel_commands.h | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/lib/librte_power/channel_commands.h b/lib/librte_power/channel_commands.h
index ee638eefa..a82724911 100644
--- a/lib/librte_power/channel_commands.h
+++ b/lib/librte_power/channel_commands.h

@@ -19,6 +19,7 @@ extern "C" {
  #define CPU_POWER               1
  #define CPU_POWER_CONNECT       2
  #define PKT_POLICY              3
+#define PKT_POLICY_REMOVE       4
  
  /* CPU Power Command Scaling */
  #define CPU_POWER_SCALE_UP      1

@@ -58,6 +59,8 @@ struct traffic {
  	uint32_t max_max_packet_thresh;
  };
  
+enum core_type { VIRTUAL = 0, PHYSICAL };
+

Why this enum, looks like a boolean to me.

I'll change this to a 'bool' in the next version.

Thanks,
Dave.