diff mbox

odp_ring.c: remove redundant initalization

Message ID 1410988495-51244-1-git-send-email-mike.holmes@linaro.org
State Accepted
Commit e56a0303f42938c46867042c66d7ac5f73ea6908
Headers show

Commit Message

Mike Holmes Sept. 17, 2014, 9:14 p.m. UTC
Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
---

TAILQ_FOREACH reinitalizises r

 platform/linux-generic/odp_ring.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Maxim Uvarov Sept. 22, 2014, 9:22 a.m. UTC | #1
Good,  it's part of my IPC patch :)

Reviewed-by: Maxim Uvarov <maxim.uvarov@linaro.org>

And applying.

Maxim.

On 09/18/2014 01:14 AM, Mike Holmes wrote:
> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>
> ---
>
> TAILQ_FOREACH reinitalizises r
>
>   platform/linux-generic/odp_ring.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/platform/linux-generic/odp_ring.c b/platform/linux-generic/odp_ring.c
> index 720d8db..d6af5ad 100644
> --- a/platform/linux-generic/odp_ring.c
> +++ b/platform/linux-generic/odp_ring.c
> @@ -549,7 +549,7 @@ void odph_ring_list_dump(void)
>   /* search a ring from its name */
>   odph_ring_t *odph_ring_lookup(const char *name)
>   {
> -	odph_ring_t *r = odp_shm_lookup(name);
> +	odph_ring_t *r;
>   
>   	odp_rwlock_read_lock(&qlock);
>   	TAILQ_FOREACH(r, &odp_ring_list, next) {
diff mbox

Patch

diff --git a/platform/linux-generic/odp_ring.c b/platform/linux-generic/odp_ring.c
index 720d8db..d6af5ad 100644
--- a/platform/linux-generic/odp_ring.c
+++ b/platform/linux-generic/odp_ring.c
@@ -549,7 +549,7 @@  void odph_ring_list_dump(void)
 /* search a ring from its name */
 odph_ring_t *odph_ring_lookup(const char *name)
 {
-	odph_ring_t *r = odp_shm_lookup(name);
+	odph_ring_t *r;
 
 	odp_rwlock_read_lock(&qlock);
 	TAILQ_FOREACH(r, &odp_ring_list, next) {