From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wolfgang Denk <wd@denx.de>
Date: Tue, 25 Jan 2011 22:34:19 +0100
Subject: [U-Boot] [PATCH V2 4/5] mvmfp: add MFP configuration support
	for PANTHEON
In-Reply-To: <1294632087-8025-5-git-send-email-leiwen@marvell.com>
References: <1294632087-8025-1-git-send-email-leiwen@marvell.com>
	<1294632087-8025-5-git-send-email-leiwen@marvell.com>
Message-ID: <20110125213419.5C9431365DB@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 Lei Wen,

In message <1294632087-8025-5-git-send-email-leiwen@marvell.com> you wrote:
> This patch adds the support MFP support for Marvell PANTHEON SoCs

"the support MFP support" ??

Please fix. Also, please explain what MFP is. It may be obvious to
you, it ain't so for me.

> +/*
> + * Frequently used MFP Configuration macros for all PANTHEON family of
> +SoCs

Line wrapped - please fix.


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
"Probably the best operating system in the world  is  the  [operating
system] made for the PDP-11 by Bell Laboratories."
                                           - Ted Nelson, October 1977