> -----Original Message----- > From: lng-odp-boun...@lists.linaro.org [mailto:lng-odp- > boun...@lists.linaro.org] On Behalf Of ext Ola Liljedahl > Sent: Tuesday, February 03, 2015 6:48 PM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCHv5 08/18] api: odp_init.h: updated return > descriptions > > Updated doxygen descriptions, particularly the @return/@retval > descriptions. > No change of implementation necessary. > > Signed-off-by: Ola Liljedahl <ola.liljed...@linaro.org> > --- > (This document/code contribution attached is provided under the terms of > agreement LES-LTM-21309) > > include/odp/api/init.h | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/include/odp/api/init.h b/include/odp/api/init.h > index c73b321..c14769f 100644 > --- a/include/odp/api/init.h > +++ b/include/odp/api/init.h > @@ -66,8 +66,8 @@ typedef enum odp_log_level { > * @param[in] level Log level > * @param[in] fmt printf-style message format > * > - * @return The number of characters logged if succeeded. Otherwise > returns > - * a negative number. > + * @return The number of characters logged on success > + * @retval <0 on failure > */ > int odp_override_log(odp_log_level_e level, const char *fmt, ...); > > @@ -108,8 +108,8 @@ typedef struct odp_platform_init_t { > * @param[in] platform_params Those parameters that are passed without > * interpretation by the ODP API to the implementation. > * > - * @retval 0 if successful > - * @retval -1 on failure > + * @retval 0 on success > + * @retval <0 on failure > */ > int odp_init_global(odp_init_t *params, odp_platform_init_t > *platform_params); > > @@ -133,8 +133,8 @@ int odp_init_global(odp_init_t *params, > odp_platform_init_t *platform_params); > * @sa odp_init_global() > * @sa odp_term_local() which must have been called prior to this. > * > - * @retval 0 if successful > - * @retval -1 on failure > + * @retval 0 on success > + * @retval <0 on failure > */ > int odp_term_global(void); > > @@ -147,8 +147,8 @@ int odp_term_global(void); > * @sa odp_term_local() > * @sa odp_init_global() which must have been called prior to this. > * > - * @retval 0 if successful > - * @retval -1 on failure > + * @retval 0 on success > + * @retval <0 on failure > */ > int odp_init_local(void); > > @@ -170,8 +170,8 @@ int odp_init_local(void); > * @warning The unwinding of HW resources to allow them to be re used > without reseting > * the device is a complex task that the application is expected to > coordinate. > * > - * @retval 1 if successful and more ODP thread exists > - * @retval 0 if successful and it was the last ODP thread > + * @retval 1 on success and more ODP threads exist > + * @retval 0 on success and no more ODP threads exist
The existing documentation is better. The _last_ thread calls odp_term_global(). -Petri > * @retval -1 on failure > */ > int odp_term_local(void); > -- > 1.9.1 > > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org http://lists.linaro.org/mailman/listinfo/lng-odp