[PATCH 1/2] staging: ath6kl: Cast variable to size_t to avoid compile warning

[Luis R. Rodriguez]

On Mon, Feb 28, 2011 at 10:25 AM, Greg KH <greg at kroah.com> 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 <martinez.javier at gmail.com>
>> ---
>>  .../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