On 02.10.2018, at 16:01, Manivannan Sadhasivam manivannan.sadhasivam@linaro.org wrote:
Hi Simon,
On Tue, Oct 02, 2018 at 04:21:38AM -0700, Simon Glass wrote:
On 27 September 2018 at 12:02, Manivannan Sadhasivam manivannan.sadhasivam@linaro.org wrote:
Rock960 is a family of boards based on Rockchip RK3399 SoC from Vamrs. It consists of Rock960 (Consumer Edition) and Ficus (Enterprise Edition) 96Boards.
Below are some of the key differences between both Rock960 and Ficus boards:
- Different host enable GPIO for USB
- Different power and reset GPIO for PCI-E
- No Ethernet port on Rock960
The common board support will be utilized by both boards. The device tree has been organized in such a way that only the properties which differ between both boards are placed in the board specific dts and the reset of the nodes are placed in common dtsi file.
Signed-off-by: Manivannan Sadhasivam manivannan.sadhasivam@linaro.org [Added instructions for SD card boot] Signed-off-by: Ezequiel Garcia ezequiel@collabora.com
Changes in v3: Added instruction for copying prebuilt bl31.elf for SPL
Changes in v2: None
arch/arm/dts/rk3399-rock960.dtsi | 506 ++++++++++++++++++++ arch/arm/mach-rockchip/rk3399/Kconfig | 26 + board/vamrs/rock960_rk3399/Kconfig | 15 + board/vamrs/rock960_rk3399/MAINTAINERS | 6 + board/vamrs/rock960_rk3399/Makefile | 6 + board/vamrs/rock960_rk3399/README | 152 ++++++ board/vamrs/rock960_rk3399/rock960-rk3399.c | 50 ++ include/configs/rock960_rk3399.h | 15 + 8 files changed, 776 insertions(+) create mode 100644 arch/arm/dts/rk3399-rock960.dtsi create mode 100644 board/vamrs/rock960_rk3399/Kconfig create mode 100644 board/vamrs/rock960_rk3399/MAINTAINERS create mode 100644 board/vamrs/rock960_rk3399/Makefile create mode 100644 board/vamrs/rock960_rk3399/README create mode 100644 board/vamrs/rock960_rk3399/rock960-rk3399.c create mode 100644 include/configs/rock960_rk3399.h
Reviewed-by: Simon Glass sjg@chromium.org
Could you also add a note to README.rockchip? Some of your docs seem to duplicate what is there.
Thanks for your review!
You mean, I should skip the duplicate instructions and add a pointer to relevant sections in Rockchip README?
I also had had a similar comment on an earlier series: we should avoid just copying these instructions verbatim into every new board: if these are indeed identical to Rockchip’s EVB (i.e. if the board-vendor completely relies on the chip-vendor's tools), they should reference back to the EVB’s README instead of duplicating the content.
Regards, Mani