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

Remove useless frame dimension check for VP9 #214

Merged
merged 3 commits into from Jul 13, 2017
Merged

Remove useless frame dimension check for VP9 #214

merged 3 commits into from Jul 13, 2017

Conversation

@fhvwy
Copy link
Contributor

@fhvwy fhvwy commented Jun 28, 2017

Fixes #213 - encoding with arbitrary sizes works on Kaby Lake after this change.

Signed-off-by: Mark Thompson <sw@jkqxz.net>
@pengche1 pengche1 requested a review from YuJiankang Jun 29, 2017
@xhaihao xhaihao self-requested a review Jun 30, 2017
Copy link
Contributor

@xhaihao xhaihao left a comment

LGTM, however I didn't test it.

@QuPengfei
Copy link
Contributor

@QuPengfei QuPengfei commented Jul 4, 2017

@YuJiankang does the media kernel have the limitation about widh/height? would you help double confirm?

@xhaihao xhaihao merged commit ae8831f into intel:master Jul 13, 2017
7 checks passed
7 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
intel/ci/fedora/24/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
intel/ci/fedora/25/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
intel/ci/ubuntu/trusty/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
intel/ci/ubuntu/xenial/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
intel/ci/ubuntu/yakkety/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
intel/ci/ubuntu/zesty/pr Build intel-vaapi-driver-3da9cb1b8ad7 finished with status SUCCESSFUL
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants