All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: oe-kbuild-all@lists.linux.dev
Subject: [freescale-fslc:pr/639 1629/24603] drivers/dma/mxs-dma.c:914:5: warning: no previous declaration for 'mxs_dma_runtime_suspend'
Date: Fri, 1 Sep 2023 06:40:08 +0800	[thread overview]
Message-ID: <202309010652.npy3mDe5-lkp@intel.com> (raw)

tree:   https://github.com/Freescale/linux-fslc pr/639
head:   857fbf7cebaba3b1ffccc558deee1d13ac0e11d7
commit: 1b639120f160cd2a1a0fa6528d4cb721b94708c3 [1629/24603] LF-251-1: dma: mxs-dma: enable runtime PM for mxs-dma
config: x86_64-buildonly-randconfig-004-20230831 (https://download.01.org/0day-ci/archive/20230901/202309010652.npy3mDe5-lkp@intel.com/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230901/202309010652.npy3mDe5-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/202309010652.npy3mDe5-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/dma/mxs-dma.c: In function 'mxs_dma_remove':
   drivers/dma/mxs-dma.c:879:2: error: implicit declaration of function 'dma_pool_destroy'; did you mean 'device_destroy'? [-Werror=implicit-function-declaration]
     dma_pool_destroy(mxs_dma->mxs_chans[0].ccw_pool);
     ^~~~~~~~~~~~~~~~
     device_destroy
   drivers/dma/mxs-dma.c:879:41: error: 'struct mxs_dma_chan' has no member named 'ccw_pool'; did you mean 'ccw_phys'?
     dma_pool_destroy(mxs_dma->mxs_chans[0].ccw_pool);
                                            ^~~~~~~~
                                            ccw_phys
   drivers/dma/mxs-dma.c:885:13: error: 'struct mxs_dma_chan' has no member named 'ccw_pool'; did you mean 'ccw_phys'?
      mxs_chan->ccw_pool = NULL;
                ^~~~~~~~
                ccw_phys
   drivers/dma/mxs-dma.c: At top level:
>> drivers/dma/mxs-dma.c:914:5: warning: no previous declaration for 'mxs_dma_runtime_suspend' [-Wmissing-declarations]
    int mxs_dma_runtime_suspend(struct device *dev)
        ^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/dma/mxs-dma.c:923:5: warning: no previous declaration for 'mxs_dma_runtime_resume' [-Wmissing-declarations]
    int mxs_dma_runtime_resume(struct device *dev)
        ^~~~~~~~~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors


vim +/mxs_dma_runtime_suspend +914 drivers/dma/mxs-dma.c

   872	
   873	static int mxs_dma_remove(struct platform_device *pdev)
   874	{
   875		struct mxs_dma_engine *mxs_dma = platform_get_drvdata(pdev);
   876		int i;
   877	
   878		dma_async_device_unregister(&mxs_dma->dma_device);
 > 879		dma_pool_destroy(mxs_dma->mxs_chans[0].ccw_pool);
   880	
   881		for (i = 0; i < MXS_DMA_CHANNELS; i++) {
   882			struct mxs_dma_chan *mxs_chan = &mxs_dma->mxs_chans[i];
   883	
   884			tasklet_kill(&mxs_chan->tasklet);
 > 885			mxs_chan->ccw_pool = NULL;
   886		}
   887	
   888		return 0;
   889	}
   890	
   891	#ifdef CONFIG_PM_SLEEP
   892	static int mxs_dma_pm_suspend(struct device *dev)
   893	{
   894		int ret;
   895	
   896		ret = pm_runtime_force_suspend(dev);
   897	
   898		return ret;
   899	}
   900	
   901	static int mxs_dma_pm_resume(struct device *dev)
   902	{
   903		struct mxs_dma_engine *mxs_dma = dev_get_drvdata(dev);
   904		int ret;
   905	
   906		ret = mxs_dma_init(mxs_dma);
   907		if (ret)
   908			return ret;
   909	
   910		return 0;
   911	}
   912	#endif
   913	
 > 914	int mxs_dma_runtime_suspend(struct device *dev)
   915	{
   916		struct mxs_dma_engine *mxs_dma = dev_get_drvdata(dev);
   917	
   918		clk_disable_unprepare(mxs_dma->clk);
   919	
   920		return 0;
   921	}
   922	
 > 923	int mxs_dma_runtime_resume(struct device *dev)
   924	{
   925		struct mxs_dma_engine *mxs_dma = dev_get_drvdata(dev);
   926		int ret;
   927	
   928		ret = clk_prepare_enable(mxs_dma->clk);
   929		if (ret) {
   930			dev_err(&mxs_dma->pdev->dev, "failed to enable the clock\n");
   931			return ret;
   932		}
   933	
   934		return 0;
   935	}
   936	

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

                 reply	other threads:[~2023-08-31 22:40 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=202309010652.npy3mDe5-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=otavio@ossystems.com.br \
    /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.