Hi Marko,

On 28/11/2017 1:22 PM, Marko Kovacevic wrote:
Signed-off-by: Marko Kovacevic <marko.kovace...@intel.com>
---
  lib/librte_power/power_acpi_cpufreq.c                   | 2 +-
  lib/librte_power/{rte_power_common.h => power_common.h} | 6 +++---
  lib/librte_power/rte_power.c                            | 2 +-
  3 files changed, 5 insertions(+), 5 deletions(-)
  rename lib/librte_power/{rte_power_common.h => power_common.h} (95%)

diff --git a/lib/librte_power/power_acpi_cpufreq.c 
b/lib/librte_power/power_acpi_cpufreq.c
index 165ec97..fd1931f 100644
--- a/lib/librte_power/power_acpi_cpufreq.c
+++ b/lib/librte_power/power_acpi_cpufreq.c
@@ -45,7 +45,7 @@
  #include <rte_atomic.h>
#include "power_acpi_cpufreq.h"
-#include "rte_power_common.h"
+#include "power_common.h"
#ifdef RTE_LIBRTE_POWER_DEBUG
  #define POWER_DEBUG_TRACE(fmt, args...) do { \
diff --git a/lib/librte_power/rte_power_common.h 
b/lib/librte_power/power_common.h
similarity index 95%
rename from lib/librte_power/rte_power_common.h
rename to lib/librte_power/power_common.h


--snip--

Could I suggest adding the following into the commit message?

"Rename private header file rte_power_common.h to power_common.h to prevent private functions from leaking into the documentation."

Apart from that, if you make those changes, you can include my Ack in the next revision.

Series-Acked-By: David Hunt <david.h...@intel.com>

Reply via email to