[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.

[Marek Vasut]

On Thursday, December 17, 2015 at 07:59:34 AM, ShengjiangWu wrote:
> Hi Marek Vasut,

Hi,

> I will submit it later, thanks for your reminder.

Thanks.

Also, please do not top-post and please keep the entire Cc list intact.
Also keep the ML in the list.

> Best Regards
> Shengjiang Wu
> 
> 
> 
> 
> ------------------ Original ------------------
> From:  "marex";<marex at denx.de>;
> Date:  Thu, Dec 17, 2015 11:46 AM
> To:  "ShengjiangWu"<shengjiangwu at qq.com>;
> Cc:  "u-boot"<u-boot at lists.denx.de>;
> "shengjiangwu"<shengjiangwu at gmail.com>; "clsee"<clsee at altera.com>;
> "dinguyen"<dinguyen at opensource.altera.com>;
> "dinh.linux"<dinh.linux at gmail.com>; "pavel"<pavel at denx.de>;
> "sr"<sr at denx.de>; Subject:  Re: [PATCH] arm: socfpga: Fixed Phy not found
> error.               Fixed QSPI flash not found error              
> Disabled Hardware watch dog by default.               Updated MMC clock.
> 
> 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 at gmail.com>
> >     Cc: Chin Liang See <clsee at altera.com>
> >     Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> >     Cc: Dinh Nguyen <dinh.linux at gmail.com>
> >     Cc: Pavel Machek <pavel at denx.de>
> >     Cc: Marek Vasut <marex at denx.de>
> >     Cc: Stefan Roese <sr at 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

Best regards,
Marek Vasut