-
Notifications
You must be signed in to change notification settings - Fork 12
Hdcpd 15 #30
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
Hdcpd 15 #30
Conversation
|
Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on |
|
Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on |
1 similar comment
|
Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on |
|
Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on |
|
Program name for this pr is not compatable with other dependent prs, for more details please check tracked_on |
1. Enable daemon for hdcp1.4/hdcp2.2 2. Add hdcp Test Utility Tracked-On: OAM-130842 Signed-off-by: Jia, Lin A <lin.a.jia@intel.com>
Change to Android.bp for hdcp build Tracked-On: OAM-130842 Signed-off-by: Jia, Lin A <lin.a.jia@intel.com>
Open virtio_gpu drm device for acrn support Tracked-On: OAM-130842 Signed-off-by: Jia, Lin A <lin.a.jia@intel.com>
1. Remove hdcpd srm function due to avoid hdcpd build fail 2. Delete unused build code Tracked-On: OAM-130842 Signed-off-by: Jia, Lin A <lin.a.jia@intel.com>
daemon/Android.mk
Outdated
|
|
||
| LOCAL_CPPFLAGS := \ | ||
| -DANDROID \ | ||
| -DUSES_IA_HWCOMPOSER \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why still need use USES_IA_HWCOMPOSER?
daemon/portmanager_android.cpp
Outdated
| { | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable"); | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable, value =%d", value); | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable, propValue =%d", propValue); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can combine together as one message
daemon/portmanager_android.cpp
Outdated
| //This is only for HDCP2.2 | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable Type 1"); | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable Type, value =%d", value); | ||
| HDCP_ASSERTMESSAGE("Attempting HDCP Enable Type, propValue =%d", propValue); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can combine together as one message
test/CMakeLists.txt
Outdated
| # OTHER DEALINGS IN THE SOFTWARE. | ||
|
|
||
|
|
||
| add_subdirectory(hdcpTestUtility) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why need this?
test/hdcpTestUtility/CMakeLists.txt
Outdated
|
|
||
| target_compile_options(${PROJECT_NAME} PRIVATE ${HDCP_CXX_FLAGS}) | ||
|
|
||
| target_link_libraries(${PROJECT_NAME} hdcpsdk) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it it necessary?
| { | ||
| if (nullptr == ctx) | ||
| { | ||
| printf("hdcp context pointer is null\n"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If need support both linux and Andorid, we need wrap it, call printf for linux, call LOGD for Android
1. Remove unsed source code 2. Rename USES_IA_HWCOMPOSER define Tracked-On: OAM-130842 Signed-off-by: Jia, Lin A <lin.a.jia@intel.com>
|
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
feijiang1
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
|
FAILURE: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details. |
|
SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has completed MERGE Build for this pr, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has completed MERGE Build for this pr, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon-merge/1087 |
No description provided.