On Sat, 8 Apr 2017, Hans de Goede wrote:
> Hi Julia,
>
> On 08-04-17 19:10, Julia Lawall wrote:
> > kfree returns immediately on NULL so the tests are not needed.
> >
> > Generated by: scripts/coccinelle/free/ifnullfree.cocci
> >
> > CC: Hans de Goede <hdego...@redhat.com>
> > Signed-off-by: Julia Lawall <julia.law...@lip6.fr>
> > Signed-off-by: Fengguang Wu <fengguang...@intel.com>
>
> Thank you for these patches. Larry Finger has just
> posted a large cleanup series for the rtl8723bs
> driver, which will likely conflict with these.
>
> Can you rebase and resend these patches once Larry's
> series has been merged please?
OK, thanks for the update. There were a bunch of other coccicheck
warnings. But I will wait for the improved version.
julia
>
> Regards,
>
> Hans
>
>
>
> > ---
> >
> > rtw_cmd.c | 6 ++----
> > 1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > --- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
> > +++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
> > @@ -248,11 +248,9 @@ void _rtw_free_evt_priv(struct evt_priv
> > void _rtw_free_cmd_priv(struct cmd_priv *pcmdpriv)
> > {
> > if (pcmdpriv) {
> > - if (pcmdpriv->cmd_allocated_buf)
> > - kfree(pcmdpriv->cmd_allocated_buf);
> > + kfree(pcmdpriv->cmd_allocated_buf);
> >
> > - if (pcmdpriv->rsp_allocated_buf)
> > - kfree(pcmdpriv->rsp_allocated_buf);
> > + kfree(pcmdpriv->rsp_allocated_buf);
> >
> > mutex_destroy(&pcmdpriv->sctx_mutex);
> > }
> >
>
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
