Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-06 Thread Pavel Machek
On Wed 2014-11-05 07:02:23, Marek Vasut wrote: > On Wednesday, November 05, 2014 at 06:06:09 AM, Masahiro Yamada wrote: > > Hi Pavel, > > > > Thanks for your close checking. > > > > > > On Tue, 4 Nov 2014 20:50:13 +0100 > > > > Pavel Machek wrote: > > > On Tue 2014-11-04 20:26:26, Masahiro Yam

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-06 Thread Lukasz Majewski
Hi Masahiro, > U-Boot has never cared about the type when we get max/min of two > values, but Linux Kernel does. This commit gets min, max, min3, max3 > macros synced with the kernel introduing type checks. > > Many of references of those macros must be fixed to suppress warnings. > We have two

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-05 Thread Marek Vasut
On Wednesday, November 05, 2014 at 09:12:40 AM, Masahiro Yamada wrote: > Hi Marek, Hi Masahiro, > On Wed, 5 Nov 2014 08:57:50 +0100 > > Marek Vasut wrote: > > On Wednesday, November 05, 2014 at 08:48:19 AM, Masahiro Yamada wrote: > > > Hi Marek, > > > > Hi! > > > > > (reduced Cc) > > > > > >

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-05 Thread Masahiro Yamada
Hi Marek, On Wed, 5 Nov 2014 08:57:50 +0100 Marek Vasut wrote: > On Wednesday, November 05, 2014 at 08:48:19 AM, Masahiro Yamada wrote: > > Hi Marek, > > Hi! > > > (reduced Cc) > > > > On Wed, 5 Nov 2014 07:02:23 +0100 > > > > Marek Vasut wrote: > > > On Wednesday, November 05, 2014 at 06:0

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Marek Vasut
On Wednesday, November 05, 2014 at 08:48:19 AM, Masahiro Yamada wrote: > Hi Marek, Hi! > (reduced Cc) > > On Wed, 5 Nov 2014 07:02:23 +0100 > > Marek Vasut wrote: > > On Wednesday, November 05, 2014 at 06:06:09 AM, Masahiro Yamada wrote: > > > Hi Pavel, > > > > > > Thanks for your close check

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Masahiro Yamada
Hi Marek, (reduced Cc) On Wed, 5 Nov 2014 07:02:23 +0100 Marek Vasut wrote: > On Wednesday, November 05, 2014 at 06:06:09 AM, Masahiro Yamada wrote: > > Hi Pavel, > > > > Thanks for your close checking. > > > > > > On Tue, 4 Nov 2014 20:50:13 +0100 > > > > Pavel Machek wrote: > > > On Tue 2

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Marek Vasut
On Wednesday, November 05, 2014 at 06:06:09 AM, Masahiro Yamada wrote: > Hi Pavel, > > Thanks for your close checking. > > > On Tue, 4 Nov 2014 20:50:13 +0100 > > Pavel Machek wrote: > > On Tue 2014-11-04 20:26:26, Masahiro Yamada wrote: > > > U-Boot has never cared about the type when we get

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Masahiro Yamada
Hi Pavel, Thanks for your close checking. On Tue, 4 Nov 2014 20:50:13 +0100 Pavel Machek wrote: > On Tue 2014-11-04 20:26:26, Masahiro Yamada wrote: > > U-Boot has never cared about the type when we get max/min of two > > values, but Linux Kernel does. This commit gets min, max, min3, max3 >

Re: [U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Pavel Machek
On Tue 2014-11-04 20:26:26, Masahiro Yamada wrote: > U-Boot has never cared about the type when we get max/min of two > values, but Linux Kernel does. This commit gets min, max, min3, max3 > macros synced with the kernel introduing type checks. "introducing" > Many of references of those macros

[U-Boot] [PATCH 6/6] linux/kernel.h: sync min, max, min3, max3 macros with Linux

2014-11-04 Thread Masahiro Yamada
U-Boot has never cared about the type when we get max/min of two values, but Linux Kernel does. This commit gets min, max, min3, max3 macros synced with the kernel introduing type checks. Many of references of those macros must be fixed to suppress warnings. We have two options: - Use min, max,