mirrors/linux
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git synced 2026-05-05 17:03:47 -04:00
Code Issues Packages Projects Releases Wiki Activity

staging: wfx: declare the field 'packet_id' with native byte order

Browse Source 
The field packet_id is not interpreted by the device. It is only used as
identifier for the device answer. So it is not necessary to declare it
little endian. It fixes some warnings raised by Sparse without
complexifying the code.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200512150414.267198-12-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Jérôme Pouiller
2020-05-12 17:04:08 +02:00
committed by Greg Kroah-Hartman
parent b8743c786b
commit 9fee675c2c
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/staging/wfx/hif_api_cmd.h
View File

@@ -254,7 +254,9 @@ struct hif_ht_tx_parameters {
} __packed;
struct hif_req_tx {
__le32 packet_id;
// packet_id is not interpreted by the device, so it is not necessary to
// declare it little endian
u32 packet_id;
u8 max_tx_rate;
struct hif_queue queue_id;
struct hif_data_flags data_flags;
@@ -283,7 +285,9 @@ struct hif_tx_result_flags {
struct hif_cnf_tx {
__le32 status;
__le32 packet_id;
// packet_id is copied from struct hif_req_tx without been interpreted
// by the device, so it is not necessary to declare it little endian
u32 packet_id;
u8 txed_rate;
u8 ack_failures;
struct hif_tx_result_flags tx_result_flags;