marvell: add support for Synology DS109 board
PATCH 1/1 marvell: add support for Synology DS109 board
PATCH 1/1 marvell: add support for Synology DS109 board
Hi Walter,
sorry for the late review.
Quoted 12 lines · Walter Schweizer
From
842431a0cc2acb336f5c57a3ca822ba0673e712dMon Sep 17 00:00:00 2001
From: Walter Schweizer <[email protected]>
Date: Sat, 18 Jun 2016 12:56:37 +0200
Subject: [PATCH] add support for Synology DS109 boardSynology DS109 is based on MV88F6281. The code
is based on Dreamplug code with modificatons
from Synologys open source repository.Signed-off-by Walter Schweizer <[email protected]>---
Quoted 8 lines
arch/arm/mach-kirkwood/Kconfig | 4 + board/Synology/ds109/Kconfig | 12 +++ board/Synology/ds109/Makefile | 9 ++ board/Synology/ds109/ds109.c | 169 ++++++++++++++++++++++++++++++++++++++ board/Synology/ds109/ds109.h | 44 ++++++++++ board/Synology/ds109/kwbimage.cfg | 147+++++++++++++++++++++++++++++++++
Your patch seems to have some line wraps included. Please fix this in v2.
Quoted 4 lines
board/Synology/ds109/openocd.cfg | 115 ++++++++++++++++++++++++++ configs/ds109_defconfig | 16 ++++ include/configs/ds109.h | 118 ++++++++++++++++++++++++++ 9 files changed, 634 insertions(+)
Quoted 8 lines
create mode 100644 board/Synology/ds109/Kconfig create mode 100644 board/Synology/ds109/Makefile create mode 100644 board/Synology/ds109/ds109.c create mode 100644 board/Synology/ds109/ds109.h create mode 100644 board/Synology/ds109/kwbimage.cfg create mode 100644 board/Synology/ds109/openocd.cfg create mode 100644 configs/ds109_defconfig create mode 100644 include/configs/ds109.h
MATAINERS file missing.
Please check in v2 that your patch passes checkpatch without errors. There seem to be many whitespace errors.
Thanks,
Stefan
Viele Gr??e,
Stefan
Signature · 3 lines
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de