Re: [PATCH 2/5] staging: wilc1000: remove FREE_WILC_BUFFER()

2015-08-17 Thread Dan Carpenter
On Mon, Aug 17, 2015 at 12:08:34PM -0400, Raphaël Beamonte wrote: > It was just a wrapper around kfree(), so call that instead. > > Signed-off-by: Raphaël Beamonte > --- > + kfree(exported_g_tx_buf); > + exported_g_tx_buf = NULL; No need to add these new NULL assignments. The module is

[PATCH 2/5] staging: wilc1000: remove FREE_WILC_BUFFER()

2015-08-17 Thread Raphaël Beamonte
It was just a wrapper around kfree(), so call that instead. Signed-off-by: Raphaël Beamonte --- drivers/staging/wilc1000/wilc_exported_buf.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_exported_buf.c

[PATCH 2/5] staging: wilc1000: remove FREE_WILC_BUFFER()

2015-08-17 Thread Raphaël Beamonte
It was just a wrapper around kfree(), so call that instead. Signed-off-by: Raphaël Beamonte raphael.beamo...@gmail.com --- drivers/staging/wilc1000/wilc_exported_buf.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_exported_buf.c

Re: [PATCH 2/5] staging: wilc1000: remove FREE_WILC_BUFFER()

2015-08-17 Thread Dan Carpenter
On Mon, Aug 17, 2015 at 12:08:34PM -0400, Raphaël Beamonte wrote: It was just a wrapper around kfree(), so call that instead. Signed-off-by: Raphaël Beamonte raphael.beamo...@gmail.com --- + kfree(exported_g_tx_buf); + exported_g_tx_buf = NULL; No need to add these new NULL