I think a more usual patch name would be

linux-generic: ticketlock: remove odp_sync_stores for octeon

we try to do approximate area, api, test, docs, linux-generic
implementation, followed by a tighter localisation and then a description,
when I read  "api sync" I assumed it should have gone to api-next

On 13 May 2016 at 11:04, Rizwan Ansari <[email protected]> wrote:

> Removes odp_sync_stores() call for OCTEON, Octeon
> build will fail, As this api has been already
> replaced by odp_mb_full().
>
> Signed-off-by: Rizwan Ansari <[email protected]>
> Reviewed-by: Petri Savolainen <[email protected]>
> ---
>  platform/linux-generic/odp_ticketlock.c | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/platform/linux-generic/odp_ticketlock.c
> b/platform/linux-generic/odp_ticketlock.c
> index 7b4246f..353af9a 100644
> --- a/platform/linux-generic/odp_ticketlock.c
> +++ b/platform/linux-generic/odp_ticketlock.c
> @@ -69,9 +69,6 @@ void odp_ticketlock_unlock(odp_ticketlock_t *ticketlock)
>
>         odp_atomic_store_rel_u32(&ticketlock->cur_ticket, cur + 1);
>
> -#if defined __OCTEON__
> -       odp_sync_stores(); /* SYNCW to flush write buffer */
> -#endif
>  }
>
>  int odp_ticketlock_is_locked(odp_ticketlock_t *ticketlock)
> --
> 2.5.5
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> https://lists.linaro.org/mailman/listinfo/lng-odp
>



-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
"Work should be fun and collaborative, the rest follows"
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to