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

update for fix compile warning -Wformat #10

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

jennycao
Copy link
Contributor

@jennycao jennycao commented Apr 8, 2018

update for fix compile warning : format specifies type 'unsigned long' but the argument has type '__u64' (aka 'unsigned long long') [-Wformat]

Jira: GSE-1411
Tests: compilation with warning clean on android

Kishore409 and others added 8 commits March 28, 2018 19:35
Export necessary header files used by other components for Android,
such as libva intel-driver, gralloc, hwcomposer, etc.

Signed-off-by: li zhuangzhi <[email protected]>
Those formats are used for HEVC10bit decoding
Android need gralloc these formats

Signed-off-by: Lin Johnson <[email protected]>
JIRA : None
Test : Travis CI checks getting triggered
Jira: None.
Test: Build passes on Android and no warnings reported.
Signed-off-by: renchenglei <[email protected]>
    update for fix compile warning :  format specifies type 'unsigned long' but the argument has type '__u64' (aka 'unsigned long long') [-Wformat]

    Jira: GSE-1411
    Tests: compilation with warning clean
@kalyankondapally
Copy link
Contributor

@jennycao Update commit message and send it to libdrm mailing list for review ?

@jennycao
Copy link
Contributor Author

@kalyankondapally https://patchwork.freedesktop.org/patch/223574/ is send for dri-devel mailing list for review.

@js0701
Copy link
Contributor

js0701 commented Jun 14, 2018

@Kishore409 Why push other commits into the same review? I think it cause confusion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale label for inactive open prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants