On 7/10/2020 11:07 PM, Thomas Monjalon wrote: > 10/07/2020 23:43, Ferruh Yigit: >> Using '__rte_internal' tag in 'rte_ethdev_driver.h' causing build error >> for applications and examples. Because they don't define >> 'ALLOW_INTERNAL_API' flag and '__rte_internal' causes the error. >> This patch is preparation for future '__rte_internal' usage. >> >> At first place, applications/examples should not include >> 'rte_ethdev_driver.h', this is happening because of PMD public header >> files include 'rte_ethdev_driver.h' by mistake. >> >> Updated PMD public header files to not include internal header files. >> >> But for unit test application, 'app/test', enable accessing internal >> APIs, since some unit tests need them. >> >> Fixes: ffc905f3b856 ("ethdev: separate driver APIs") >> Fixes: ec0dec44ecb9 ("net/atlantic: enable MACsec configuration") >> Cc: sta...@dpdk.org >> >> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > > Not sure to understand the title but I have no better proposal.
Agree, I will change to following: drivers/net: fix exposing internal headers > For the explanations and the code, > Acked-by: Thomas Monjalon <tho...@monjalon.net> > >