Should be odpdrv_*
I was obviously blind when writting these comments :-)
Will fix.
On 19 December 2016 at 08:20, Forrest Shi <forrest....@linaro.org> wrote:
> I'm a bit curious why not odp_drv_enum?
>
> On 19 December 2016 at 14:04, Yi He <yi...@linaro.org> wrote:
>
>>
>>
>> On 8 December 2016 at 17:20, Christophe Milard <
>> christophe.mil...@linaro.org> wrote:
>>
>>> The linux implementation for the enumerator class registration function,
>>> enumerator instance registration function,
>>> devio and driver registration functions (stub)
>>>
>>> Signed-off-by: Christophe Milard <christophe.mil...@linaro.org>
>>> ---
>>> include/odp_drv.h | 1 +
>>> platform/linux-generic/Makefile.am | 2 ++
>>> platform/linux-generic/drv_driver.c | 43
>>> +++++++++++++++++++++++++
>>> platform/linux-generic/include/odp/drv/driver.h | 27 ++++++++++++++++
>>> 4 files changed, 73 insertions(+)
>>> create mode 100644 platform/linux-generic/drv_driver.c
>>> create mode 100644 platform/linux-generic/include/odp/drv/driver.h
>>>
>>> diff --git a/include/odp_drv.h b/include/odp_drv.h
>>> index 0959879..96d81ba 100644
>>> --- a/include/odp_drv.h
>>> +++ b/include/odp_drv.h
>>> @@ -23,6 +23,7 @@ extern C {
>>> #include <odp/drv/barrier.h>
>>> #include <odp/drv/byteorder.h>
>>> #include <odp/drv/compiler.h>
>>> +#include <odp/drv/driver.h>
>>> #include <odp/drv/shm.h>
>>> #include <odp/drv/spinlock.h>
>>> #include <odp/drv/std_types.h>
>>> diff --git a/platform/linux-generic/Makefile.am
>>> b/platform/linux-generic/Makefile.am
>>> index adbe24d..2ed8043 100644
>>> --- a/platform/linux-generic/Makefile.am
>>> +++ b/platform/linux-generic/Makefile.am
>>> @@ -107,6 +107,7 @@ odpdrvinclude_HEADERS = \
>>> $(srcdir)/include/odp/drv/barrier.h \
>>> $(srcdir)/include/odp/drv/byteorder.h \
>>> $(srcdir)/include/odp/drv/compiler.h \
>>> + $(srcdir)/include/odp/drv/driver.h \
>>> $(srcdir)/include/odp/drv/shm.h \
>>> $(srcdir)/include/odp/drv/spinlock.h \
>>> $(srcdir)/include/odp/drv/std_types.h \
>>> @@ -225,6 +226,7 @@ __LIB__libodp_linux_la_SOURCES = \
>>> odp_weak.c \
>>> drv_atomic.c \
>>> drv_barrier.c \
>>> + drv_driver.c \
>>> drv_shm.c \
>>> drv_spinlock.c \
>>> arch/@ARCH_DIR@/odp_cpu_arch.c \
>>> diff --git a/platform/linux-generic/drv_driver.c
>>> b/platform/linux-generic/drv_driver.c
>>> new file mode 100644
>>> index 0000000..c520d1d
>>> --- /dev/null
>>> +++ b/platform/linux-generic/drv_driver.c
>>> @@ -0,0 +1,43 @@
>>> +/* Copyright (c) 2016, Linaro Limited
>>> + * All rights reserved.
>>> + *
>>> + * SPDX-License-Identifier: BSD-3-Clause
>>> + */
>>> +
>>> +#include <odp_config_internal.h>
>>> +#include <odp/api/std_types.h>
>>> +#include <odp/api/debug.h>
>>> +#include <odp_debug_internal.h>
>>> +#include <odp/drv/driver.h>
>>> +
>>> +int odrdrv_enum_class_register(odpdrv_enum_class_t *enum_class)
>>>
>>
>> odrdrv_enum_ -> odpdrv_enum?
>>
>>
>>> +{
>>> + ODP_ERR("NOT Supported yet! Enumerator Class %s
>>> Registration!\n.",
>>> + enum_class->name);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +int odpdrv_enum_register(odpdrv_enum_t *enumerator)
>>> +{
>>> + ODP_ERR("NOT Supported yet! Enumerator instance %s
>>> Registration!\n.",
>>> + enumerator->class_name);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +int odpdrv_devio_register(odpdrv_devio_t *devio)
>>> +{
>>> + ODP_ERR("NOT Supported yet! Driver %s Registration!\n.",
>>> + devio->devio_api_name);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +int odpdrv_driver_register(odpdrv_driver_t *driver)
>>> +{
>>> + ODP_ERR("NOT Supported yet! Driver %s Registration!\n.",
>>> + driver->name);
>>> +
>>> + return 0;
>>> +}
>>> diff --git a/platform/linux-generic/include/odp/drv/driver.h
>>> b/platform/linux-generic/include/odp/drv/driver.h
>>> new file mode 100644
>>> index 0000000..9eb6962
>>> --- /dev/null
>>> +++ b/platform/linux-generic/include/odp/drv/driver.h
>>> @@ -0,0 +1,27 @@
>>> +/* Copyright (c) 2016, Linaro Limited
>>> + * All rights reserved.
>>> + *
>>> + * SPDX-License-Identifier: BSD-3-Clause
>>> + */
>>> +
>>> +/**
>>> + * @file
>>> + *
>>> + * ODPDRV drivers
>>> + */
>>> +
>>> +#ifndef ODPDRV_PLAT_DRIVER_H_
>>> +#define ODPDRV_PLAT_DRIVER_H_
>>> +
>>> +#ifdef __cplusplus
>>> +extern "C" {
>>> +#endif
>>> +
>>> +#include <odp/drv/std_types.h>
>>> +#include <odp/drv/spec/driver.h>
>>> +
>>> +#ifdef __cplusplus
>>> +}
>>> +#endif
>>> +
>>> +#endif
>>> --
>>> 2.7.4
>>>
>>>
>>
>
