[dpdk-dev] [PATCH v3 03/12] linuxapp: eal: arm: Always return 0 for rte_eal_iopl_init()

Stephen Hemminger stephen at networkplumber.org
Thu Jan 7 19:14:07 CET 2016


On Thu,  7 Jan 2016 22:03:00 +0530
Santosh Shukla <sshukla at mvista.com> wrote:

>  #else
> +#if defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64)
> +	return 0; /* iopl syscall not supported for ARM/ARM64 */
> +#endif
>  	return -1;
>  #endif

Minor net why not:

#elif defined(RTE_ARCH_ARM) || defined(RTE_ARCH_ARM64)
	return -1
#else

That way you won't generate two return statements and potentially
trigger warnings from static checkers.


More information about the dev mailing list