[U-Boot] [PATCH v3 09/10] mvebu: Support Synology DS414

Tom Rini trini at konsulko.com
Wed Jan 13 16:41:25 CET 2016


On Wed, Jan 13, 2016 at 04:24:42PM +0100, Stefan Roese wrote:
> Hi Tom,
> 
> On 13.01.2016 16:13, Tom Rini wrote:
> >On Fri, Dec 25, 2015 at 02:41:25PM +0100, Phil Sutter wrote:
> >
> >>This adds support for the MV78230 based DS414 NAS by Synology. The
> >>relevant bits have been extracted from the 'synogpl-5004-armadaxp'
> >>package Synology kindly published, garnished with a fair amount of
> >>trial-and-error.
> >
> >So, for the code:
> >
> >Reviewed-by: Tom Rini <trini at konsulko.com>
> >
> >But most of what follows should be in the "log" side, ie after the ---
> >rather than the message itself.  This can be fixed when applying, but if
> >you end up needing to respin please reword this part, thanks!
> 
> I've already prepared a git branch, with all the pending Marvell
> mvebu related patches included. For Luka to pull from into the
> official Marvell U-boot git repo. And I've added Phil's patches
> as well. My personal feeling is, that all this additional text
> could be quite useful. That's why I would like to keep it in.
> But if you really feel that we should remove it, then I can
> definitely drop those passages.
> 
> Just let me know what you think.

I'll live with it in the commit log, thanks.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160113/a1cd1e3a/attachment.sig>


More information about the U-Boot mailing list