On Tue, Jun 30, 2015 at 06:28:08PM +0300, Ivan Uskov wrote: > Hello Michael, > > I'm sorry, what exactly documentation should be added regarding this > patch? As I can see there is no any standalone topic about QSV into > current /doc. Can modifications of Changelog be enough?
your patch adds comments like "translate MSDK error codes to ffmpeg error codes"
these should be in a seperate patch as they are unrelated
to " enable QSV acceleration under linux platform"
sorry if i wasnt clear
>
> Tuesday, June 30, 2015, 5:42:56 PM, you wrote:
>
> MN> On Tue, Jun 30, 2015 at 05:24:54PM +0300, Ivan Uskov wrote:
> >> +
> >> +/**
> >> + * @brief translate MSDK error codes to ffmpeg error codes
> >> + */
> >> int ff_qsv_error(int mfx_err)
> >> {
> >> switch (mfx_err) {
> >> @@ -77,6 +89,22 @@ int ff_qsv_error(int mfx_err)
> >> }
>
> MN> The documentation addition should be in a seperate patch
>
>
> >> }
> >>
>
> --
> Best regards,
> Ivan mailto:ivan.us...@nablet.com
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
