| Message ID | 1416266343-7601-2-git-send-email-bill.fischofer@linaro.org |
|---|---|
| State | New |
| Headers | show |
Both patches: Reviewed-by: Petri Savolainen <petri.savolainen@linaro.org> > -----Original Message----- > From: lng-odp-bounces@lists.linaro.org [mailto:lng-odp- > bounces@lists.linaro.org] On Behalf Of ext Bill Fischofer > Sent: Tuesday, November 18, 2014 1:19 AM > To: lng-odp@lists.linaro.org > Subject: [lng-odp] [PATCH 2/2] API name changes for odp_barrier.h for ODP > v1.0 > > This patch reflects API name changes for odp_barrier.h for ODP v1.0. > > Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> > --- > platform/linux-generic/include/api/odp_barrier.h | 4 ++-- > platform/linux-generic/odp_barrier.c | 4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/platform/linux-generic/include/api/odp_barrier.h > b/platform/linux-generic/include/api/odp_barrier.h > index fb02a9d..1790ea3 100644 > --- a/platform/linux-generic/include/api/odp_barrier.h > +++ b/platform/linux-generic/include/api/odp_barrier.h > @@ -42,7 +42,7 @@ typedef struct odp_barrier_t { > * @param barrier Barrier > * @param count Thread count > */ > -void odp_barrier_init_count(odp_barrier_t *barrier, int count); > +void odp_barrier_init(odp_barrier_t *barrier, int count); > > > /** > @@ -50,7 +50,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int > count); > * > * @param barrier Barrier > */ > -void odp_barrier_sync(odp_barrier_t *barrier); > +void odp_barrier_wait(odp_barrier_t *barrier); > > /** > * @} > diff --git a/platform/linux-generic/odp_barrier.c b/platform/linux- > generic/odp_barrier.c > index f4a87c8..87be2a1 100644 > --- a/platform/linux-generic/odp_barrier.c > +++ b/platform/linux-generic/odp_barrier.c > @@ -8,7 +8,7 @@ > #include <odp_sync.h> > #include <odp_spin_internal.h> > > -void odp_barrier_init_count(odp_barrier_t *barrier, int count) > +void odp_barrier_init(odp_barrier_t *barrier, int count) > { > barrier->count = count; > barrier->bar = 0; > @@ -28,7 +28,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int > count) > * barrier crosses to the other half of the cycle. > */ > > -void odp_barrier_sync(odp_barrier_t *barrier) > +void odp_barrier_wait(odp_barrier_t *barrier) > { > uint32_t count; > int wasless; > -- > 1.8.3.2 > > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/lng-odp
Merged both patches as single commit to be git bisect'able: b931829 API name changes for odp_barrier.h for ODP v1.0 Thanks, Maxim. On 11/20/2014 11:39 AM, Savolainen, Petri (NSN - FI/Espoo) wrote: > Both patches: > > Reviewed-by: Petri Savolainen <petri.savolainen@linaro.org> > > >> -----Original Message----- >> From: lng-odp-bounces@lists.linaro.org [mailto:lng-odp- >> bounces@lists.linaro.org] On Behalf Of ext Bill Fischofer >> Sent: Tuesday, November 18, 2014 1:19 AM >> To: lng-odp@lists.linaro.org >> Subject: [lng-odp] [PATCH 2/2] API name changes for odp_barrier.h for ODP >> v1.0 >> >> This patch reflects API name changes for odp_barrier.h for ODP v1.0. >> >> Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> >> --- >> platform/linux-generic/include/api/odp_barrier.h | 4 ++-- >> platform/linux-generic/odp_barrier.c | 4 ++-- >> 2 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/platform/linux-generic/include/api/odp_barrier.h >> b/platform/linux-generic/include/api/odp_barrier.h >> index fb02a9d..1790ea3 100644 >> --- a/platform/linux-generic/include/api/odp_barrier.h >> +++ b/platform/linux-generic/include/api/odp_barrier.h >> @@ -42,7 +42,7 @@ typedef struct odp_barrier_t { >> * @param barrier Barrier >> * @param count Thread count >> */ >> -void odp_barrier_init_count(odp_barrier_t *barrier, int count); >> +void odp_barrier_init(odp_barrier_t *barrier, int count); >> >> >> /** >> @@ -50,7 +50,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int >> count); >> * >> * @param barrier Barrier >> */ >> -void odp_barrier_sync(odp_barrier_t *barrier); >> +void odp_barrier_wait(odp_barrier_t *barrier); >> >> /** >> * @} >> diff --git a/platform/linux-generic/odp_barrier.c b/platform/linux- >> generic/odp_barrier.c >> index f4a87c8..87be2a1 100644 >> --- a/platform/linux-generic/odp_barrier.c >> +++ b/platform/linux-generic/odp_barrier.c >> @@ -8,7 +8,7 @@ >> #include <odp_sync.h> >> #include <odp_spin_internal.h> >> >> -void odp_barrier_init_count(odp_barrier_t *barrier, int count) >> +void odp_barrier_init(odp_barrier_t *barrier, int count) >> { >> barrier->count = count; >> barrier->bar = 0; >> @@ -28,7 +28,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int >> count) >> * barrier crosses to the other half of the cycle. >> */ >> >> -void odp_barrier_sync(odp_barrier_t *barrier) >> +void odp_barrier_wait(odp_barrier_t *barrier) >> { >> uint32_t count; >> int wasless; >> -- >> 1.8.3.2 >> >> >> _______________________________________________ >> 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
Opps, looks like I did merge the same time you wrote your comments. Please check if I need to correct description while this commit is on top. Maxim. On 20 November 2014 12:04, Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > Merged both patches as single commit to be git bisect'able: > b931829 API name changes for odp_barrier.h for ODP v1.0 > > Thanks, > Maxim. > > > On 11/20/2014 11:39 AM, Savolainen, Petri (NSN - FI/Espoo) wrote: > >> Both patches: >> >> Reviewed-by: Petri Savolainen <petri.savolainen@linaro.org> >> >> >> -----Original Message----- >>> From: lng-odp-bounces@lists.linaro.org [mailto:lng-odp- >>> bounces@lists.linaro.org] On Behalf Of ext Bill Fischofer >>> Sent: Tuesday, November 18, 2014 1:19 AM >>> To: lng-odp@lists.linaro.org >>> Subject: [lng-odp] [PATCH 2/2] API name changes for odp_barrier.h for ODP >>> v1.0 >>> >>> This patch reflects API name changes for odp_barrier.h for ODP v1.0. >>> >>> Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> >>> --- >>> platform/linux-generic/include/api/odp_barrier.h | 4 ++-- >>> platform/linux-generic/odp_barrier.c | 4 ++-- >>> 2 files changed, 4 insertions(+), 4 deletions(-) >>> >>> diff --git a/platform/linux-generic/include/api/odp_barrier.h >>> b/platform/linux-generic/include/api/odp_barrier.h >>> index fb02a9d..1790ea3 100644 >>> --- a/platform/linux-generic/include/api/odp_barrier.h >>> +++ b/platform/linux-generic/include/api/odp_barrier.h >>> @@ -42,7 +42,7 @@ typedef struct odp_barrier_t { >>> * @param barrier Barrier >>> * @param count Thread count >>> */ >>> -void odp_barrier_init_count(odp_barrier_t *barrier, int count); >>> +void odp_barrier_init(odp_barrier_t *barrier, int count); >>> >>> >>> /** >>> @@ -50,7 +50,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, >>> int >>> count); >>> * >>> * @param barrier Barrier >>> */ >>> -void odp_barrier_sync(odp_barrier_t *barrier); >>> +void odp_barrier_wait(odp_barrier_t *barrier); >>> >>> /** >>> * @} >>> diff --git a/platform/linux-generic/odp_barrier.c b/platform/linux- >>> generic/odp_barrier.c >>> index f4a87c8..87be2a1 100644 >>> --- a/platform/linux-generic/odp_barrier.c >>> +++ b/platform/linux-generic/odp_barrier.c >>> @@ -8,7 +8,7 @@ >>> #include <odp_sync.h> >>> #include <odp_spin_internal.h> >>> >>> -void odp_barrier_init_count(odp_barrier_t *barrier, int count) >>> +void odp_barrier_init(odp_barrier_t *barrier, int count) >>> { >>> barrier->count = count; >>> barrier->bar = 0; >>> @@ -28,7 +28,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, >>> int >>> count) >>> * barrier crosses to the other half of the cycle. >>> */ >>> >>> -void odp_barrier_sync(odp_barrier_t *barrier) >>> +void odp_barrier_wait(odp_barrier_t *barrier) >>> { >>> uint32_t count; >>> int wasless; >>> -- >>> 1.8.3.2 >>> >>> >>> _______________________________________________ >>> 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 >> > >
diff --git a/platform/linux-generic/include/api/odp_barrier.h b/platform/linux-generic/include/api/odp_barrier.h index fb02a9d..1790ea3 100644 --- a/platform/linux-generic/include/api/odp_barrier.h +++ b/platform/linux-generic/include/api/odp_barrier.h @@ -42,7 +42,7 @@ typedef struct odp_barrier_t { * @param barrier Barrier * @param count Thread count */ -void odp_barrier_init_count(odp_barrier_t *barrier, int count); +void odp_barrier_init(odp_barrier_t *barrier, int count); /** @@ -50,7 +50,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int count); * * @param barrier Barrier */ -void odp_barrier_sync(odp_barrier_t *barrier); +void odp_barrier_wait(odp_barrier_t *barrier); /** * @} diff --git a/platform/linux-generic/odp_barrier.c b/platform/linux-generic/odp_barrier.c index f4a87c8..87be2a1 100644 --- a/platform/linux-generic/odp_barrier.c +++ b/platform/linux-generic/odp_barrier.c @@ -8,7 +8,7 @@ #include <odp_sync.h> #include <odp_spin_internal.h> -void odp_barrier_init_count(odp_barrier_t *barrier, int count) +void odp_barrier_init(odp_barrier_t *barrier, int count) { barrier->count = count; barrier->bar = 0; @@ -28,7 +28,7 @@ void odp_barrier_init_count(odp_barrier_t *barrier, int count) * barrier crosses to the other half of the cycle. */ -void odp_barrier_sync(odp_barrier_t *barrier) +void odp_barrier_wait(odp_barrier_t *barrier) { uint32_t count; int wasless;
This patch reflects API name changes for odp_barrier.h for ODP v1.0. Signed-off-by: Bill Fischofer <bill.fischofer@linaro.org> --- platform/linux-generic/include/api/odp_barrier.h | 4 ++-- platform/linux-generic/odp_barrier.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-)