From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v5 3/4] net: add arm64 neon version of CRC compute APIs Date: Mon, 03 Jul 2017 23:06:04 +0200 Message-ID: <2605665.7CG6vINj2E@xps> References: <20170427140618.35897-1-ashwin.sekhar@caviumnetworks.com> <20170512101535.7221-1-ashwin.sekhar@caviumnetworks.com> <20170512101535.7221-4-ashwin.sekhar@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, jerin.jacob@caviumnetworks.com, viktorin@rehivetech.com, jianbo.liu@linaro.org, jasvinder.singh@intel.com To: Ashwin Sekhar T K Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id A82512A58 for ; Mon, 3 Jul 2017 23:06:05 +0200 (CEST) In-Reply-To: <20170512101535.7221-4-ashwin.sekhar@caviumnetworks.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 12/05/2017 12:15, Ashwin Sekhar T K: > +#elif defined(ARM64_NEON_PMULL) > + case RTE_NET_CRC_NEON: > + if (rte_cpu_get_flag_enabled(RTE_CPUFLAG_PMULL)) { > + handlers = handlers_neon; > + break; > + } > + //-fallthrough > +#endif > case RTE_NET_CRC_SCALAR: > + //-fallthrough > default: These fallthrough comments are not in the right coding style.