kernel-janitors.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Drivers: Stagins: memrar: cleanup
@ 2010-05-31 15:04 Spyros Papageorgiou
  2010-05-31 17:37 ` Jiri Slaby
  0 siblings, 1 reply; 2+ messages in thread
From: Spyros Papageorgiou @ 2010-05-31 15:04 UTC (permalink / raw)
  To: gregkh; +Cc: ossama.othman, devel, Kernel development list, kernel-janitors

From:Spyros Papageorgiou <spyros.papageorgiou@gmai.com>

Removed an unneeded struct specifier in function call.

Signed-off-by: Spyros Papageorgiou <spyros.papageorgiou@gmai.com>
---
 drivers/staging/memrar/memrar_allocator.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/memrar/memrar_allocator.c b/drivers/staging/memrar/memrar_allocator.c
index a4f8c58..db37c6a 100644
--- a/drivers/staging/memrar/memrar_allocator.c
+++ b/drivers/staging/memrar/memrar_allocator.c
@@ -276,7 +276,7 @@ long memrar_allocator_free(struct memrar_allocator *allocator,
 
 		struct memrar_address_range * const chunk  			&list_entry(pos,
-				    struct memrar_address_ranges,
+				    memrar_address_ranges,
 				    list)->range;
 
 		/* Extend size of existing free adjacent chunk. */
@@ -300,7 +300,7 @@ long memrar_allocator_free(struct memrar_allocator *allocator,
 
 			struct memrar_address_ranges const * const next  				list_entry(pos->next,
-					   struct memrar_address_ranges,
+					   memrar_address_ranges,
 					   list);
 
 			chunk->end = handle->end;
@@ -352,7 +352,7 @@ long memrar_allocator_free(struct memrar_allocator *allocator,
 
 			struct memrar_address_ranges const * const prev  				list_entry(pos->prev,
-					   struct memrar_address_ranges,
+					   memrar_address_ranges,
 					   list);
 
 			chunk->begin = handle->begin;
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] Drivers: Stagins: memrar: cleanup
  2010-05-31 15:04 [PATCH] Drivers: Stagins: memrar: cleanup Spyros Papageorgiou
@ 2010-05-31 17:37 ` Jiri Slaby
  0 siblings, 0 replies; 2+ messages in thread
From: Jiri Slaby @ 2010-05-31 17:37 UTC (permalink / raw)
  To: Spyros Papageorgiou
  Cc: gregkh, devel, kernel-janitors, ossama.othman,
	Kernel development list

On 05/31/2010 05:04 PM, Spyros Papageorgiou wrote:
> From:Spyros Papageorgiou <spyros.papageorgiou@gmai.com>
> 
> Removed an unneeded struct specifier in function call.
> 
> Signed-off-by: Spyros Papageorgiou <spyros.papageorgiou@gmai.com>
> ---
>  drivers/staging/memrar/memrar_allocator.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/memrar/memrar_allocator.c b/drivers/staging/memrar/memrar_allocator.c
> index a4f8c58..db37c6a 100644
> --- a/drivers/staging/memrar/memrar_allocator.c
> +++ b/drivers/staging/memrar/memrar_allocator.c
> @@ -276,7 +276,7 @@ long memrar_allocator_free(struct memrar_allocator *allocator,
>  
>  		struct memrar_address_range * const chunk >  			&list_entry(pos,
> -				    struct memrar_address_ranges,
> +				    memrar_address_ranges,
>  				    list)->range;

It is not a function call, it's a macro. Did you try to compile this?

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-05-31 17:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-31 15:04 [PATCH] Drivers: Stagins: memrar: cleanup Spyros Papageorgiou
2010-05-31 17:37 ` Jiri Slaby

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).