From: kernel test robot <lkp@intel.com>
To: Baokun Li <libaokun1@huawei.com>,
mchehab@kernel.org, sakari.ailus@linux.intel.com,
gregkh@linuxfoundation.org, andriy.shevchenko@linux.intel.com,
kaixuxia@tencent.com, gustavoars@kernel.org,
linux-media@vger.kernel.org, linux-staging@lists.linux.dev,
linux-kernel@vger.kernel.org
Cc: kbuild-all@lists.01.org, weiyongjun1@huawei.com
Subject: Re: [PATCH -next v3] media: staging: atomisp: use list_splice_init in atomisp_compat_css20.c
Date: Thu, 17 Jun 2021 16:52:00 +0800 [thread overview]
Message-ID: <202106171617.BN97N8mw-lkp@intel.com> (raw)
In-Reply-To: <20210611081004.1348026-1-libaokun1@huawei.com>
[-- Attachment #1: Type: text/plain, Size: 5175 bytes --]
Hi Baokun,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on next-20210616]
url: https://github.com/0day-ci/linux/commits/Baokun-Li/media-staging-atomisp-use-list_splice_init-in-atomisp_compat_css20-c/20210617-043443
base: c7d4c1fd91ab4a6d2620497921a9c6bf54650ab8
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/05c714d7b4e1722a75f949e40c5305f3ba28d048
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Baokun-Li/media-staging-atomisp-use-list_splice_init-in-atomisp_compat_css20-c/20210617-043443
git checkout 05c714d7b4e1722a75f949e40c5305f3ba28d048
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c: In function 'atomisp_css_stop':
>> drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:2156:50: error: 'struct atomisp_sub_device' has no member named 'asd'
2156 | list_splice_init(&asd->metadata_in_css[i], &asd->asd->metadata[i]);
| ^~
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:2157:49: error: 'struct atomisp_sub_device' has no member named 'asd'
2157 | list_splice_init(&asd->metadata_ready[i], &asd->asd->metadata[i]);
| ^~
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:2107:31: warning: unused variable 'md_buf' [-Wunused-variable]
2107 | struct atomisp_metadata_buf *md_buf;
| ^~~~~~
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:2106:26: warning: unused variable 'dis_buf' [-Wunused-variable]
2106 | struct atomisp_dis_buf *dis_buf;
| ^~~~~~~
drivers/staging/media/atomisp/pci/atomisp_compat_css20.c:2105:26: warning: unused variable 's3a_buf' [-Wunused-variable]
2105 | struct atomisp_s3a_buf *s3a_buf;
| ^~~~~~~
vim +2156 drivers/staging/media/atomisp/pci/atomisp_compat_css20.c
2100
2101 void atomisp_css_stop(struct atomisp_sub_device *asd,
2102 enum ia_css_pipe_id pipe_id, bool in_reset)
2103 {
2104 struct atomisp_device *isp = asd->isp;
2105 struct atomisp_s3a_buf *s3a_buf;
2106 struct atomisp_dis_buf *dis_buf;
2107 struct atomisp_metadata_buf *md_buf;
2108 unsigned long irqflags;
2109 unsigned int i;
2110
2111 /* if is called in atomisp_reset(), force destroy stream */
2112 if (__destroy_streams(asd, true))
2113 dev_err(isp->dev, "destroy stream failed.\n");
2114
2115 /* if is called in atomisp_reset(), force destroy all pipes */
2116 if (__destroy_pipes(asd, true))
2117 dev_err(isp->dev, "destroy pipes failed.\n");
2118
2119 atomisp_init_raw_buffer_bitmap(asd);
2120
2121 /*
2122 * SP can not be stop if other streams are in use
2123 */
2124 if (atomisp_streaming_count(isp) == 0)
2125 ia_css_stop_sp();
2126
2127 if (!in_reset) {
2128 struct atomisp_stream_env *stream_env;
2129 int i, j;
2130
2131 for (i = 0; i < ATOMISP_INPUT_STREAM_NUM; i++) {
2132 stream_env = &asd->stream_env[i];
2133 for (j = 0; j < IA_CSS_PIPE_ID_NUM; j++) {
2134 ia_css_pipe_config_defaults(
2135 &stream_env->pipe_configs[j]);
2136 ia_css_pipe_extra_config_defaults(
2137 &stream_env->pipe_extra_configs[j]);
2138 }
2139 ia_css_stream_config_defaults(
2140 &stream_env->stream_config);
2141 }
2142 memset(&asd->params.config, 0, sizeof(asd->params.config));
2143 asd->params.css_update_params_needed = false;
2144 }
2145
2146 /* move stats buffers to free queue list */
2147 list_splice_init(&asd->s3a_stats_in_css, &asd->s3a_stats);
2148 list_splice_init(&asd->s3a_stats_ready, &asd->s3a_stats);
2149
2150 spin_lock_irqsave(&asd->dis_stats_lock, irqflags);
2151 list_splice_init(&asd->dis_stats_in_css, &asd->dis_stats);
2152 asd->params.dis_proj_data_valid = false;
2153 spin_unlock_irqrestore(&asd->dis_stats_lock, irqflags);
2154
2155 for (i = 0; i < ATOMISP_METADATA_TYPE_NUM; i++) {
> 2156 list_splice_init(&asd->metadata_in_css[i], &asd->asd->metadata[i]);
2157 list_splice_init(&asd->metadata_ready[i], &asd->asd->metadata[i]);
2158 }
2159
2160 atomisp_flush_params_queue(&asd->video_out_capture);
2161 atomisp_flush_params_queue(&asd->video_out_vf);
2162 atomisp_flush_params_queue(&asd->video_out_preview);
2163 atomisp_flush_params_queue(&asd->video_out_video_capture);
2164 atomisp_free_css_parameters(&asd->params.css_param);
2165 memset(&asd->params.css_param, 0, sizeof(asd->params.css_param));
2166 }
2167
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65574 bytes --]
next prev parent reply other threads:[~2021-06-17 8:52 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-11 8:10 [PATCH -next v3] media: staging: atomisp: use list_splice_init in atomisp_compat_css20.c Baokun Li
2021-06-11 9:59 ` Andy Shevchenko
2021-06-17 8:52 ` kernel test robot [this message]
2021-06-17 12:14 ` libaokun (A)
2021-06-17 12:41 ` Andy Shevchenko
2021-06-17 12:53 ` libaokun (A)
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=202106171617.BN97N8mw-lkp@intel.com \
--to=lkp@intel.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=gregkh@linuxfoundation.org \
--cc=gustavoars@kernel.org \
--cc=kaixuxia@tencent.com \
--cc=kbuild-all@lists.01.org \
--cc=libaokun1@huawei.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=mchehab@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=weiyongjun1@huawei.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).