On Sun, Dec 03, 2017 at 05:32:21PM -0800, Aman Gupta wrote: > From: Aman Gupta <a...@tmm1.net> > > --- > libavutil/Makefile | 1 + > libavutil/hwcontext.c | 4 ++++ > libavutil/hwcontext.h | 1 + > libavutil/hwcontext_mediacodec.c | 50 > ++++++++++++++++++++++++++++++++++++++++ > libavutil/hwcontext_mediacodec.h | 36 +++++++++++++++++++++++++++++ > 5 files changed, 92 insertions(+) > create mode 100644 libavutil/hwcontext_mediacodec.c > create mode 100644 libavutil/hwcontext_mediacodec.h > > diff --git a/libavutil/Makefile b/libavutil/Makefile > index 721784086c..0b73a14f0b 100644 > --- a/libavutil/Makefile > +++ b/libavutil/Makefile > @@ -166,6 +166,7 @@ OBJS-$(CONFIG_OPENCL) += > hwcontext_opencl.o > OBJS-$(CONFIG_VAAPI) += hwcontext_vaapi.o > OBJS-$(CONFIG_VIDEOTOOLBOX) += hwcontext_videotoolbox.o > OBJS-$(CONFIG_VDPAU) += hwcontext_vdpau.o > +OBJS-$(CONFIG_MEDIACODEC) += hwcontext_mediacodec.o > > OBJS += $(COMPAT_OBJS:%=../compat/%) > > diff --git a/libavutil/hwcontext.c b/libavutil/hwcontext.c > index f47158f811..31ac12807b 100644 > --- a/libavutil/hwcontext.c > +++ b/libavutil/hwcontext.c > @@ -56,6 +56,9 @@ static const HWContextType * const hw_table[] = { > #if CONFIG_VIDEOTOOLBOX > &ff_hwcontext_type_videotoolbox, > #endif > +#if CONFIG_MEDIACODEC > + &ff_hwcontext_type_mediacodec, > +#endif > NULL, > }; > > @@ -69,6 +72,7 @@ static const char *const hw_type_names[] = { > [AV_HWDEVICE_TYPE_VAAPI] = "vaapi", > [AV_HWDEVICE_TYPE_VDPAU] = "vdpau", > [AV_HWDEVICE_TYPE_VIDEOTOOLBOX] = "videotoolbox", > + [AV_HWDEVICE_TYPE_MEDIACODEC] = "mediacodec", > }; > > enum AVHWDeviceType av_hwdevice_find_type_by_name(const char *name) > diff --git a/libavutil/hwcontext.h b/libavutil/hwcontext.h > index 8d27b987df..f5a4b62387 100644 > --- a/libavutil/hwcontext.h > +++ b/libavutil/hwcontext.h > @@ -35,6 +35,7 @@ enum AVHWDeviceType { > AV_HWDEVICE_TYPE_D3D11VA, > AV_HWDEVICE_TYPE_DRM, > AV_HWDEVICE_TYPE_OPENCL, > + AV_HWDEVICE_TYPE_MEDIACODEC, > }; > > typedef struct AVHWDeviceInternal AVHWDeviceInternal; > diff --git a/libavutil/hwcontext_mediacodec.c > b/libavutil/hwcontext_mediacodec.c > new file mode 100644 > index 0000000000..b0d8993e15 > --- /dev/null > +++ b/libavutil/hwcontext_mediacodec.c > @@ -0,0 +1,50 @@ > +/* > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * FFmpeg is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with FFmpeg; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 > USA > + */ > + > +#include "config.h" > + > +#include "buffer.h" > +#include "common.h" > +#include "hwcontext.h" > +#include "hwcontext_internal.h" > +#include "hwcontext_mediacodec.h" > + > +static int mc_device_create(AVHWDeviceContext *ctx, const char *device, > + AVDictionary *opts, int flags) > +{ > + if (device && device[0]) { > + av_log(ctx, AV_LOG_ERROR, "Device selection unsupported.\n"); > + return AVERROR_UNKNOWN; > + } > + > + return 0; > +} > + > +const HWContextType ff_hwcontext_type_mediacodec = { > + .type = AV_HWDEVICE_TYPE_MEDIACODEC, > + .name = "mediacodec", > + > + .device_hwctx_size = sizeof(AVMediaCodecDeviceContext), > + > + .device_create = mc_device_create, > + > + .pix_fmts = (const enum AVPixelFormat[]){ > + AV_PIX_FMT_MEDIACODEC, > + AV_PIX_FMT_NONE > + }, > +}; > diff --git a/libavutil/hwcontext_mediacodec.h > b/libavutil/hwcontext_mediacodec.h > new file mode 100644 > index 0000000000..101a9806d5 > --- /dev/null > +++ b/libavutil/hwcontext_mediacodec.h > @@ -0,0 +1,36 @@ > +/* > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or > + * modify it under the terms of the GNU Lesser General Public > + * License as published by the Free Software Foundation; either > + * version 2.1 of the License, or (at your option) any later version. > + * > + * FFmpeg is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with FFmpeg; if not, write to the Free Software > + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 > USA > + */ > + > +#ifndef AVUTIL_HWCONTEXT_MEDIACODEC_H > +#define AVUTIL_HWCONTEXT_MEDIACODEC_H > + > +/** > + * MediaCodec details. > + * > + * Allocated as AVHWDeviceContext.hwctx > + */ > +typedef struct AVMediaCodecDeviceContext { > + /** > + * android/view/Surface handle, to be filled by the user. > + * > + * This is the default surface used by decoders on this device. > + */ > + void *surface; > +} AVMediaCodecDeviceContext; > + > +#endif /* AVUTIL_HWCONTEXT_MEDIACODEC_H */
Hi, Patch might be missing the following chunk (in order to build): +++ b/libavutil/hwcontext_internal.h @@ -166,5 +166,6 @@ extern const HWContextType ff_hwcontext_type_qsv; extern const HWContextType ff_hwcontext_type_vaapi; extern const HWContextType ff_hwcontext_type_vdpau; extern const HWContextType ff_hwcontext_type_videotoolbox; +extern const HWContextType ff_hwcontext_type_mediacodec; Best regards, -- Matthieu B. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel