From mboxrd@z Thu Jan 1 00:00:00 1970 From: Martin Kepplinger Subject: [PATCH] input: misc: yealink: define packet offset __be16 instead of u16 Date: Mon, 10 Apr 2017 10:56:17 +0200 Message-ID: <1491814577-26408-1-git-send-email-martin.kepplinger@ginzinger.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from mail.ginzinger.com ([31.193.165.229]:11461 "EHLO mail.ginzinger.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750989AbdDJJD0 (ORCPT ); Mon, 10 Apr 2017 05:03:26 -0400 Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Henk.Vergonet@gmail.com, dmitry.torokhov@gmail.com Cc: usbb2k-api-dev@nongnu.org, linux-input@vger.kernel.org, linux-kernel@vger.kernel.org, Martin Kepplinger sparse says warning: incorrect type in assignment (different base types) expected unsigned short [unsigned] [usertype] offset got restricted __be16 [usertype] for every usage of cpu_to_be16 in yealink.c. Defining it __be16 in the first place shouldn't hurt. Signed-off-by: Martin Kepplinger --- drivers/input/misc/yealink.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/misc/yealink.h b/drivers/input/misc/yealink.h index 1e0f523..934c247 100644 --- a/drivers/input/misc/yealink.h +++ b/drivers/input/misc/yealink.h @@ -28,7 +28,7 @@ struct yld_ctl_packet { u8 cmd; /* command code, see below */ u8 size; /* 1-11, size of used data bytes. */ - u16 offset; /* internal packet offset */ + __be16 offset; /* internal packet offset */ u8 data[11]; s8 sum; /* negative sum of 15 preceding bytes */ } __attribute__ ((packed)); -- 2.1.4