From: Kalle Valo <kvalo@codeaurora.org>
To: Siva Rebbagondla <siva8118@gmail.com>
Cc: kbuild test robot <lkp@intel.com>,
kbuild-all@01.org, linux-wireless@vger.kernel.org
Subject: Re: [wireless-drivers-next:master 28/45] drivers/net/wireless/rsi/rsi_91x_usb.c:229:27: sparse: restricted __le32 degrades to integer
Date: Fri, 26 Apr 2019 06:38:56 +0300 [thread overview]
Message-ID: <875zr1wi3z.fsf@kamboji.qca.qualcomm.com> (raw)
In-Reply-To: <CANGSkXQUwyAsQxKFrJSeycDJ-X5SUnoXroJt0a2tjg2btSyZvQ@mail.gmail.com> (Siva Rebbagondla's message of "Fri, 26 Apr 2019 05:50:08 +0530")
+ linux-wireless
Siva Rebbagondla <siva8118@gmail.com> writes:
> Hi Kalle,
>
> On Fri, Apr 26, 2019 at 1:06 AM kbuild test robot <lkp@intel.com> wrote:
>>
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
>> head: 147b502bda338f4f2dff19faaa5829b691305ea5
>> commit: 0a60014b76f512f18e48cfb4efc71e07c6791996 [28/45] rsi: miscallaneous changes for 9116 and common
>> reproduce:
>> # apt-get install sparse
>> git checkout 0a60014b76f512f18e48cfb4efc71e07c6791996
>> make ARCH=x86_64 allmodconfig
>> make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
>>
>> If you fix the issue, kindly add following tag
>> Reported-by: kbuild test robot <lkp@intel.com>
>>
>>
>> sparse warnings: (new ones prefixed by >>)
>>
>> >> drivers/net/wireless/rsi/rsi_91x_usb.c:229:27: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:230:27: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:231:27: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:232:27: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:238:36: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:239:35: sparse: restricted __le32 degrades to integer
>> drivers/net/wireless/rsi/rsi_91x_usb.c:354:28: sparse: expression using sizeof(void)
>> drivers/net/wireless/rsi/rsi_91x_usb.c:401:28: sparse: expression using sizeof(void)
>>
>> vim +229 drivers/net/wireless/rsi/rsi_91x_usb.c
>>
>> 203
>> 204 /**
>> 205 * rsi_usb_reg_write() - This function writes the given data into the given
>> 206 * register address.
>> 207 * @usbdev: Pointer to the usb_device structure.
>> 208 * @reg: Address of the register.
>> 209 * @value: Value to write.
>> 210 * @len: Length of data to be written.
>> 211 *
>> 212 * Return: status: 0 on success, a negative error code on failure.
>> 213 */
>> 214 static int rsi_usb_reg_write(struct usb_device *usbdev,
>> 215 u32 reg,
>> 216 u32 value,
>> 217 u16 len)
>> 218 {
>> 219 u8 *usb_reg_buf;
>> 220 int status = -ENOMEM;
>> 221
>> 222 if (len > RSI_USB_CTRL_BUF_SIZE)
>> 223 return -EINVAL;
>> 224
>> 225 usb_reg_buf = kmalloc(RSI_USB_CTRL_BUF_SIZE, GFP_KERNEL);
>> 226 if (!usb_reg_buf)
>> 227 return status;
>> 228
>> > 229 usb_reg_buf[0] = (cpu_to_le32(value) & 0x00ff);
>> 230 usb_reg_buf[1] = (cpu_to_le32(value) & 0xff00) >> 8;
>> 231 usb_reg_buf[2] = (cpu_to_le32(value) & 0x00ff0000) >> 16;
>> 232 usb_reg_buf[3] = (cpu_to_le32(value) & 0xff000000) >> 24;
>> 233
>> 234 status = usb_control_msg(usbdev,
>> 235 usb_sndctrlpipe(usbdev, 0),
>> 236 USB_VENDOR_REGISTER_WRITE,
>> 237 RSI_USB_REQ_OUT,
>> 238 ((cpu_to_le32(reg) & 0xffff0000) >> 16),
>> 239 (cpu_to_le32(reg) & 0xffff),
>> 240 (void *)usb_reg_buf,
>> 241 len,
>> 242 USB_CTRL_SET_TIMEOUT);
>> 243 if (status < 0) {
>> 244 rsi_dbg(ERR_ZONE,
>> 245 "%s: Reg write failed with error code :%d\n",
>> 246 __func__, status);
>> 247 }
>> 248 kfree(usb_reg_buf);
>> 249
>> 250 return status;
>> 251 }
>> 252
>>
>> ---
>> 0-DAY kernel test infrastructure Open Source Technology Center
>> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
> Some where went wrong. I fixed this one. I will resend you the patches
> with changes.
Your patches are now applied and cannot be changed, so no point of
resending them anymore. You need to send followup patches which fix the
issues and apply to wireless-drivers-next.
And remember to use Fixes tag so that we know which commit patch fixes.
--
Kalle Valo
next parent reply other threads:[~2019-04-26 3:39 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <201904260301.nITK1ldv%lkp@intel.com>
[not found] ` <CANGSkXQUwyAsQxKFrJSeycDJ-X5SUnoXroJt0a2tjg2btSyZvQ@mail.gmail.com>
2019-04-26 3:38 ` Kalle Valo [this message]
2019-04-26 11:34 ` [wireless-drivers-next:master 28/45] drivers/net/wireless/rsi/rsi_91x_usb.c:229:27: sparse: restricted __le32 degrades to integer Siva Rebbagondla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=875zr1wi3z.fsf@kamboji.qca.qualcomm.com \
--to=kvalo@codeaurora.org \
--cc=kbuild-all@01.org \
--cc=linux-wireless@vger.kernel.org \
--cc=lkp@intel.com \
--cc=siva8118@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox