From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v3 07/10] media: rkvdec: Add the VP9 backend
Date: Fri, 06 Aug 2021 04:14:57 +0800 [thread overview]
Message-ID: <202108060449.d5kjb5TB-lkp@intel.com> (raw)
In-Reply-To: <20210805144246.11998-8-andrzej.p@collabora.com>
[-- Attachment #1: Type: text/plain, Size: 3992 bytes --]
Hi Andrzej,
I love your patch! Perhaps something to improve:
[auto build test WARNING on bfee75f73c37a2f46a6326eaa06f5db701f76f01]
url: https://github.com/0day-ci/linux/commits/Andrzej-Pietrasiewicz/VP9-codec-V4L2-control-interface/20210805-224717
base: bfee75f73c37a2f46a6326eaa06f5db701f76f01
config: m68k-allmodconfig (attached as .config)
compiler: m68k-linux-gcc (GCC) 10.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/babc0bba621efcae4556da29717e9ae86b7238fa
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Andrzej-Pietrasiewicz/VP9-codec-V4L2-control-interface/20210805-224717
git checkout babc0bba621efcae4556da29717e9ae86b7238fa
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross ARCH=m68k
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/staging/media/rkvdec/rkvdec-vp9.c: In function 'init_intra_only_probs':
>> drivers/staging/media/rkvdec/rkvdec-vp9.c:190:36: warning: variable 'dec_params' set but not used [-Wunused-but-set-variable]
190 | const struct v4l2_ctrl_vp9_frame *dec_params;
| ^~~~~~~~~~
drivers/staging/media/rkvdec/rkvdec-vp9.c: In function 'init_inter_probs':
drivers/staging/media/rkvdec/rkvdec-vp9.c:245:36: warning: variable 'dec_params' set but not used [-Wunused-but-set-variable]
245 | const struct v4l2_ctrl_vp9_frame *dec_params;
| ^~~~~~~~~~
vim +/dec_params +190 drivers/staging/media/rkvdec/rkvdec-vp9.c
186
187 static void init_intra_only_probs(struct rkvdec_ctx *ctx,
188 const struct rkvdec_vp9_run *run)
189 {
> 190 const struct v4l2_ctrl_vp9_frame *dec_params;
191 struct rkvdec_vp9_ctx *vp9_ctx = ctx->priv;
192 struct rkvdec_vp9_priv_tbl *tbl = vp9_ctx->priv_tbl.cpu;
193 struct rkvdec_vp9_intra_only_frame_probs *rkprobs;
194 const struct v4l2_vp9_frame_context *probs;
195 unsigned int i, j, k, m;
196
197 rkprobs = &tbl->probs.intra_only;
198 dec_params = run->decode_params;
199 probs = &vp9_ctx->probability_tables;
200
201 /*
202 * intra only 149 x 128 bits ,aligned to 152 x 128 bits coeff related
203 * prob 64 x 128 bits
204 */
205 for (i = 0; i < ARRAY_SIZE(probs->coef); i++) {
206 for (j = 0; j < ARRAY_SIZE(probs->coef[0]); j++)
207 write_coeff_plane(probs->coef[i][j][0],
208 rkprobs->coef_intra[i][j]);
209 }
210
211 /* intra mode prob 80 x 128 bits */
212 for (i = 0; i < ARRAY_SIZE(v4l2_vp9_kf_y_mode_prob); i++) {
213 unsigned int byte_count = 0;
214 int idx = 0;
215
216 /* vp9_kf_y_mode_prob */
217 for (j = 0; j < ARRAY_SIZE(v4l2_vp9_kf_y_mode_prob[0]); j++) {
218 for (k = 0; k < ARRAY_SIZE(v4l2_vp9_kf_y_mode_prob[0][0]);
219 k++) {
220 u8 val = v4l2_vp9_kf_y_mode_prob[i][j][k];
221
222 rkprobs->intra_mode[i].y_mode[idx++] = val;
223 byte_count++;
224 if (byte_count == 27) {
225 byte_count = 0;
226 idx += 5;
227 }
228 }
229 }
230
231 idx = 0;
232 if (i < 4) {
233 for (m = 0; m < (i < 3 ? 23 : 21); m++) {
234 const u8 *ptr = (const u8 *)v4l2_vp9_kf_uv_mode_prob;
235
236 rkprobs->intra_mode[i].uv_mode[idx++] = ptr[i * 23 + m];
237 }
238 }
239 }
240 }
241
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 60703 bytes --]
next prev parent reply other threads:[~2021-08-05 20:14 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-05 14:42 [PATCH v3 00/10] VP9 codec V4L2 control interface Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 01/10] hantro: postproc: Fix motion vector space size Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 02/10] hantro: postproc: Introduce struct hantro_postproc_ops Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 03/10] hantro: Simplify postprocessor Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 04/10] hantro: Add quirk for NV12/NV12_4L4 capture format Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 19:29 ` kernel test robot
2021-08-05 19:29 ` kernel test robot
2021-08-05 19:29 ` kernel test robot
2021-08-05 19:29 ` kernel test robot
2021-08-05 14:42 ` [PATCH v3 05/10] media: uapi: Add VP9 stateless decoder controls Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-09 18:04 ` Jernej Škrabec
2021-08-09 18:04 ` Jernej Škrabec
2021-08-09 18:04 ` Jernej Škrabec
2021-08-09 18:36 ` Ezequiel Garcia
2021-08-09 18:36 ` Ezequiel Garcia
2021-08-09 18:36 ` Ezequiel Garcia
2021-08-09 18:36 ` Ezequiel Garcia
2021-08-17 17:49 ` Nicolas Dufresne
2021-08-17 17:49 ` Nicolas Dufresne
2021-08-17 17:49 ` Nicolas Dufresne
2021-08-17 17:49 ` Nicolas Dufresne
2021-08-05 14:42 ` [PATCH v3 06/10] media: Add VP9 v4l2 library Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 07/10] media: rkvdec: Add the VP9 backend Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 20:14 ` kernel test robot [this message]
2021-08-05 14:42 ` [PATCH v3 08/10] media: hantro: Prepare for other G2 codecs Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` [PATCH v3 09/10] media: hantro: Support VP9 on the G2 core Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-09 17:07 ` Jernej Škrabec
2021-08-09 17:07 ` Jernej Škrabec
2021-08-09 17:07 ` Jernej Škrabec
2021-08-05 14:42 ` [PATCH v3 10/10] media: hantro: Support NV12 " Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
2021-08-05 14:42 ` Andrzej Pietrasiewicz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202108060449.d5kjb5TB-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.