This patch is against the tip, not the spec. It would be reorganized as part of implementing the spec.
For v1.0 there may well be changes needed here as well, but I'd prefer to get this in now so that we can delta off it as needed rather than keep more patches in limbo. Bill On Tue, Nov 18, 2014 at 9:16 AM, Taras Kondratiuk < [email protected]> wrote: > On 11/14/2014 04:18 AM, Bill Fischofer wrote: > >> diff --git a/platform/linux-generic/include/api/odp_impl_types.h >> b/platform/linux-generic/include/api/odp_impl_types.h >> new file mode 100644 >> index 0000000..f4deedb >> --- /dev/null >> +++ b/platform/linux-generic/include/api/odp_impl_types.h >> @@ -0,0 +1,72 @@ >> +/* Copyright (c) 2014, Linaro Limited >> + * All rights reserved. >> + * >> + * SPDX-License-Identifier: BSD-3-Clause >> + */ >> + >> + >> +/** >> + * @file >> + * ODP implementation types >> + * This file contains all of the implementation-defined types for ODP >> + * abstract definitions. Having this in one file means that other ODP >> + * API files are implementation-independent and avoids circular >> + * dependencies for files that refer to types managed by other >> + * components. Included here are typedefs and related typed constants >> + * that are referenced by other ODP API files. >> + */ >> + >> +#ifndef ODP_IMPL_TYPES_H_ >> +#define ODP_IMPL_TYPES_H_ >> + >> +/** @defgroup odp_types ODP TYPES >> + * Implementation definitions for ODP abstract types. >> + * @{ >> + */ >> + >> +/** ODP Buffer pool */ >> +typedef uint32_t odp_buffer_pool_t; >> + >> +/** ODP buffer */ >> +typedef uint32_t odp_buffer_t; >> + >> +/** Invalid buffer */ >> +#define ODP_BUFFER_INVALID (0xffffffff) >> + >> +/** ODP packet */ >> +typedef odp_buffer_t odp_packet_t; >> + >> +/** Invalid packet */ >> +#define ODP_PACKET_INVALID ODP_BUFFER_INVALID >> + >> +/** Invalid offset */ >> +#define ODP_PACKET_OFFSET_INVALID ((uint32_t)-1) >> + >> +/** ODP packet segment */ >> +typedef int odp_packet_seg_t; >> + >> +/** Invalid packet segment */ >> +#define ODP_PACKET_SEG_INVALID -1 >> + >> +/** ODP packet segment info */ >> +typedef struct odp_packet_seg_info_t { >> + void *addr; /**< Segment start address */ >> + size_t size; /**< Segment maximum data size */ >> + void *data; /**< Segment data address */ >> + size_t data_len; /**< Segment data length */ >> +} odp_packet_seg_info_t; >> > > Sorry for a late comment. > odp_packet_seg_info_t is a public structure visible to a user. > It shouldn't be changed by implementation. >
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
