1163557 - [aries-l] camera does not work

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: GonkIntegration, defect, P1)

Comment 1

[:gerard-majax]

This will work once Alin completes the needed kernel work :)

Comment 2

[Peter Bylenga [:PBylenga]]

[Blocking Requested - why for this release]:
This is needed for lollipop support of Aries device and fails smoke/sanity tests.

Comment 3

[Bobby Chien]

hayden, could you help on this?

Comment 4

[:gerard-majax]

Bobby, do you expect hayden to do Sony's work? Support on kernel side is not yet finished.

Comment 5

[Hayden Huang [:haydenh]]

Alexandre is right, Sony hasn't release necessary libraries and kernel support for camera to work on Aries-l.

Comment 6

[Bobby Chien]

Thanks for update, Hayden and Alex. Let's wait for Alin's response.

Comment 7

[:gerard-majax]

FYI Alin is back from holidays and Camera is his top priority :)

Comment 8

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Hey Hema?  Can you assign this to a dev to follow up with Alin please?

Comment 9

[Hema Koka [:hema]]

Alin is currently working on this with top priority.

Comment 10

[Hema Koka [:hema]]

Alin, any update on this?

Comment 11

[:gerard-majax]

(In reply to Hema Koka [:hema] from comment #10)
> Alin, any update on this?

There is ongoing work which is progressing:
 - https://github.com/sonyxperiadev/kernel/commits/aosp/AU_LINUX_ANDROID_LA.BF64.1.2.1_RB1.05.00.02.019.067
 - https://github.com/sonyxperiadev/device-sony-shinano/commits/l-mr1

Comment 12

[Ken Chang[:kenkai]]

2.5 won't use L.

Comment 13

[Al Tsai [:atsai]]

remove from smoketest blockers due to not support with L

Comment 14

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I believe we're switching over to L as soon as possible because our vendor doesn't support kk, they support L?  We need to do some L testing to ensure that we don't have L bugs.

Comment 15

[:gerard-majax]

We are getting closer, Alin merged a commit that adds Camera HAL on Shinano and Yukon platforms:
 - https://github.com/sonyxperiadev/device-sony-shinano/commit/71d457e7e142004a4843f2501e0e0ac1fd35abeb
 - https://github.com/sonyxperiadev/device-sony-yukon/commit/a1d9824812800283f93fe62ac12c5ed8537124e8

Comment 16

[Bobby Chien]

ni ken for Aries-L porting in further release.

Comment 17

[Munro Mengjue Chiang [:mchiang] Be Mozillian since 1/19/2018]

camera related blob has not been released yet.

Comment 18

[Ken Chang[:kenkai]]

Munro is following up this bug.

Comment 19

[:gerard-majax]

Experimental code is public now! http://developer.sonymobile.com/2015/11/18/experimental-aosp-camera-available-for-experienced-developers/

Comment 20

[:gerard-majax]

Added bug 1236535 and bug 1236536 to land what we can already land. This will leave the only moving piece for that to be the kernel, thus lowering the barrier to testers

Comment 21

[:gerard-majax]

Michael, can you have a look at the reviews for bug 1236535 and bug 1236536? Thanks!

Comment 22

[:gerard-majax]

FYI, current builds just need a different kernel. Branch can be found at: https://github.com/lissyx/codeaurora_kernel_msm/commits/sony_v1.1-rb1.27_camera_20160113

Comment 23

[:gerard-majax]

Attachment: PR for gonk-misc master

Comment 24

[:gerard-majax]

Everything should be in tree now to support the experimental camera driver from Sony AOSP. It might be possible there are stil bugs. For example, you might need to disable front camera this way otherwise it might crash mm-qcamera-daemon:

diff --git a/dom/camera/GonkCameraManager.cpp b/dom/camera/GonkCameraManager.cpp
index bd74870..4e75dfc 100644
--- a/dom/camera/GonkCameraManager.cpp
+++ b/dom/camera/GonkCameraManager.cpp
@@ -75,7 +75,7 @@ ICameraControl::GetCameraName(uint32_t aDeviceNum, nsCString& aDeviceName)
 nsresult
 ICameraControl::GetListOfCameras(nsTArray<nsString>& aList)
 {
-  int32_t count = android::Camera::getNumberOfCameras();
+  int32_t count = 1; // android::Camera::getNumberOfCameras();
   DOM_CAMERA_LOGI("getListOfCameras : getNumberOfCameras() returned %d\n", count);
   if (count <= 0) {
     aList.Clear();