Opened 11 years ago

Closed 11 years ago

#2859 closed enhancement (invalid)

Changeset [7705] by norbi20

Reported by: norbi20 Owned by: norbi20
Priority: Please fill in Component: Protocol - CCCam
Severity: high Keywords:
Cc: Sensitive: yes

Description

CCcam ecm fix !
reader fix !
cliens fix !

Attachments (5)

module-cccam.c (98.0 KB ) - added by norbi20 11 years ago.
oscam-reader.c (17.0 KB ) - added by norbi20 11 years ago.
module-newcamd.c (40.9 KB ) - added by norbi20 11 years ago.
module-dvbapi.c (73.2 KB ) - added by norbi20 11 years ago.
norbi.patch (6.7 KB ) - added by theparasol 11 years ago.
Norbi's changes in a diff patch

Download all attachments as: .zip

Change History (19)

by norbi20, 11 years ago

Attachment: module-cccam.c added

by norbi20, 11 years ago

Attachment: oscam-reader.c added

by norbi20, 11 years ago

Attachment: module-newcamd.c added

by norbi20, 11 years ago

Attachment: module-dvbapi.c added

comment:1 by norbi20, 11 years ago

Owner: set to norbi20
Priority: majorPlease fill in
Status: newassigned
Type: taskenhancement

comment:2 by norbi20, 11 years ago

Status: assignednew

comment:3 by Deas, 11 years ago

please provide a PATCH and not full files.

comment:4 by norbi20, 11 years ago

copy in oscam-svn modified data files
The in terminal
make
make sh4
oscam easy build create oscam file

http://kepfeltoltes.hu/121013/K_perny_k_p_-_2012-10-13_-_15.28.15_www.kepfeltoltes.hu_.png

Last edited 11 years ago by norbi20 (previous) (diff)

comment:5 by dropsman, 11 years ago

@norbi20:

Please create a patch for every fix, coz if there are new revisions out and people will test your changes the compilation goes wrong.

comment:6 by theparasol, 11 years ago

I was curious to examine his changes so I created the diff patch.
Seems he changed some timeouts to let it work in his oscam card system.
Perhaps we can use it as usefull input to make those timings userconfigable?

@Norbi: If you create tickets next time please try harder to make a description or write it in your native language and use google translate to get it into english and copy paste that! Your input can be useful but ignored due to bad language = bad improvements...

Last edited 11 years ago by theparasol (previous) (diff)

by theparasol, 11 years ago

Attachment: norbi.patch added

Norbi's changes in a diff patch

comment:7 by gf, 11 years ago

What fix? This just lowers all timeouts to 1/4 of their values. What that this fix?

comment:8 by norbi20, 11 years ago

thx yes that's it :)

comment:9 by theparasol, 11 years ago

Norbi, GF asks what this patch of your exactly fixes. Can you describe it to us?

comment:10 by Deas, 11 years ago

@norbi20

i know how to build oscam - you don´t need to explain this to me...

but a PATCH makes it very easy to check what you changed. just click on the file theparasol created for you and you see excactly what was changed. the way you did it first is - sorry - useless.

comment:11 by norbi20, 11 years ago

ecm timeout on cccam kliens smallər

Version 1, edited 11 years ago by norbi20 (previous) (next) (diff)

comment:12 by gf, 11 years ago

Look I understand why you are lowering timeouts, the thing that is not clear is why the timeouts are set the way they are. What are you breaking by lowering the timeouts?

Also your patch is not only touching cccam, it changes the timeouts all over the place. Config items that are defined as seconds are changed to 250 ms which is certainly NOT ok. As for lowering sleep times, that can be discussed on case by case basis but a patch like the one posted above is just too wrong for the small gain it'll give you in certain cases.

Last edited 11 years ago by gf (previous) (diff)

comment:14 by Deas, 11 years ago

Resolution: invalid
Status: newclosed

as the cccam protocol is maintained by corsair in streamboard - please contact him and discuss this change with him. nobody will commit this without corsair says that it is ok.

Note: See TracTickets for help on using tickets.