diff options
author | Krzysztof Kozlowski <krzk@kernel.org> | 2020-07-20 13:03:01 +0200 |
---|---|---|
committer | Krzysztof Kozlowski <krzk@kernel.org> | 2020-07-24 09:42:44 +0200 |
commit | c4f16e96d8fdd62ef12898fc0965c42093bed237 (patch) | |
tree | 1542119585f269111908c8f0e4505aa5416d9157 /drivers/memory/pl353-smc.c | |
parent | b3a9e3b9622ae10064826dccb4f7a52bd88c7407 (diff) |
memory: samsung: exynos5422-dmc: Do not ignore return code of regmap_read()
Check for regmap_read() return code before using the read value in
following write in exynos5_switch_timing_regs(). Pass reading error
code to the callers.
This does not introduce proper error handling for such failed reads (and
obviously regmap_write() error is still ignored) because the driver
ignored this in all places. Therefor it only fixes reported issue while
matching current driver coding style:
drivers/memory/samsung/exynos5422-dmc.c: In function 'exynos5_switch_timing_regs':
>> drivers/memory/samsung/exynos5422-dmc.c:216:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Diffstat (limited to 'drivers/memory/pl353-smc.c')
0 files changed, 0 insertions, 0 deletions