Hi, On Mon, Jul 27, 2015 at 7:59 AM, <shivraj.pa...@imgtec.com> wrote:
> From: Shivraj Patil <shivraj.pa...@imgtec.com> > > Signed-off-by: Shivraj Patil <shivraj.pa...@imgtec.com> > --- > libavcodec/mips/vp9_mc_msa.c | 2123 > ++++++++++++++++++++++++++++++++++++ > libavcodec/mips/vp9dsp_init_mips.c | 2 + > libavcodec/mips/vp9dsp_mips.h | 32 + > 3 files changed, 2157 insertions(+) > [..] > +void ff_avg_bilin_4h_msa(uint8_t *dst, ptrdiff_t dst_stride, > + const uint8_t *src, ptrdiff_t src_stride, > + int height, int mx, int my) > +{ > + const int8_t *filter = vp9_bilinear_filters_msa[mx - 1]; > + > + if (4 == height) { > + common_hz_2t_and_aver_dst_4x4_msa(src, src_stride, dst, > dst_stride, > + filter); > + } else if (8 == height) { > + common_hz_2t_and_aver_dst_4x8_msa(src, src_stride, dst, > dst_stride, > + filter); > + } > +} You're using this construct in various places, how much does it help? (Otherwise no comments, basically lgtm % the above.) Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel