diff options
| author | Richard Fitzgerald <rf@opensource.cirrus.com> | 2026-06-11 18:12:34 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-06-11 22:39:06 +0300 |
| commit | 3073eb1f1143deabbda6a043238ab9d99672e7c8 (patch) | |
| tree | bf57ae5ffb579eaea93682d40fd53b9dfd4e9e85 /include/linux/timerqueue.h | |
| parent | 0f54ce994b23875aad771064b53483f7f791efbf (diff) | |
| download | linux-3073eb1f1143deabbda6a043238ab9d99672e7c8.tar.xz | |
ASoC: cs35l56: Fix wrong error test on simple_write_to_buffer()
In cs35l56_cal_data_debugfs_write() fix the if statement that checks for
error return to only check for negative values.
Reported by Sashiko:
simple_write_to_buffer() returns the positive number of bytes copied
on success. Since the condition returns immediately on any non-zero
value, is it possible that the written calibration data is discarded
and cs35l56_stash_calibration() is never called?
Fixes: f7097161e94c ("ASoC: cs35l56: Add common code for factory calibration")
Reported-by: Sashiko <sashiko-bot@kernel.org>
Link: https://sashiko.dev/#/patchset/20260610093432.557375-1-rf%40opensource.cirrus.com
Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://patch.msgid.link/20260611151234.1111153-1-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include/linux/timerqueue.h')
0 files changed, 0 insertions, 0 deletions
