All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: "Jyothi Kumar Seerapu" <quic_jseerapu@quicinc.com>,
	"Vinod Koul" <vkoul@kernel.org>,
	"Mukesh Kumar Savaliya" <quic_msavaliy@quicinc.com>,
	"Viken Dadhaniya" <quic_vdadhani@quicinc.com>,
	"Andi Shyti" <andi.shyti@kernel.org>,
	"Sumit Semwal" <sumit.semwal@linaro.org>,
	"Christian König" <christian.koenig@amd.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-arm-msm@vger.kernel.org,
	dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-i2c@vger.kernel.org, linux-media@vger.kernel.org,
	dri-devel@lists.freedesktop.org, linaro-mm-sig@lists.linaro.org,
	quic_vtanuku@quicinc.com
Subject: Re: [PATCH v7 2/2] i2c: i2c-qcom-geni: Add Block event interrupt support
Date: Sat, 6 Sep 2025 20:49:26 +0800	[thread overview]
Message-ID: <202509062008.lSOdhd4U-lkp@intel.com> (raw)
In-Reply-To: <20250903073059.2151837-3-quic_jseerapu@quicinc.com>

Hi Jyothi,

kernel test robot noticed the following build warnings:

[auto build test WARNING on vkoul-dmaengine/next]
[also build test WARNING on andi-shyti/i2c/i2c-host linus/master v6.17-rc4 next-20250905]
[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/Jyothi-Kumar-Seerapu/dmaengine-qcom-gpi-Add-GPI-Block-event-interrupt-support/20250903-153745
base:   https://git.kernel.org/pub/scm/linux/kernel/git/vkoul/dmaengine.git next
patch link:    https://lore.kernel.org/r/20250903073059.2151837-3-quic_jseerapu%40quicinc.com
patch subject: [PATCH v7 2/2] i2c: i2c-qcom-geni: Add Block event interrupt support
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20250906/202509062008.lSOdhd4U-lkp@intel.com/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250906/202509062008.lSOdhd4U-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/202509062008.lSOdhd4U-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/i2c/busses/i2c-qcom-geni.c: In function 'geni_i2c_gpi_multi_desc_unmap':
>> drivers/i2c/busses/i2c-qcom-geni.c:576:42: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     576 |                                    NULL, (dma_addr_t)NULL);
         |                                          ^


vim +576 drivers/i2c/busses/i2c-qcom-geni.c

   555	
   556	/**
   557	 * geni_i2c_gpi_multi_desc_unmap() - Unmaps DMA buffers post multi message TX transfers
   558	 * @gi2c: I2C dev handle
   559	 * @msgs: Array of I2C  messages
   560	 * @peripheral: Pointer to gpi_i2c_config
   561	 */
   562	static void geni_i2c_gpi_multi_desc_unmap(struct geni_i2c_dev *gi2c, struct i2c_msg msgs[],
   563						  struct gpi_i2c_config *peripheral)
   564	{
   565		u32 msg_xfer_cnt, wr_idx = 0;
   566		struct geni_i2c_gpi_multi_desc_xfer *tx_multi_xfer = &gi2c->i2c_multi_desc_config;
   567	
   568		msg_xfer_cnt = gi2c->err ? tx_multi_xfer->msg_idx_cnt : tx_multi_xfer->irq_cnt;
   569	
   570		/* Unmap the processed DMA buffers based on the received interrupt count */
   571		for (; tx_multi_xfer->unmap_msg_cnt < msg_xfer_cnt; tx_multi_xfer->unmap_msg_cnt++) {
   572			wr_idx = tx_multi_xfer->unmap_msg_cnt;
   573			geni_i2c_gpi_unmap(gi2c, &msgs[wr_idx],
   574					   tx_multi_xfer->dma_buf[wr_idx],
   575					   tx_multi_xfer->dma_addr[wr_idx],
 > 576					   NULL, (dma_addr_t)NULL);
   577	
   578			if (tx_multi_xfer->unmap_msg_cnt == gi2c->num_msgs - 1) {
   579				kfree(tx_multi_xfer->dma_buf);
   580				kfree(tx_multi_xfer->dma_addr);
   581				break;
   582			}
   583		}
   584	}
   585	

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

      reply	other threads:[~2025-09-06 12:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-03  7:30 [PATCH v7 0/2] i2c: i2c-qcom-geni: Add Block event interrupt support Jyothi Kumar Seerapu
2025-09-03  7:30 ` [PATCH v7 1/2] dmaengine: qcom: gpi: Add GPI " Jyothi Kumar Seerapu
2025-09-03  9:25   ` Dmitry Baryshkov
2025-09-03  9:42     ` Jyothi Kumar Seerapu
2025-09-03  7:30 ` [PATCH v7 2/2] i2c: i2c-qcom-geni: Add " Jyothi Kumar Seerapu
2025-09-06 12:49   ` kernel test robot [this message]

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=202509062008.lSOdhd4U-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=andi.shyti@kernel.org \
    --cc=christian.koenig@amd.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=quic_jseerapu@quicinc.com \
    --cc=quic_msavaliy@quicinc.com \
    --cc=quic_vdadhani@quicinc.com \
    --cc=quic_vtanuku@quicinc.com \
    --cc=sumit.semwal@linaro.org \
    --cc=vkoul@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.