This patch does away with cast on void * and the if as it is unnecessary. The following Coccinelle semantic patch was used for making the change: @r@ expression x; void* e; type T; identifier f; @@ ( *((T *)e) | ((T *)x)[...] | ((T *)x)->f | - (T *) e ) Signed-off-by: Himangi Saraogi <himangi774@xxxxxxxxx> --- To send to: Matthew Wilcox <matthew@xxxxxx>,"James E.J. Bottomley" <JBottomley@xxxxxxxxxxxxx>,linux-scsi@xxxxxxxxxxxxxxx,linux-kernel@xxxxxxxxxxxxxxx drivers/scsi/sym53c8xx_2/sym_malloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/sym53c8xx_2/sym_malloc.c b/drivers/scsi/sym53c8xx_2/sym_malloc.c index 6f9af0d..1dde1e9 100644 --- a/drivers/scsi/sym53c8xx_2/sym_malloc.c +++ b/drivers/scsi/sym53c8xx_2/sym_malloc.c @@ -95,7 +95,7 @@ static void *___sym_malloc(m_pool_p mp, int size) } } #ifdef DEBUG - printf("___sym_malloc(%d) = %p\n", size, (void *) a); + printf("___sym_malloc(%d) = %p\n", size, a); #endif return a; } -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html