1748 replies to this topic

[christophecvr]

You can omit the 13, enum numbering continues after the 12.
But I think we have to wait for betacentauri's comments, he knows the exact reasoning behind the 12, and he can tell you whether 13 is a good idea (or perhaps we should skip even more values to avoid conflicts)

 

Thank's I was not shure what it would have done whitout the =13 behind.

 

But in the mean time I already did experiment and tested with the cDDPPID=13

 

vuduo2 ok also after reboot channel audio stream setting is retained selected and works.

 

For that I have here the patch 3

 

Now it's just waiting on betacentauri's comments if it's ok the 2 patch or the three posted here.

Attached Files

•  ac3plusCacheEnigma2-3.patch   7.69KB   12 downloads

[betacentauri]

Thank's I was not shure what it would have done whitout the =13 behind.

 

But in the mean time I already did experiment and tested with the cDDPPID=13

 

vuduo2 ok also after reboot channel audio stream setting is retained selected and works.

 

For that I have here the patch 3

 

Now it's just waiting on betacentauri's comments if it's ok the 2 patch or the three posted here.

 

Version 3 of the patch looks good (I didn't review the whole patch. Only the enum). As pieterg already said the "=13" is not necessary. But it's also no error.

 

Yes, there was a reason why 11 should not be used. Here is the discussion about it:

http://openpli.org/f...db/#entry399570

In short: Dreamboxes use 11 and if we also use it with another meaning, it's not possible for tools to support dreambox images and OpenPli based images at the same time.

Edited by betacentauri, 27 February 2014 - 16:40.

[pieterg]

thx, applied (minus the '= 13')

[christophecvr]

Thank's I now make a build for my two boxes from scratch. To check all (for my dm8000 I wel add the extra decoder.cpp patch off athoik)

[theparasol]

Next time better make a comment in the source?
Avoids such situations

[christophecvr]

Next time better make a comment in the source?
Avoids such situations

What do You mean ?

[theparasol]

If something is not tot be used:

*/ dont use xxx for compatbility with yyyyy */

[christophecvr]

If something is not tot be used:

*/ dont use xxx for compatbility with yyyyy */

Yes that's I nice idea.

[WanWizard]

What is this? Documentation? In Enigma? Now that is something new....

[christophecvr]

All tests done ac3+ caching is now working perfect on my vuduo2 (basic pli4)

and on dm8000 also 100 % perfect with the ac3+ by pass patch applied to enigma2.

[dhwz]

thx for keeping compatibility (at least partially) 

c:13 will also be added to the upcoming dreamboxEDIT release.

[littlesat]

(at least partially)

Is there still an incompatebility?

Edited by littlesat, 28 February 2014 - 16:22.

[christophecvr]

(at least partially)

Is there still an incompatebility?

I think he means the ac3+ by pass with partially.

 

Just one question.

 

I noticed it would be A nice thing to also have the choice into the menu in Dutch it is called Automatische taalkeuzeselectie gues Automatic Languageselcetion in English.

To have by ac3 preference now it's no or yes,

 

But modifie it into (I will now only show the dutch part as I do have ducth setup)

 Ac3 voorkeur                                            nee/ja (as choice)

Ac3 of Ac3+(ddp) voorkeur                      nee/ac3/ddp  (as choice) ddp is the ec3+ stream.

 

I now that I have to add an  extra else if into the pmt.cpp  so that config parameter into enigma2/settings :

config.autolanguage.audio_defaultddp=true is followed as the config.autolanguage.audio_defaultac3=true

 

But I can't find the file where to modifie the menu Automatische taalkeuzeselectie

[Persian Prince]

regarding this commit http://sourceforge.n...08386ed67f43c8/

 

we have 3 more :

 

/proc/stb/info/hwmodel

/proc/stb/info/gbmodel

/proc/stb/info/gmmodel

 

and that's all for current brands/models

[Taapat]

And after this commit http://sourceforge.n...4c3303b6167e05/
please remove import os in 23 line in http://sourceforge.n...rtWizard.py#l23

It is also no longer necessary.

[littlesat]

Thanks!

[mirakels]

regarding this commit http://sourceforge.n...08386ed67f43c8/

 

we have 3 more :

 

/proc/stb/info/hwmodel

/proc/stb/info/gbmodel

/proc/stb/info/gmmodel

 

and that's all for current brands/models

 

Will add.

What brands are these?

Do these files describe full model name (like et boxes) or only the subtypes (like vu boxes)?

 

Would be useful to collect more data from such boxes.

The following would collect usefull info:

$ cd /proc/stb
$ grep  -r  ""  *

[mirakels]

Bump. No intetest in gathering data?

[Persian Prince]

regarding this commit http://sourceforge.n...08386ed67f43c8/

 

we have 3 more :

 

/proc/stb/info/hwmodel

/proc/stb/info/gbmodel

/proc/stb/info/gmmodel

 

and that's all for current brands/models

 

Will add.

What brands are these?

Do these files describe full model name (like et boxes) or only the subtypes (like vu boxes)?

 

Would be useful to collect more data from such boxes.

The following would collect usefull info:

$ cd /proc/stb
$ grep  -r  ""  *

 

sorry for late answer , i didn't check the thread

 

"hwmodel" is for some brands like technomate - iqon - edision - jepssen - sogno

 

example : tmtwinoe - ios100hd - optimussos1 - mediabox - sogno-8800hd

 

"gbmodel" is only for gigablue

 

example : gb800se - gbquad

 

"gmmodel" is only for golden media hypercube which gives "su980" the model name

[mirakels]

Thanks,

But please lets try collect the /proc/stb info for all these (anmd other)  boxes. We can put them in the wiki for everybody to see.

 

So run these commands on the specific box and copy paste the result her (or capture it in a file and post as an attachment):

 

 

$ cd /proc/stb
$ grep  -r  ""  *
 

 

We already have info for the following boxes (not yet in the wiki yet. thinking about how to do that...)

 

dm8000 (using drivers with 2.6.18 kernel)
dm800se
et4000
et6500
et9000
vuduo
vuduo2
vusolo2
vuultimo
vuuno
xp1000
 

Please add your contributions...