[dpdk-dev] librte_power: channel_commands: Remove duplicate definition

Message ID 1445630637-32293-1-git-send-email-aconole@redhat.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Aaron Conole Oct. 23, 2015, 8:03 p.m. UTC
  The CHANNEL_CMDS_MAX_VM_CHANNELS is duplicated in the channel_commands
header file. This commit removes that duplication.

fixes 210c383e247b5335cd95223fac1c0880ebbd5c96 ("power: packet format for ...")

Signed-off-by: Aaron Conole <aconole@redhat.com>
---
 lib/librte_power/channel_commands.h | 3 ---
 1 file changed, 3 deletions(-)
  

Comments

Bruce Richardson Oct. 27, 2015, 11:12 a.m. UTC | #1
On Fri, Oct 23, 2015 at 04:03:57PM -0400, Aaron Conole wrote:
> The CHANNEL_CMDS_MAX_VM_CHANNELS is duplicated in the channel_commands
> header file. This commit removes that duplication.
> 
> fixes 210c383e247b5335cd95223fac1c0880ebbd5c96 ("power: packet format for ...")
> 
> Signed-off-by: Aaron Conole <aconole@redhat.com>

Acked-by: Bruce Richardson <bruce.richardson@intel.com>
  
Thomas Monjalon Dec. 7, 2015, 3:21 a.m. UTC | #2
> > The CHANNEL_CMDS_MAX_VM_CHANNELS is duplicated in the channel_commands
> > header file. This commit removes that duplication.
> > 
> > fixes 210c383e247b5335cd95223fac1c0880ebbd5c96 ("power: packet format for ...")
> > 
> > Signed-off-by: Aaron Conole <aconole@redhat.com>
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks
  

Patch

diff --git a/lib/librte_power/channel_commands.h b/lib/librte_power/channel_commands.h
index 7e78a8b..de6d926 100644
--- a/lib/librte_power/channel_commands.h
+++ b/lib/librte_power/channel_commands.h
@@ -50,9 +50,6 @@  extern "C" {
 /* Maximum number of channels per VM */
 #define CHANNEL_CMDS_MAX_VM_CHANNELS 64
 
-/* Maximum number of channels per VM */
-#define CHANNEL_CMDS_MAX_VM_CHANNELS 64
-
 /* Valid Commands */
 #define CPU_POWER               1
 #define CPU_POWER_CONNECT       2