All of lore.kernel.org
 help / color / mirror / Atom feed
* linux-next: build warning after merge of the spi tree
@ 2021-04-06  6:48 Stephen Rothwell
  2021-04-06  7:38 ` Jay Fang
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2021-04-06  6:48 UTC (permalink / raw)
  To: Mark Brown; +Cc: Jay Fang, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

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

drivers/spi/spi-hisi-kunpeng.c: In function 'hisi_spi_transfer_one':
drivers/spi/spi-hisi-kunpeng.c:361:9: warning: conversion from 'long unsigned int' to 'unsigned int' changes value from '18446744073709551600' to '4294967280' [-Woverflow]
  361 |  writel(~IMR_MASK, hs->regs + HISI_SPI_IMR);
      |         ^

Introduced by commit

  c770d8631e18 ("spi: Add HiSilicon SPI Controller Driver for Kunpeng SoCs")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2021-04-06  6:48 linux-next: build warning after merge of the spi tree Stephen Rothwell
@ 2021-04-06  7:38 ` Jay Fang
  0 siblings, 0 replies; 43+ messages in thread
From: Jay Fang @ 2021-04-06  7:38 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown; +Cc: Linux Next Mailing List

On 2021/4/6 14:48, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the spi tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> drivers/spi/spi-hisi-kunpeng.c: In function 'hisi_spi_transfer_one':
> drivers/spi/spi-hisi-kunpeng.c:361:9: warning: conversion from 'long unsigned int' to 'unsigned int' changes value from '18446744073709551600' to '4294967280' [-Woverflow]
>   361 |  writel(~IMR_MASK, hs->regs + HISI_SPI_IMR);
>       |         ^
> 
> Introduced by commit
> 
>   c770d8631e18 ("spi: Add HiSilicon SPI Controller Driver for Kunpeng SoCs")
> 
I will fix this warning ASAP.


Thanks,
Jay



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

* Re: linux-next: build warning after merge of the spi tree
  2024-02-16  3:11 Stephen Rothwell
@ 2024-02-19 11:18 ` Dhruva Gole
  0 siblings, 0 replies; 43+ messages in thread
From: Dhruva Gole @ 2024-02-19 11:18 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Linux Kernel Mailing List, Linux Next Mailing List, Randy Dunlap

Hi,

On 16/02/24 08:41, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the spi tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> Documentation/spi/spi-summary.rst:274: WARNING: Title underline too short.
> 
> Declare target Devices
> ^^^^^^^^^^^^^^^^^^^^^
> 
> Introduced by commit
> 
>    99769a52464d ("spi: Update the "master/slave" terminology in documentation")
> 

This is a known issue with a Fix pending to be merged [0]. Thanks for
reporting.

[0] 
https://lore.kernel.org/linux-spi/20240216051637.10920-1-rdunlap@infradead.org/

-- 
Thanks and Regards,
Dhruva Gole

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

* linux-next: build warning after merge of the spi tree
@ 2024-02-16  3:11 Stephen Rothwell
  2024-02-19 11:18 ` Dhruva Gole
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2024-02-16  3:11 UTC (permalink / raw)
  To: Mark Brown
  Cc: Dhruva Gole, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

After merging the spi tree, today's linux-next build (htmldocs) produced
this warning:

Documentation/spi/spi-summary.rst:274: WARNING: Title underline too short.

Declare target Devices
^^^^^^^^^^^^^^^^^^^^^

Introduced by commit

  99769a52464d ("spi: Update the "master/slave" terminology in documentation")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2024-02-08 13:32 ` Amadeusz Sławiński
@ 2024-02-08 21:59   ` Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2024-02-08 21:59 UTC (permalink / raw)
  To: Amadeusz Sławiński
  Cc: Mark Brown, Cezary Rojewski, Linux Kernel Mailing List,
	Linux Next Mailing List

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

Hi Amadeusz,

On Thu, 8 Feb 2024 14:32:15 +0100 Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com> wrote:
>
> Ah! The avs path should've been:
> /sys/devices/pci0000:00/<dev>/avs/fw_version
> Will "avs/" in path this fix the error?

Presumably, since the path string will then be unique.

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2024-02-08  3:52 Stephen Rothwell
  2024-02-08 11:43 ` Mark Brown
@ 2024-02-08 13:32 ` Amadeusz Sławiński
  2024-02-08 21:59   ` Stephen Rothwell
  1 sibling, 1 reply; 43+ messages in thread
From: Amadeusz Sławiński @ 2024-02-08 13:32 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Cezary Rojewski, Linux Kernel Mailing List, Linux Next Mailing List

On 2/8/2024 4:52 AM, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the spi tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> Warning: /sys/devices/pci0000:00/<dev>/fw_version is defined 2 times:  Documentation/ABI/testing/sysfs-bus-pci-devices-avs:0  Documentation/ABI/testing/sysfs-bus-pci-devices-catpt:0
> 
> Introduced by commit
> 
>    148b93a5062d ("ASoC: Intel: avs: Expose FW version with sysfs")
> 
> You need to make the "<dev>" part unique.
> 

Ah! The avs path should've been:
/sys/devices/pci0000:00/<dev>/avs/fw_version
Will "avs/" in path this fix the error?
As far as I know there are at least two possible devices, that's why we 
used <dev> in path.

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

* Re: linux-next: build warning after merge of the spi tree
  2024-02-08  3:52 Stephen Rothwell
@ 2024-02-08 11:43 ` Mark Brown
  2024-02-08 13:32 ` Amadeusz Sławiński
  1 sibling, 0 replies; 43+ messages in thread
From: Mark Brown @ 2024-02-08 11:43 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Amadeusz Sławiński, Cezary Rojewski,
	Linux Kernel Mailing List, Linux Next Mailing List

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

On Thu, Feb 08, 2024 at 02:52:00PM +1100, Stephen Rothwell wrote:
> Introduced by commit

>   148b93a5062d ("ASoC: Intel: avs: Expose FW version with sysfs")

> You need to make the "<dev>" part unique.

Since it ended up in the wrong tree anyway I'll just drop this.

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

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

* linux-next: build warning after merge of the spi tree
@ 2024-02-08  3:52 Stephen Rothwell
  2024-02-08 11:43 ` Mark Brown
  2024-02-08 13:32 ` Amadeusz Sławiński
  0 siblings, 2 replies; 43+ messages in thread
From: Stephen Rothwell @ 2024-02-08  3:52 UTC (permalink / raw)
  To: Mark Brown
  Cc: Amadeusz Sławiński, Cezary Rojewski,
	Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

After merging the spi tree, today's linux-next build (htmldocs) produced
this warning:

Warning: /sys/devices/pci0000:00/<dev>/fw_version is defined 2 times:  Documentation/ABI/testing/sysfs-bus-pci-devices-avs:0  Documentation/ABI/testing/sysfs-bus-pci-devices-catpt:0

Introduced by commit

  148b93a5062d ("ASoC: Intel: avs: Expose FW version with sysfs")

You need to make the "<dev>" part unique.

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2022-06-08  7:30 ` David Jander
@ 2022-06-08 10:05   ` Mark Brown
  0 siblings, 0 replies; 43+ messages in thread
From: Mark Brown @ 2022-06-08 10:05 UTC (permalink / raw)
  To: David Jander
  Cc: Stephen Rothwell, Linux Kernel Mailing List, Linux Next Mailing List

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

On Wed, Jun 08, 2022 at 09:30:52AM +0200, David Jander wrote:
> Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> > 
> > include/linux/spi/spi.h:82: warning: Function parameter or member 'syncp' not described in 'spi_statistics'
> > include/linux/spi/spi.h:213: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_device'
> > include/linux/spi/spi.h:676: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_controller'

> > Introduced by commit

> >   6598b91b5ac3 ("spi: spi.c: Convert statistics to per-cpu u64_stats_t")

> This is my fault. @Mark: Should/must I do something about this now that it
> already hit the next tree? How to proceed?

It'd be great if you could send an incremental fix adding the
appropriate documentation.

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

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

* Re: linux-next: build warning after merge of the spi tree
  2022-06-08  4:30 Stephen Rothwell
@ 2022-06-08  7:30 ` David Jander
  2022-06-08 10:05   ` Mark Brown
  0 siblings, 1 reply; 43+ messages in thread
From: David Jander @ 2022-06-08  7:30 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Mark Brown, Linux Kernel Mailing List, Linux Next Mailing List


Hi Mark, Stephen,

On Wed, 8 Jun 2022 14:30:38 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> After merging the spi tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> include/linux/spi/spi.h:82: warning: Function parameter or member 'syncp' not described in 'spi_statistics'
> include/linux/spi/spi.h:213: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_device'
> include/linux/spi/spi.h:676: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_controller'
> 
> Introduced by commit
> 
>   6598b91b5ac3 ("spi: spi.c: Convert statistics to per-cpu u64_stats_t")
> 

This is my fault. @Mark: Should/must I do something about this now that it
already hit the next tree? How to proceed?

Best regards,

-- 
David Jander

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

* linux-next: build warning after merge of the spi tree
@ 2022-06-08  4:30 Stephen Rothwell
  2022-06-08  7:30 ` David Jander
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2022-06-08  4:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: David Jander, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

After merging the spi tree, today's linux-next build (htmldocs) produced
this warning:

include/linux/spi/spi.h:82: warning: Function parameter or member 'syncp' not described in 'spi_statistics'
include/linux/spi/spi.h:213: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_device'
include/linux/spi/spi.h:676: warning: Function parameter or member 'pcpu_statistics' not described in 'spi_controller'

Introduced by commit

  6598b91b5ac3 ("spi: spi.c: Convert statistics to per-cpu u64_stats_t")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2021-08-10  6:29 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2021-08-10  6:29 UTC (permalink / raw)
  To: Mark Brown
  Cc: Zhengxun Li, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

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

/home/sfr/next/next/drivers/spi/spi-mxic.c: In function 'mxic_spi_mem_exec_op':
/home/sfr/next/next/drivers/spi/spi-mxic.c:401:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
  401 |   if (op->data.dir == SPI_MEM_DATA_IN)
      |   ^~
/home/sfr/next/next/drivers/spi/spi-mxic.c:403:4: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
  403 |    if (op->data.dtr)
      |    ^~

Introduced by commit

  d05aaa66ba3c ("spi: mxic: patch for octal DTR mode support")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2021-08-09  5:30 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2021-08-09  5:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: Mason Zhang, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

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

In file included from include/linux/kernel.h:16,
                 from include/linux/clk.h:13,
                 from drivers/spi/spi-mt65xx.c:7:
drivers/spi/spi-mt65xx.c: In function 'mtk_spi_set_hw_cs_timing':
include/linux/minmax.h:20:28: warning: comparison of distinct pointer types lacks a cast
   20 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:26:4: note: in expansion of macro '__typecheck'
   26 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:36:24: note: in expansion of macro '__safe_cmp'
   36 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:45:19: note: in expansion of macro '__careful_cmp'
   45 | #define min(x, y) __careful_cmp(x, y, <)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-mt65xx.c:242:10: note: in expansion of macro 'min'
  242 |   hold = min(hold, 0xffff);
      |          ^~~
include/linux/minmax.h:20:28: warning: comparison of distinct pointer types lacks a cast
   20 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:26:4: note: in expansion of macro '__typecheck'
   26 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:36:24: note: in expansion of macro '__safe_cmp'
   36 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:45:19: note: in expansion of macro '__careful_cmp'
   45 | #define min(x, y) __careful_cmp(x, y, <)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-mt65xx.c:243:11: note: in expansion of macro 'min'
  243 |   setup = min(setup, 0xffff);
      |           ^~~
include/linux/minmax.h:20:28: warning: comparison of distinct pointer types lacks a cast
   20 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:26:4: note: in expansion of macro '__typecheck'
   26 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:36:24: note: in expansion of macro '__safe_cmp'
   36 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:45:19: note: in expansion of macro '__careful_cmp'
   45 | #define min(x, y) __careful_cmp(x, y, <)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-mt65xx.c:251:10: note: in expansion of macro 'min'
  251 |   hold = min(hold, 0xff);
      |          ^~~
include/linux/minmax.h:20:28: warning: comparison of distinct pointer types lacks a cast
   20 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:26:4: note: in expansion of macro '__typecheck'
   26 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:36:24: note: in expansion of macro '__safe_cmp'
   36 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:45:19: note: in expansion of macro '__careful_cmp'
   45 | #define min(x, y) __careful_cmp(x, y, <)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-mt65xx.c:252:11: note: in expansion of macro 'min'
  252 |   setup = min(setup, 0xff);
      |           ^~~
include/linux/minmax.h:20:28: warning: comparison of distinct pointer types lacks a cast
   20 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:26:4: note: in expansion of macro '__typecheck'
   26 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:36:24: note: in expansion of macro '__safe_cmp'
   36 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:45:19: note: in expansion of macro '__careful_cmp'
   45 | #define min(x, y) __careful_cmp(x, y, <)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-mt65xx.c:261:13: note: in expansion of macro 'min'
  261 |  inactive = min(inactive, 0xff);
      |             ^~~

Introduced by commit

  04e6bb0d6bb1 ("spi: modify set_cs_timing parameter")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2021-04-20  3:48 Stephen Rothwell
@ 2021-04-20  7:56 ` Patrice CHOTARD
  0 siblings, 0 replies; 43+ messages in thread
From: Patrice CHOTARD @ 2021-04-20  7:56 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Linux Kernel Mailing List, Linux Next Mailing List

Hi Mark

I will send a patch to fix this warning, sorry for that.

Patrice.

On 4/20/21 5:48 AM, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the spi tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> In file included from include/linux/printk.h:409,
>                  from include/linux/kernel.h:16,
>                  from include/linux/clk.h:13,
>                  from drivers/spi/spi-stm32-qspi.c:7:
> drivers/spi/spi-stm32-qspi.c: In function 'stm32_qspi_dirmap_read':
> drivers/spi/spi-stm32-qspi.c:481:21: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
>   481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
>       |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
>   129 |   func(&id, ##__VA_ARGS__);  \
>       |               ^~~~~~~~~~~
> include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
>   161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
>       |  ^~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
>   123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
>       |  ^~~~~~~~~~~~~~~
> include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
>   123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
>       |                       ^~~~~~~
> drivers/spi/spi-stm32-qspi.c:481:2: note: in expansion of macro 'dev_dbg'
>   481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
>       |  ^~~~~~~
> drivers/spi/spi-stm32-qspi.c:481:34: note: format string is defined here
>   481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
>       |                                 ~^
>       |                                  |
>       |                                  unsigned int
>       |                                 %lx
> 
> Introduced by commit
> 
>   18674dee3cd6 ("spi: stm32-qspi: Add dirmap support")
> 

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

* linux-next: build warning after merge of the spi tree
@ 2021-04-20  3:48 Stephen Rothwell
  2021-04-20  7:56 ` Patrice CHOTARD
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2021-04-20  3:48 UTC (permalink / raw)
  To: Mark Brown
  Cc: Patrice Chotard, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

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

In file included from include/linux/printk.h:409,
                 from include/linux/kernel.h:16,
                 from include/linux/clk.h:13,
                 from drivers/spi/spi-stm32-qspi.c:7:
drivers/spi/spi-stm32-qspi.c: In function 'stm32_qspi_dirmap_read':
drivers/spi/spi-stm32-qspi.c:481:21: warning: format '%x' expects argument of type 'unsigned int', but argument 5 has type 'size_t' {aka 'long unsigned int'} [-Wformat=]
  481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
      |                     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:129:15: note: in definition of macro '__dynamic_func_call'
  129 |   func(&id, ##__VA_ARGS__);  \
      |               ^~~~~~~~~~~
include/linux/dynamic_debug.h:161:2: note: in expansion of macro '_dynamic_func_call'
  161 |  _dynamic_func_call(fmt,__dynamic_dev_dbg,   \
      |  ^~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:123:2: note: in expansion of macro 'dynamic_dev_dbg'
  123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
      |  ^~~~~~~~~~~~~~~
include/linux/dev_printk.h:123:23: note: in expansion of macro 'dev_fmt'
  123 |  dynamic_dev_dbg(dev, dev_fmt(fmt), ##__VA_ARGS__)
      |                       ^~~~~~~
drivers/spi/spi-stm32-qspi.c:481:2: note: in expansion of macro 'dev_dbg'
  481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
      |  ^~~~~~~
drivers/spi/spi-stm32-qspi.c:481:34: note: format string is defined here
  481 |  dev_dbg(qspi->dev, "%s len = 0x%x offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
      |                                 ~^
      |                                  |
      |                                  unsigned int
      |                                 %lx

Introduced by commit

  18674dee3cd6 ("spi: stm32-qspi: Add dirmap support")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2021-01-11  1:09 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2021-01-11  1:09 UTC (permalink / raw)
  To: Mark Brown; +Cc: Yanteng Si, Linux Kernel Mailing List, Linux Next Mailing List

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

Hi all,

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

In file included from include/linux/kernel.h:14,
                 from include/linux/clk.h:13,
                 from drivers/spi/spi-cadence-quadspi.c:9:
drivers/spi/spi-cadence-quadspi.c: In function 'cqspi_direct_read_execute':
include/linux/minmax.h:18:28: warning: comparison of distinct pointer types lacks a cast
   18 |  (!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
      |                            ^~
include/linux/minmax.h:32:4: note: in expansion of macro '__typecheck'
   32 |   (__typecheck(x, y) && __no_side_effects(x, y))
      |    ^~~~~~~~~~~
include/linux/minmax.h:42:24: note: in expansion of macro '__safe_cmp'
   42 |  __builtin_choose_expr(__safe_cmp(x, y), \
      |                        ^~~~~~~~~~
include/linux/minmax.h:58:19: note: in expansion of macro '__careful_cmp'
   58 | #define max(x, y) __careful_cmp(x, y, >)
      |                   ^~~~~~~~~~~~~
drivers/spi/spi-cadence-quadspi.c:1153:24: note: in expansion of macro 'max'
 1153 |       msecs_to_jiffies(max(len, 500U)))) {
      |                        ^~~

Introduced by commit

  8728a81b8f10 ("spi: Fix distinct pointer types warning for ARCH=mips")

-- 
Cheers,
Stephen Rothwell

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

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

* RE: linux-next: build warning after merge of the spi tree
  2021-01-04  0:06 Stephen Rothwell
@ 2021-01-04  7:56 ` Ardelean, Alexandru
  0 siblings, 0 replies; 43+ messages in thread
From: Ardelean, Alexandru @ 2021-01-04  7:56 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Andy Shevchenko, Linux Kernel Mailing List, Linux Next Mailing List



> -----Original Message-----
> From: Stephen Rothwell <sfr@canb.auug.org.au>
> Sent: Monday, January 4, 2021 2:07 AM
> To: Mark Brown <broonie@kernel.org>
> Cc: Ardelean, Alexandru <alexandru.Ardelean@analog.com>; Andy Shevchenko
> <andy.shevchenko@gmail.com>; Linux Kernel Mailing List <linux-
> kernel@vger.kernel.org>; Linux Next Mailing List <linux-next@vger.kernel.org>
> Subject: linux-next: build warning after merge of the spi tree
> 
> [External]
> 
> Hi all,
> 
> After merging the spi tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
> 

Apologies for this.
Will send a fix.


> In file included from include/linux/device.h:15,
>                  from include/linux/dmaengine.h:8,
>                  from drivers/spi/spi-stm32.c:11:
> drivers/spi/spi-stm32.c: In function 'stm32_spi_prepare_msg':
> drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type
> 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
>    19 | #define dev_fmt(fmt) fmt
>       |                      ^~~
> drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |  ^~~~~~~
> drivers/spi/spi-stm32.c:1030:27: note: format string is defined here
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                          ~^
>       |                           |
>       |                           int
>       |                          %ld
> In file included from include/linux/device.h:15,
>                  from include/linux/dmaengine.h:8,
>                  from drivers/spi/spi-stm32.c:11:
> drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type
> 'int', but argument 5 has type 'long unsigned int' [-Wformat=]
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
>    19 | #define dev_fmt(fmt) fmt
>       |                      ^~~
> drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |  ^~~~~~~
> drivers/spi/spi-stm32.c:1030:35: note: format string is defined here
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                                  ~^
>       |                                   |
>       |                                   int
>       |                                  %ld
> In file included from include/linux/device.h:15,
>                  from include/linux/dmaengine.h:8,
>                  from drivers/spi/spi-stm32.c:11:
> drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type
> 'int', but argument 6 has type 'long unsigned int' [-Wformat=]
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
>    19 | #define dev_fmt(fmt) fmt
>       |                      ^~~
> drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |  ^~~~~~~
> drivers/spi/spi-stm32.c:1030:48: note: format string is defined here
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                                               ~^
>       |                                                |
>       |                                                int
>       |                                               %ld
> In file included from include/linux/device.h:15,
>                  from include/linux/dmaengine.h:8,
>                  from drivers/spi/spi-stm32.c:11:
> drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type
> 'int', but argument 7 has type 'long unsigned int' [-Wformat=]
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
>    19 | #define dev_fmt(fmt) fmt
>       |                      ^~~
> drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |  ^~~~~~~
> drivers/spi/spi-stm32.c:1030:59: note: format string is defined here
>  1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
>       |                                                          ~^
>       |                                                           |
>       |                                                           int
>       |                                                          %ld
> 
> Introduced by commit
> 
>   f7005142dace ("spi: uapi: unify SPI modes into a single spi.h header")
> 
> --
> Cheers,
> Stephen Rothwell

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

* linux-next: build warning after merge of the spi tree
@ 2021-01-04  0:06 Stephen Rothwell
  2021-01-04  7:56 ` Ardelean, Alexandru
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2021-01-04  0:06 UTC (permalink / raw)
  To: Mark Brown
  Cc: Alexandru Ardelean, Andy Shevchenko, Linux Kernel Mailing List,
	Linux Next Mailing List

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

Hi all,

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

In file included from include/linux/device.h:15,
                 from include/linux/dmaengine.h:8,
                 from drivers/spi/spi-stm32.c:11:
drivers/spi/spi-stm32.c: In function 'stm32_spi_prepare_msg':
drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long unsigned int' [-Wformat=]
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
   19 | #define dev_fmt(fmt) fmt
      |                      ^~~
drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |  ^~~~~~~
drivers/spi/spi-stm32.c:1030:27: note: format string is defined here
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                          ~^
      |                           |
      |                           int
      |                          %ld
In file included from include/linux/device.h:15,
                 from include/linux/dmaengine.h:8,
                 from drivers/spi/spi-stm32.c:11:
drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type 'int', but argument 5 has type 'long unsigned int' [-Wformat=]
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
   19 | #define dev_fmt(fmt) fmt
      |                      ^~~
drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |  ^~~~~~~
drivers/spi/spi-stm32.c:1030:35: note: format string is defined here
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                                  ~^
      |                                   |
      |                                   int
      |                                  %ld
In file included from include/linux/device.h:15,
                 from include/linux/dmaengine.h:8,
                 from drivers/spi/spi-stm32.c:11:
drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int' [-Wformat=]
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
   19 | #define dev_fmt(fmt) fmt
      |                      ^~~
drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |  ^~~~~~~
drivers/spi/spi-stm32.c:1030:48: note: format string is defined here
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                                               ~^
      |                                                |
      |                                                int
      |                                               %ld
In file included from include/linux/device.h:15,
                 from include/linux/dmaengine.h:8,
                 from drivers/spi/spi-stm32.c:11:
drivers/spi/spi-stm32.c:1030:20: warning: format '%d' expects argument of type 'int', but argument 7 has type 'long unsigned int' [-Wformat=]
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:19:22: note: in definition of macro 'dev_fmt'
   19 | #define dev_fmt(fmt) fmt
      |                      ^~~
drivers/spi/spi-stm32.c:1030:2: note: in expansion of macro 'dev_dbg'
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |  ^~~~~~~
drivers/spi/spi-stm32.c:1030:59: note: format string is defined here
 1030 |  dev_dbg(spi->dev, "cpol=%d cpha=%d lsb_first=%d cs_high=%d\n",
      |                                                          ~^
      |                                                           |
      |                                                           int
      |                                                          %ld

Introduced by commit

  f7005142dace ("spi: uapi: unify SPI modes into a single spi.h header")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2020-07-09  4:10 Stephen Rothwell
@ 2020-07-09  5:29 ` Peng Fan
  0 siblings, 0 replies; 43+ messages in thread
From: Peng Fan @ 2020-07-09  5:29 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List

Very sorry for that, I will re-send v2 later.

&quot;Stephen Rothwell&quot; &lt;sfr@canb.auug.org.au&gt;wrote:
> Hi all,
> 
> After merging the spi tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
> 
> drivers/spi/spi-atmel.c: In function 'atmel_spi_probe':
> drivers/spi/spi-atmel.c:1680:1: warning: label 'out_free' defined but not used [-Wunused-label]
>  1680 | out_free:
>       | ^~~~~~~~
> 
> Introduced by commit
> 
>   2d9a744685bc ("spi: atmel: No need to call spi_master_put() if spi_alloc_master() failed")
> 
> -- 
> Cheers,
> Stephen Rothwell

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

* linux-next: build warning after merge of the spi tree
@ 2020-07-09  4:10 Stephen Rothwell
  2020-07-09  5:29 ` Peng Fan
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2020-07-09  4:10 UTC (permalink / raw)
  To: Mark Brown; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, Peng Fan

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

Hi all,

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

drivers/spi/spi-atmel.c: In function 'atmel_spi_probe':
drivers/spi/spi-atmel.c:1680:1: warning: label 'out_free' defined but not used [-Wunused-label]
 1680 | out_free:
      | ^~~~~~~~

Introduced by commit

  2d9a744685bc ("spi: atmel: No need to call spi_master_put() if spi_alloc_master() failed")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2020-04-28  5:01 ` Sanjay R Mehta
@ 2020-04-28  9:38   ` Mark Brown
  0 siblings, 0 replies; 43+ messages in thread
From: Mark Brown @ 2020-04-28  9:38 UTC (permalink / raw)
  To: Sanjay R Mehta
  Cc: Stephen Rothwell, Linux Next Mailing List,
	Linux Kernel Mailing List, Sanjay R Mehta

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

On Tue, Apr 28, 2020 at 10:31:56AM +0530, Sanjay R Mehta wrote:
> Hi Stephen,
> 
> I have just sent a fixup patch for this. Could it still reach 5.8?

Yes, it'll be fine - thanks for sending the fix.

Please don't top post, reply in line with needed context.  This allows
readers to readily follow the flow of conversation and understand what
you are talking about and also helps ensure that everything in the
discussion is being addressed.

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

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

* Re: linux-next: build warning after merge of the spi tree
  2020-04-28  4:16 Stephen Rothwell
@ 2020-04-28  5:01 ` Sanjay R Mehta
  2020-04-28  9:38   ` Mark Brown
  0 siblings, 1 reply; 43+ messages in thread
From: Sanjay R Mehta @ 2020-04-28  5:01 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sanjay R Mehta

Hi Stephen,

I have just sent a fixup patch for this. Could it still reach 5.8?

Thanks & regards
Sanjay Mehta

On 4/28/2020 9:46 AM, Stephen Rothwell wrote:
> [CAUTION: External Email]
> 

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

* linux-next: build warning after merge of the spi tree
@ 2020-04-28  4:16 Stephen Rothwell
  2020-04-28  5:01 ` Sanjay R Mehta
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2020-04-28  4:16 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Sanjay R Mehta

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

Hi all,

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

drivers/spi/spi-amd.c: In function 'amd_spi_fifo_xfer':
drivers/spi/spi-amd.c:171:17: warning: unused variable 'opcode' [-Wunused-variable]
  171 |  u8 cmd_opcode, opcode = 0;
      |                 ^~~~~~

Introduced by commit

  bbb336f39efc ("spi: spi-amd: Add AMD SPI controller driver support")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2020-01-15 10:27   ` Stephen Rothwell
@ 2020-01-15 11:38     ` Jean Pihet
  0 siblings, 0 replies; 43+ messages in thread
From: Jean Pihet @ 2020-01-15 11:38 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List

Stephen,

On Wed, Jan 15, 2020 at 11:28 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi Jean,
>
> On Wed, 15 Jan 2020 10:10:09 +0100 Jean Pihet <jean.pihet@newoldbits.com> wrote:
> >
> > I did not have this warning, it may be a combination of compiler
> > version and build flags. Do you need a fixup patch for it?
>
> That is up to Mark, really.  Unfortunately, Linus will probably get
> this warning during the merge window which he will complain about.

Ok I just sent a fixup patch to Mark (on spi ML). Could it still reach 5.6?

>
> It is really weird, as there are three other references to rx_wlen
> immediately before the one complained about ... so maybe this is a bug
> in the compiler I am using (gcc 9.2.1 from Debian, cross compiler
> ppc64le hosted).
Yes indeed it is weird. Only some compiler versions are throwing the warning.

Thanks,
Jean

>
> --
> Cheers,
> Stephen Rothwell

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

* Re: linux-next: build warning after merge of the spi tree
  2020-01-15  9:10 ` Jean Pihet
@ 2020-01-15 10:27   ` Stephen Rothwell
  2020-01-15 11:38     ` Jean Pihet
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2020-01-15 10:27 UTC (permalink / raw)
  To: Jean Pihet; +Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List

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

Hi Jean,

On Wed, 15 Jan 2020 10:10:09 +0100 Jean Pihet <jean.pihet@newoldbits.com> wrote:
>
> I did not have this warning, it may be a combination of compiler
> version and build flags. Do you need a fixup patch for it?

That is up to Mark, really.  Unfortunately, Linus will probably get
this warning during the merge window which he will complain about.

It is really weird, as there are three other references to rx_wlen
immediately before the one complained about ... so maybe this is a bug
in the compiler I am using (gcc 9.2.1 from Debian, cross compiler
ppc64le hosted).

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2020-01-15  2:23 Stephen Rothwell
@ 2020-01-15  9:10 ` Jean Pihet
  2020-01-15 10:27   ` Stephen Rothwell
  0 siblings, 1 reply; 43+ messages in thread
From: Jean Pihet @ 2020-01-15  9:10 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List

Hi Stephen,

I did not have this warning, it may be a combination of compiler
version and build flags. Do you need a fixup patch for it?

Thanks,
Jean

On Wed, Jan 15, 2020 at 3:23 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi all,
>
> After merging the spi tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
>
> drivers/spi/spi-ti-qspi.c: In function 'ti_qspi_start_transfer_one':
> drivers/spi/spi-ti-qspi.c:392:8: warning: 'rx_wlen' may be used uninitialized in this function [-Wmaybe-uninitialized]
>   392 |     if (rx_wlen >= 32)
>       |        ^
> drivers/spi/spi-ti-qspi.c:318:12: note: 'rx_wlen' was declared here
>   318 |  u8 rxlen, rx_wlen;
>       |            ^~~~~~~
>
> Introduced by commit
>
>   e7cc5cfbea4c ("spi: spi-ti-qspi: optimize byte-transfers")
>
> Looks like it may be a false positive.
>
> --
> Cheers,
> Stephen Rothwell

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

* linux-next: build warning after merge of the spi tree
@ 2020-01-15  2:23 Stephen Rothwell
  2020-01-15  9:10 ` Jean Pihet
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2020-01-15  2:23 UTC (permalink / raw)
  To: Mark Brown; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, Jean Pihet

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

Hi all,

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

drivers/spi/spi-ti-qspi.c: In function 'ti_qspi_start_transfer_one':
drivers/spi/spi-ti-qspi.c:392:8: warning: 'rx_wlen' may be used uninitialized in this function [-Wmaybe-uninitialized]
  392 |     if (rx_wlen >= 32)
      |        ^
drivers/spi/spi-ti-qspi.c:318:12: note: 'rx_wlen' was declared here
  318 |  u8 rxlen, rx_wlen;
      |            ^~~~~~~

Introduced by commit

  e7cc5cfbea4c ("spi: spi-ti-qspi: optimize byte-transfers")

Looks like it may be a false positive.

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2019-04-29  4:30 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2019-04-29  4:30 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Linus Walleij

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

Hi Mark,

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

drivers/spi/spi-ep93xx.c: In function 'ep93xx_spi_probe':
drivers/spi/spi-ep93xx.c:654:6: warning: unused variable 'i' [-Wunused-variable]
  int i;
      ^

Introduced by commit

  06a391b1621e ("spi: ep93xx: Convert to use CS GPIO descriptors")

-- 
Cheers,
Stephen Rothwell

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

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

* RE: linux-next: build warning after merge of the spi tree
  2019-01-29  8:26   ` Stephen Rothwell
@ 2019-01-29  8:39     ` Yogesh Narayan Gaur
  0 siblings, 0 replies; 43+ messages in thread
From: Yogesh Narayan Gaur @ 2019-01-29  8:39 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List,
	Frieder Schrempf, Boris Brezillon

Hi Stephen,

> -----Original Message-----
> From: Stephen Rothwell [mailto:sfr@canb.auug.org.au]
> Sent: Tuesday, January 29, 2019 1:57 PM
> To: Yogesh Narayan Gaur <yogeshnarayan.gaur@nxp.com>
> Cc: Mark Brown <broonie@kernel.org>; Linux Next Mailing List <linux-
> next@vger.kernel.org>; Linux Kernel Mailing List <linux-kernel@vger.kernel.org>;
> Frieder Schrempf <frieder.schrempf@kontron.de>; Boris Brezillon
> <bbrezillon@kernel.org>
> Subject: Re: linux-next: build warning after merge of the spi tree
> 
> Hi Yogesh,
> 
> On Tue, 29 Jan 2019 06:57:51 +0000 Yogesh Narayan Gaur
> <yogeshnarayan.gaur@nxp.com> wrote:
> >
> > > WARNING: modpost: missing MODULE_LICENSE() in
> > > drivers/spi/spi-nxp-fspi.o see include/linux/module.h for more
> > > information
> > >
> > Can someone help in pointing out that why above warning message is coming
> for mentioned commit.
> > Below MODULE_LICENSE string is present in the above mentioned file,
> drivers/spi/spi-nxp-fspi.c.
> >     MODULE_LICENSE("GPL v2");
> 
> The version in linux-next has no such line.
> 
Yes, somehow this missing line was appended to the commit message instead of the actual in patchwork, also being pointed out by Frieder in his reply.
Sorry for this.

I have send fix patch for this warning message [1]. Please check the patch and apply.

--
Regards,
Yogesh Gaur

[1] https://patchwork.ozlabs.org/patch/1032586/

> --
> Cheers,
> Stephen Rothwell

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

* Re: linux-next: build warning after merge of the spi tree
  2019-01-29  6:57 ` Yogesh Narayan Gaur
  2019-01-29  7:25   ` Schrempf Frieder
@ 2019-01-29  8:26   ` Stephen Rothwell
  2019-01-29  8:39     ` Yogesh Narayan Gaur
  1 sibling, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2019-01-29  8:26 UTC (permalink / raw)
  To: Yogesh Narayan Gaur
  Cc: Mark Brown, Linux Next Mailing List, Linux Kernel Mailing List,
	Frieder Schrempf, Boris Brezillon

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

Hi Yogesh,

On Tue, 29 Jan 2019 06:57:51 +0000 Yogesh Narayan Gaur <yogeshnarayan.gaur@nxp.com> wrote:
>
> > WARNING: modpost: missing MODULE_LICENSE() in drivers/spi/spi-nxp-fspi.o
> > see include/linux/module.h for more information
> >   
> Can someone help in pointing out that why above warning message is coming for mentioned commit.
> Below MODULE_LICENSE string is present in the above mentioned file, drivers/spi/spi-nxp-fspi.c.
>     MODULE_LICENSE("GPL v2");

The version in linux-next has no such line.

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2019-01-29  6:57 ` Yogesh Narayan Gaur
@ 2019-01-29  7:25   ` Schrempf Frieder
  2019-01-29  8:26   ` Stephen Rothwell
  1 sibling, 0 replies; 43+ messages in thread
From: Schrempf Frieder @ 2019-01-29  7:25 UTC (permalink / raw)
  To: Yogesh Narayan Gaur, Stephen Rothwell, Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Boris Brezillon

On 29.01.19 07:57, Yogesh Narayan Gaur wrote:
> Hi,
> 
>> -----Original Message-----
>> From: Stephen Rothwell [mailto:sfr@canb.auug.org.au]
>> Sent: Tuesday, January 29, 2019 7:33 AM
>> To: Mark Brown <broonie@kernel.org>
>> Cc: Linux Next Mailing List <linux-next@vger.kernel.org>; Linux Kernel Mailing
>> List <linux-kernel@vger.kernel.org>; Yogesh Narayan Gaur
>> <yogeshnarayan.gaur@nxp.com>; Frieder Schrempf
>> <frieder.schrempf@kontron.de>; Boris Brezillon <bbrezillon@kernel.org>;
>> Ashish Kumar <ashish.kumar@nxp.com>
>> Subject: linux-next: build warning after merge of the spi tree
>>
>> Hi all,
>>
>> After merging the spi tree, today's linux-next build (KCONFIG_NAME) produced
>> this warning:
>>
>> WARNING: modpost: missing MODULE_LICENSE() in drivers/spi/spi-nxp-fspi.o
>> see include/linux/module.h for more information
>>
> Can someone help in pointing out that why above warning message is coming for mentioned commit.
> Below MODULE_LICENSE string is present in the above mentioned file, drivers/spi/spi-nxp-fspi.c.
>      MODULE_LICENSE("GPL v2");

No, it is not present. Look at your patches [1] or at spi-next [2]. 
MODULE_LICENSE is definitely missing.

In patchwork the missing line was appended to the commit message instead 
of the actual patch for some reason.

[1] https://patchwork.kernel.org/patch/10764347/
[2] 
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git/commit/?h=for-next&id=a5356aef6a907c2e2aed0caaa2b88b6021394471

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

* RE: linux-next: build warning after merge of the spi tree
  2019-01-29  2:02 Stephen Rothwell
@ 2019-01-29  6:57 ` Yogesh Narayan Gaur
  2019-01-29  7:25   ` Schrempf Frieder
  2019-01-29  8:26   ` Stephen Rothwell
  0 siblings, 2 replies; 43+ messages in thread
From: Yogesh Narayan Gaur @ 2019-01-29  6:57 UTC (permalink / raw)
  To: Stephen Rothwell, Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List,
	Frieder Schrempf, Boris Brezillon

Hi,

> -----Original Message-----
> From: Stephen Rothwell [mailto:sfr@canb.auug.org.au]
> Sent: Tuesday, January 29, 2019 7:33 AM
> To: Mark Brown <broonie@kernel.org>
> Cc: Linux Next Mailing List <linux-next@vger.kernel.org>; Linux Kernel Mailing
> List <linux-kernel@vger.kernel.org>; Yogesh Narayan Gaur
> <yogeshnarayan.gaur@nxp.com>; Frieder Schrempf
> <frieder.schrempf@kontron.de>; Boris Brezillon <bbrezillon@kernel.org>;
> Ashish Kumar <ashish.kumar@nxp.com>
> Subject: linux-next: build warning after merge of the spi tree
> 
> Hi all,
> 
> After merging the spi tree, today's linux-next build (KCONFIG_NAME) produced
> this warning:
> 
> WARNING: modpost: missing MODULE_LICENSE() in drivers/spi/spi-nxp-fspi.o
> see include/linux/module.h for more information
> 
Can someone help in pointing out that why above warning message is coming for mentioned commit.
Below MODULE_LICENSE string is present in the above mentioned file, drivers/spi/spi-nxp-fspi.c.
    MODULE_LICENSE("GPL v2");

--
Regards,
Yogesh Gaur
> Introduced by commit
> 
>   a5356aef6a90 ("spi: spi-mem: Add driver for NXP FlexSPI controller")
> 
> --
> Cheers,
> Stephen Rothwell

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

* linux-next: build warning after merge of the spi tree
@ 2019-01-29  2:02 Stephen Rothwell
  2019-01-29  6:57 ` Yogesh Narayan Gaur
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2019-01-29  2:02 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List,
	Yogesh Narayan Gaur, Frieder Schrempf, Boris Brezillon,
	Ashish Kumar

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

Hi all,

After merging the spi tree, today's linux-next build (KCONFIG_NAME)
produced this warning:

WARNING: modpost: missing MODULE_LICENSE() in drivers/spi/spi-nxp-fspi.o
see include/linux/module.h for more information

Introduced by commit

  a5356aef6a90 ("spi: spi-mem: Add driver for NXP FlexSPI controller")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2019-01-10  0:44 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2019-01-10  0:44 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Linus Walleij

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

Hi Mark,

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

drivers/spi/spi-dw.c: In function 'dw_spi_setup':
drivers/spi/spi-dw.c:400:6: warning: unused variable 'ret' [-Wunused-variable]
  int ret;
      ^~~

Introduced by commit

  9400c41e77b8 ("spi: dw: Convert to use CS GPIO descriptors")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2019-01-10  0:41 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2019-01-10  0:41 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Linus Walleij

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

Hi Mark,

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

drivers/spi/spi-davinci.c: In function 'davinci_spi_setup':
drivers/spi/spi-davinci.c:426:21: warning: unused variable 'master' [-Wunused-variable]
  struct spi_master *master = spi->master;
                     ^~~~~~

Introduced by commit

  101a68e74fe0 ("spi: davinci: Convert to use CS GPIO descriptors")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
       [not found] ` <CAP6Zq1gp+hmNkvk8D2uv8FUqTXE4+ZMPhMB7A7vLMaM6swHy4g@mail.gmail.com>
@ 2018-11-15 19:26   ` Mark Brown
  0 siblings, 0 replies; 43+ messages in thread
From: Mark Brown @ 2018-11-15 19:26 UTC (permalink / raw)
  To: Tomer Maimon; +Cc: sfr, linux-next, Linux Kernel Mailing List

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

On Thu, Nov 15, 2018 at 05:56:19PM +0200, Tomer Maimon wrote:
> Hi Mark and Stephen,
> 
> I saw the warning from the kbuilt, in this matter the ioread() function
> will call every time (and the val parameter will used) because rsize can be
> 1 or 2.
> 
> rsize = min(bytes_per_word(priv->bits_per_word), priv->rx_bytes);
> bytes_per_word(priv->bits_per_word)function return 1 or 2.
> npcm_pspi_recv()function will called only if parameter priv->rx_bytes is
> not 0.
> 
> please let me know if you think that I still need to modify the npcm_pspi_recv
> to fix kbuilt warning.

Please, it's going to create noise for people working on the kernel if
they get a spurious warning.

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

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

* linux-next: build warning after merge of the spi tree
@ 2018-11-14  2:18 Stephen Rothwell
       [not found] ` <CAP6Zq1gp+hmNkvk8D2uv8FUqTXE4+ZMPhMB7A7vLMaM6swHy4g@mail.gmail.com>
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2018-11-14  2:18 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List, Tomer Maimon

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

Hi Mark,

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

drivers/spi/spi-npcm-pspi.c: In function 'npcm_pspi_recv':
drivers/spi/spi-npcm-pspi.c:226:17: warning: 'val' may be used uninitialized in this function [-Wmaybe-uninitialized]
   *priv->rx_buf = val;
   ~~~~~~~~~~~~~~^~~~~

Introduced by commit

  2a22f1b30cee ("spi: npcm: add NPCM PSPI controller driver")

-- 
Cheers,
Stephen Rothwell

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

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

* linux-next: build warning after merge of the spi tree
@ 2018-08-03  4:37 Stephen Rothwell
  0 siblings, 0 replies; 43+ messages in thread
From: Stephen Rothwell @ 2018-08-03  4:37 UTC (permalink / raw)
  To: Mark Brown
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Frieder Schrempf, Boris Brezillon

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

Hi Mark,

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

drivers/spi/spi-mem.c: In function 'spi_mem_probe':
drivers/spi/spi-mem.c:375:13: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
   mem->name = ctlr->mem_ops->get_name(mem);
             ^
drivers/spi/spi-mem.c:377:13: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
   mem->name = dev_name(&spi->dev);
             ^

Introduced by commit

  5d27a9c8ea9e ("spi: spi-mem: Extend the SPI mem interface to set a custom memory name")

"const" is good :-)

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the spi tree
  2017-03-27  2:41 Stephen Rothwell
@ 2017-03-27  5:33   ` Vignesh R
  0 siblings, 0 replies; 43+ messages in thread
From: Vignesh R @ 2017-03-27  5:33 UTC (permalink / raw)
  To: Mark Brown; +Cc: Stephen Rothwell, linux-next, linux-kernel



On Monday 27 March 2017 08:11 AM, Stephen Rothwell wrote:
> Hi Mark,
> 
> After merging the spi tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> drivers/spi/spi-ti-qspi.c: In function 'ti_qspi_dma_xfer':
> drivers/spi/spi-ti-qspi.c:398:21: warning: unused variable 'dma_dev' [-Wunused-variable]
>   struct dma_device *dma_dev = chan->device;
>                      ^
> 
> Introduced by commit
> 
>   1351aaeb50b2 ("spi: spi-ti-qspi: Use dma_engine wrapper for dma memcpy call")
> 

Sorry, I have posted a fix for the same.

-- 
Regards
Vignesh

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

* Re: linux-next: build warning after merge of the spi tree
@ 2017-03-27  5:33   ` Vignesh R
  0 siblings, 0 replies; 43+ messages in thread
From: Vignesh R @ 2017-03-27  5:33 UTC (permalink / raw)
  To: Mark Brown; +Cc: Stephen Rothwell, linux-next, linux-kernel



On Monday 27 March 2017 08:11 AM, Stephen Rothwell wrote:
> Hi Mark,
> 
> After merging the spi tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> drivers/spi/spi-ti-qspi.c: In function 'ti_qspi_dma_xfer':
> drivers/spi/spi-ti-qspi.c:398:21: warning: unused variable 'dma_dev' [-Wunused-variable]
>   struct dma_device *dma_dev = chan->device;
>                      ^
> 
> Introduced by commit
> 
>   1351aaeb50b2 ("spi: spi-ti-qspi: Use dma_engine wrapper for dma memcpy call")
> 

Sorry, I have posted a fix for the same.

-- 
Regards
Vignesh

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

* linux-next: build warning after merge of the spi tree
@ 2017-03-27  2:41 Stephen Rothwell
  2017-03-27  5:33   ` Vignesh R
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2017-03-27  2:41 UTC (permalink / raw)
  To: Mark Brown; +Cc: linux-next, linux-kernel, Vignesh R

Hi Mark,

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

drivers/spi/spi-ti-qspi.c: In function 'ti_qspi_dma_xfer':
drivers/spi/spi-ti-qspi.c:398:21: warning: unused variable 'dma_dev' [-Wunused-variable]
  struct dma_device *dma_dev = chan->device;
                     ^

Introduced by commit

  1351aaeb50b2 ("spi: spi-ti-qspi: Use dma_engine wrapper for dma memcpy call")

-- 
Cheers,
Stephen Rothwell

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

* Re: linux-next: build warning after merge of the spi tree
  2010-10-14  1:59 Stephen Rothwell
@ 2010-10-14  5:00 ` Grant Likely
  0 siblings, 0 replies; 43+ messages in thread
From: Grant Likely @ 2010-10-14  5:00 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: linux-next, linux-kernel, Masayuki Ohtake

On Thu, Oct 14, 2010 at 12:59:52PM +1100, Stephen Rothwell wrote:
> Hi Grant,
> 
> After merging the rcu tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> drivers/spi/spi_topcliff_pch.c: In function 'pch_spi_process_messages':
> drivers/spi/spi_topcliff_pch.c:752: warning: 'data' is used uninitialized in this function
> 
> and, indeed it is ...

Oops, so it is.  I messed that up in the refactoring patch.
Fortunately this is a new driver so it is not a regression.  I'll push
out the fix this evening.

g.



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

* linux-next: build warning after merge of the spi tree
@ 2010-10-14  1:59 Stephen Rothwell
  2010-10-14  5:00 ` Grant Likely
  0 siblings, 1 reply; 43+ messages in thread
From: Stephen Rothwell @ 2010-10-14  1:59 UTC (permalink / raw)
  To: Grant Likely; +Cc: linux-next, linux-kernel, Masayuki Ohtake

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

Hi Grant,

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

drivers/spi/spi_topcliff_pch.c: In function 'pch_spi_process_messages':
drivers/spi/spi_topcliff_pch.c:752: warning: 'data' is used uninitialized in this function

and, indeed it is ...
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]

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

end of thread, other threads:[~2024-02-19 11:18 UTC | newest]

Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-06  6:48 linux-next: build warning after merge of the spi tree Stephen Rothwell
2021-04-06  7:38 ` Jay Fang
  -- strict thread matches above, loose matches on Subject: below --
2024-02-16  3:11 Stephen Rothwell
2024-02-19 11:18 ` Dhruva Gole
2024-02-08  3:52 Stephen Rothwell
2024-02-08 11:43 ` Mark Brown
2024-02-08 13:32 ` Amadeusz Sławiński
2024-02-08 21:59   ` Stephen Rothwell
2022-06-08  4:30 Stephen Rothwell
2022-06-08  7:30 ` David Jander
2022-06-08 10:05   ` Mark Brown
2021-08-10  6:29 Stephen Rothwell
2021-08-09  5:30 Stephen Rothwell
2021-04-20  3:48 Stephen Rothwell
2021-04-20  7:56 ` Patrice CHOTARD
2021-01-11  1:09 Stephen Rothwell
2021-01-04  0:06 Stephen Rothwell
2021-01-04  7:56 ` Ardelean, Alexandru
2020-07-09  4:10 Stephen Rothwell
2020-07-09  5:29 ` Peng Fan
2020-04-28  4:16 Stephen Rothwell
2020-04-28  5:01 ` Sanjay R Mehta
2020-04-28  9:38   ` Mark Brown
2020-01-15  2:23 Stephen Rothwell
2020-01-15  9:10 ` Jean Pihet
2020-01-15 10:27   ` Stephen Rothwell
2020-01-15 11:38     ` Jean Pihet
2019-04-29  4:30 Stephen Rothwell
2019-01-29  2:02 Stephen Rothwell
2019-01-29  6:57 ` Yogesh Narayan Gaur
2019-01-29  7:25   ` Schrempf Frieder
2019-01-29  8:26   ` Stephen Rothwell
2019-01-29  8:39     ` Yogesh Narayan Gaur
2019-01-10  0:44 Stephen Rothwell
2019-01-10  0:41 Stephen Rothwell
2018-11-14  2:18 Stephen Rothwell
     [not found] ` <CAP6Zq1gp+hmNkvk8D2uv8FUqTXE4+ZMPhMB7A7vLMaM6swHy4g@mail.gmail.com>
2018-11-15 19:26   ` Mark Brown
2018-08-03  4:37 Stephen Rothwell
2017-03-27  2:41 Stephen Rothwell
2017-03-27  5:33 ` Vignesh R
2017-03-27  5:33   ` Vignesh R
2010-10-14  1:59 Stephen Rothwell
2010-10-14  5:00 ` Grant Likely

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.