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.

Sign up for GitHub

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

Jump to bottom

build error with SlimRoms/frameworks_av #2

Open
moonlightly opened this issue Apr 15, 2014 · 5 comments
Open

build error with SlimRoms/frameworks_av #2

moonlightly opened this issue Apr 15, 2014 · 5 comments

Comments

@moonlightly
Copy link

target thumb C++: libcameraservice <= frameworks/av/services/camera/libcameraservice/common/CameraDeviceBase.cpp
frameworks/av/media/libmedia/AudioParameter.cpp:39:50: error: 'AUDIO_PARAMETER_KEY_HANDLE_FM' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:40:51: error: 'AUDIO_PARAMETER_KEY_VOIP_CHECK' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:41:53: error: 'AUDIO_PARAMETER_KEY_FLUENCE_TYPE' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:42:45: error: 'AUDIO_PARAMETER_KEY_SSR' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:43:58: error: 'AUDIO_PARAMETER_KEY_HANDLE_A2DP_DEVICE' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:44:52: error: 'AUDIO_PARAMETER_KEY_ADSP_STATUS' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:45:54: error: 'AUDIO_CAN_OPEN_PROXY' was not declared in this scope
frameworks/av/media/libmedia/AudioParameter.cpp:46:50: error: 'AUDIO_PARAMETER_KEY_FM_VOLUME' was not declared in this scope
make: *** [/home/mc/slim/out/target/product/xt925/obj/STATIC_LIBRARIES/libmedia_helper_intermediates/AudioParameter.o] error 1
@moonlightly moonlightly changed the title build build error with this code Apr 15, 2014
@moonlightly moonlightly changed the title build error with this code build error with SlimRoms/frameworks_av Apr 15, 2014
@moonlightly
Copy link
Author

when build xt926 & htc one & d2vzw and etc ROMS always tell these errors

@REV3NT3CH
Copy link

was there ever a fix in build for this

@bobobo1618
Copy link

Getting the same on Peregrine (Moto G LTE). Any fix would be wonderful.

@bobobo1618
Copy link

The problem is in here: https://github.com/SlimRoms/hardware_libhardware/blob/kk4.4-caf/include/hardware/audio.h#L160

In the default (kk4.4) branch, those lines are missing. Switching to kk4.4-caf should fix it. Switching now.

@bobobo1618
Copy link

After that, there's a new issue though.

CyanogenMod/android_external_stagefright-plugins@0d37b03 isn't compatible with this version of frameworks_av, which lacks this line https://github.com/CyanogenMod/android_frameworks_av/blob/cm-11.0/include/media/stagefright/MetaData.h#L190

Further, the default CyanogenMod remote in the default Slim manifest doesn't specify a fixed revision and assumes the cm-11.0 branch will be compatible. All up to date builds of Slim will be failing right now I'm fairly sure. You need to update the AV stuff as CM seems to be doing a lot of work there, particularly regarding audio.

fusionjack pushed a commit to SlimSaber/frameworks_av that referenced this issue Sep 26, 2014
@fusionjack
Fix inline assembly for clang
521e81c 
add r8, r0, asl SlimRoms#2
and
add r8, r8, r0, asl SlimRoms#2

are identical in gcc inline assembly, but clang accepts only the latter
variant.

Change-Id: I922367984b6ae2f7268ce99facaf8da425c39d7e
Signed-off-by: Bernhard Rosenkränzer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bobobo1618 @moonlightly @REV3NT3CH