Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong stb_type detected for sf8008 #11

Open
LraiZer opened this issue May 4, 2024 · 2 comments
Open

Wrong stb_type detected for sf8008 #11

LraiZer opened this issue May 4, 2024 · 2 comments

Comments

@LraiZer
Copy link

LraiZer commented May 4, 2024

So i was confused for a while as to why some aio-grab code changes i was testing on an Octagon sf8008 test box would not work, but it turns out the STB thinks it is a BRCM7400

With the current order of detection as such, and only the last check being detected at all, which stb_type category is best to add an earlier correct hierarchy detection for this STB instead of this box being wrongly detected as a dm8000?

root@sf8008:~# cat /proc/stb/info/boxtype
sf8008
root@sf8008:~# cat /proc/stb/info/chipset
3798mv200
root@sf8008:~# cat /proc/stb/info/model
dm8000

Correct detection: stb_type = HISIL_ARM

LraiZer referenced this issue in oe-alliance/oe-alliance-core May 4, 2024
see: https://git.openembedded.org/openemb...d666a282f6a27f
Grabbing 32bit Framebuffer ...
... Framebuffer-Size: 1920 x 1080
Grabbing Video ...
Mainmemory: <Memmapping failed>
Resizing Video to 1920 x 1080 ...
Merge Video with Framebuffer ...
@jbleyel
Copy link
Contributor

jbleyel commented May 5, 2024

Do you know that sf8008 build is using a closed source binary for grab?
And this binary also supports 4K video.

https://github.com/oe-alliance/oe-alliance-core/blob/806fea7b82c2c60d4a3695c75a9cf33b92c83c51/meta-brands/meta-octagon/recipes-bsp/octagon-grab-hisi3798mv300.bb#L14

Sf8008 in only one example because there are many other HISI boxes with closed source code for grab.

@jbleyel
Copy link
Contributor

jbleyel commented May 5, 2024

If you don’t use a shared binary then you can remove the detection code and make that via configure.ac

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants