diff mbox

[dpdk-dev] hash: add switch fall-through comments for arm64

Message ID 20170509094541.8301-1-jerin.jacob@caviumnetworks.com (mailing list archive)
State Accepted, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Jerin Jacob May 9, 2017, 9:45 a.m. UTC 
  This fixes compiler warnings with GCC 7 for arm64 build.
Fixes: da8dcc27f644 ("hash: use armv8-a CRC32 instructions")

Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
---
 lib/librte_hash/rte_crc_arm64.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Thomas Monjalon May 10, 2017, 2:31 p.m. UTC | #1 
09/05/2017 11:45, Jerin Jacob:
> This fixes compiler warnings with GCC 7 for arm64 build.
> Fixes: da8dcc27f644 ("hash: use armv8-a CRC32 instructions")
> 
> Signed-off-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

Applied, thanks
diff mbox

Patch

diff --git a/lib/librte_hash/rte_crc_arm64.h b/lib/librte_hash/rte_crc_arm64.h
index 7dd6334ee..2abe42ab8 100644
--- a/lib/librte_hash/rte_crc_arm64.h
+++ b/lib/librte_hash/rte_crc_arm64.h
@@ -110,8 +110,10 @@  rte_hash_crc_set_alg(uint8_t alg)
 	case CRC32_ARM64:
 		if (!rte_cpu_get_flag_enabled(RTE_CPUFLAG_CRC32))
 			alg = CRC32_SW;
+		/* fall-through */
 	case CRC32_SW:
 		crc32_alg = alg;
+		/* fall-through */
 	default:
 		break;
 	}