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

[loire] Camera lacks of full Camera 2 API support #275

Closed
Gray47Maxx opened this issue Dec 19, 2018 · 11 comments
Closed

[loire] Camera lacks of full Camera 2 API support #275

Gray47Maxx opened this issue Dec 19, 2018 · 11 comments

Comments

@Gray47Maxx
Copy link
Contributor

Gray47Maxx commented Dec 19, 2018

Platform: loire
Device: suzu
Kernel version: 4.9
Android version: 9.0

Problem:
The camera is not supporting full C2API, which leads to problems with using third-party cameras like Google Camera. Binaries: v3 and possibly v4.

The bug is not presented with kernel 4.4 and latest blobs for it, that's why I thought this is a bug.

How to reproduce:
Try to probe it using something like Camera 2 Probe app.

Screenshot by @lubik1

Screenshot by @lubik1

@RobbieBnt
Copy link

Also confirming this issue on loire. Guess it's true for all platforms?

@Gray47Maxx
Copy link
Contributor Author

Maybe. Can someone check, please?

@bartcubbins
Copy link
Contributor

Only on loire

@Gray47Maxx
Copy link
Contributor Author

Hmm... Can this be fixed, @jerpelea?

@jerpelea
Copy link
Collaborator

i will check

@RobbieBnt
Copy link

@jerpelea, sorry for bothering you, but could you please tell us the status of this problem? I (and, I guess, not only me) would appreciate your immediate attention to this problem, as it's one of the most important things related to the most important part in every custom ROM - user experience:)

@jerpelea
Copy link
Collaborator

jerpelea commented Jan 9, 2019

@RobbieBnt
I checked the kugo device with both android 8.1 and 9.0
The device has the same limited camera for the main camera and we are looking for the cause of this issue

NOTE:
all other platforms have a full camera for the main camera

@jerpelea
Copy link
Collaborator

initStaticMetadata: 8785: SRA LIMITED CHECK: no per-frame 0, yuv 0, mono 0, supportBurst 0
supportBurst 0 has to be supported for FULL API2

@jerpelea
Copy link
Collaborator

workaround found
it will be released in v5

@Gray47Maxx
Copy link
Contributor Author

Gray47Maxx commented Jan 12, 2019

@jerpelea and btw a quick question - can we improve the ISO on platforms? Sometimes 1200 is kinda not enough...

@RobbieBnt told me something about noises and stuff, but I guess it can be fixed by proper software. Or we need something like camera mitigation for it as well?

@jerpelea
Copy link
Collaborator

jerpelea commented Jan 12, 2019

@Gray47Maxx this is all that can be done witout increasing the noise

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

4 participants