Commit a98ccfcf authored by Antti Palosaari's avatar Antti Palosaari Committed by Mauro Carvalho Chehab

[media] rtl2832_sdr: remove plain 64-bit divisions

Commit 0ba2aeb6
(v4l2-ctrls: increase internal min/max/step/def to 64 bit)
changes v4l2 controls to 64-bit. Driver it not working on 32-bit
arch as it uses directly control 'step' which is changed to 64-bit.
Reported-by: default avatarkbuild test robot <fengguang.wu@intel.com>
Cc: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarAntti Palosaari <crope@iki.fi>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent 9aabd95a
...@@ -1364,17 +1364,16 @@ static int rtl2832_sdr_s_ctrl(struct v4l2_ctrl *ctrl) ...@@ -1364,17 +1364,16 @@ static int rtl2832_sdr_s_ctrl(struct v4l2_ctrl *ctrl)
/* TODO: these controls should be moved to tuner drivers */ /* TODO: these controls should be moved to tuner drivers */
if (s->bandwidth_auto->val) { if (s->bandwidth_auto->val) {
/* Round towards the closest legal value */ /* Round towards the closest legal value */
s32 val = s->f_adc + s->bandwidth->step / 2; s32 val = s->f_adc + div_u64(s->bandwidth->step, 2);
u32 offset; u32 offset;
val = clamp_t(s32, val, s->bandwidth->minimum, val = clamp_t(s32, val, s->bandwidth->minimum,
s->bandwidth->maximum); s->bandwidth->maximum);
offset = val - s->bandwidth->minimum; offset = val - s->bandwidth->minimum;
offset = s->bandwidth->step * offset = s->bandwidth->step *
(offset / s->bandwidth->step); div_u64(offset, s->bandwidth->step);
s->bandwidth->val = s->bandwidth->minimum + offset; s->bandwidth->val = s->bandwidth->minimum + offset;
} }
c->bandwidth_hz = s->bandwidth->val; c->bandwidth_hz = s->bandwidth->val;
if (!test_bit(POWER_ON, &s->flags)) if (!test_bit(POWER_ON, &s->flags))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment