mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Russ Weight <russell.h.weight@intel.com>,
	mdf@kernel.org, linux-fpga@vger.kernel.org,
	linux-kernel@vger.kernel.org
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org, trix@redhat.com,
	marpagan@redhat.com, lgoncalv@redhat.com, yilun.xu@intel.com,
	hao.wu@intel.com, matthew.gerlach@linux.intel.com,
	tianfei.zhang@intel.com, Russ Weight <russell.h.weight@intel.com>
Subject: Re: [PATCH v1 1/1] fpga: dfl: fix the kernel warning when release/assign ports for SRIOV
Date: Tue, 26 Apr 2022 05:08:16 +0800	[thread overview]
Message-ID: <202204260456.dOkQIjjg-lkp@intel.com> (raw)
In-Reply-To: <20220425174827.89819-1-russell.h.weight@intel.com>

Hi Russ,

I love your patch! Perhaps something to improve:

[auto build test WARNING on linus/master]
[also build test WARNING on v5.18-rc4 next-20220422]
[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]

url:    https://github.com/intel-lab-lkp/linux/commits/Russ-Weight/fpga-dfl-fix-the-kernel-warning-when-release-assign-ports-for-SRIOV/20220426-014949
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git af2d861d4cd2a4da5137f795ee3509e6f944a25b
config: hexagon-randconfig-r041-20220425 (https://download.01.org/0day-ci/archive/20220426/202204260456.dOkQIjjg-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 1cddcfdc3c683b393df1a5c9063252eb60e52818)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/4b4db40cd1b608457d6e1fcf95443bd940d01f3c
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Russ-Weight/fpga-dfl-fix-the-kernel-warning-when-release-assign-ports-for-SRIOV/20220426-014949
        git checkout 4b4db40cd1b608457d6e1fcf95443bd940d01f3c
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/fpga/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/fpga/dfl.c:54: warning: cannot understand function prototype: 'struct dfl_dev_info '
   drivers/fpga/dfl.c:74: warning: cannot understand function prototype: 'struct dfl_chardev_info '
   drivers/fpga/dfl.c:154: warning: Function parameter or member 'fdata' not described in 'dfl_fpga_port_ops_get'
   drivers/fpga/dfl.c:154: warning: Excess function parameter 'pdev' description in 'dfl_fpga_port_ops_get'
   drivers/fpga/dfl.c:220: warning: Function parameter or member 'fdata' not described in 'dfl_fpga_check_port_id'
   drivers/fpga/dfl.c:220: warning: Excess function parameter 'pdev' description in 'dfl_fpga_check_port_id'
   drivers/fpga/dfl.c:716: warning: Function parameter or member 'revision' not described in 'dfl_feature_info'
>> drivers/fpga/dfl.c:1509: warning: expecting prototype for __dfl_fpga_cdev_find_port(). Prototype was for __dfl_fpga_cdev_find_port_data() instead


vim +1509 drivers/fpga/dfl.c

543be3d8c999b3 Wu Hao   2018-06-30  1490  
5d56e117001996 Wu Hao   2018-06-30  1491  /**
5d56e117001996 Wu Hao   2018-06-30  1492   * __dfl_fpga_cdev_find_port - find a port under given container device
5d56e117001996 Wu Hao   2018-06-30  1493   *
5d56e117001996 Wu Hao   2018-06-30  1494   * @cdev: container device
5d56e117001996 Wu Hao   2018-06-30  1495   * @data: data passed to match function
5d56e117001996 Wu Hao   2018-06-30  1496   * @match: match function used to find specific port from the port device list
5d56e117001996 Wu Hao   2018-06-30  1497   *
5d56e117001996 Wu Hao   2018-06-30  1498   * Find a port device under container device. This function needs to be
5d56e117001996 Wu Hao   2018-06-30  1499   * invoked with lock held.
5d56e117001996 Wu Hao   2018-06-30  1500   *
5d56e117001996 Wu Hao   2018-06-30  1501   * Return: pointer to port's platform device if successful, NULL otherwise.
5d56e117001996 Wu Hao   2018-06-30  1502   *
5d56e117001996 Wu Hao   2018-06-30  1503   * NOTE: you will need to drop the device reference with put_device() after use.
5d56e117001996 Wu Hao   2018-06-30  1504   */
4b4db40cd1b608 Xu Yilun 2022-04-25  1505  struct dfl_feature_dev_data *
4b4db40cd1b608 Xu Yilun 2022-04-25  1506  __dfl_fpga_cdev_find_port_data(struct dfl_fpga_cdev *cdev, void *data,
4b4db40cd1b608 Xu Yilun 2022-04-25  1507  			       int (*match)(struct dfl_feature_dev_data *,
4b4db40cd1b608 Xu Yilun 2022-04-25  1508  					    void *))
5d56e117001996 Wu Hao   2018-06-30 @1509  {
4b4db40cd1b608 Xu Yilun 2022-04-25  1510  	struct dfl_feature_dev_data *fdata;
5d56e117001996 Wu Hao   2018-06-30  1511  
4b4db40cd1b608 Xu Yilun 2022-04-25  1512  	list_for_each_entry(fdata, &cdev->port_dev_list, node) {
4b4db40cd1b608 Xu Yilun 2022-04-25  1513  		if (match(fdata, data))
4b4db40cd1b608 Xu Yilun 2022-04-25  1514  			return fdata;
5d56e117001996 Wu Hao   2018-06-30  1515  	}
5d56e117001996 Wu Hao   2018-06-30  1516  
5d56e117001996 Wu Hao   2018-06-30  1517  	return NULL;
5d56e117001996 Wu Hao   2018-06-30  1518  }
4b4db40cd1b608 Xu Yilun 2022-04-25  1519  EXPORT_SYMBOL_GPL(__dfl_fpga_cdev_find_port_data);
5d56e117001996 Wu Hao   2018-06-30  1520  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

  reply	other threads:[~2022-04-25 21:08 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-25 17:48 Russ Weight
2022-04-25 21:08 ` kernel test robot [this message]
2022-04-26  2:00 ` Wu, Hao

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=202204260456.dOkQIjjg-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=hao.wu@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=lgoncalv@redhat.com \
    --cc=linux-fpga@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=llvm@lists.linux.dev \
    --cc=marpagan@redhat.com \
    --cc=matthew.gerlach@linux.intel.com \
    --cc=mdf@kernel.org \
    --cc=russell.h.weight@intel.com \
    --cc=tianfei.zhang@intel.com \
    --cc=trix@redhat.com \
    --cc=yilun.xu@intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®