On Mon, Feb 28, 2011 at 10:25 AM, Greg KH <[email protected]> wrote: > On Sat, Feb 26, 2011 at 07:45:03PM +0100, Javier Martinez Canillas wrote: >> The min() macro does strict type-checking so we have to cast >> MAX_BDADDR_FORMAT_LENGTH - 1 to size_t to silence a compile warning >> >> Signed-off-by: Javier Martinez Canillas <[email protected]> >> --- >> .../staging/ath6kl/miscdrv/ar3kps/ar3kpsconfig.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/staging/ath6kl/miscdrv/ar3kps/ar3kpsconfig.c >> b/drivers/staging/ath6kl/miscdrv/ar3kps/ar3kpsconfig.c >> index 67e6d5e..778e4fd 100644 >> --- a/drivers/staging/ath6kl/miscdrv/ar3kps/ar3kpsconfig.c >> +++ b/drivers/staging/ath6kl/miscdrv/ar3kps/ar3kpsconfig.c >> @@ -360,7 +360,7 @@ int PSSendOps(void *arg) >> status = 1; >> goto complete; >> } >> - len = min(firmware->size, MAX_BDADDR_FORMAT_LENGTH - 1); >> + len = min(firmware->size, (size_t)MAX_BDADDR_FORMAT_LENGTH - 1); > > Shouldn't you use min_t() instead of casting here?
I'll fix this and resend this as part of my series. Luis _______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
