On Tue, Nov 16, 2021 at 10:44 AM Joyce Kong <joyce.k...@arm.com> wrote: > > Remove the unnecessary rte_atomic.h included in app modules. > > Signed-off-by: Joyce Kong <joyce.k...@arm.com> > Reviewed-by: Ruifeng Wang <ruifeng.w...@arm.com>
After patch, I still see: $ git grep rte_atomic.h app/ app/test/commands.c:#include <rte_atomic.h> app/test/test_atomic.c:#include <rte_atomic.h> app/test/test_event_timer_adapter.c:#include <rte_atomic.h> I can undertand why the test_atomic would depend on rte_atomic.h :-) but not the rest. Is there a reason? or is it just a miss? -- David Marchand