Since most of them are only used in the C file and are not needed elsewhere, no. But NBR_CACHE_TYPE_* and NBR_STATUS_* do not belong here since they are used by the publicly available ndp_neighbor_cache_t and its functions, that is right.