[FFmpeg-devel] Intel QSV MPEG2 video decoder

Ivan Uskov ivan.uskov at nablet.com
Mon Jun 29 23:45:35 CEST 2015


Hello Hendrik,

Thank you very much for the review, we will try to split one big invalid
patch to sequence of small valid changes.

Tuesday, June 30, 2015, 12:03:10 AM, you wrote:

HL> On Mon, Jun 29, 2015 at 10:57 PM, Ivan Uskov <ivan.uskov at nablet.com> wrote:
>> Hello All,
>>
>> I'm software developer of Nablet GmbH. We were requested by Intel to
>> integrate all possible QSV-accelerated codecs and VPP modules into ffmpeg.
>> This is my first commit which adding the QSV MPEG2 video decoder codec, I hope I did all or at least all most right.
>> The 0001-Patch-for-QSV-components-by-Intel-applied.patch was received
>> from Intel, so unfortunately it is not possible to split it to
>> separate small patches.
>>

HL> The first patch is full of errors. It reverts changes from ffmpeg
HL> mainline, it contains a dupe file (configure.orig), it contains
HL> non-GPL compatible license headers.
HL> Its not possible to apply it in any form unless it is severly cleaned
HL> up and split - no matter if its from Intel or anyone, clean patches
HL> are mandatory.

HL> - Hendrik
HL> _______________________________________________
HL> ffmpeg-devel mailing list
HL> ffmpeg-devel at ffmpeg.org
HL> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel



-- 
Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com



More information about the ffmpeg-devel mailing list