lkml.org 
[lkml]   [2020]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 05/17] staging: wfx: fix output of rx_stats on big endian hosts
    Date
    From: Jérôme Pouiller <jerome.pouiller@silabs.com>

    The struct hif_rx_stats contains only little endian values. Thus, it is
    necessary to fix byte ordering before to use them.

    Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
    ---
    drivers/staging/wfx/debug.c | 11 +++++++----
    1 file changed, 7 insertions(+), 4 deletions(-)

    diff --git a/drivers/staging/wfx/debug.c b/drivers/staging/wfx/debug.c
    index 2fae6c913b01..846a0b29f8c9 100644
    --- a/drivers/staging/wfx/debug.c
    +++ b/drivers/staging/wfx/debug.c
    @@ -155,7 +155,7 @@ static int wfx_rx_stats_show(struct seq_file *seq, void *v)
    mutex_lock(&wdev->rx_stats_lock);
    seq_printf(seq, "Timestamp: %dus\n", st->date);
    seq_printf(seq, "Low power clock: frequency %uHz, external %s\n",
    - st->pwr_clk_freq,
    + le32_to_cpu(st->pwr_clk_freq),
    st->is_ext_pwr_clk ? "yes" : "no");
    seq_printf(seq,
    "Num. of frames: %d, PER (x10e4): %d, Throughput: %dKbps/s\n",
    @@ -165,9 +165,12 @@ static int wfx_rx_stats_show(struct seq_file *seq, void *v)
    for (i = 0; i < ARRAY_SIZE(channel_names); i++) {
    if (channel_names[i])
    seq_printf(seq, "%5s %8d %8d %8d %8d %8d\n",
    - channel_names[i], st->nb_rx_by_rate[i],
    - st->per[i], st->rssi[i] / 100,
    - st->snr[i] / 100, st->cfo[i]);
    + channel_names[i],
    + le32_to_cpu(st->nb_rx_by_rate[i]),
    + le16_to_cpu(st->per[i]),
    + (s16)le16_to_cpu(st->rssi[i]) / 100,
    + (s16)le16_to_cpu(st->snr[i]) / 100,
    + (s16)le16_to_cpu(st->cfo[i]));
    }
    mutex_unlock(&wdev->rx_stats_lock);

    --
    2.26.2
    \
     
     \ /
      Last update: 2020-05-11 17:52    [W:5.690 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site