[ofw] [PATCH] incorrect application of '&'

Smith, Stan stan.smith at intel.com
Wed Jul 13 09:21:21 PDT 2011


Done.
Revision: 3238
Author: stansmith
Date: 9:11:29 AM, Wednesday, July 13, 2011
Message:
[IBBUS] remove unneeded/confusing address-of operator '&'.
----
Modified : /gen1/trunk/core/bus/kernel/bus_driver.c



>-----Original Message-----
>From: ofw-bounces at lists.openfabrics.org [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Smith, Stan
>Sent: Monday, July 11, 2011 4:55 PM
>To: Leonid Keller
>Cc: ofw at lists.openfabrics.org
>Subject: [ofw] [PATCH] incorrect application of '&'
>
>Hello,
>  'g_bus_filters' is defined as ' extern bus_filter_t g_bus_filters[MAX_BUS_FILTERS]' hence the '&' operator is unnecessary and misleading.
>Remove the unnecessary '&' operator.
>
>Signed-off-by: stan smith <stan.smith at intel.com>
>
>--- core/bus/kernel/bus_driver.c	Mon Jul 11 16:22:10 2011
>+++ core/bus/kernel/bus_driver.c	Wed Jun 29 15:26:49 2011
>@@ -1086,7 +1086,7 @@
>
> 	KeInitializeEvent(&g_ControlEvent, SynchronizationEvent, TRUE);
> 	g_bfi_InstanceCount = 0;
>-	memset( (void*)&g_bus_filters, 0, sizeof(g_bus_filters) );
>+	memset( (void*)g_bus_filters, 0, sizeof(g_bus_filters) );
>
> 	BUS_TRACE_EXIT( BUS_DBG_DRV,
> 		("=====> IBBUS: Driver exited\n") );
>
>_______________________________________________
>ofw mailing list
>ofw at lists.openfabrics.org
>http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw



More information about the ofw mailing list