From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751099AbdIKWvU (ORCPT ); Mon, 11 Sep 2017 18:51:20 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:37340 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751028AbdIKWvT (ORCPT ); Mon, 11 Sep 2017 18:51:19 -0400 X-Google-Smtp-Source: ADKCNb4WwdwPKMcG8l60Mo2K5iONt69D3kJVB1NnWBUqLT1EcxK7+RQnonl+q4y6QpUETkxnRff7sQ== To: Greg KH , sergio.paracuellos@gmail.com, juliana.orod@gmail.com Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org From: Aviya Erenfeld Subject: [PATCH] staging: wlan-ng: Avoid bogus endianness Message-ID: <83e35218-eb69-7f83-6fe1-faec8ec755ce@gmail.com> Date: Tue, 12 Sep 2017 01:51:10 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The linkstatus variable in the info struct received as __le16 but handled in every other place in the driver as u16 Fix that and remove the sparse warning that occurred due to it: prism2sta.c:1450:29: warning: incorrect type in assignment (different base types) prism2sta.c:1450:29: expected unsigned short [unsigned] [usertype] link_status_new prism2sta.c:1450:29: got restricted __le16 [usertype] linkstatus Signed-off-by: Aviya Erenfeld --- drivers/staging/wlan-ng/prism2sta.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c index c9df450..070a237 100644 --- a/drivers/staging/wlan-ng/prism2sta.c +++ b/drivers/staging/wlan-ng/prism2sta.c @@ -1447,7 +1447,7 @@ static void prism2sta_inf_linkstatus(struct wlandevice *wlandev, { struct hfa384x *hw = wlandev->priv; - hw->link_status_new = inf->info.linkstatus.linkstatus; + hw->link_status_new = le16_to_cpu(inf->info.linkstatus.linkstatus); schedule_work(&hw->link_bh); } -- 2.7.4