Re: [PATCH 09/12] staging: ks7010: remove unnecessary cast
On Tue, Mar 14, 2017 at 08:07:15AM +0800, Greg Kroah-Hartman wrote: > On Tue, Mar 14, 2017 at 09:54:07AM +1100, Tobin C. Harding wrote: > > Return value from kmalloc() does not require a cast. > > > > Remove unnecessary cast. > > > > Signed-off-by: Tobin C. Harding > > --- > > drivers/staging/ks7010/ks_wlan_net.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/staging/ks7010/ks_wlan_net.c > > b/drivers/staging/ks7010/ks_wlan_net.c > > index 1451ba7..e1e4a31 100644 > > --- a/drivers/staging/ks7010/ks_wlan_net.c > > +++ b/drivers/staging/ks7010/ks_wlan_net.c > > @@ -2561,7 +2561,7 @@ static int ks_wlan_data_write(struct net_device *dev, > > if (priv->sleep_mode == SLP_SLEEP) > > return -EPERM; > > /* for SLEEP MODE */ > > - wbuff = (unsigned char *)kmalloc(dwrq->length, GFP_ATOMIC); > > + wbuff = kmalloc(dwrq->length, GFP_ATOMIC); > > if (!wbuff) > > return -EFAULT; > > This should also return -ENOMEM, but that's a separate fix :) Good catch. I'll make a note of this and when/if the patch set gets merged do a separate patch with this fix since it is not checkpatch related. thanks, Tobin. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH 09/12] staging: ks7010: remove unnecessary cast
On Tue, Mar 14, 2017 at 09:54:07AM +1100, Tobin C. Harding wrote: > Return value from kmalloc() does not require a cast. > > Remove unnecessary cast. > > Signed-off-by: Tobin C. Harding > --- > drivers/staging/ks7010/ks_wlan_net.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/ks7010/ks_wlan_net.c > b/drivers/staging/ks7010/ks_wlan_net.c > index 1451ba7..e1e4a31 100644 > --- a/drivers/staging/ks7010/ks_wlan_net.c > +++ b/drivers/staging/ks7010/ks_wlan_net.c > @@ -2561,7 +2561,7 @@ static int ks_wlan_data_write(struct net_device *dev, > if (priv->sleep_mode == SLP_SLEEP) > return -EPERM; > /* for SLEEP MODE */ > - wbuff = (unsigned char *)kmalloc(dwrq->length, GFP_ATOMIC); > + wbuff = kmalloc(dwrq->length, GFP_ATOMIC); > if (!wbuff) > return -EFAULT; This should also return -ENOMEM, but that's a separate fix :) thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH 09/12] staging: ks7010: remove unnecessary cast
Return value from kmalloc() does not require a cast. Remove unnecessary cast. Signed-off-by: Tobin C. Harding --- drivers/staging/ks7010/ks_wlan_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ks7010/ks_wlan_net.c b/drivers/staging/ks7010/ks_wlan_net.c index 1451ba7..e1e4a31 100644 --- a/drivers/staging/ks7010/ks_wlan_net.c +++ b/drivers/staging/ks7010/ks_wlan_net.c @@ -2561,7 +2561,7 @@ static int ks_wlan_data_write(struct net_device *dev, if (priv->sleep_mode == SLP_SLEEP) return -EPERM; /* for SLEEP MODE */ - wbuff = (unsigned char *)kmalloc(dwrq->length, GFP_ATOMIC); + wbuff = kmalloc(dwrq->length, GFP_ATOMIC); if (!wbuff) return -EFAULT; memcpy(wbuff, extra, dwrq->length); -- 2.7.4 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel