On Thu, 2016-11-10 at 15:08 -0800, Daniel Charles wrote: > This is a re-send of patch 1/2 and a new patch that modifies the current > unittests to properly test the unsupported profile condition. > > As mentioned in the commit messages, UNSUPPORTED_PROFILE will be returned > when a queried profile is not supported by any entrypoint. > > If squashing the patches is preferred, let me know and I'll resubmit. >
lgtm, added Artie's review tag, applied. Thanks, Sean > v2: do not change the checks for specific h/w, i.e. IS_SKL as it will > change the purpose of the test > > v3: fix the config tests for all components available so far using the same > purpose as the original test. > > > Daniel Charles (2): > i965_validate_config: return unsupported profile > i965_test_config: return properly unsupported profile > > src/i965_drv_video.c | 49 +++++++++++++++++++++----- > test/i965_avcd_config_test.cpp | 71 +++++++++++++++++++++++++++++-------- > test/i965_avce_config_test.cpp | 78 > ++++++++++++++++++++++++++++++++++------- > test/i965_jpegd_config_test.cpp | 29 +++++++++++++-- > test/i965_jpege_config_test.cpp | 29 ++++++++++++--- > 5 files changed, 214 insertions(+), 42 deletions(-) > >
_______________________________________________ Libva mailing list Libva@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libva
