From: kernel test robot <lkp@intel.com>
To: Hao Yao <hao.yao@intel.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
"Дамјан Георгиевски" <gdamjan@gmail.com>
Subject: [gdamjan:wip/ipu7-imx471 2/4] drivers/media/pci/intel/ipu7/ipu7-isys-queue.c:717:22: warning: unused variable 'isp'
Date: Sun, 29 Jun 2025 06:24:52 +0800 [thread overview]
Message-ID: <202506290640.PNXIsrIB-lkp@intel.com> (raw)
tree: https://github.com/gdamjan/linux wip/ipu7-imx471
head: 5340515315a1fa34442850491121fae1a3b0f42a
commit: 90640d505b4f1fe8958277855c11d0cfdd0c5e1f [2/4] media: pci: intel: Add IPU7 Kconfig & Makefile
config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20250629/202506290640.PNXIsrIB-lkp@intel.com/config)
compiler: clang version 20.1.7 (https://github.com/llvm/llvm-project 6146a88f60492b520a36f8f8f3231e15f3cc6082)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250629/202506290640.PNXIsrIB-lkp@intel.com/reproduce)
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>
| Closes: https://lore.kernel.org/oe-kbuild-all/202506290640.PNXIsrIB-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/media/pci/intel/ipu7/ipu7-isys-queue.c:717:22: warning: unused variable 'isp' [-Wunused-variable]
717 | struct ipu7_device *isp = adev->isp;
| ^~~
>> drivers/media/pci/intel/ipu7/ipu7-isys-queue.c:721:6: warning: unused variable 'ret' [-Wunused-variable]
721 | int ret, i, j;
| ^~~
2 warnings generated.
vim +/isp +717 drivers/media/pci/intel/ipu7/ipu7-isys-queue.c
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 711
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 712 static int ipu_isys_reset(struct ipu7_isys_video *self_av,
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 713 struct ipu7_isys_stream *self_stream)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 714 {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 715 struct ipu7_isys *isys = self_av->isys;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 716 struct ipu7_bus_device *adev = isys->adev;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 @717 struct ipu7_device *isp = adev->isp;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 718 struct ipu7_isys_video *av = NULL;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 719 struct ipu7_isys_stream *stream = NULL;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 720 struct device *dev = &adev->auxdev.dev;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 @721 int ret, i, j;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 722 int has_streaming = 0;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 723 const struct ipu7_isys_internal_csi2_pdata *csi2_pdata =
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 724 &isys->pdata->ipdata->csi2;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 725
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 726 dev_dbg(dev, "%s\n", __func__);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 727
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 728 mutex_lock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 729 if (isys->in_reset) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 730 mutex_unlock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 731 return 0;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 732 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 733 isys->in_reset = true;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 734
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 735 while (isys->in_stop_streaming) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 736 dev_dbg(dev, "isys reset: %s: wait for stop\n",
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 737 self_av->vdev.name);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 738 mutex_unlock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 739 usleep_range(10000, 11000);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 740 mutex_lock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 741 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 742
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 743 mutex_unlock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 744
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 745 dev_dbg(dev, "reset stop streams\n");
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 746 for (i = 0; i < csi2_pdata->nports; i++) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 747 for (j = 0; j < NR_OF_CSI2_SRC_PADS; j++) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 748 av = &isys->csi2[i].av[j];
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 749 if (av == self_av)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 750 continue;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 751
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 752 stream = av->stream;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 753 if (!stream || stream == self_stream)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 754 continue;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 755
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 756 if (!stream->streaming && !stream->nr_streaming)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 757 continue;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 758
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 759 av->reset = true;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 760 has_streaming = true;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 761 reset_stop_streaming(av);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 762 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 763 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 764
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 765 if (!has_streaming)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 766 goto end_of_reset;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 767
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 768 ipu7_cleanup_fw_msg_bufs(isys);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 769
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 770 dev_dbg(dev, "reset start streams\n");
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 771
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 772 for (j = 0; j < csi2_pdata->nports; j++) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 773 for (i = 0; i < NR_OF_CSI2_SRC_PADS; i++) {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 774 av = &isys->csi2[j].av[i];
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 775 if (!av->reset)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 776 continue;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 777
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 778 av->reset = false;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 779 reset_start_streaming(av);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 780 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 781 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 782
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 783 end_of_reset:
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 784 mutex_lock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 785 isys->in_reset = false;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 786 mutex_unlock(&isys->reset_mutex);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 787 dev_dbg(dev, "reset done\n");
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 788
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 789 return 0;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 790 }
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 791 #endif
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 792 static void stop_streaming(struct vb2_queue *q)
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 793 {
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 794 struct ipu7_isys_queue *aq = vb2_queue_to_isys_queue(q);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 795 struct ipu7_isys_video *av = ipu7_isys_queue_to_video(aq);
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 796 struct ipu7_isys_stream *stream = av->stream;
8427c76ace11f4 Дамјан Георгиевски 2025-06-26 797
:::::: The code at line 717 was first introduced by commit
:::::: 8427c76ace11f4a0292dc052cab902f2c4c3a188 copy drivers from github.com/intel/ipu7-drivers
:::::: TO: Дамјан Георгиевски <gdamjan@gmail.com>
:::::: CC: Дамјан Георгиевски <gdamjan@gmail.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2025-06-28 22:25 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202506290640.PNXIsrIB-lkp@intel.com \
--to=lkp@intel.com \
--cc=gdamjan@gmail.com \
--cc=hao.yao@intel.com \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@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.