From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id BED354061F for ; Sat, 26 Feb 2022 07:18:19 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 67ED368A8D9; Sat, 26 Feb 2022 09:18:16 +0200 (EET) Received: from mail-pg1-f175.google.com (mail-pg1-f175.google.com [209.85.215.175]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 4FBCF68996A for ; Sat, 26 Feb 2022 09:18:10 +0200 (EET) Received: by mail-pg1-f175.google.com with SMTP id t14so3756382pgr.3 for ; Fri, 25 Feb 2022 23:18:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=date:from:to:subject:message-id:mail-followup-to:references :mime-version:content-disposition:in-reply-to; bh=ES5m6I4J4xzQ1O6sM9xTO/QCl7VBXW//E+HhQWJBm8E=; b=QsUGCmX+w7zFPo8O5k89t3tlLp6bZOZOIJquljwsLKjvJrLzVJ5NRqC+Imlw8Wofr+ Nl/tuEkClMCeWK/p6d6d64QiQyxMZfAqNw241+AeP2ot/+OmC3AAz6IC3UzbCBtujYqS IpDp5t+sbR5c/xpv3F0LCr2gUchPFsMiKhHknmXOb+h2DITB5c5eGxxiqmc5ZoK6jALJ vQTGlWW81AC07UcUF+t9I3GxQqnD0K6SRgK6oR4EyNI3tz+LMnAXM2LsCpRXFiGgeA1j JfY2YIXfeyhsk89/ykT7Qwg/T2gsNxFIRSINW/SqawJeKu7fE2ZK2bk31CtOdXyG1ovt DF1w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:subject:message-id:mail-followup-to :references:mime-version:content-disposition:in-reply-to; bh=ES5m6I4J4xzQ1O6sM9xTO/QCl7VBXW//E+HhQWJBm8E=; b=zzuuq3t958L3JhaeAM06ns6nqXafeLL4+r7osYZjwXlhgpXruwbzrIdeq3/7gRL92U BhZ98j4QAx4/0O4uc1Syg2zlA95lsVjOwgy6mi/oP9nyiSImwnyZAmDAMuMKhTH1pSoe 5Y6hkrz7IE2OdmrnfxLzX5Vpdw/ZOYrAHUjictUtq7imboI8ozp+xqB06vIGCycsbRYr R0bsbua7Uoq9AFCvBqNtU6bc0b3dl/9bQBjrwjwF0HT6Qj4uw4/glxUI8vhqwQ5AjC7S VTAzkzIXgobmAKyHi7ynjvIq6j19+RBaioZApZqy+A6MDYAiRdP1bGCBm5aLtcqEtJ1t ydyg== X-Gm-Message-State: AOAM533SNn1sE0adHlb1CKAwN69JqLKcAeo7Zmt6HRmutjtn89NJxlKY h7SBkW2fXwhp9AQj7lo+ZAhnMUmYMPo= X-Google-Smtp-Source: ABdhPJy/d6zz0Bq2lVU3EqOSxIDzh3zSHRWiDeHRt0/jAPc6QoQdCUNJzx7y2BWgjxLiX3SUzirLkg== X-Received: by 2002:a05:6a00:1aca:b0:4e1:a2b6:5b9 with SMTP id f10-20020a056a001aca00b004e1a2b605b9mr11447409pfv.4.1645859888111; Fri, 25 Feb 2022 23:18:08 -0800 (PST) Received: from gmail.com ([161.117.202.209]) by smtp.gmail.com with ESMTPSA id mq5-20020a17090b380500b001bc770d1df4sm4407234pjb.55.2022.02.25.23.18.07 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 25 Feb 2022 23:18:07 -0800 (PST) Date: Sat, 26 Feb 2022 15:18:04 +0800 From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Message-ID: <20220226071804.GA16790@gmail.com> Mail-Followup-To: ffmpeg-devel@ffmpeg.org References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: Subject: Re: [FFmpeg-devel] [PATCH] AVS3: add support for AVS3 High profile - same syntax as AVS3 Main profile X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Sat, Feb 26, 2022 at 05:50:39AM +0000, Paul Higgs wrote: > This patch adds high level syntax support for parsing AVS3 High profile bitstreams. > Latest AVS3 specification including High profile is available at http://www.avs.org.cn/AVS3_download/en_index.asp > > Signed-off-by: Paul Higgs > --- > libavcodec/avs3.h | 2 ++ > libavcodec/avs3_parser.c | 8 ++++---- > libavformat/avs3dec.c | 2 +- > 3 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/avs3.h b/libavcodec/avs3.h > index 4189d9b583..c8caa58b0a 100644 > --- a/libavcodec/avs3.h > +++ b/libavcodec/avs3.h > @@ -35,6 +35,8 @@ > #define AVS3_FIRST_SLICE_START_CODE 0x00 > #define AVS3_PROFILE_BASELINE_MAIN 0x20 > #define AVS3_PROFILE_BASELINE_MAIN10 0x22 > +#define AVS3_PROFILE_BASELINE_HIGH 0x30 > +#define AVS3_PROFILE_BASELINE_HIGH10 0x32 > > #define AVS3_ISPIC(x) ((x) == AVS3_INTRA_PIC_START_CODE || (x) == AVS3_INTER_PIC_START_CODE) > #define AVS3_ISUNIT(x) ((x) == AVS3_SEQ_START_CODE || AVS3_ISPIC(x)) > diff --git a/libavcodec/avs3_parser.c b/libavcodec/avs3_parser.c > index d04d96a03a..483b38fe76 100644 > --- a/libavcodec/avs3_parser.c > +++ b/libavcodec/avs3_parser.c > @@ -91,11 +91,11 @@ static void parse_avs3_nal_units(AVCodecParserContext *s, const uint8_t *buf, > // sampe_precision(3) > skip_bits(&gb, 47); > > - if (profile == AVS3_PROFILE_BASELINE_MAIN10) { > - int sample_precision = get_bits(&gb, 3); > - if (sample_precision == 1) { > + if (profile == AVS3_PROFILE_BASELINE_MAIN10 || profile == AVS3_PROFILE_BASELINE_HIGH10) { > + int encoding_precision = get_bits(&gb, 3); > + if (encoding_precision == 1) { Please keep sample_precision name, it's unrelated cosmetic change in this patch. > avctx->pix_fmt = AV_PIX_FMT_YUV420P; > - } else if (sample_precision == 2) { > + } else if (encoding_precision == 2) { > avctx->pix_fmt = AV_PIX_FMT_YUV420P10LE; > } else { > avctx->pix_fmt = AV_PIX_FMT_NONE; > diff --git a/libavformat/avs3dec.c b/libavformat/avs3dec.c > index 2395df171b..335b5409f5 100644 > --- a/libavformat/avs3dec.c > +++ b/libavformat/avs3dec.c > @@ -47,7 +47,7 @@ static int avs3video_probe(const AVProbeData *p) > } > if (state == AVS3_SEQ_START_CODE) { > seq++; > - if (*ptr != AVS3_PROFILE_BASELINE_MAIN && *ptr != AVS3_PROFILE_BASELINE_MAIN10) > + if (*ptr != AVS3_PROFILE_BASELINE_MAIN && *ptr != AVS3_PROFILE_BASELINE_MAIN10 && *ptr != AVS3_PROFILE_BASELINE_HIGH && *ptr != AVS3_PROFILE_BASELINE_HIGH10) The line is too long. > return 0; > } else if (AVS3_ISPIC(state)) { > pic++; > -- > 2.30.0.windows.2 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". -- Thanks, Limin Wang _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".