diff options
| author | Bryan O'Donoghue | 2015-07-22 00:50:10 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2015-07-22 19:46:53 +0200 |
| commit | 583cbf50e0a4c8918811f245860922353420a378 (patch) | |
| tree | 09a0d82fc6ac3b3ad65aa61ab2e112fbee148ef7 /drivers/staging/greybus/loopback.c | |
| parent | greybus: loopback: add commentary to sysfs variables (diff) | |
| download | kernel-qcow2-linux-583cbf50e0a4c8918811f245860922353420a378.tar.gz kernel-qcow2-linux-583cbf50e0a4c8918811f245860922353420a378.tar.xz kernel-qcow2-linux-583cbf50e0a4c8918811f245860922353420a378.zip | |
greybus: loopback: rename frequency to requests
The name frequency does not adequately describe the data-point tracking the
number of greybus operations performed in a second by the loopback code.
Firmware team is moving towards calling this variable requests-per-second
or similar. This patch renames to keep the namespace consistent.
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Patrick Titiano <ptitiano@baylibre.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/loopback.c')
| -rw-r--r-- | drivers/staging/greybus/loopback.c | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index cc19f1be7e2f..50eacd1e11a3 100644 --- a/drivers/staging/greybus/loopback.c +++ b/drivers/staging/greybus/loopback.c @@ -41,7 +41,7 @@ struct gb_loopback { struct gb_loopback_stats latency; struct gb_loopback_stats throughput; - struct gb_loopback_stats frequency; + struct gb_loopback_stats requests_per_second; struct timeval ts; struct timeval te; u64 elapsed_nsecs; @@ -133,8 +133,8 @@ static void gb_loopback_check_attr(struct gb_loopback *gb) /* Time to send and receive one message */ gb_loopback_stats_attrs(latency); -/* Number of packet sent per second on this cport */ -gb_loopback_stats_attrs(frequency); +/* Number of requests sent per second on this cport */ +gb_loopback_stats_attrs(requests_per_second); /* Quantity of data sent and received on this cport */ gb_loopback_stats_attrs(throughput); /* Number of errors encountered during loop */ @@ -166,7 +166,7 @@ gb_loopback_attr(iteration_max, u); static struct attribute *loopback_attrs[] = { dev_stats_attrs(latency), - dev_stats_attrs(frequency), + dev_stats_attrs(requests_per_second), dev_stats_attrs(throughput), &dev_attr_type.attr, &dev_attr_size.attr, @@ -320,7 +320,8 @@ static void gb_loopback_reset_stats(struct gb_loopback *gb) }; memcpy(&gb->latency, &reset, sizeof(struct gb_loopback_stats)); memcpy(&gb->throughput, &reset, sizeof(struct gb_loopback_stats)); - memcpy(&gb->frequency, &reset, sizeof(struct gb_loopback_stats)); + memcpy(&gb->requests_per_second, &reset, + sizeof(struct gb_loopback_stats)); memset(&gb->ts, 0, sizeof(struct timeval)); } @@ -336,12 +337,12 @@ static void gb_loopback_update_stats(struct gb_loopback_stats *stats, u64 val) do_div(stats->avg, stats->count); } -static void gb_loopback_frequency_update(struct gb_loopback *gb, u32 latency) +static void gb_loopback_requests_update(struct gb_loopback *gb, u32 latency) { - u32 freq = USEC_PER_SEC; + u32 req = USEC_PER_SEC; - do_div(freq, latency); - gb_loopback_update_stats(&gb->frequency, freq); + do_div(req, latency); + gb_loopback_update_stats(&gb->requests_per_second, req); } static void gb_loopback_throughput_update(struct gb_loopback *gb, u32 latency) @@ -386,9 +387,9 @@ static void gb_loopback_calculate_stats(struct gb_loopback *gb, /* Log latency stastic */ gb_loopback_update_stats(&gb->latency, lat); - /* Log throughput and frequency using latency as benchmark */ + /* Log throughput and requests using latency as benchmark */ gb_loopback_throughput_update(gb, lat); - gb_loopback_frequency_update(gb, lat); + gb_loopback_requests_update(gb, lat); } static int gb_loopback_fn(void *data) |