From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-pa0-f42.google.com ([209.85.220.42]:34204 "EHLO mail-pa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751141AbbGAHGI (ORCPT ); Wed, 1 Jul 2015 03:06:08 -0400 Received: by pabvl15 with SMTP id vl15so18480065pab.1 for ; Wed, 01 Jul 2015 00:06:08 -0700 (PDT) Date: Wed, 1 Jul 2015 12:36:01 +0530 From: Sudip Mukherjee To: Michel von Czettritz Cc: gregkh@linux.com, devel@driverdev.osuosl.org, Jes.Sorensen@redhat.com, linux-wireless@vger.kernel.org, Larry.Finger@lwfinger.net Subject: Re: [PATCH 2/3] staging: rtl8723au: fix "incorrect type in assignment" Message-ID: <20150701070601.GB2411@sudip-PC> (sfid-20150701_090613_355562_E4D3F587) References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Jun 30, 2015 at 11:54:02PM +0200, Michel von Czettritz wrote: > Writing the output of cpu_to_le32 to an u32 or *(u32*) is an implicit > cast and results in an sparse warning. > > Since param and mask won't be changed, the implicit cast can be avoided > by creating local variables. > > Signed-off-by: Michel von Czettritz > --- > drivers/staging/rtl8723au/hal/rtl8723a_cmd.c | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c > index 1003365..a5e6726 100644 > --- a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c > +++ b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c > @@ -115,9 +115,10 @@ exit: > > int rtl8723a_set_rssi_cmd(struct rtw_adapter *padapter, u32 *param) > { > - *((u32 *)param) = cpu_to_le32(*(param)); > + __le32 param_le; checkpatch warns about "Missing a blank line after declarations" regards sudip