On Tue, Oct 22, 2019 at 12:23:12PM +0300, Dan Carpenter wrote:
> Hi Corentin,
> 
> url:    
> https://github.com/0day-ci/linux/commits/Corentin-Labbe/crypto-add-sun8i-ce-driver-for-Allwinner-crypto-engine/20191014-104401
> base:   
> https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git 
> master
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <l...@intel.com>
> Reported-by: Dan Carpenter <dan.carpen...@oracle.com>
> 
> smatch warnings:
> drivers/crypto/allwinner/sun8i-ce/sun8i-ce-core.c:371 
> sun8i_ce_allocate_chanlist() error: uninitialized symbol 'err'.
> 
> # 
> https://github.com/0day-ci/linux/commit/f113059e7b4f94c545994aeafdc809a3e4907ae4
> git remote add linux-review https://github.com/0day-ci/linux
> git remote update linux-review
> git checkout f113059e7b4f94c545994aeafdc809a3e4907ae4
> vim +/err +371 drivers/crypto/allwinner/sun8i-ce/sun8i-ce-core.c
> 
> f113059e7b4f94 Corentin Labbe 2019-10-12  334  static int 
> sun8i_ce_allocate_chanlist(struct sun8i_ce_dev *ce)
> f113059e7b4f94 Corentin Labbe 2019-10-12  335  {
> f113059e7b4f94 Corentin Labbe 2019-10-12  336         int i, err;
> f113059e7b4f94 Corentin Labbe 2019-10-12  337  
> f113059e7b4f94 Corentin Labbe 2019-10-12  338         ce->chanlist = 
> devm_kcalloc(ce->dev, MAXFLOW,
> f113059e7b4f94 Corentin Labbe 2019-10-12  339                                 
>     sizeof(struct sun8i_ce_flow), GFP_KERNEL);
> f113059e7b4f94 Corentin Labbe 2019-10-12  340         if (!ce->chanlist)
> f113059e7b4f94 Corentin Labbe 2019-10-12  341                 return -ENOMEM;
> f113059e7b4f94 Corentin Labbe 2019-10-12  342  
> f113059e7b4f94 Corentin Labbe 2019-10-12  343         for (i = 0; i < 
> MAXFLOW; i++) {
> f113059e7b4f94 Corentin Labbe 2019-10-12  344                 
> init_completion(&ce->chanlist[i].complete);
> f113059e7b4f94 Corentin Labbe 2019-10-12  345  
> f113059e7b4f94 Corentin Labbe 2019-10-12  346                 
> ce->chanlist[i].engine = crypto_engine_alloc_init(ce->dev, true);
> f113059e7b4f94 Corentin Labbe 2019-10-12  347                 if 
> (!ce->chanlist[i].engine) {
> f113059e7b4f94 Corentin Labbe 2019-10-12  348                         
> dev_err(ce->dev, "Cannot allocate engine\n");
> f113059e7b4f94 Corentin Labbe 2019-10-12  349                         i--;
> f113059e7b4f94 Corentin Labbe 2019-10-12  350                         goto 
> error_engine;
> 
> err = -ENOMEM;
> 
> f113059e7b4f94 Corentin Labbe 2019-10-12  351                 }
> f113059e7b4f94 Corentin Labbe 2019-10-12  352                 err = 
> crypto_engine_start(ce->chanlist[i].engine);
> f113059e7b4f94 Corentin Labbe 2019-10-12  353                 if (err) {
> f113059e7b4f94 Corentin Labbe 2019-10-12  354                         
> dev_err(ce->dev, "Cannot start engine\n");
> f113059e7b4f94 Corentin Labbe 2019-10-12  355                         goto 
> error_engine;
> f113059e7b4f94 Corentin Labbe 2019-10-12  356                 }
> f113059e7b4f94 Corentin Labbe 2019-10-12  357                 
> ce->chanlist[i].tl = dma_alloc_coherent(ce->dev,
> f113059e7b4f94 Corentin Labbe 2019-10-12  358                                 
>                         sizeof(struct ce_task),
> f113059e7b4f94 Corentin Labbe 2019-10-12  359                                 
>                         &ce->chanlist[i].t_phy,
> f113059e7b4f94 Corentin Labbe 2019-10-12  360                                 
>                         GFP_KERNEL);
> f113059e7b4f94 Corentin Labbe 2019-10-12  361                 if 
> (!ce->chanlist[i].tl) {
> f113059e7b4f94 Corentin Labbe 2019-10-12  362                         
> dev_err(ce->dev, "Cannot get DMA memory for task %d\n",
> f113059e7b4f94 Corentin Labbe 2019-10-12  363                                 
> i);
> f113059e7b4f94 Corentin Labbe 2019-10-12  364                         err = 
> -ENOMEM;
> f113059e7b4f94 Corentin Labbe 2019-10-12  365                         goto 
> error_engine;
> f113059e7b4f94 Corentin Labbe 2019-10-12  366                 }
> f113059e7b4f94 Corentin Labbe 2019-10-12  367         }
> f113059e7b4f94 Corentin Labbe 2019-10-12  368         return 0;
> f113059e7b4f94 Corentin Labbe 2019-10-12  369  error_engine:
> f113059e7b4f94 Corentin Labbe 2019-10-12  370         
> sun8i_ce_free_chanlist(ce, i);
> f113059e7b4f94 Corentin Labbe 2019-10-12 @371         return err;
> f113059e7b4f94 Corentin Labbe 2019-10-12  372  }
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

fixed.

Thanks

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
To view this discussion on the web, visit 
https://groups.google.com/d/msgid/linux-sunxi/20191022132212.GA28813%40Red.

Reply via email to