diff options
| author |   Peter Ujfalusi <peter.ujfalusi@linux.intel.com> | 2022-01-28 14:06:27 +0200 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2022-01-28 13:06:48 +0000 |
| commit | 7bd04b8d46b9362fb1ade63b99cd6ddee0740af4 (patch) | |
| tree | 153b01395215719ecf72772ddbe1b5ca62fb71b6 /sound/soc/sof/trace.c | |
| parent | ASoC: dt-bindings: realtek,rt5682s: Drop Tegra specifics from example (diff) | |
| download | wireguard-linux-7bd04b8d46b9362fb1ade63b99cd6ddee0740af4.tar.xz wireguard-linux-7bd04b8d46b9362fb1ade63b99cd6ddee0740af4.zip | |
ASoC: SOF: trace: Simplify count adjustment in trace_read
The first count check and fixup against "buffer - lpos" can be removed as
we will do the adjustment later against the "avail" in
sof_dfsentry_trace_read()
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/20220128120627.18443-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/sof/trace.c')
| -rw-r--r-- | sound/soc/sof/trace.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sound/soc/sof/trace.c b/sound/soc/sof/trace.c index f13024c8ebf2..9b505c4fe794 100644 --- a/sound/soc/sof/trace.c +++ b/sound/soc/sof/trace.c @@ -308,9 +308,6 @@ static ssize_t sof_dfsentry_trace_read(struct file *file, char __user *buffer, lpos_64 = lpos; lpos = do_div(lpos_64, buffer_size); - if (count > buffer_size - lpos) /* min() not used to avoid sparse warnings */ - count = buffer_size - lpos; - /* get available count based on current host offset */ avail = sof_wait_trace_avail(sdev, lpos, buffer_size); if (sdev->dtrace_error) { @@ -319,7 +316,8 @@ static ssize_t sof_dfsentry_trace_read(struct file *file, char __user *buffer, } /* make sure count is <= avail */ - count = avail > count ? count : avail; + if (count > avail) + count = avail; /* copy available trace data to debugfs */ rem = copy_to_user(buffer, ((u8 *)(dfse->buf) + lpos), count); |