josteink / device_asus_tf101

cyanogenmod compatible device tree for asus tf101
12 stars 6 forks source link

Switching to front facing camera causes camera-app to crash. #13

Closed josteink closed 11 years ago

josteink commented 11 years ago

Determine if source is device tree code or gapps related.

davebuk commented 11 years ago

Confirmed as not working, and am now unable to return to rear camera :-) (20130224 nightly.)

Tried WIP 0.8.0 Gapps and Timduru Gapps with the same result. 'Camera ICS' app from the app store showed rear camera, went into front and crashed. After a re-install of 'Camera ICS' I have the rear camera back, but nothing from stock camera, just crashes.

Clearing cache/data for 'Gallery' brings stock camera back to rear facing working again.

davebuk commented 11 years ago

Both cameras work in video mode in Skype.

josteink commented 11 years ago

That's definitely interesting. So probably a gapps issue then.

Thanks for reporting.

davebuk commented 11 years ago

I don't know if there is any code difference between still and video mode, but if not, I also guess its a gapps issue?

sidelong commented 11 years ago

Still an issue with 20130306

josteink commented 11 years ago

@sidelong Thanks for the report. Unfortunately that is unsurprising, since no work has been done on this, since I haven't had much time to look into bugs in general.

I might just try to make a gapps without the new gallery (since Photosphere doesnt work either) and see if it works with stock camera.

sidelong commented 11 years ago

So the new gallery works on other devices running cm10.1, but not on the TF101?

From: Jostein Kjønigsen [mailto:notifications@github.com] Sent: 07 March 2013 11:41 To: josteink/device_asus_tf101 Cc: sidelong Subject: Re: [device_asus_tf101] Switching to front facing camera causes camera-app to crash. (#13)

@sidelong https://github.com/sidelong Thanks for the report. Unfortunately that is unsurprising, since no work has been done on this, since I haven't had much time to look into bugs in general.

I might just try to make a gapps without the new gallery (since Photosphere doesnt work either) and see if it works with stock camera.

— Reply to this email directly or view it on GitHub https://github.com/josteink/device_asus_tf101/issues/13#issuecomment-14555341 . https://github.com/notifications/beacon/HF06bm-8Hs86FXTP6doWmQIcRMgyRZmpmNoHmFmAqMKU0FmW0H-AbwREePwlnUrh.gif

josteink commented 11 years ago

I'm honestly not sure where the problem lies, since, like I said, we haven't had time or resources to look into this.

That said: I don't think there's anything fundamental about the TF101 which makes it impossible for this to work. It's just that in its current incarnation, and at least on this ROM, it doesn't right now.

Until we have time to look into it, find out what goes wrong, and if that is something under our control, and then fix that, it wont work.

sidelong commented 11 years ago

I really wish I could help, but I don’t have the skills. I can only tell you whether it works or not and if it goes wrong, how it goes wrong. I do respect and appreciate all the effort that you have and are putting the TF101 fw. It enables me to satisfy my craving to have the latest and best fw on mine. I also have 4.2.2 running on my HTC HD2, and custom fw on my SGS3 and my wife’s SGS2. So I suppose that makes me an unmitigated flashaholic. Anyway, thanks for taking the trouble to respond to my concerns,

From: Jostein Kjønigsen [mailto:notifications@github.com] Sent: 07 March 2013 13:58 To: josteink/device_asus_tf101 Cc: sidelong Subject: Re: [device_asus_tf101] Switching to front facing camera causes camera-app to crash. (#13)

I'm honestly not sure where the problem lies, since we haven't had time or resources to look into this.

That said: I don't think there's anything fundamental about the TF101 which makes it impossible for this to work. It's just that in its current incarnation, and at least on this ROM, it doesn't work.

Until we have time to look into it, find out what goes wrong, and if that is something under our control, and then fix that, it wont work.

— Reply to this email directly or view it on GitHub https://github.com/josteink/device_asus_tf101/issues/13#issuecomment-14561676 . https://github.com/notifications/beacon/HF06bm-8Hs86FXTP6doWmQIcRMgyRZmpmNoHmFmAqMKU0FmW0H-AbwREePwlnUrh.gif

davebuk commented 11 years ago

It still could be just a Gapps issue as there seems to be a limited amount of versions that work with 4.2.2. Seeing as its the 'Galley' that crashes, and its the same error as trying to use photoshpere, it maybe that there is a clash between the CM10.1 and Gapps versions. I guess its the same reason that the full function of the 4.2.2 on screen keyboard either crashes with correction and no guesture typing or guesture not working with correction on?

Am I correct in saying that the EOS ROMs are AOSP not CM based?

I see that 4.2.2 has gone AOSP: (http://www.androidcentral.com/android-422-code-now-pushing-android-open-source-project) and that some binaries are available. These maybe what are missing and causing some issues with hardware on non nexus devcies?

Or am I talking rubbish Jostein? :-)

josteink commented 11 years ago

Those binaries are probably very device-specific. And not for non-Nexus devices like the TF101.

On a lighter note, I've done some light research. Flashing Banks gapps (which does not include Google Camera w/Photosphere and keeps AOSP Camera/Gallery) the front-facing camera works.

For now I'll cook up a new Gapps-package for this ROM, without Photosphere since it doesn't work anyway, and then at least this issue should be fixed.

dlumberg commented 11 years ago

Photosphere is only technically supported on nexus devices. People have modded it to work on others.

josteink commented 11 years ago

Yeah. I don't think I have the skills or time for that, so I'm just going to "fix" both these issues by dropping Photosphere.

If people complain, they can try fixing it themsleves and send me a pull-request :)

josteink commented 11 years ago

Fixed. New gapps uploaded and linked in thread.