Thread (12 messages) 12 messages, 5 authors, 2012-09-11

Re: [PATCH] dvb_frontend: Multistream support

From: Antti Palosaari <hidden>
Date: 2012-08-17 18:52:55

On 08/17/2012 08:21 PM, CrazyCat wrote:
16.08.2012, 21:11, "Antti Palosaari" [off-list ref]:
quoted
quoted
  - /* ISDB-T specifics */
  - u32 isdbs_ts_id;
  -
  - /* DVB-T2 specifics */
  - u32                     dvbt2_plp_id;
  + /* Multistream specifics */
  + u32 stream_id;
u32 == 32 bit long unsigned number. See next comment.
quoted
  - c->isdbs_ts_id = 0;
  - c->dvbt2_plp_id = 0;
  + c->stream_id = -1;
unsigned number cannot be -1. It can be only 0 or bigger. Due to that
this is wrong.
so maybe better declare in as int ? depend from standard valid stream id (for DVB is 0-255) and any another value (-1) disable stream filtering in demod.
I agree that. Actually I was thinking same. For DVB-T2 valid values are 
0-255, I haven't looked others but surely int maximum should enough for all.

regards
Antti

-- 
http://palosaari.fi/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help