From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Thu, 17 Dec 2015 04:46:55 +0100
Subject: [U-Boot] [PATCH] arm: socfpga: Fixed Phy not found error. Fixed
	QSPI flash not found error Disabled Hardware watch dog by
	default. Updated MMC clock.
In-Reply-To: <tencent_69482D7B010BD104370EB028@qq.com>
References: <tencent_69482D7B010BD104370EB028@qq.com>
Message-ID: <201512170446.55816.marex@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

On Thursday, December 17, 2015 at 04:25:28 AM, ShengjiangWu wrote:
> arm: socfpga: Fixed Phy not found error.
>                   Fixed QSPI flash not found error
>                   Disabled Hardware watch dog by default.
>                   Updated MMC clock.

Fix your mailer please.

>     Updated pinmux and clock for EMAC1 and QSPI flash.
>     U-BOOT-2015-10 can run on Cyclone V SoC Development
>     Kit (SoCDK) with this fix.
> 
>     Signed-off-by: swu <shengjiangwu@gmail.com>
>     Cc: Chin Liang See <clsee@altera.com>
>     Cc: Dinh Nguyen <dinguyen@opensource.altera.com>
>     Cc: Dinh Nguyen <dinh.linux@gmail.com>
>     Cc: Pavel Machek <pavel@denx.de>
>     Cc: Marek Vasut <marex@denx.de>
>     Cc: Stefan Roese <sr@denx.de>

Please repost the patch using git send-email , details are at [1] and the 
process is very similar to Linux.

I briefly skimmed across the patch. I don't understand why do you replace
almost the entire pinmux just to fix ethernet issue. (ethernet works on
SoCDK I believe, which is another odd thing).

Furthermore, if you're fixing multiple things, split them into multiple
patches.

Finally, do not disable HW watchdog.

[1] www.denx.de/wiki/U-Boot/Patches

Best regards,
Marek Vasut