05/06/2018 08:20, Shreyansh Jain: > On 6/4/2018 8:22 AM, Jerin Jacob wrote: > > ccache gcc -Idrivers/drivers@@tmp_rte_pmd_dpaa_sec@sta -Idrivers > > In file included from ../drivers/bus/dpaa/include/fsl_usd.h:11, > > from ../drivers/crypto/dpaa_sec/dpaa_sec.c:27: > > ../drivers/bus/dpaa/include/compat.h:53: > > error: "__packed" redefined [-Werror] > > #define __packed __rte_packed > > > > In file included from /usr/include/bsd/string.h:39, > > from ../lib/librte_eal/common/include/rte_string_fns.h:71, > > from ../drivers/crypto/dpaa_sec/dpaa_sec.c:25: > > /usr/include/bsd/sys/cdefs.h:120: note: this is the location > > of the previous definition > > # define __packed __attribute__((__packed__)) > > > > Cc: sta...@dpdk.org > > Fixes: 39f373cf015a ("bus/dpaa: add compatibility and helper macros") > > > > Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> > > --- > > Thanks Jerin. > Somehow, this didn't appear on my build environment (maybe because LIB > BSD is not enabled?). But, the change looks harmless. > > Acked-by: Shreyansh Jain <shreyansh.j...@nxp.com>
I missed this patch and sent a more complete one recently: https://patches.dpdk.org/patch/41314/