Le 12/02/2016 à 03:42 PM, Maxim Uvarov a écrit :
> if you provide NULL is has to be initialized to default, do not see
> reason for separate api for that.
>
> Maxim.
Because if you want to provide only 1 settings among many, you needs to make
sure all other are set to defaults
Nicolas
>
> On 12/02/16 13:25, Nicolas Morey-Chaisemartin wrote:
>> Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu>
>> ---
>> include/odp/api/spec/init.h | 8 ++++++++
>> platform/linux-generic/odp_init.c | 5 +++++
>> 2 files changed, 13 insertions(+)
>>
>> diff --git a/include/odp/api/spec/init.h b/include/odp/api/spec/init.h
>> index 154cdf8..0600691 100644
>> --- a/include/odp/api/spec/init.h
>> +++ b/include/odp/api/spec/init.h
>> @@ -165,6 +165,14 @@ typedef struct odp_init_t {
>> * passing any required platform specific data.
>> */
>>
>> +/**
>> + * Initialize platform init structure
>> + *
>> + * Initialize an odp_platform_init_t to its default values for all fields
>> + *
>> + * @param platform_params Address of the odp_platform_init_t to be
>> initialized
>> + */
>> +void odp_platform_init_init(odp_platform_init_t *platform_params);
>>
>> /**
>> * Global ODP initialization
>> diff --git a/platform/linux-generic/odp_init.c
>> b/platform/linux-generic/odp_init.c
>> index d40a83c..d74ab4c 100644
>> --- a/platform/linux-generic/odp_init.c
>> +++ b/platform/linux-generic/odp_init.c
>> @@ -415,3 +415,8 @@ int _odp_term_local(enum init_stage stage)
>>
>> return rc;
>> }
>> +
>> +void odp_platform_init_init(odp_platform_init_t *platform_params)
>> +{
>> + platform_params->ipc_ns = 0;
>> +}
>>
