Re: [ulf.hansson-mmc:next 84/84] drivers/mmc/host/dw_mmc.c:1953:9: sparse: incompatible types in comparison expression (different type sizes)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2 November 2017 at 02:43, Doug Anderson <dianders@xxxxxxxxxxxx> wrote:
> Hi,
>
> On Wed, Nov 1, 2017 at 11:19 AM, kbuild test robot
> <fengguang.wu@xxxxxxxxx> wrote:
>> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/ulfh/mmc.git next
>> head:   80a2a5a7745b1f65198fb11da9a8d1d38c3003d5
>> commit: 80a2a5a7745b1f65198fb11da9a8d1d38c3003d5 [84/84] mmc: dw_mmc: Cleanup the DTO timer like the CTO one
>> reproduce:
>>         # apt-get install sparse
>>         git checkout 80a2a5a7745b1f65198fb11da9a8d1d38c3003d5
>>         make ARCH=x86_64 allmodconfig
>>         make C=1 CF=-D__CHECK_ENDIAN__
>>
>>
>> sparse warnings: (new ones prefixed by >>)
>>
>>
>> vim +1953 drivers/mmc/host/dw_mmc.c
>>
>>   1935
>>   1936  static void dw_mci_set_drto(struct dw_mci *host)
>>   1937  {
>>   1938          unsigned int drto_clks;
>>   1939          unsigned int drto_div;
>>   1940          unsigned int drto_ms;
>>   1941          unsigned int irqflags;
>
> Ulf: this looks like it came from your manual fixup of my patches.
> The above should be an "unsigned long", AKA:
>
>   unsigned long irqflags;

:-)

Thanks for pointing out - fixed it!

Kind regards
Uffe
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux