On 9/3/2017 12:05 PM, Paras Chadha wrote:
> Signed-off-by: Paras Chadha <paraschadh...@gmail.com>
> ---
> This fixes the failed fate tests on ppc64be:
> http://fate.ffmpeg.org/report.cgi?time=20170903050743&slot=ppc64be-RHEL7.0-gcc-4.8.5-ibmcrl
>
> libavcodec/fitsdec.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/fitsdec.c b/libavcodec/fitsdec.c
> index ec15248..97b3d13 100644
> --- a/libavcodec/fitsdec.c
> +++ b/libavcodec/fitsdec.c
> @@ -205,9 +205,9 @@ static int fits_decode_frame(AVCodecContext *avctx, void
> *data, int *got_frame,
> }
> } else if (header.bitpix == 16) {
> if (header.naxisn[2] == 3) {
> - avctx->pix_fmt = AV_PIX_FMT_GBRP16;
> + avctx->pix_fmt = AV_PIX_FMT_GBRP16LE;
Just wondering, the encoder and muxer use the big endian pixel formats
variants, and you're using AV_RB* helper macros everywhere. It seems
unusual then having the decoder output frames in little endian formats.
Wouldn't it make more sense to set this to BE and update the fate tests
accordingly instead?
> } else {
> - avctx->pix_fmt = AV_PIX_FMT_GBRAP16;
> + avctx->pix_fmt = AV_PIX_FMT_GBRAP16LE;
> }
> } else {
> av_log(avctx, AV_LOG_ERROR, "unsupported BITPIX = %d\n",
> header.bitpix);
> @@ -217,7 +217,7 @@ static int fits_decode_frame(AVCodecContext *avctx, void
> *data, int *got_frame,
> if (header.bitpix == 8) {
> avctx->pix_fmt = AV_PIX_FMT_GRAY8;
> } else {
> - avctx->pix_fmt = AV_PIX_FMT_GRAY16;
> + avctx->pix_fmt = AV_PIX_FMT_GRAY16LE;
> }
> }
>
> --
> 2.4.11
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
