All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Kyrie Wu <kyrie.wu@mediatek.com>,
	Hans Verkuil <hverkuil@kernel.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Nicolas Dufresne <nicolas.dufresne@collabora.com>,
	Conor Dooley <conor+dt@kernel.org>,
	Matthias Brugger <matthias.bgg@gmail.com>,
	AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org
Cc: oe-kbuild-all@lists.linux.dev, linux-media@vger.kernel.org
Subject: Re: [PATCH v12 05/12] media: mediatek: jpeg: Fix multi-core clk suspend and resume setting
Date: Fri, 23 Jan 2026 20:26:08 +0800	[thread overview]
Message-ID: <202601232059.fCMMbALw-lkp@intel.com> (raw)
In-Reply-To: <20260123031713.14621-6-kyrie.wu@mediatek.com>

Hi Kyrie,

kernel test robot noticed the following build warnings:

[auto build test WARNING on robh/for-next]
[also build test WARNING on linuxtv-media-pending/master media-tree/master linus/master v6.19-rc6 next-20260122]
[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/Kyrie-Wu/media-mediatek-jpeg-fix-jpeg-cores-amounts-setting/20260123-112425
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
patch link:    https://lore.kernel.org/r/20260123031713.14621-6-kyrie.wu%40mediatek.com
patch subject: [PATCH v12 05/12] media: mediatek: jpeg: Fix multi-core clk suspend and resume setting
config: parisc-allyesconfig (https://download.01.org/0day-ci/archive/20260123/202601232059.fCMMbALw-lkp@intel.com/config)
compiler: hppa-linux-gcc (GCC) 15.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260123/202601232059.fCMMbALw-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/202601232059.fCMMbALw-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c:429:12: warning: 'mtk_jpegenc_resume' defined but not used [-Wunused-function]
     429 | static int mtk_jpegenc_resume(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c:420:12: warning: 'mtk_jpegenc_suspend' defined but not used [-Wunused-function]
     420 | static int mtk_jpegenc_suspend(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c:412:12: warning: 'mtk_jpegenc_pm_resume' defined but not used [-Wunused-function]
     412 | static int mtk_jpegenc_pm_resume(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c:402:12: warning: 'mtk_jpegenc_pm_suspend' defined but not used [-Wunused-function]
     402 | static int mtk_jpegenc_pm_suspend(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~~~~
--
   drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c: In function 'mtk_jpegdec_hw_irq_handler':
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c:549:30: warning: variable 'ctx' set but not used [-Wunused-but-set-variable]
     549 |         struct mtk_jpeg_ctx *ctx;
         |                              ^~~
   drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c: At top level:
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c:707:12: warning: 'mtk_jpegdec_resume' defined but not used [-Wunused-function]
     707 | static int mtk_jpegdec_resume(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c:698:12: warning: 'mtk_jpegdec_suspend' defined but not used [-Wunused-function]
     698 | static int mtk_jpegdec_suspend(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c:690:12: warning: 'mtk_jpegdec_pm_resume' defined but not used [-Wunused-function]
     690 | static int mtk_jpegdec_pm_resume(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~~~
>> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c:680:12: warning: 'mtk_jpegdec_pm_suspend' defined but not used [-Wunused-function]
     680 | static int mtk_jpegdec_pm_suspend(struct device *dev)
         |            ^~~~~~~~~~~~~~~~~~~~~~


vim +/mtk_jpegenc_resume +429 drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c

   401	
 > 402	static int mtk_jpegenc_pm_suspend(struct device *dev)
   403	{
   404		struct mtk_jpegenc_comp_dev *jpeg = dev_get_drvdata(dev);
   405	
   406		clk_bulk_disable_unprepare(jpeg->venc_clk.clk_num,
   407					   jpeg->venc_clk.clks);
   408	
   409		return 0;
   410	}
   411	
 > 412	static int mtk_jpegenc_pm_resume(struct device *dev)
   413	{
   414		struct mtk_jpegenc_comp_dev *jpeg = dev_get_drvdata(dev);
   415	
   416		return clk_bulk_prepare_enable(jpeg->venc_clk.clk_num,
   417					       jpeg->venc_clk.clks);
   418	}
   419	
 > 420	static int mtk_jpegenc_suspend(struct device *dev)
   421	{
   422		struct mtk_jpegenc_comp_dev *jpeg = dev_get_drvdata(dev);
   423	
   424		v4l2_m2m_suspend(jpeg->master_dev->m2m_dev);
   425	
   426		return pm_runtime_force_suspend(dev);
   427	}
   428	
 > 429	static int mtk_jpegenc_resume(struct device *dev)
   430	{
   431		struct mtk_jpegenc_comp_dev *jpeg = dev_get_drvdata(dev);
   432		int ret;
   433	
   434		ret = pm_runtime_force_resume(dev);
   435		if (ret < 0)
   436			return ret;
   437	
   438		v4l2_m2m_resume(jpeg->master_dev->m2m_dev);
   439	
   440		return 0;
   441	}
   442	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki


  reply	other threads:[~2026-01-23 12:27 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-23  3:17 [PATCH v12 00/12] Enable jpeg enc & dec multi-hardwares for MT8196 Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 01/12] media: mediatek: jpeg: fix jpeg cores' amounts setting Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 02/12] media: mediatek: jpeg: fix jpeg buffer payload size setting Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 03/12] media: mediatek: jpeg: fix buffer structure size and layout Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 04/12] media: mediatek: jpeg: Fix buffer completion on multi-core streaming stop Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 05/12] media: mediatek: jpeg: Fix multi-core clk suspend and resume setting Kyrie Wu
2026-01-23 12:26   ` kernel test robot [this message]
2026-01-23  3:17 ` [PATCH v12 06/12] media: mediatek: jpeg: fix decoding buffer number setting timing issue Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 07/12] media: mediatek: jpeg: fix resolution change event handling in decoder Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 08/12] media: mediatek: jpeg: fix remove buffer removal timing for multi-core Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 09/12] media: dt-bindings: mediatek,jpeg: Add mediatek, mt8196-jpgdec compatible Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 10/12] media: dt-bindings: mediatek,jpeg: Add mediatek, mt8196-jpgenc compatible Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 11/12] media: mediatek: jpeg: add jpeg compatible Kyrie Wu
2026-01-23  3:17 ` [PATCH v12 12/12] media: mediatek: jpeg: add jpeg smmu sid setting Kyrie Wu

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=202601232059.fCMMbALw-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=hverkuil@kernel.org \
    --cc=krzk@kernel.org \
    --cc=kyrie.wu@mediatek.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=matthias.bgg@gmail.com \
    --cc=mchehab@kernel.org \
    --cc=nicolas.dufresne@collabora.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=robh@kernel.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.