From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com, Dan Carpenter <error27@gmail.com>
Subject: Re: [PATCH 1/2] dmaengine: fsl-edma: request per-channel IRQ only when channel is allocated
Date: Sun, 2 Jun 2024 01:48:12 +0800 [thread overview]
Message-ID: <202406020102.cYYPfac7-lkp@intel.com> (raw)
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20240531210152.1878443-1-Frank.Li@nxp.com>
References: <20240531210152.1878443-1-Frank.Li@nxp.com>
TO: Frank Li <Frank.Li@nxp.com>
TO: Vinod Koul <vkoul@kernel.org>
TO: "open list:FREESCALE eDMA DRIVER" <imx@lists.linux.dev>
TO: open list <linux-kernel@vger.kernel.org>
Hi Frank,
kernel test robot noticed the following build warnings:
[auto build test WARNING on vkoul-dmaengine/next]
[also build test WARNING on linus/master v6.10-rc1 next-20240531]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Frank-Li/dmaengine-fsl-edma-remove-redundant-idle-field-from-fsl_chan/20240601-050401
base: https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next
patch link: https://lore.kernel.org/r/20240531210152.1878443-1-Frank.Li%40nxp.com
patch subject: [PATCH 1/2] dmaengine: fsl-edma: request per-channel IRQ only when channel is allocated
:::::: branch date: 21 hours ago
:::::: commit date: 21 hours ago
config: i386-randconfig-141-20240601 (https://download.01.org/0day-ci/archive/20240602/202406020102.cYYPfac7-lkp@intel.com/config)
compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202406020102.cYYPfac7-lkp@intel.com/
New smatch warnings:
drivers/dma/fsl-edma-common.c:828 fsl_edma_alloc_chan_resources() warn: 'fsl_chan->clk' from clk_prepare_enable() not released on lines: 824.
Old smatch warnings:
drivers/dma/fsl-edma-common.c:439 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:439 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:441 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:441 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:442 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:442 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:444 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:444 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:445 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:445 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:447 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:447 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:448 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:448 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:450 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:450 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:451 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:451 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:452 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:452 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:454 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:454 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:460 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:460 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:475 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:475 fsl_edma_set_tcd_regs() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:495 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:495 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:496 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:496 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:498 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:498 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:500 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:500 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:517 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:517 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:518 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:518 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:520 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:520 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:521 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:521 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:523 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:523 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:525 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:525 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:542 fsl_edma_fill_tcd() warn: statement has no effect 31
drivers/dma/fsl-edma-common.c:542 fsl_edma_fill_tcd() warn: statement has no effect 31
vim +828 drivers/dma/fsl-edma-common.c
9d831528a6567d Angelo Dureghello 2018-08-19 804
9d831528a6567d Angelo Dureghello 2018-08-19 805 int fsl_edma_alloc_chan_resources(struct dma_chan *chan)
9d831528a6567d Angelo Dureghello 2018-08-19 806 {
9d831528a6567d Angelo Dureghello 2018-08-19 807 struct fsl_edma_chan *fsl_chan = to_fsl_edma_chan(chan);
cd750c6413ce8a Frank Li 2024-05-31 808 int ret;
9d831528a6567d Angelo Dureghello 2018-08-19 809
d8d4355861d874 Joy Zou 2024-03-23 810 if (fsl_edma_drvflags(fsl_chan) & FSL_EDMA_DRV_HAS_CHCLK)
d8d4355861d874 Joy Zou 2024-03-23 811 clk_prepare_enable(fsl_chan->clk);
d8d4355861d874 Joy Zou 2024-03-23 812
9d831528a6567d Angelo Dureghello 2018-08-19 813 fsl_chan->tcd_pool = dma_pool_create("tcd_pool", chan->device->dev,
de7d9cb3b064fd Frank Li 2023-12-21 814 fsl_edma_drvflags(fsl_chan) & FSL_EDMA_DRV_TCD64 ?
de7d9cb3b064fd Frank Li 2023-12-21 815 sizeof(struct fsl_edma_hw_tcd64) : sizeof(struct fsl_edma_hw_tcd),
9d831528a6567d Angelo Dureghello 2018-08-19 816 32, 0);
cd750c6413ce8a Frank Li 2024-05-31 817
cd750c6413ce8a Frank Li 2024-05-31 818 if (fsl_chan->txirq) {
cd750c6413ce8a Frank Li 2024-05-31 819 ret = request_irq(fsl_chan->txirq, fsl_chan->irq_handler, IRQF_SHARED,
cd750c6413ce8a Frank Li 2024-05-31 820 fsl_chan->chan_name, fsl_chan);
cd750c6413ce8a Frank Li 2024-05-31 821
cd750c6413ce8a Frank Li 2024-05-31 822 if (ret) {
cd750c6413ce8a Frank Li 2024-05-31 823 dma_pool_destroy(fsl_chan->tcd_pool);
cd750c6413ce8a Frank Li 2024-05-31 824 return ret;
cd750c6413ce8a Frank Li 2024-05-31 825 }
cd750c6413ce8a Frank Li 2024-05-31 826 }
cd750c6413ce8a Frank Li 2024-05-31 827
9d831528a6567d Angelo Dureghello 2018-08-19 @828 return 0;
9d831528a6567d Angelo Dureghello 2018-08-19 829 }
9d831528a6567d Angelo Dureghello 2018-08-19 830
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2024-06-01 17:48 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-01 17:48 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2024-05-31 21:01 [PATCH 1/2] dmaengine: fsl-edma: request per-channel IRQ only when channel is allocated Frank Li
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=202406020102.cYYPfac7-lkp@intel.com \
--to=lkp@intel.com \
--cc=error27@gmail.com \
--cc=oe-kbuild@lists.linux.dev \
/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.