diff options
| author | Jia-Ju Bai <baijiaju1990@gmail.com> | 2018-07-27 04:58:43 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-07-27 14:11:13 +0200 |
| commit | 2b5c5798750180a34cb1f39cf648d8a8bceec287 (patch) | |
| tree | e61aebc09199fb5676d25cb3cb35e21e7edcc4df /usr | |
| parent | media: video-i2c: hwmon: fix return value from amg88xx_hwmon_init() (diff) | |
| download | linux-2b5c5798750180a34cb1f39cf648d8a8bceec287.tar.xz linux-2b5c5798750180a34cb1f39cf648d8a8bceec287.zip | |
media: i2c: adv7842: Replace mdelay() with msleep() and usleep_range() in adv7842_ddr_ram_test()
adv7842_ddr_ram_test() is never called in atomic context.
It only calls from:
adv7842_ddr_ram_test() <- adv7842_command_ram_test() <- adv7842_ioctl()
adv7842_ddr_ram_test() calls mdelay() to busily wait,
which is not necessary.
mdelay() can be replaced with msleep() and usleep_range().
This is found by a static analysis tool named DCNS written by myself.
Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'usr')
0 files changed, 0 insertions, 0 deletions