Re: [PATCH] iwlwifi: fix build warnings with format string

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

 



(please cc linux-wireless on all wireless patches)

Akinobu Mita <akinobu.mita@xxxxxxxxx> writes:

> This fixes build warnings introduced by commit "iwlegacy: use
> <linux/units.h> helpers" (iwlegacy-use-linux-unitsh-helpers.patch in -mm)
>
> The format '%d' has to be changed to '%ld' because the return type of
> kelvin_to_celsius() is 'long'.
>
> Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> Link: https://lore.kernel.org/r/20200106171452.201c3b4c@xxxxxxxxxxxxxxxx
> Cc: Kalle Valo <kvalo@xxxxxxxxxxxxxx>
> Cc: Stanislaw Gruszka <stf_xl@xxxxx>
> Signed-off-by: Akinobu Mita <akinobu.mita@xxxxxxxxx>

I don't know what's Andrew's workflow, but shouldn't the original patch
be fixed instead?

-- 
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches



[Index of Archives]     [Linux Kernel]     [Linux USB Development]     [Yosemite News]     [Linux SCSI]

  Powered by Linux