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 <martinez.javier@xxxxxxxxx> > --- > .../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? thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
