* [Buildroot] [PATCH] board: Add nanopi-m1 Support
@ 2017-06-15 17:24 Chakra Divi
2017-06-15 20:15 ` Thomas Petazzoni
2017-06-15 21:53 ` Arnout Vandecappelle
0 siblings, 2 replies; 7+ messages in thread
From: Chakra Divi @ 2017-06-15 17:24 UTC (permalink / raw)
To: buildroot
Add initial support for nanopi-m1 board
with below features
- U-Boot 2017.07-rc1
- Linux 4.11
- Default packages from buildroot
Created a new folder nanopi under buildroot
board; any new board for nanopi can be added
here
Signed-off-by: Chakra Divi <chakra@openedev.com>
---
board/nanopi/nanopi-m1/boot.cmd | 8 ++++++++
board/nanopi/nanopi-m1/genimage.cfg | 36 ++++++++++++++++++++++++++++++++++
board/nanopi/nanopi-m1/post-build.sh | 1 +
board/nanopi/nanopi-m1/post-image.sh | 1 +
board/nanopi/nanopi-m1/readme.txt | 29 +++++++++++++++++++++++++++
board/nanopi/post-build.sh | 11 +++++++++++
board/nanopi/post-image.sh | 14 +++++++++++++
configs/nanopi_m1_defconfig | 29 +++++++++++++++++++++++++++
8 files changed, 129 insertions(+)
create mode 100644 board/nanopi/nanopi-m1/boot.cmd
create mode 100644 board/nanopi/nanopi-m1/genimage.cfg
create mode 120000 board/nanopi/nanopi-m1/post-build.sh
create mode 120000 board/nanopi/nanopi-m1/post-image.sh
create mode 100644 board/nanopi/nanopi-m1/readme.txt
create mode 100755 board/nanopi/post-build.sh
create mode 100755 board/nanopi/post-image.sh
create mode 100644 configs/nanopi_m1_defconfig
diff --git a/board/nanopi/nanopi-m1/boot.cmd b/board/nanopi/nanopi-m1/boot.cmd
new file mode 100644
index 0000000..b4d612e
--- /dev/null
+++ b/board/nanopi/nanopi-m1/boot.cmd
@@ -0,0 +1,8 @@
+setenv fdt_high ffffffff
+
+setenv bootargs console=ttyS0,115200 earlyprintk root=/dev/mmcblk0p2 rootwait
+
+fatload mmc 0 $kernel_addr_r zImage
+fatload mmc 0 $fdt_addr_r sun8i-h3-nanopi-m1.dtb
+
+bootz $kernel_addr_r - $fdt_addr_r
diff --git a/board/nanopi/nanopi-m1/genimage.cfg b/board/nanopi/nanopi-m1/genimage.cfg
new file mode 100644
index 0000000..23c9c7d
--- /dev/null
+++ b/board/nanopi/nanopi-m1/genimage.cfg
@@ -0,0 +1,36 @@
+# Minimal SD card image for the NanoPi M1
+#
+image boot.vfat {
+ vfat {
+ files = {
+ "zImage",
+ "sun8i-h3-nanopi-m1.dtb",
+ "boot.scr"
+ }
+ }
+ size = 10M
+}
+
+image sdcard.img {
+ hdimage {
+ }
+
+ partition u-boot {
+ in-partition-table = "no"
+ image = "u-boot-sunxi-with-spl.bin"
+ offset = 8192
+ size = 1040384 # 1MB - 8192
+ }
+
+ partition boot {
+ partition-type = 0xC
+ bootable = "true"
+ image = "boot.vfat"
+ }
+
+ partition rootfs {
+ partition-type = 0x83
+ image = "rootfs.ext4"
+ size = 512M
+ }
+}
diff --git a/board/nanopi/nanopi-m1/post-build.sh b/board/nanopi/nanopi-m1/post-build.sh
new file mode 120000
index 0000000..cd7f116
--- /dev/null
+++ b/board/nanopi/nanopi-m1/post-build.sh
@@ -0,0 +1 @@
+../post-build.sh
\ No newline at end of file
diff --git a/board/nanopi/nanopi-m1/post-image.sh b/board/nanopi/nanopi-m1/post-image.sh
new file mode 120000
index 0000000..ae6294f
--- /dev/null
+++ b/board/nanopi/nanopi-m1/post-image.sh
@@ -0,0 +1 @@
+../post-image.sh
\ No newline at end of file
diff --git a/board/nanopi/nanopi-m1/readme.txt b/board/nanopi/nanopi-m1/readme.txt
new file mode 100644
index 0000000..8b10e4b
--- /dev/null
+++ b/board/nanopi/nanopi-m1/readme.txt
@@ -0,0 +1,29 @@
+NanoPi M1
+
+Intro
+=====
+
+This default configuration will allow you to start experimenting with the
+buildroot environment for the NanoPi M1. With the current configuration
+it will bring-up the board, and allow access through the serial console.
+
+How to build it
+===============
+
+Configure Buildroot:
+
+ $ make nanopi_m1_defconfig
+
+Compile everything and build the SD card image:
+
+ $ make
+
+How to write the SD card
+========================
+
+Once the build process is finished you will have an image called "sdcard.img"
+in the output/images/ directory.
+
+Copy the bootable "sdcard.img" onto an SD card with "dd":
+
+ $ sudo dd if=output/images/sdcard.img of=/dev/sdX
diff --git a/board/nanopi/post-build.sh b/board/nanopi/post-build.sh
new file mode 100755
index 0000000..a544a19
--- /dev/null
+++ b/board/nanopi/post-build.sh
@@ -0,0 +1,11 @@
+#!/bin/sh
+# post-build.sh for OrangePi taken from CubieBoard's post-build.sh
+# 2013, Carlo Caione <carlo.caione@gmail.com>
+
+BOARD_DIR="$(dirname $0)"
+MKIMAGE=$HOST_DIR/usr/bin/mkimage
+BOOT_CMD=$BOARD_DIR/boot.cmd
+BOOT_CMD_H=$BINARIES_DIR/boot.scr
+
+# U-Boot script
+$MKIMAGE -C none -A arm -T script -d $BOOT_CMD $BOOT_CMD_H
diff --git a/board/nanopi/post-image.sh b/board/nanopi/post-image.sh
new file mode 100755
index 0000000..359b4c9
--- /dev/null
+++ b/board/nanopi/post-image.sh
@@ -0,0 +1,14 @@
+#!/bin/sh
+
+BOARD_DIR="$(dirname $0)"
+GENIMAGE_CFG="${BOARD_DIR}/genimage.cfg"
+GENIMAGE_TMP="${BUILD_DIR}/genimage.tmp"
+
+rm -rf "${GENIMAGE_TMP}"
+
+genimage \
+ --rootpath "${TARGET_DIR}" \
+ --tmppath "${GENIMAGE_TMP}" \
+ --inputpath "${BINARIES_DIR}" \
+ --outputpath "${BINARIES_DIR}" \
+ --config "${GENIMAGE_CFG}"
diff --git a/configs/nanopi_m1_defconfig b/configs/nanopi_m1_defconfig
new file mode 100644
index 0000000..b1443f8
--- /dev/null
+++ b/configs/nanopi_m1_defconfig
@@ -0,0 +1,29 @@
+BR2_arm=y
+BR2_cortex_a7=y
+BR2_ARM_FPU_VFPV4=y
+BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_11=y
+BR2_TARGET_GENERIC_HOSTNAME="nanopi-m1"
+BR2_TARGET_GENERIC_ISSUE="Welcome to Buildroot for the NanoPi M1"
+BR2_ROOTFS_POST_BUILD_SCRIPT="board/nanopi/nanopi-m1/post-build.sh"
+BR2_ROOTFS_POST_IMAGE_SCRIPT="board/nanopi/nanopi-m1/post-image.sh"
+BR2_LINUX_KERNEL=y
+BR2_LINUX_KERNEL_CUSTOM_VERSION=y
+BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="4.11"
+BR2_LINUX_KERNEL_DEFCONFIG="sunxi"
+BR2_LINUX_KERNEL_DTS_SUPPORT=y
+BR2_LINUX_KERNEL_INTREE_DTS_NAME="sun8i-h3-nanopi-m1"
+BR2_TARGET_ROOTFS_EXT2=y
+BR2_TARGET_ROOTFS_EXT2_4=y
+# BR2_TARGET_ROOTFS_TAR is not set
+BR2_TARGET_UBOOT=y
+BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
+BR2_TARGET_UBOOT_CUSTOM_VERSION=y
+BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2017.07-rc1"
+BR2_TARGET_UBOOT_BOARD_DEFCONFIG="nanopi_m1"
+BR2_TARGET_UBOOT_NEEDS_DTC=y
+BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
+BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot-sunxi-with-spl.bin"
+BR2_PACKAGE_HOST_DOSFSTOOLS=y
+BR2_PACKAGE_HOST_GENIMAGE=y
+BR2_PACKAGE_HOST_MTOOLS=y
+BR2_PACKAGE_HOST_UBOOT_TOOLS=y
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-15 17:24 [Buildroot] [PATCH] board: Add nanopi-m1 Support Chakra Divi
@ 2017-06-15 20:15 ` Thomas Petazzoni
2017-06-16 16:13 ` Chakra D
2017-06-15 21:53 ` Arnout Vandecappelle
1 sibling, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2017-06-15 20:15 UTC (permalink / raw)
To: buildroot
Hello,
On Thu, 15 Jun 2017 22:54:25 +0530, Chakra Divi wrote:
> Add initial support for nanopi-m1 board
> with below features
> - U-Boot 2017.07-rc1
Are you using this -rc1 because it's the first U-Boot version with
support for this board?
> - Linux 4.11
> - Default packages from buildroot
>
> Created a new folder nanopi under buildroot
> board; any new board for nanopi can be added
> here
We already have the support for another nanopi board
in board/nanopi-neo/. Perhaps some refactoring is needed?
Also is "nanopi" really the vendor name? We normally like to have
board/<vendor>/<boardname>/.
> diff --git a/board/nanopi/post-image.sh b/board/nanopi/post-image.sh
> new file mode 100755
> index 0000000..359b4c9
> --- /dev/null
> +++ b/board/nanopi/post-image.sh
> @@ -0,0 +1,14 @@
> +#!/bin/sh
> +
> +BOARD_DIR="$(dirname $0)"
> +GENIMAGE_CFG="${BOARD_DIR}/genimage.cfg"
> +GENIMAGE_TMP="${BUILD_DIR}/genimage.tmp"
> +
> +rm -rf "${GENIMAGE_TMP}"
> +
> +genimage \
> + --rootpath "${TARGET_DIR}" \
> + --tmppath "${GENIMAGE_TMP}" \
> + --inputpath "${BINARIES_DIR}" \
> + --outputpath "${BINARIES_DIR}" \
> + --config "${GENIMAGE_CFG}"
Could you use support/scripts/genimage.sh instead ?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-15 17:24 [Buildroot] [PATCH] board: Add nanopi-m1 Support Chakra Divi
2017-06-15 20:15 ` Thomas Petazzoni
@ 2017-06-15 21:53 ` Arnout Vandecappelle
2017-06-16 16:22 ` Chakra D
1 sibling, 1 reply; 7+ messages in thread
From: Arnout Vandecappelle @ 2017-06-15 21:53 UTC (permalink / raw)
To: buildroot
Hi Chakra,
Thanks you for this contribution! I have a few more comments in addition to
Thomas's. But it looks pretty good!
On 15-06-17 19:24, Chakra Divi wrote:
> Add initial support for nanopi-m1 board
> with below features
> - U-Boot 2017.07-rc1
> - Linux 4.11
> - Default packages from buildroot
>
> Created a new folder nanopi under buildroot
> board; any new board for nanopi can be added
> here
Will you also provide a patch that moves nanopi-neo there, and reuses the same
scripts?
> Signed-off-by: Chakra Divi <chakra@openedev.com>
> ---
> board/nanopi/nanopi-m1/boot.cmd | 8 ++++++++
> board/nanopi/nanopi-m1/genimage.cfg | 36 ++++++++++++++++++++++++++++++++++
> board/nanopi/nanopi-m1/post-build.sh | 1 +
> board/nanopi/nanopi-m1/post-image.sh | 1 +
> board/nanopi/nanopi-m1/readme.txt | 29 +++++++++++++++++++++++++++
> board/nanopi/post-build.sh | 11 +++++++++++
> board/nanopi/post-image.sh | 14 +++++++++++++
> configs/nanopi_m1_defconfig | 29 +++++++++++++++++++++++++++
> 8 files changed, 129 insertions(+)
> create mode 100644 board/nanopi/nanopi-m1/boot.cmd
> create mode 100644 board/nanopi/nanopi-m1/genimage.cfg
> create mode 120000 board/nanopi/nanopi-m1/post-build.sh
> create mode 120000 board/nanopi/nanopi-m1/post-image.sh
> create mode 100644 board/nanopi/nanopi-m1/readme.txt
> create mode 100755 board/nanopi/post-build.sh
> create mode 100755 board/nanopi/post-image.sh
> create mode 100644 configs/nanopi_m1_defconfig
When adding a defconfig, please also run 'make .gitlab-ci.yml' to update the
Gitlab-CI configuration. Yes, we should update the documentation to mention this :-)
Also, please add yourself to the DEVELOPERS file for this board (both
config/xxx_defconfig and board/nanopi/).
[snip]
> diff --git a/board/nanopi/nanopi-m1/genimage.cfg b/board/nanopi/nanopi-m1/genimage.cfg
> new file mode 100644
> index 0000000..23c9c7d
> --- /dev/null
> +++ b/board/nanopi/nanopi-m1/genimage.cfg
> @@ -0,0 +1,36 @@
> +# Minimal SD card image for the NanoPi M1
> +#
> +image boot.vfat {
> + vfat {
> + files = {
> + "zImage",
> + "sun8i-h3-nanopi-m1.dtb",
> + "boot.scr"
> + }
> + }
> + size = 10M
> +}
> +
> +image sdcard.img {
> + hdimage {
> + }
> +
> + partition u-boot {
> + in-partition-table = "no"
> + image = "u-boot-sunxi-with-spl.bin"
> + offset = 8192
> + size = 1040384 # 1MB - 8192
> + }
> +
> + partition boot {
> + partition-type = 0xC
> + bootable = "true"
> + image = "boot.vfat"
> + }
> +
> + partition rootfs {
> + partition-type = 0x83
> + image = "rootfs.ext4"
> + size = 512M
No need to specify the size, genimage will take it from the ext4 file. Yes, I
know, many .cfg files don't specify this correctly.
> + }
> +}
> diff --git a/board/nanopi/nanopi-m1/post-build.sh b/board/nanopi/nanopi-m1/post-build.sh
> new file mode 120000
> index 0000000..cd7f116
> --- /dev/null
> +++ b/board/nanopi/nanopi-m1/post-build.sh
> @@ -0,0 +1 @@
> +../post-build.sh
> \ No newline at end of file
> diff --git a/board/nanopi/nanopi-m1/post-image.sh b/board/nanopi/nanopi-m1/post-image.sh
As mentioned by Thomas, the post-image script isn't needed, instead you can
call support/script/genimage.sh directly. You have to add
BR2_ROOTFS_POST_SCRIPT_ARGS="-c board/nanopi/nanopi-m1/genimage.cfg"
[snip]
> diff --git a/board/nanopi/post-build.sh b/board/nanopi/post-build.sh
> new file mode 100755
> index 0000000..a544a19
> --- /dev/null
> +++ b/board/nanopi/post-build.sh
> @@ -0,0 +1,11 @@
> +#!/bin/sh
> +# post-build.sh for OrangePi taken from CubieBoard's post-build.sh
I don't think this is relevant anymore...
> +# 2013, Carlo Caione <carlo.caione@gmail.com>
> +
> +BOARD_DIR="$(dirname $0)"
> +MKIMAGE=$HOST_DIR/usr/bin/mkimage
> +BOOT_CMD=$BOARD_DIR/boot.cmd
> +BOOT_CMD_H=$BINARIES_DIR/boot.scr
> +
> +# U-Boot script
> +$MKIMAGE -C none -A arm -T script -d $BOOT_CMD $BOOT_CMD_H
[snip]
> diff --git a/configs/nanopi_m1_defconfig b/configs/nanopi_m1_defconfig
> new file mode 100644
> index 0000000..b1443f8
> --- /dev/null
> +++ b/configs/nanopi_m1_defconfig
> @@ -0,0 +1,29 @@
> +BR2_arm=y
> +BR2_cortex_a7=y
> +BR2_ARM_FPU_VFPV4=y
> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_11=y
> +BR2_TARGET_GENERIC_HOSTNAME="nanopi-m1"
> +BR2_TARGET_GENERIC_ISSUE="Welcome to Buildroot for the NanoPi M1"
> +BR2_ROOTFS_POST_BUILD_SCRIPT="board/nanopi/nanopi-m1/post-build.sh"
> +BR2_ROOTFS_POST_IMAGE_SCRIPT="board/nanopi/nanopi-m1/post-image.sh"
> +BR2_LINUX_KERNEL=y
> +BR2_LINUX_KERNEL_CUSTOM_VERSION=y
> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="4.11"
Why not 4.11.5?
Regards,
Arnout
> +BR2_LINUX_KERNEL_DEFCONFIG="sunxi"
> +BR2_LINUX_KERNEL_DTS_SUPPORT=y
> +BR2_LINUX_KERNEL_INTREE_DTS_NAME="sun8i-h3-nanopi-m1"
> +BR2_TARGET_ROOTFS_EXT2=y
> +BR2_TARGET_ROOTFS_EXT2_4=y
> +# BR2_TARGET_ROOTFS_TAR is not set
> +BR2_TARGET_UBOOT=y
> +BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
> +BR2_TARGET_UBOOT_CUSTOM_VERSION=y
> +BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2017.07-rc1"
> +BR2_TARGET_UBOOT_BOARD_DEFCONFIG="nanopi_m1"
> +BR2_TARGET_UBOOT_NEEDS_DTC=y
> +BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
> +BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot-sunxi-with-spl.bin"
> +BR2_PACKAGE_HOST_DOSFSTOOLS=y
> +BR2_PACKAGE_HOST_GENIMAGE=y
> +BR2_PACKAGE_HOST_MTOOLS=y
> +BR2_PACKAGE_HOST_UBOOT_TOOLS=y
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-15 20:15 ` Thomas Petazzoni
@ 2017-06-16 16:13 ` Chakra D
0 siblings, 0 replies; 7+ messages in thread
From: Chakra D @ 2017-06-16 16:13 UTC (permalink / raw)
To: buildroot
Hi Thomas,
Thanks for the feedback - please find my answers inline
On Fri, Jun 16, 2017 at 1:45 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Hello,
>
> On Thu, 15 Jun 2017 22:54:25 +0530, Chakra Divi wrote:
>> Add initial support for nanopi-m1 board
>> with below features
>> - U-Boot 2017.07-rc1
>
> Are you using this -rc1 because it's the first U-Boot version with
> support for this board?
Yes i have verified and found that nanopi-m1 is supported from rc1. Is
this an issue ?
>
>> - Linux 4.11
>> - Default packages from buildroot
>>
>> Created a new folder nanopi under buildroot
>> board; any new board for nanopi can be added
>> here
>
> We already have the support for another nanopi board
> in board/nanopi-neo/. Perhaps some refactoring is needed?
As Arnout suggested, will move "nanopi-neo" also under this vendor folder
> Also is "nanopi" really the vendor name? We normally like to have
> board/<vendor>/<boardname>/.
vendor is "friendlyarm"
yuefei , can you please confirm this - shall i go ahead with folder
name "friendlyarm" ?
>> diff --git a/board/nanopi/post-image.sh b/board/nanopi/post-image.sh
>> new file mode 100755
>> index 0000000..359b4c9
>> --- /dev/null
>> +++ b/board/nanopi/post-image.sh
>> @@ -0,0 +1,14 @@
>> +#!/bin/sh
>> +
>> +BOARD_DIR="$(dirname $0)"
>> +GENIMAGE_CFG="${BOARD_DIR}/genimage.cfg"
>> +GENIMAGE_TMP="${BUILD_DIR}/genimage.tmp"
>> +
>> +rm -rf "${GENIMAGE_TMP}"
>> +
>> +genimage \
>> + --rootpath "${TARGET_DIR}" \
>> + --tmppath "${GENIMAGE_TMP}" \
>> + --inputpath "${BINARIES_DIR}" \
>> + --outputpath "${BINARIES_DIR}" \
>> + --config "${GENIMAGE_CFG}"
>
> Could you use support/scripts/genimage.sh instead ?
>
Ok will update
> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-15 21:53 ` Arnout Vandecappelle
@ 2017-06-16 16:22 ` Chakra D
2017-06-16 16:36 ` Chakra D
2017-06-16 21:08 ` Arnout Vandecappelle
0 siblings, 2 replies; 7+ messages in thread
From: Chakra D @ 2017-06-16 16:22 UTC (permalink / raw)
To: buildroot
Hi Arnout,
Thanks for the feedback....happy to see your comments as it is my
first patch in the buildroot
On Fri, Jun 16, 2017 at 3:23 AM, Arnout Vandecappelle <arnout@mind.be> wrote:
> Hi Chakra,
>
> Thanks you for this contribution! I have a few more comments in addition to
> Thomas's. But it looks pretty good!
>
> On 15-06-17 19:24, Chakra Divi wrote:
>> Add initial support for nanopi-m1 board
>> with below features
>> - U-Boot 2017.07-rc1
>> - Linux 4.11
>> - Default packages from buildroot
>>
>> Created a new folder nanopi under buildroot
>> board; any new board for nanopi can be added
>> here
>
> Will you also provide a patch that moves nanopi-neo there, and reuses the same
> scripts?
>
Yes will move it under vendor folder "friendlyarm"
>
>> Signed-off-by: Chakra Divi <chakra@openedev.com>
>> ---
>> board/nanopi/nanopi-m1/boot.cmd | 8 ++++++++
>> board/nanopi/nanopi-m1/genimage.cfg | 36 ++++++++++++++++++++++++++++++++++
>> board/nanopi/nanopi-m1/post-build.sh | 1 +
>> board/nanopi/nanopi-m1/post-image.sh | 1 +
>> board/nanopi/nanopi-m1/readme.txt | 29 +++++++++++++++++++++++++++
>> board/nanopi/post-build.sh | 11 +++++++++++
>> board/nanopi/post-image.sh | 14 +++++++++++++
>> configs/nanopi_m1_defconfig | 29 +++++++++++++++++++++++++++
>> 8 files changed, 129 insertions(+)
>> create mode 100644 board/nanopi/nanopi-m1/boot.cmd
>> create mode 100644 board/nanopi/nanopi-m1/genimage.cfg
>> create mode 120000 board/nanopi/nanopi-m1/post-build.sh
>> create mode 120000 board/nanopi/nanopi-m1/post-image.sh
>> create mode 100644 board/nanopi/nanopi-m1/readme.txt
>> create mode 100755 board/nanopi/post-build.sh
>> create mode 100755 board/nanopi/post-image.sh
>> create mode 100644 configs/nanopi_m1_defconfig
>
> When adding a defconfig, please also run 'make .gitlab-ci.yml' to update the
> Gitlab-CI configuration. Yes, we should update the documentation to mention this :-)
>
Yes will do and send updated patch
> Also, please add yourself to the DEVELOPERS file for this board (both
> config/xxx_defconfig and board/nanopi/).
>
Yes will do -
> [snip]
>> diff --git a/board/nanopi/nanopi-m1/genimage.cfg b/board/nanopi/nanopi-m1/genimage.cfg
>> new file mode 100644
>> index 0000000..23c9c7d
>> --- /dev/null
>> +++ b/board/nanopi/nanopi-m1/genimage.cfg
>> @@ -0,0 +1,36 @@
>> +# Minimal SD card image for the NanoPi M1
>> +#
>> +image boot.vfat {
>> + vfat {
>> + files = {
>> + "zImage",
>> + "sun8i-h3-nanopi-m1.dtb",
>> + "boot.scr"
>> + }
>> + }
>> + size = 10M
>> +}
>> +
>> +image sdcard.img {
>> + hdimage {
>> + }
>> +
>> + partition u-boot {
>> + in-partition-table = "no"
>> + image = "u-boot-sunxi-with-spl.bin"
>> + offset = 8192
>> + size = 1040384 # 1MB - 8192
>> + }
>> +
>> + partition boot {
>> + partition-type = 0xC
>> + bootable = "true"
>> + image = "boot.vfat"
>> + }
>> +
>> + partition rootfs {
>> + partition-type = 0x83
>> + image = "rootfs.ext4"
>> + size = 512M
>
> No need to specify the size, genimage will take it from the ext4 file. Yes, I
> know, many .cfg files don't specify this correctly.
>
Will make the change
>> + }
>> +}
>> diff --git a/board/nanopi/nanopi-m1/post-build.sh b/board/nanopi/nanopi-m1/post-build.sh
>> new file mode 120000
>> index 0000000..cd7f116
>> --- /dev/null
>> +++ b/board/nanopi/nanopi-m1/post-build.sh
>> @@ -0,0 +1 @@
>> +../post-build.sh
>> \ No newline at end of file
>> diff --git a/board/nanopi/nanopi-m1/post-image.sh b/board/nanopi/nanopi-m1/post-image.sh
>
> As mentioned by Thomas, the post-image script isn't needed, instead you can
> call support/script/genimage.sh directly. You have to add
> BR2_ROOTFS_POST_SCRIPT_ARGS="-c board/nanopi/nanopi-m1/genimage.cfg"
>
Do you mean add the following config def config file ?
BR2_ROOTFS_POST_SCRIPT_ARGS="-c support/script/genimage.sh'"
> [snip]
>> diff --git a/board/nanopi/post-build.sh b/board/nanopi/post-build.sh
>> new file mode 100755
>> index 0000000..a544a19
>> --- /dev/null
>> +++ b/board/nanopi/post-build.sh
>> @@ -0,0 +1,11 @@
>> +#!/bin/sh
>> +# post-build.sh for OrangePi taken from CubieBoard's post-build.sh
>
> I don't think this is relevant anymore...
>
Ok will remove it. I thought post-build.sh might be needed if a specific board
want to implement something different from normal flow.
>> +# 2013, Carlo Caione <carlo.caione@gmail.com>
>> +
>> +BOARD_DIR="$(dirname $0)"
>> +MKIMAGE=$HOST_DIR/usr/bin/mkimage
>> +BOOT_CMD=$BOARD_DIR/boot.cmd
>> +BOOT_CMD_H=$BINARIES_DIR/boot.scr
>> +
>> +# U-Boot script
>> +$MKIMAGE -C none -A arm -T script -d $BOOT_CMD $BOOT_CMD_H
> [snip]
>> diff --git a/configs/nanopi_m1_defconfig b/configs/nanopi_m1_defconfig
>> new file mode 100644
>> index 0000000..b1443f8
>> --- /dev/null
>> +++ b/configs/nanopi_m1_defconfig
>> @@ -0,0 +1,29 @@
>> +BR2_arm=y
>> +BR2_cortex_a7=y
>> +BR2_ARM_FPU_VFPV4=y
>> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_11=y
>> +BR2_TARGET_GENERIC_HOSTNAME="nanopi-m1"
>> +BR2_TARGET_GENERIC_ISSUE="Welcome to Buildroot for the NanoPi M1"
>> +BR2_ROOTFS_POST_BUILD_SCRIPT="board/nanopi/nanopi-m1/post-build.sh"
>> +BR2_ROOTFS_POST_IMAGE_SCRIPT="board/nanopi/nanopi-m1/post-image.sh"
>> +BR2_LINUX_KERNEL=y
>> +BR2_LINUX_KERNEL_CUSTOM_VERSION=y
>> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="4.11"
>
> Why not 4.11.5?
>
Will change it 4.11.5
> Regards,
> Arnout
>
>> +BR2_LINUX_KERNEL_DEFCONFIG="sunxi"
>> +BR2_LINUX_KERNEL_DTS_SUPPORT=y
>> +BR2_LINUX_KERNEL_INTREE_DTS_NAME="sun8i-h3-nanopi-m1"
>> +BR2_TARGET_ROOTFS_EXT2=y
>> +BR2_TARGET_ROOTFS_EXT2_4=y
>> +# BR2_TARGET_ROOTFS_TAR is not set
>> +BR2_TARGET_UBOOT=y
>> +BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
>> +BR2_TARGET_UBOOT_CUSTOM_VERSION=y
>> +BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2017.07-rc1"
>> +BR2_TARGET_UBOOT_BOARD_DEFCONFIG="nanopi_m1"
>> +BR2_TARGET_UBOOT_NEEDS_DTC=y
>> +BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
>> +BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot-sunxi-with-spl.bin"
>> +BR2_PACKAGE_HOST_DOSFSTOOLS=y
>> +BR2_PACKAGE_HOST_GENIMAGE=y
>> +BR2_PACKAGE_HOST_MTOOLS=y
>> +BR2_PACKAGE_HOST_UBOOT_TOOLS=y
>>
>
> --
> Arnout Vandecappelle arnout at mind be
> Senior Embedded Software Architect +32-16-286500
> Essensium/Mind http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-16 16:22 ` Chakra D
@ 2017-06-16 16:36 ` Chakra D
2017-06-16 21:08 ` Arnout Vandecappelle
1 sibling, 0 replies; 7+ messages in thread
From: Chakra D @ 2017-06-16 16:36 UTC (permalink / raw)
To: buildroot
Hi Arnout,
On Fri, Jun 16, 2017 at 9:52 PM, Chakra D <2chakrass@gmail.com> wrote:
> Hi Arnout,
>
> Thanks for the feedback....happy to see your comments as it is my
> first patch in the buildroot
>
> On Fri, Jun 16, 2017 at 3:23 AM, Arnout Vandecappelle <arnout@mind.be> wrote:
>> Hi Chakra,
>>
>> Thanks you for this contribution! I have a few more comments in addition to
>> Thomas's. But it looks pretty good!
>>
>> On 15-06-17 19:24, Chakra Divi wrote:
>>> Add initial support for nanopi-m1 board
>>> with below features
>>> - U-Boot 2017.07-rc1
>>> - Linux 4.11
>>> - Default packages from buildroot
>>>
>>> Created a new folder nanopi under buildroot
>>> board; any new board for nanopi can be added
>>> here
>>
>> Will you also provide a patch that moves nanopi-neo there, and reuses the same
>> scripts?
>>
>
> Yes will move it under vendor folder "friendlyarm"
>
>>
>>> Signed-off-by: Chakra Divi <chakra@openedev.com>
>>> ---
>>> board/nanopi/nanopi-m1/boot.cmd | 8 ++++++++
>>> board/nanopi/nanopi-m1/genimage.cfg | 36 ++++++++++++++++++++++++++++++++++
>>> board/nanopi/nanopi-m1/post-build.sh | 1 +
>>> board/nanopi/nanopi-m1/post-image.sh | 1 +
>>> board/nanopi/nanopi-m1/readme.txt | 29 +++++++++++++++++++++++++++
>>> board/nanopi/post-build.sh | 11 +++++++++++
>>> board/nanopi/post-image.sh | 14 +++++++++++++
>>> configs/nanopi_m1_defconfig | 29 +++++++++++++++++++++++++++
>>> 8 files changed, 129 insertions(+)
>>> create mode 100644 board/nanopi/nanopi-m1/boot.cmd
>>> create mode 100644 board/nanopi/nanopi-m1/genimage.cfg
>>> create mode 120000 board/nanopi/nanopi-m1/post-build.sh
>>> create mode 120000 board/nanopi/nanopi-m1/post-image.sh
>>> create mode 100644 board/nanopi/nanopi-m1/readme.txt
>>> create mode 100755 board/nanopi/post-build.sh
>>> create mode 100755 board/nanopi/post-image.sh
>>> create mode 100644 configs/nanopi_m1_defconfig
>>
>> When adding a defconfig, please also run 'make .gitlab-ci.yml' to update the
>> Gitlab-CI configuration. Yes, we should update the documentation to mention this :-)
>>
>
> Yes will do and send updated patch
>
>> Also, please add yourself to the DEVELOPERS file for this board (both
>> config/xxx_defconfig and board/nanopi/).
>>
>
> Yes will do -
>
>> [snip]
>>> diff --git a/board/nanopi/nanopi-m1/genimage.cfg b/board/nanopi/nanopi-m1/genimage.cfg
>>> new file mode 100644
>>> index 0000000..23c9c7d
>>> --- /dev/null
>>> +++ b/board/nanopi/nanopi-m1/genimage.cfg
>>> @@ -0,0 +1,36 @@
>>> +# Minimal SD card image for the NanoPi M1
>>> +#
>>> +image boot.vfat {
>>> + vfat {
>>> + files = {
>>> + "zImage",
>>> + "sun8i-h3-nanopi-m1.dtb",
>>> + "boot.scr"
>>> + }
>>> + }
>>> + size = 10M
>>> +}
>>> +
>>> +image sdcard.img {
>>> + hdimage {
>>> + }
>>> +
>>> + partition u-boot {
>>> + in-partition-table = "no"
>>> + image = "u-boot-sunxi-with-spl.bin"
>>> + offset = 8192
>>> + size = 1040384 # 1MB - 8192
>>> + }
>>> +
>>> + partition boot {
>>> + partition-type = 0xC
>>> + bootable = "true"
>>> + image = "boot.vfat"
>>> + }
>>> +
>>> + partition rootfs {
>>> + partition-type = 0x83
>>> + image = "rootfs.ext4"
>>> + size = 512M
>>
>> No need to specify the size, genimage will take it from the ext4 file. Yes, I
>> know, many .cfg files don't specify this correctly.
>>
>
> Will make the change
>
>>> + }
>>> +}
>>> diff --git a/board/nanopi/nanopi-m1/post-build.sh b/board/nanopi/nanopi-m1/post-build.sh
>>> new file mode 120000
>>> index 0000000..cd7f116
>>> --- /dev/null
>>> +++ b/board/nanopi/nanopi-m1/post-build.sh
>>> @@ -0,0 +1 @@
>>> +../post-build.sh
>>> \ No newline at end of file
>>> diff --git a/board/nanopi/nanopi-m1/post-image.sh b/board/nanopi/nanopi-m1/post-image.sh
>>
>> As mentioned by Thomas, the post-image script isn't needed, instead you can
>> call support/script/genimage.sh directly. You have to add
>> BR2_ROOTFS_POST_SCRIPT_ARGS="-c board/nanopi/nanopi-m1/genimage.cfg"
>>
>
> Do you mean add the following config def config file ?
> BR2_ROOTFS_POST_SCRIPT_ARGS="-c support/script/genimage.sh'"
>
Sorry about this statement...Understood now will make the changes
>> [snip]
>>> diff --git a/board/nanopi/post-build.sh b/board/nanopi/post-build.sh
>>> new file mode 100755
>>> index 0000000..a544a19
>>> --- /dev/null
>>> +++ b/board/nanopi/post-build.sh
>>> @@ -0,0 +1,11 @@
>>> +#!/bin/sh
>>> +# post-build.sh for OrangePi taken from CubieBoard's post-build.sh
>>
>> I don't think this is relevant anymore...
>>
> Ok will remove it. I thought post-build.sh might be needed if a specific board
> want to implement something different from normal flow.
>
>>> +# 2013, Carlo Caione <carlo.caione@gmail.com>
>>> +
>>> +BOARD_DIR="$(dirname $0)"
>>> +MKIMAGE=$HOST_DIR/usr/bin/mkimage
>>> +BOOT_CMD=$BOARD_DIR/boot.cmd
>>> +BOOT_CMD_H=$BINARIES_DIR/boot.scr
>>> +
>>> +# U-Boot script
>>> +$MKIMAGE -C none -A arm -T script -d $BOOT_CMD $BOOT_CMD_H
>> [snip]
>>> diff --git a/configs/nanopi_m1_defconfig b/configs/nanopi_m1_defconfig
>>> new file mode 100644
>>> index 0000000..b1443f8
>>> --- /dev/null
>>> +++ b/configs/nanopi_m1_defconfig
>>> @@ -0,0 +1,29 @@
>>> +BR2_arm=y
>>> +BR2_cortex_a7=y
>>> +BR2_ARM_FPU_VFPV4=y
>>> +BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_4_11=y
>>> +BR2_TARGET_GENERIC_HOSTNAME="nanopi-m1"
>>> +BR2_TARGET_GENERIC_ISSUE="Welcome to Buildroot for the NanoPi M1"
>>> +BR2_ROOTFS_POST_BUILD_SCRIPT="board/nanopi/nanopi-m1/post-build.sh"
>>> +BR2_ROOTFS_POST_IMAGE_SCRIPT="board/nanopi/nanopi-m1/post-image.sh"
>>> +BR2_LINUX_KERNEL=y
>>> +BR2_LINUX_KERNEL_CUSTOM_VERSION=y
>>> +BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="4.11"
>>
>> Why not 4.11.5?
>>
>
> Will change it 4.11.5
>
>> Regards,
>> Arnout
>>
>>> +BR2_LINUX_KERNEL_DEFCONFIG="sunxi"
>>> +BR2_LINUX_KERNEL_DTS_SUPPORT=y
>>> +BR2_LINUX_KERNEL_INTREE_DTS_NAME="sun8i-h3-nanopi-m1"
>>> +BR2_TARGET_ROOTFS_EXT2=y
>>> +BR2_TARGET_ROOTFS_EXT2_4=y
>>> +# BR2_TARGET_ROOTFS_TAR is not set
>>> +BR2_TARGET_UBOOT=y
>>> +BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
>>> +BR2_TARGET_UBOOT_CUSTOM_VERSION=y
>>> +BR2_TARGET_UBOOT_CUSTOM_VERSION_VALUE="2017.07-rc1"
>>> +BR2_TARGET_UBOOT_BOARD_DEFCONFIG="nanopi_m1"
>>> +BR2_TARGET_UBOOT_NEEDS_DTC=y
>>> +BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
>>> +BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot-sunxi-with-spl.bin"
>>> +BR2_PACKAGE_HOST_DOSFSTOOLS=y
>>> +BR2_PACKAGE_HOST_GENIMAGE=y
>>> +BR2_PACKAGE_HOST_MTOOLS=y
>>> +BR2_PACKAGE_HOST_UBOOT_TOOLS=y
>>>
>>
>> --
>> Arnout Vandecappelle arnout at mind be
>> Senior Embedded Software Architect +32-16-286500
>> Essensium/Mind http://www.mind.be
>> G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
>> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
>> GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] board: Add nanopi-m1 Support
2017-06-16 16:22 ` Chakra D
2017-06-16 16:36 ` Chakra D
@ 2017-06-16 21:08 ` Arnout Vandecappelle
1 sibling, 0 replies; 7+ messages in thread
From: Arnout Vandecappelle @ 2017-06-16 21:08 UTC (permalink / raw)
To: buildroot
On 16-06-17 18:22, Chakra D wrote:
>>> +++ b/board/nanopi/post-build.sh
>>> @@ -0,0 +1,11 @@
>>> +#!/bin/sh
>>> +# post-build.sh for OrangePi taken from CubieBoard's post-build.sh
>> I don't think this is relevant anymore...
>>
> Ok will remove it. I thought post-build.sh might be needed if a specific board
> want to implement something different from normal flow.
I meant that the comment is not relevant anymore (in particular the OrangePi part).
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-06-16 21:08 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-15 17:24 [Buildroot] [PATCH] board: Add nanopi-m1 Support Chakra Divi
2017-06-15 20:15 ` Thomas Petazzoni
2017-06-16 16:13 ` Chakra D
2017-06-15 21:53 ` Arnout Vandecappelle
2017-06-16 16:22 ` Chakra D
2017-06-16 16:36 ` Chakra D
2017-06-16 21:08 ` Arnout Vandecappelle
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.