Re: [PATCH] venus: vdec: fix format enumeration

2018-03-19 Thread Alexandre Courbot
On Thu, Mar 15, 2018 at 6:52 PM, Stanimir Varbanov wrote: > Hi Alex, > > Thanks for the patch! > > On 13.03.2018 11:11, Alexandre Courbot wrote: >> find_format_by_index() stops enumerating formats as soon as the index >> matches, and returns NULL if

Re: [PATCH] venus: vdec: fix format enumeration

2018-03-19 Thread Alexandre Courbot
On Thu, Mar 15, 2018 at 6:52 PM, Stanimir Varbanov wrote: > Hi Alex, > > Thanks for the patch! > > On 13.03.2018 11:11, Alexandre Courbot wrote: >> find_format_by_index() stops enumerating formats as soon as the index >> matches, and returns NULL if venus_helper_check_codec() finds out that >>

Re: [PATCH] venus: vdec: fix format enumeration

2018-03-15 Thread Stanimir Varbanov
Hi Alex, Thanks for the patch! On 13.03.2018 11:11, Alexandre Courbot wrote: > find_format_by_index() stops enumerating formats as soon as the index > matches, and returns NULL if venus_helper_check_codec() finds out that > the format is not supported. This prevents formats to be properly >

Re: [PATCH] venus: vdec: fix format enumeration

2018-03-15 Thread Stanimir Varbanov
Hi Alex, Thanks for the patch! On 13.03.2018 11:11, Alexandre Courbot wrote: > find_format_by_index() stops enumerating formats as soon as the index > matches, and returns NULL if venus_helper_check_codec() finds out that > the format is not supported. This prevents formats to be properly >

[PATCH] venus: vdec: fix format enumeration

2018-03-13 Thread Alexandre Courbot
find_format_by_index() stops enumerating formats as soon as the index matches, and returns NULL if venus_helper_check_codec() finds out that the format is not supported. This prevents formats to be properly enumerated if a non-supported format is present, as the enumeration will end with it. Fix

[PATCH] venus: vdec: fix format enumeration

2018-03-13 Thread Alexandre Courbot
find_format_by_index() stops enumerating formats as soon as the index matches, and returns NULL if venus_helper_check_codec() finds out that the format is not supported. This prevents formats to be properly enumerated if a non-supported format is present, as the enumeration will end with it. Fix