Linux-Next Archive on lore.kernel.org
 help / color / Atom feed
* linux-next: build warning after merge of the slave-dma tree
@ 2015-03-30  6:20 Stephen Rothwell
  2015-03-30  6:26 ` Appana Durga Kedareswara Rao
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2015-03-30  6:20 UTC (permalink / raw)
  To: Vinod Koul; +Cc: linux-next, linux-kernel, Kedareswara rao Appana


[-- Attachment #1: Type: text/plain, Size: 1445 bytes --]

Hi Vinod,

After merging the slave-dma tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

In file included from drivers/dma/xilinx/xilinx_vdma.c:26:0:
include/linux/dmapool.h:18:4: warning: 'struct device' declared inside parameter list
    size_t size, size_t align, size_t allocation);
    ^
include/linux/dmapool.h:18:4: warning: its scope is only this definition or declaration, which is probably not what you want
include/linux/dmapool.h:31:7: warning: 'struct device' declared inside parameter list
       size_t size, size_t align, size_t allocation);
       ^
drivers/dma/xilinx/xilinx_vdma.c: In function 'xilinx_vdma_alloc_chan_resources':
drivers/dma/xilinx/xilinx_vdma.c:501:20: warning: passing argument 2 of 'dma_pool_create' from incompatible pointer type
  chan->desc_pool = dma_pool_create("xilinx_vdma_desc_pool",
                    ^
In file included from drivers/dma/xilinx/xilinx_vdma.c:26:0:
include/linux/dmapool.h:17:18: note: expected 'struct device *' but argument is of type 'struct device *'
 struct dma_pool *dma_pool_create(const char *name, struct device *dev, 
                  ^

Presumable caused by the include order change in commit 937abe88aea3
("dmaengine: xilinx-dma: move header file to common location").

I guess that dmapool.h needs a "struct device;" line near the top.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2016-06-09  1:18 Stephen Rothwell
  2016-06-09  6:04 ` Appana Durga Kedareswara Rao
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2016-06-09  1:18 UTC (permalink / raw)
  To: Vinod Koul; +Cc: linux-next, linux-kernel, Kedareswara rao Appana

Hi Vinod,

After merging the slave-dma tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

drivers/dma/xilinx/xilinx_vdma.c: In function 'xilinx_dma_prep_dma_cyclic':
drivers/dma/xilinx/xilinx_vdma.c:1808:23: warning: 'segment' may be used uninitialized in this function [-Wmaybe-uninitialized]
   segment->hw.control |= XILINX_DMA_BD_SOP;

Introduced by commit

  92d794dfb699 ("dmaengine: vdma: Add support for cyclic dma mode")

I am not sure if this is tecnically a problem, but gcc certainly cannot
tell.

-- 
Cheers,
Stephen Rothwell

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2018-07-04  3:30 Stephen Rothwell
  2018-07-04  4:20 ` Vinod
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2018-07-04  3:30 UTC (permalink / raw)
  To: Vinod Koul
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Angelo Dureghello


[-- Attachment #1: Type: text/plain, Size: 362 bytes --]

Hi Vinod,

After merging the slave-dma tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

WARNING: modpost: missing MODULE_LICENSE() in drivers/dma/fsl-edma-common.o

Introduced by commit

  6ad069123f03 ("dmaengine: fsl-edma: extract common fsl-edma code (no changes in behavior intended)")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2018-09-13  2:22 Stephen Rothwell
  2018-09-13  7:17 ` Angelo Dureghello
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2018-09-13  2:22 UTC (permalink / raw)
  To: Vinod Koul
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Angelo Dureghello, Krzysztof Kozlowski


[-- Attachment #1: Type: text/plain, Size: 511 bytes --]

Hi Vinod,

After merging the slave-dma tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/dma/mcf-edma.c: In function 'mcf_edma_filter_fn':
drivers/dma/mcf-edma.c:296:33: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
   return (mcf_chan->slave_id == (u32)param);
                                 ^

Introduced by commit

  e7a3ff92eaf1 ("dmaengine: fsl-edma: add ColdFire mcf5441x edma support")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2018-11-26  5:59 Stephen Rothwell
  2018-11-26  8:09 ` Vinod Koul
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2018-11-26  5:59 UTC (permalink / raw)
  To: Vinod Koul; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, Jia-Ju Bai


[-- Attachment #1: Type: text/plain, Size: 437 bytes --]

Hi Vinod,

After merging the slave-dma tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

drivers/dma/coh901318.c: In function 'coh901318_config':
drivers/dma/coh901318.c:1805:16: warning: unused variable 'flags' [-Wunused-variable]
  unsigned long flags;
                ^~~~~

Introduced by commit

  627469e4445b ("dmaengine: coh901318: Fix a double-lock bug")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2019-08-12  7:14 Stephen Rothwell
  2019-08-12  9:14 ` Arnd Bergmann
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2019-08-12  7:14 UTC (permalink / raw)
  To: Vinod Koul
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Arnd Bergmann


[-- Attachment #1: Type: text/plain, Size: 455 bytes --]

Hi all,

After merging the slave-dma tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

WARNING: "omap_dma_filter_fn" [vmlinux] is a static EXPORT_SYMBOL_GPL
WARNING: "edma_filter_fn" [vmlinux] is a static EXPORT_SYMBOL

Introduced by commits

  9c71b9eb3cb2 ("dmaengine: omap-dma: make omap_dma_filter_fn private")
  d2bfe7b5d182 ("dmaengine: edma: make edma_filter_fn private")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread
* linux-next: build warning after merge of the slave-dma tree
@ 2020-02-14  0:00 Stephen Rothwell
  2020-02-14  4:41 ` Vinod Koul
  0 siblings, 1 reply; 17+ messages in thread
From: Stephen Rothwell @ 2020-02-14  0:00 UTC (permalink / raw)
  To: Vinod Koul; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, YueHaibing


[-- Attachment #1: Type: text/plain, Size: 822 bytes --]

Hi all,

After merging the slave-dma tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

drivers/dma/sun4i-dma.c: In function 'sun4i_dma_prep_dma_cyclic':
drivers/dma/sun4i-dma.c:30:51: warning: statement with no effect [-Wunused-value]
   30 | #define SUN4I_DMA_CFG_SRC_ADDR_MODE(mode) ((mode) << 5)
      |                                           ~~~~~~~~^~~~~
drivers/dma/sun4i-dma.c:701:8: note: in expansion of macro 'SUN4I_DMA_CFG_SRC_ADDR_MODE'
  701 |        SUN4I_DMA_CFG_SRC_ADDR_MODE(linear_mode);
      |        ^~~~~~~~~~~~~~~~~~~~~~~~~~~

Introduced by commit

  6ebb827f7aad ("dmaengine: sun4i: use 'linear_mode' in sun4i_dma_prep_dma_cyclic")

Please do not ignore/dismiss new warnings ... this one points out a real issue.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, back to index

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-30  6:20 linux-next: build warning after merge of the slave-dma tree Stephen Rothwell
2015-03-30  6:26 ` Appana Durga Kedareswara Rao
2016-06-09  1:18 Stephen Rothwell
2016-06-09  6:04 ` Appana Durga Kedareswara Rao
2018-07-04  3:30 Stephen Rothwell
2018-07-04  4:20 ` Vinod
2018-07-05  2:47   ` Stephen Rothwell
2018-07-05  5:01     ` Vinod
2018-09-13  2:22 Stephen Rothwell
2018-09-13  7:17 ` Angelo Dureghello
2018-11-26  5:59 Stephen Rothwell
2018-11-26  8:09 ` Vinod Koul
2019-08-12  7:14 Stephen Rothwell
2019-08-12  9:14 ` Arnd Bergmann
2019-08-12  9:21   ` Stephen Rothwell
2020-02-14  0:00 Stephen Rothwell
2020-02-14  4:41 ` Vinod Koul

Linux-Next Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-next/0 linux-next/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-next linux-next/ https://lore.kernel.org/linux-next \
		linux-next@vger.kernel.org
	public-inbox-index linux-next

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-next


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git