From: Daniel W. S. Almeida <dwlsalmeida@xxxxxxxxx> Fix the following error for builds on 32bit architectures: ERROR: modpost: "__udivdi3" [drivers/media/test-drivers/vidtv/dvb-vidtv-bridge.ko] undefined! Which is due to 64bit divisions that did not go through the helpers in linux/math64.h As vidtv_mux_check_mux_rate was not operational in its current form, drop the entire function while it is not fixed properly. For now, call vidtv_mux_pad_with_nulls with a constant number of packets to avoid warnings due to unused functions when building this driver. Fixes: f90cf6079bf67988 ("media: vidtv: add a bridge driver") Signed-off-by: Daniel W. S. Almeida <dwlsalmeida@xxxxxxxxx> --- drivers/media/test-drivers/vidtv/vidtv_mux.c | 34 +------------------ .../media/test-drivers/vidtv/vidtv_s302m.c | 4 +-- 2 files changed, 3 insertions(+), 35 deletions(-) diff --git a/drivers/media/test-drivers/vidtv/vidtv_mux.c b/drivers/media/test-drivers/vidtv/vidtv_mux.c index 5d1a275d504b..6e402a880fdc 100644 --- a/drivers/media/test-drivers/vidtv/vidtv_mux.c +++ b/drivers/media/test-drivers/vidtv/vidtv_mux.c @@ -336,38 +336,6 @@ static u32 vidtv_mux_pad_with_nulls(struct vidtv_mux *m, u32 npkts) return nbytes; } -static u32 vidtv_mux_check_mux_rate(struct vidtv_mux *m) -{ - /* - * attempt to maintain a constant mux rate, padding with null packets - * if needed - */ - - u32 nbytes = 0; /* the number of bytes written by this function */ - - u64 nbytes_expected; /* the number of bytes we should have written */ - u64 nbytes_streamed; /* the number of bytes we actually wrote */ - u32 num_null_pkts; /* number of null packets to bridge the gap */ - - u64 elapsed_time_msecs = jiffies_to_usecs(m->timing.current_jiffies - - m->timing.past_jiffies); - - elapsed_time_msecs = min(elapsed_time_msecs, (u64)VIDTV_MAX_SLEEP_USECS / 1000); - nbytes_expected = div64_u64(m->mux_rate_kbytes_sec * 1000, MSEC_PER_SEC); - nbytes_expected *= elapsed_time_msecs; - - nbytes_streamed = m->mux_buf_offset; - - if (nbytes_streamed < nbytes_expected) { - /* can't write half a packet: roundup to a 188 multiple */ - nbytes_expected = roundup(nbytes_expected - nbytes_streamed, TS_PACKET_LEN); - num_null_pkts = nbytes_expected / TS_PACKET_LEN; - nbytes += vidtv_mux_pad_with_nulls(m, num_null_pkts); - } - - return nbytes; -} - static void vidtv_mux_clear(struct vidtv_mux *m) { /* clear the packets currently in the mux */ @@ -397,7 +365,7 @@ static void vidtv_mux_tick(struct work_struct *work) nbytes += vidtv_mux_push_si(m); nbytes += vidtv_mux_poll_encoders(m); - nbytes += vidtv_mux_check_mux_rate(m); + nbytes += vidtv_mux_pad_with_nulls(m, 256); npkts = nbytes / TS_PACKET_LEN; diff --git a/drivers/media/test-drivers/vidtv/vidtv_s302m.c b/drivers/media/test-drivers/vidtv/vidtv_s302m.c index f8049cdf564a..e3290facf57b 100644 --- a/drivers/media/test-drivers/vidtv/vidtv_s302m.c +++ b/drivers/media/test-drivers/vidtv/vidtv_s302m.c @@ -285,12 +285,12 @@ static void vidtv_s302m_compute_pts(struct vidtv_encoder *e) { u64 count = e->sample_count; struct vidtv_access_unit *au = e->access_units; + u32 duration = CLOCK_UNIT_90KHZ / e->sampling_rate_hz; while (au) { count += au->num_samples; - au->pts = count * - CLOCK_UNIT_90KHZ / e->sampling_rate_hz; + au->pts = count * duration; au = au->next; } -- 2.28.0