From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Sat, 06 Feb 2010 15:24:59 +0100
Subject: [U-Boot] [PATCH ARM 4/4] Change s3c24x0 register struct members
	to lower case
In-Reply-To: <1f3430fb1002060418ydd28583sb6051dd1517fa864@mail.gmail.com>
References: <4B3936EF.7030002@fearnside-systems.co.uk>
	<1f3430fb1002060418ydd28583sb6051dd1517fa864@mail.gmail.com>
Message-ID: <20100206142459.4541024F09@gemini.denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Minkyu Kang,

In message <1f3430fb1002060418ydd28583sb6051dd1517fa864@mail.gmail.com> you wrote:
> 
> > Changes the names of the s3c24x0 register struct members from upper-case
> > to lower-case.
...
> Sorry for late reply.
> I tried to review your patch series,
> but your patch is too confusion, because of files are scattered in
> every patches.

Well, but that's only natural, as the use of these names is scattered
all over the place.

> If possible, could you please remake patches?
> Please make to one file's modification by one patch.

This request makes no sense at all. If you change a variable name, you
*must* change all places where it hgets used within the same commit.

The splitting you suggest would make things worse.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"And they told us, what they wanted... Was a sound  that  could  kill
some-one, from a distance."                               - Kate Bush