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

Masahiro Yamada yamada.m at jp.panasonic.com
Wed Nov 5 09:12:40 CET 2014


Hi Marek,

On Wed, 5 Nov 2014 08:57:50 +0100
Marek Vasut <marex at denx.de> 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 <marex at denx.de> 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 <pavel at denx.de> 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 macros synced with the kernel introduing type checks.
> > > > > 
> > > > > "introducing"
> > > > 
> > > > I will fix this.
> > > 
> > > Are you linting the patches with aspell or something ?
> > 
> > No.
> > Going forward I will do that. Thaks for your advice!
> 
> I was just curious what Pavel does there, since I also got some spelling 
> corrections from him. Please don't take it like I want to add more work
> on your plate.
> 

I was not sure if your question was addressed to me or Pavel,
but never mind.

I use nano editor for inputting git-log.
It is just to press Ctrl-T to invoke aspell,
so it will not load me with much work.
Resending a patch just for a typo is a more tedious work.
(Pavel, don't get me wrong.  All you pointed out are good things.)


Best Regards
Masahiro Yamada



More information about the U-Boot mailing list