Linux-Next Archive on lore.kernel.org
 help / color / Atom feed
* linux-next: build warning after merge of the mmc tree
@ 2019-10-22  0:47 Stephen Rothwell
  2019-10-22  7:31 ` Ulf Hansson
  0 siblings, 1 reply; 6+ messages in thread
From: Stephen Rothwell @ 2019-10-22  0:47 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Chaotian Jing

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

Hi all,

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

drivers/mmc/core/block.c: In function '__mmc_blk_ioctl_cmd':
drivers/mmc/core/block.c:500:6: warning: unused variable 'status' [-Wunused-variable]
  500 |  u32 status = 0;
      |      ^~~~~~

Introduced by commit

  05224f7e4975 ("mmc: block: Add CMD13 polling for MMC IOCTLS with R1B response")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the mmc tree
  2019-10-22  0:47 linux-next: build warning after merge of the mmc tree Stephen Rothwell
@ 2019-10-22  7:31 ` Ulf Hansson
  0 siblings, 0 replies; 6+ messages in thread
From: Ulf Hansson @ 2019-10-22  7:31 UTC (permalink / raw)
  To: Stephen Rothwell, Chaotian Jing
  Cc: Linux Next Mailing List, Linux Kernel Mailing List

On Tue, 22 Oct 2019 at 02:47, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> Hi all,
>
> After merging the mmc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> drivers/mmc/core/block.c: In function '__mmc_blk_ioctl_cmd':
> drivers/mmc/core/block.c:500:6: warning: unused variable 'status' [-Wunused-variable]
>   500 |  u32 status = 0;
>       |      ^~~~~~
>
> Introduced by commit
>
>   05224f7e4975 ("mmc: block: Add CMD13 polling for MMC IOCTLS with R1B response")
>
> --
> Cheers,
> Stephen Rothwell

Thanks for the report. I have amended the patch to address the warning.

Kind regards
Uffe

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

* linux-next: build warning after merge of the mmc tree
@ 2019-03-19  1:15 Stephen Rothwell
  0 siblings, 0 replies; 6+ messages in thread
From: Stephen Rothwell @ 2019-03-19  1:15 UTC (permalink / raw)
  To: Ulf Hansson; +Cc: Linux Next Mailing List, Linux Kernel Mailing List, Shawn Lin

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

Hi all,

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

drivers/mmc/host/dw_mmc.c: In function 'dw_mci_wait_hw_unbusy':
drivers/mmc/host/dw_mmc.c:239:16: warning: unused variable 'irqflags' [-Wunused-variable]
  unsigned long irqflags;
                ^~~~~~~~

Introduced by commit

  705a0f288113 ("mmc: dw_mmc: Add hardware unbusy interrupt support")

-- 
Cheers,
Stephen Rothwell

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

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

* Re: linux-next: build warning after merge of the mmc tree
  2017-11-02  1:47 ` Doug Anderson
@ 2017-11-02 15:03   ` Ulf Hansson
  0 siblings, 0 replies; 6+ messages in thread
From: Ulf Hansson @ 2017-11-02 15:03 UTC (permalink / raw)
  To: Doug Anderson
  Cc: Stephen Rothwell, Linux-Next Mailing List,
	Linux Kernel Mailing List, Shawn Lin

On 2 November 2017 at 02:47, Doug Anderson <dianders@chromium.org> wrote:
> Hi,
>
> On Wed, Nov 1, 2017 at 6:36 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>> Hi Ulf,
>>
>> After merging the mmc tree, today's linux-next build (arm
>> multi_v7_defconfig) produced this warning:
>>
>> In file included from include/linux/irqflags.h:14:0,
>>                  from arch/arm/include/asm/bitops.h:27,
>>                  from include/linux/bitops.h:37,
>>                  from include/linux/kernel.h:10,
>>                  from include/asm-generic/bug.h:15,
>>                  from arch/arm/include/asm/bug.h:59,
>>                  from include/linux/bug.h:4,
>>                  from include/linux/thread_info.h:11,
>>                  from include/asm-generic/current.h:4,
>>                  from ./arch/arm/include/generated/asm/current.h:1,
>>                  from include/linux/sched.h:11,
>>                  from include/linux/blkdev.h:4,
>>                  from drivers/mmc/host/dw_mmc.c:14:
>> drivers/mmc/host/dw_mmc.c: In function 'dw_mci_set_drto':
>> include/linux/typecheck.h:11:18: warning: comparison of distinct pointer types lacks a cast
>>   (void)(&__dummy == &__dummy2); \
>>                   ^
>> include/linux/spinlock.h:221:3: note: in expansion of macro 'typecheck'
>>    typecheck(unsigned long, flags); \
>>    ^
>> include/linux/spinlock.h:346:2: note: in expansion of macro 'raw_spin_lock_irqsave'
>>   raw_spin_lock_irqsave(spinlock_check(lock), flags); \
>>   ^
>> drivers/mmc/host/dw_mmc.c:1953:2: note: in expansion of macro 'spin_lock_irqsave'
>>   spin_lock_irqsave(&host->irq_lock, irqflags);
>>   ^
>>
>> Introduced by commit
>>
>>   80a2a5a7745b ("mmc: dw_mmc: Cleanup the DTO timer like the CTO one")
>
> Yes, the 0-day bot already complained about this.  The short answer is
> that Ulf kindly volunteered to manually fix up my patch series where I
> had introduced a variable in the wrong patch.  ...but in doing so he
> introduced a small bug.
>
> It should be very easy to fix.  Just need to change irqflags from
> "unsigned int" to "unsigned long".

Yes, problem fixed!

Kind regards
Uffe

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

* Re: linux-next: build warning after merge of the mmc tree
  2017-11-02  1:36 Stephen Rothwell
@ 2017-11-02  1:47 ` Doug Anderson
  2017-11-02 15:03   ` Ulf Hansson
  0 siblings, 1 reply; 6+ messages in thread
From: Doug Anderson @ 2017-11-02  1:47 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Ulf Hansson, Linux-Next Mailing List, Linux Kernel Mailing List,
	Shawn Lin

Hi,

On Wed, Nov 1, 2017 at 6:36 PM, Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> Hi Ulf,
>
> After merging the mmc tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> In file included from include/linux/irqflags.h:14:0,
>                  from arch/arm/include/asm/bitops.h:27,
>                  from include/linux/bitops.h:37,
>                  from include/linux/kernel.h:10,
>                  from include/asm-generic/bug.h:15,
>                  from arch/arm/include/asm/bug.h:59,
>                  from include/linux/bug.h:4,
>                  from include/linux/thread_info.h:11,
>                  from include/asm-generic/current.h:4,
>                  from ./arch/arm/include/generated/asm/current.h:1,
>                  from include/linux/sched.h:11,
>                  from include/linux/blkdev.h:4,
>                  from drivers/mmc/host/dw_mmc.c:14:
> drivers/mmc/host/dw_mmc.c: In function 'dw_mci_set_drto':
> include/linux/typecheck.h:11:18: warning: comparison of distinct pointer types lacks a cast
>   (void)(&__dummy == &__dummy2); \
>                   ^
> include/linux/spinlock.h:221:3: note: in expansion of macro 'typecheck'
>    typecheck(unsigned long, flags); \
>    ^
> include/linux/spinlock.h:346:2: note: in expansion of macro 'raw_spin_lock_irqsave'
>   raw_spin_lock_irqsave(spinlock_check(lock), flags); \
>   ^
> drivers/mmc/host/dw_mmc.c:1953:2: note: in expansion of macro 'spin_lock_irqsave'
>   spin_lock_irqsave(&host->irq_lock, irqflags);
>   ^
>
> Introduced by commit
>
>   80a2a5a7745b ("mmc: dw_mmc: Cleanup the DTO timer like the CTO one")

Yes, the 0-day bot already complained about this.  The short answer is
that Ulf kindly volunteered to manually fix up my patch series where I
had introduced a variable in the wrong patch.  ...but in doing so he
introduced a small bug.

It should be very easy to fix.  Just need to change irqflags from
"unsigned int" to "unsigned long".


-Doug

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

* linux-next: build warning after merge of the mmc tree
@ 2017-11-02  1:36 Stephen Rothwell
  2017-11-02  1:47 ` Doug Anderson
  0 siblings, 1 reply; 6+ messages in thread
From: Stephen Rothwell @ 2017-11-02  1:36 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Linux-Next Mailing List, Linux Kernel Mailing List,
	Douglas Anderson, Shawn Lin

Hi Ulf,

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

In file included from include/linux/irqflags.h:14:0,
                 from arch/arm/include/asm/bitops.h:27,
                 from include/linux/bitops.h:37,
                 from include/linux/kernel.h:10,
                 from include/asm-generic/bug.h:15,
                 from arch/arm/include/asm/bug.h:59,
                 from include/linux/bug.h:4,
                 from include/linux/thread_info.h:11,
                 from include/asm-generic/current.h:4,
                 from ./arch/arm/include/generated/asm/current.h:1,
                 from include/linux/sched.h:11,
                 from include/linux/blkdev.h:4,
                 from drivers/mmc/host/dw_mmc.c:14:
drivers/mmc/host/dw_mmc.c: In function 'dw_mci_set_drto':
include/linux/typecheck.h:11:18: warning: comparison of distinct pointer types lacks a cast
  (void)(&__dummy == &__dummy2); \
                  ^
include/linux/spinlock.h:221:3: note: in expansion of macro 'typecheck'
   typecheck(unsigned long, flags); \
   ^
include/linux/spinlock.h:346:2: note: in expansion of macro 'raw_spin_lock_irqsave'
  raw_spin_lock_irqsave(spinlock_check(lock), flags); \
  ^
drivers/mmc/host/dw_mmc.c:1953:2: note: in expansion of macro 'spin_lock_irqsave'
  spin_lock_irqsave(&host->irq_lock, irqflags);
  ^

Introduced by commit

  80a2a5a7745b ("mmc: dw_mmc: Cleanup the DTO timer like the CTO one")

-- 
Cheers,
Stephen Rothwell

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

end of thread, back to index

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-22  0:47 linux-next: build warning after merge of the mmc tree Stephen Rothwell
2019-10-22  7:31 ` Ulf Hansson
  -- strict thread matches above, loose matches on Subject: below --
2019-03-19  1:15 Stephen Rothwell
2017-11-02  1:36 Stephen Rothwell
2017-11-02  1:47 ` Doug Anderson
2017-11-02 15:03   ` Ulf Hansson

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