On 30 June 2015 at 16:37, Angelo Dureghello <ang...@sysam.it> wrote:
> Hi Jagan,
>
>
> On 29/06/2015 22:00, Jagan Teki wrote:
>>
>> On 22 June 2015 at 12:49, Angelo Dureghello <ang...@sysam.it> wrote:
>>>
>>>
>>>
>>> On 22/06/2015 08:44, Jagan Teki wrote:
>>>>
>>>>
>>>> On 22 June 2015 at 03:10, Angelo Dureghello <ang...@sysam.it> wrote:
>>>>>
>>>>>
>>>>> Fix compilation warnings for redefined 'clamp' macro and non-uniform
>>>>> clamp macro types.
>>>>>
>>>>> Signed-off-by: Angelo Dureghello <ang...@sysam.it>
>>>>> ---
>>>>> drivers/spi/cf_qspi.c | 3 +--
>>>>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/drivers/spi/cf_qspi.c b/drivers/spi/cf_qspi.c
>>>>> index 834c5bd..e57e63e 100644
>>>>> --- a/drivers/spi/cf_qspi.c
>>>>> +++ b/drivers/spi/cf_qspi.c
>>>>> @@ -19,7 +19,6 @@
>>>>>
>>>>> DECLARE_GLOBAL_DATA_PTR;
>>>>>
>>>>> -#define clamp(x, low, high) (min(max(low, x), high))
>>>>> #define to_cf_qspi_slave(s) container_of(s, struct cf_qspi_slave,
>>>>> slave)
>>>>>
>>>>> struct cf_qspi_slave {
>>>>> @@ -120,7 +119,7 @@ struct spi_slave *spi_setup_slave(unsigned int bus,
>>>>> unsigned int cs,
>>>>> dev->qmr = 2u;
>>>>> else /* Get the closest baud rate */
>>>>> dev->qmr = clamp(((gd->bus_clk >> 2) + max_hz -
>>>>> 1)/max_hz,
>>>>> - 2u, 255u);
>>>>> + 2lu, 255lu);
>>
>>
>> replacing u with lu - any reason?
>
>
> Compiling this file you will get compilation warnings.
> The reason is that the "unnecessary type check" throws the warnings since
> clamp (and so included min max) uses different types.
>
> u is unsigned int
> lu uniforms types to gd->bus_clk, that is unsigned long.
Applied to u-boot-spi/master
thanks!
--
Jagan | openedev.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
