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

[Stephen Hemminger]

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.