mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-05 02:21:52 +09:00
media: fc2580: fix tuning failure on 32-bit arch
commit 8845cc6415 upstream.
There was some frequency calculation overflows which caused tuning
failure on 32-bit architecture. Use 64-bit numbers where needed in
order to avoid calculation overflows.
Thanks for the Finnish person, who asked remain anonymous, reporting,
testing and suggesting the fix.
Signed-off-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
0a4e3565df
commit
4f792a2972
@@ -195,7 +195,7 @@ static int fc2580_set_params(struct dvb_frontend *fe)
|
||||
|
||||
f_ref = 2UL * priv->cfg->clock / r_val;
|
||||
n_val = div_u64_rem(f_vco, f_ref, &k_val);
|
||||
k_val_reg = 1UL * k_val * (1 << 20) / f_ref;
|
||||
k_val_reg = div_u64(1ULL * k_val * (1 << 20), f_ref);
|
||||
|
||||
ret = fc2580_wr_reg(priv, 0x18, r18_val | ((k_val_reg >> 16) & 0xff));
|
||||
if (ret < 0)
|
||||
@@ -348,8 +348,8 @@ static int fc2580_set_params(struct dvb_frontend *fe)
|
||||
if (ret < 0)
|
||||
goto err;
|
||||
|
||||
ret = fc2580_wr_reg(priv, 0x37, 1UL * priv->cfg->clock * \
|
||||
fc2580_if_filter_lut[i].mul / 1000000000);
|
||||
ret = fc2580_wr_reg(priv, 0x37, div_u64(1ULL * priv->cfg->clock *
|
||||
fc2580_if_filter_lut[i].mul, 1000000000));
|
||||
if (ret < 0)
|
||||
goto err;
|
||||
|
||||
|
||||
@@ -22,6 +22,7 @@
|
||||
#define FC2580_PRIV_H
|
||||
|
||||
#include "fc2580.h"
|
||||
#include <linux/math64.h>
|
||||
|
||||
struct fc2580_reg_val {
|
||||
u8 reg;
|
||||
|
||||
Reference in New Issue
Block a user