[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Luca Ceresoli]

Hi,

this patchset adds support to the DualLite versions of the i.MX6 SABRESD board
in addition to the Quad variant that is already supported. It also documentats
how to generate a bootable SD.

Patch 4 by Vincent makes it possible to boot from USB without having to
manually modify the target rootfs during the build process.

Changes since v3:
 - rebased on current master;
 - added Vincent patch for USB boot.

Luca Ceresoli (3):
  configs/freescale_imx6qsabresd: rename imx6 to imx6q
  configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD
  freescale/imx6-sabresd: document how to create a bootable SD

Vincent Stehl? (1):
  imx6sabresd: boot to /init in mfgtools mode

 board/freescale/imx6sabresd/create-boot-sd.sh      | 95 ++++++++++++++++++++++
 board/freescale/imx6sabresd/readme.txt             | 67 +++++++++++++++
 ...ommon-boot-Linux-to-init-in-mfgtools-mode.patch | 36 ++++++++
 configs/freescale_imx6dlsabresd_defconfig          | 28 +++++++
 configs/freescale_imx6qsabresd_defconfig           | 28 +++++++
 configs/freescale_imx6sabresd_defconfig            | 27 ------
 6 files changed, 254 insertions(+), 27 deletions(-)
 create mode 100755 board/freescale/imx6sabresd/create-boot-sd.sh
 create mode 100644 board/freescale/imx6sabresd/readme.txt
 create mode 100644 board/freescale/imx6sabresd/uboot-0001-mx6qsabre_common-boot-Linux-to-init-in-mfgtools-mode.patch
 create mode 100644 configs/freescale_imx6dlsabresd_defconfig
 create mode 100644 configs/freescale_imx6qsabresd_defconfig
 delete mode 100644 configs/freescale_imx6sabresd_defconfig

-- 
Luca

-- 
1.9.1

[Buildroot] [PATCH v4 1/4] configs/freescale_imx6qsabresd: rename imx6 to imx6q

[Luca Ceresoli]

This defconfig is specific to the SABRESD board equipped with the iMX6Q SoC.
Rename it for clarity in order to avoid confusion with the upcoming
defconfig for the iMX6DL version.

Suggested by Gary Bisson.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gilles Talis <gilles.talis@gmail.com>
Cc: Gary Bisson <bisson.gary@gmail.com>
Cc: Vincent Stehl? <vincent.stehle@freescale.com>

---

Changes since v3:
 - rebased on current master.

This patch was added in v3.

---
 configs/freescale_imx6qsabresd_defconfig | 27 +++++++++++++++++++++++++++
 configs/freescale_imx6sabresd_defconfig  | 27 ---------------------------
 2 files changed, 27 insertions(+), 27 deletions(-)
 create mode 100644 configs/freescale_imx6qsabresd_defconfig
 delete mode 100644 configs/freescale_imx6sabresd_defconfig

diff --git a/configs/freescale_imx6qsabresd_defconfig b/configs/freescale_imx6qsabresd_defconfig
new file mode 100644
index 0000000..16e4d0b
--- /dev/null
+++ b/configs/freescale_imx6qsabresd_defconfig
@@ -0,0 +1,27 @@
+# architecture
+BR2_arm=y
+BR2_cortex_a9=y
+
+# system
+BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
+
+# kernel
+BR2_LINUX_KERNEL=y
+BR2_LINUX_KERNEL_CUSTOM_GIT=y
+BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
+BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
+BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
+BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_DTS_SUPPORT=y
+BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6q-sabresd"
+
+# filesystem
+BR2_TARGET_ROOTFS_EXT2=y
+
+# bootloader
+BR2_TARGET_UBOOT=y
+BR2_TARGET_UBOOT_BOARDNAME="mx6qsabresd"
+BR2_TARGET_UBOOT_FORMAT_IMX=y
+BR2_TARGET_UBOOT_CUSTOM_GIT=y
+BR2_TARGET_UBOOT_CUSTOM_REPO_URL="git://git.freescale.com/imx/uboot-imx.git"
+BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
diff --git a/configs/freescale_imx6sabresd_defconfig b/configs/freescale_imx6sabresd_defconfig
deleted file mode 100644
index 16e4d0b..0000000
--- a/configs/freescale_imx6sabresd_defconfig
+++ /dev/null
@@ -1,27 +0,0 @@
-# architecture
-BR2_arm=y
-BR2_cortex_a9=y
-
-# system
-BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
-
-# kernel
-BR2_LINUX_KERNEL=y
-BR2_LINUX_KERNEL_CUSTOM_GIT=y
-BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
-BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
-BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
-BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
-BR2_LINUX_KERNEL_DTS_SUPPORT=y
-BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6q-sabresd"
-
-# filesystem
-BR2_TARGET_ROOTFS_EXT2=y
-
-# bootloader
-BR2_TARGET_UBOOT=y
-BR2_TARGET_UBOOT_BOARDNAME="mx6qsabresd"
-BR2_TARGET_UBOOT_FORMAT_IMX=y
-BR2_TARGET_UBOOT_CUSTOM_GIT=y
-BR2_TARGET_UBOOT_CUSTOM_REPO_URL="git://git.freescale.com/imx/uboot-imx.git"
-BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
-- 
1.9.1

[Buildroot] [PATCH v4 2/4] configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD

[Luca Ceresoli]

This is very similar to freescale_imx6qsabresd_defconfig, which supports the
i.MX6Q ("Quad", quad core CPU) version of the board, except it supports the
i.MX6DL (DualLite, dual core CPU) version.

The differences are in the U-boot configuration and the device tree file.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gilles Talis <gilles.talis@gmail.com>
Cc: Gary Bisson <bisson.gary@gmail.com>
Cc: Vincent Stehl? <vincent.stehle@freescale.com>

---
Changes since v3:
 - rebased on current master.

Changes since v2: nothing.
Changes since v1: nothing.
---
 configs/freescale_imx6dlsabresd_defconfig | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 configs/freescale_imx6dlsabresd_defconfig

diff --git a/configs/freescale_imx6dlsabresd_defconfig b/configs/freescale_imx6dlsabresd_defconfig
new file mode 100644
index 0000000..4177e16
--- /dev/null
+++ b/configs/freescale_imx6dlsabresd_defconfig
@@ -0,0 +1,27 @@
+# architecture
+BR2_arm=y
+BR2_cortex_a9=y
+
+# system
+BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
+
+# kernel
+BR2_LINUX_KERNEL=y
+BR2_LINUX_KERNEL_CUSTOM_GIT=y
+BR2_LINUX_KERNEL_CUSTOM_REPO_URL="git://git.freescale.com/imx/linux-2.6-imx.git"
+BR2_LINUX_KERNEL_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
+BR2_LINUX_KERNEL_DEFCONFIG="imx_v7"
+BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
+BR2_LINUX_KERNEL_DTS_SUPPORT=y
+BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6dl-sabresd"
+
+# filesystem
+BR2_TARGET_ROOTFS_EXT2=y
+
+# bootloader
+BR2_TARGET_UBOOT=y
+BR2_TARGET_UBOOT_BOARDNAME="mx6dlsabresd"
+BR2_TARGET_UBOOT_FORMAT_IMX=y
+BR2_TARGET_UBOOT_CUSTOM_GIT=y
+BR2_TARGET_UBOOT_CUSTOM_REPO_URL="git://git.freescale.com/imx/uboot-imx.git"
+BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
-- 
1.9.1

[Buildroot] [PATCH v4 3/4] freescale/imx6-sabresd: document how to create a bootable SD

[Luca Ceresoli]

Document how to create a bootable SD card for the two supported SABRESD cards:
i.MX6Q and i.MX6DL.

The SD card creation relies on an ad-hoc script.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gilles Talis <gilles.talis@gmail.com>
Cc: Gary Bisson <bisson.gary@gmail.com>
Cc: Vincent Stehl? <vincent.stehle@freescale.com>

---

Changes since v3:
 - rebased on current master.

Changes since v2:
 - when initially wiping the SD, also wipe the ext2 partition to prevent it
   from being automounted just after partitioning on desktop systems;
 - readme.txt: rename freescale_imx6{,q}sabresd_defconfig to take into account
   the added patch that renames the defconfig.

Changes since v1:
 - properly handle SD/MMC devices that don't need the "p" prefix before
   the partition number (reported by Vincent Stehl?);
 - mention serial terminal parameters in readme.txt (suggested by Vincent
   Stehl?).
---
 board/freescale/imx6sabresd/create-boot-sd.sh | 95 +++++++++++++++++++++++++++
 board/freescale/imx6sabresd/readme.txt        | 67 +++++++++++++++++++
 2 files changed, 162 insertions(+)
 create mode 100755 board/freescale/imx6sabresd/create-boot-sd.sh
 create mode 100644 board/freescale/imx6sabresd/readme.txt

diff --git a/board/freescale/imx6sabresd/create-boot-sd.sh b/board/freescale/imx6sabresd/create-boot-sd.sh
new file mode 100755
index 0000000..9930b1e
--- /dev/null
+++ b/board/freescale/imx6sabresd/create-boot-sd.sh
@@ -0,0 +1,95 @@
+#!/bin/sh
+
+set -u
+set -e
+
+PROGNAME=$(basename $0)
+
+usage()
+{
+    echo "Create an SD card that boots on an i.MX6DL SABRESD board."
+    echo "It is supposed to work also for the i.MX6Q SABRESD board, but"
+    echo "it this is not tested."
+    echo
+    echo "Note: all data on the the card will be completely deleted!"
+    echo "Use with care!"
+    echo "Superuser permissions may be required to write to the device."
+    echo
+    echo "Usage: ${PROGNAME} <sd_block_device>"
+    echo "Arguments:"
+    echo "  <sd_block_device>     The device to be written to"
+    echo
+    echo "Example: ${PROGNAME} /dev/mmcblk0"
+    echo
+}
+
+if [ $# -ne 1 ]; then
+    usage
+    exit 1
+fi
+
+if [ $(id -u) -ne 0 ]; then
+    echo "${PROGNAME} must be run as root"
+    exit 1
+fi
+
+DEV=${1}
+
+# The partition name prefix depends on the device name:
+# - /dev/sde -> /dev/sde1
+# - /dev/mmcblk0 -> /dev/mmcblk0p1
+if echo ${DEV}|grep -q mmcblk ; then
+    PART="p"
+else
+    PART=""
+fi
+
+PART1=${DEV}${PART}1
+PART2=${DEV}${PART}2
+
+# Unmount the partitions if mounted
+umount ${PART1} || true
+umount ${PART2} || true
+
+# First, clear the card
+dd if=/dev/zero of=${DEV} bs=1M count=20
+
+sync
+
+# Partition the card.
+# SD layout for i.MX6 boot:
+# - Bootloader at offset 1024
+# - FAT partition starting at 1MB offset, containing uImage and *.dtb
+# - ext2/3 partition formatted as ext2 or ext3, containing the root filesystem.
+sfdisk ${DEV} <<EOF
+32,480,b
+512,,L
+EOF
+
+sync
+
+# Copy the bootloader@offset 1024
+dd if=output/images/u-boot.imx of=${DEV} obs=512 seek=2
+
+# Prepare a temp dir for mounting partitions
+TMPDIR=$(mktemp -d)
+
+# FAT partition: kernel and DTBs
+mkfs.vfat ${PART1}
+mount ${PART1} ${TMPDIR}
+cp output/images/uImage ${TMPDIR}/
+cp output/images/*.dtb  ${TMPDIR}/
+sync
+umount ${TMPDIR}
+
+# ext2 partition: root filesystem
+mkfs.ext2 ${PART2}
+mount ${PART2} ${TMPDIR}
+tar -C ${TMPDIR}/ -xf output/images/rootfs.tar
+sync
+umount ${TMPDIR}
+
+# Cleanup
+rmdir ${TMPDIR}
+sync
+echo Done
diff --git a/board/freescale/imx6sabresd/readme.txt b/board/freescale/imx6sabresd/readme.txt
new file mode 100644
index 0000000..583ef40
--- /dev/null
+++ b/board/freescale/imx6sabresd/readme.txt
@@ -0,0 +1,67 @@
+*******************************************************
+Freescale i.MX6Q and i.MX6DL SABRESD development boards
+*******************************************************
+
+This file documents the Buildroot support for the Freescale SABRE Board for
+Smart Devices Based on the i.MX 6 Series (SABRESD).
+
+Read the SABRESD Quick Start Guide for an introduction to the board:
+http://cache.freescale.com/files/32bit/doc/quick_start_guide/SABRESDB_IMX6_QSG.pdf
+
+Build
+=====
+
+First, configure Buildroot for your SABRESD board.
+For i.MX6Q:
+
+  make freescale_imx6qsabresd_defconfig
+
+For i.MX6DL:
+
+  make freescale_imx6dlsabresd_defconfig
+
+Build all components:
+
+  make
+
+You will find in ./output/images/ the following files:
+  - imx6dl-sabresd.dtb or imx6q-sabresd.dtb
+  - rootfs.ext2
+  - rootfs.tar
+  - u-boot.imx
+  - uImage
+
+Create a bootable SD card
+=========================
+
+To determine the device associated to the SD card have a look in the
+/proc/partitions file:
+
+  cat /proc/partitions
+
+Run the following script as root on your SD card. This will partition the card
+and copy the bootloader, kernel, DTBs and root filesystem as needed.
+
+*** WARNING! The script will destroy all the card content. Use with care! ***
+
+  ./board/freescale/imx6sabresd/create-boot-sd.sh <your-sd-device>
+
+Boot the SABRESD board
+======================
+
+To boot your newly created system (refer to the SABRESD Quick Start Guide for
+guidance):
+- insert the SD card in the SD3 slot of the board;
+- locate the BOOT dip switches (SW6), set dips 2 and 7 to ON, all others to OFF;
+- connect a Micro USB cable to Debug Port and connect using a terminal emulator
+  at 115200 bps, 8n1;
+- power on the board.
+
+Enjoy!
+
+References
+==========
+
+https://community.freescale.com/docs/DOC-95015
+https://community.freescale.com/docs/DOC-95017
+https://community.freescale.com/docs/DOC-99218
-- 
1.9.1

[Buildroot] [PATCH v4 4/4] imx6sabresd: boot to /init in mfgtools mode

[Luca Ceresoli]

From: Vincent Stehl? <vincent.stehle@freescale.com>

Buildroot supplies a nice /init wrapper script to use when booting from a
ramdisk.

We add a patch to u-boot to tell the kernel to boot into /init (instead of
/linuxrc) on i.MX6, when booting in mfgtools mode. This way we can boot a
buildroot system entirely through USB.

Signed-off-by: Vincent Stehl? <vincent.stehle at freescale.com
[Luca: rebase on top of iMX6DL patches and patch iMX6DL defconfig as well]
Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
[Luca: build- and run-tested on i.MX6DL SABRESD]
Tested-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Gary Bisson <bisson.gary@gmail.com>
Cc: Thomas De Schampheleire <patrickdepinguin+buildroot@gmail.com>
Cc: Gilles Talis <gilles.talis@gmail.com>
---

The original version of this patch is here:
  http://patchwork.ozlabs.org/patch/379269/

This patch was sent added to this patch series in v4, and it has been
rebased and adapted to work also for i.MX6DL.

---

 ...ommon-boot-Linux-to-init-in-mfgtools-mode.patch | 36 ++++++++++++++++++++++
 configs/freescale_imx6dlsabresd_defconfig          |  1 +
 configs/freescale_imx6qsabresd_defconfig           |  1 +
 3 files changed, 38 insertions(+)
 create mode 100644 board/freescale/imx6sabresd/uboot-0001-mx6qsabre_common-boot-Linux-to-init-in-mfgtools-mode.patch

diff --git a/board/freescale/imx6sabresd/uboot-0001-mx6qsabre_common-boot-Linux-to-init-in-mfgtools-mode.patch b/board/freescale/imx6sabresd/uboot-0001-mx6qsabre_common-boot-Linux-to-init-in-mfgtools-mode.patch
new file mode 100644
index 0000000..c01df4e
--- /dev/null
+++ b/board/freescale/imx6sabresd/uboot-0001-mx6qsabre_common-boot-Linux-to-init-in-mfgtools-mode.patch
@@ -0,0 +1,36 @@
+From 4a7cd7c5b165317dccf45cfc235da3e14bc339e8 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Vincent=20Stehl=C3=A9?= <vincent.stehle@freescale.com>
+Date: Tue, 12 Aug 2014 10:17:31 +0200
+Subject: [PATCH] mx6qsabre_common: boot Linux to /init in mfgtools mode
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Buildroot supplies a nice /init wrapper script to use when booting from a
+ramdisk.
+
+This patch tells u-boot to tell the kernel to boot into /init (instead of
+/linuxrc) on i.MX6, when booting in mfgtools mode. This way we can boot a
+buildroot system entirely through USB.
+
+Signed-off-by: Vincent Stehl? <vincent.stehle@freescale.com>
+---
+ include/configs/mx6qsabre_common.h | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/include/configs/mx6qsabre_common.h b/include/configs/mx6qsabre_common.h
+index 69ff0df..ec7b665 100644
+--- a/include/configs/mx6qsabre_common.h
++++ b/include/configs/mx6qsabre_common.h
+@@ -129,7 +129,7 @@
+ 
+ #define CONFIG_MFG_ENV_SETTINGS \
+ 	"mfgtool_args=setenv bootargs console=" CONFIG_CONSOLE_DEV ",115200 " \
+-		"rdinit=/linuxrc " \
++		"rdinit=/init " \
+ 		"g_mass_storage.stall=0 g_mass_storage.removable=1 " \
+ 		"g_mass_storage.idVendor=0x066F g_mass_storage.idProduct=0x37FF "\
+ 		"g_mass_storage.iSerialNumber=\"\" "\
+-- 
+2.0.1
+
diff --git a/configs/freescale_imx6dlsabresd_defconfig b/configs/freescale_imx6dlsabresd_defconfig
index 4177e16..62caf79 100644
--- a/configs/freescale_imx6dlsabresd_defconfig
+++ b/configs/freescale_imx6dlsabresd_defconfig
@@ -25,3 +25,4 @@ BR2_TARGET_UBOOT_FORMAT_IMX=y
 BR2_TARGET_UBOOT_CUSTOM_GIT=y
 BR2_TARGET_UBOOT_CUSTOM_REPO_URL="git://git.freescale.com/imx/uboot-imx.git"
 BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
+BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="board/freescale/imx6sabresd"
diff --git a/configs/freescale_imx6qsabresd_defconfig b/configs/freescale_imx6qsabresd_defconfig
index 16e4d0b..bc0aa46 100644
--- a/configs/freescale_imx6qsabresd_defconfig
+++ b/configs/freescale_imx6qsabresd_defconfig
@@ -25,3 +25,4 @@ BR2_TARGET_UBOOT_FORMAT_IMX=y
 BR2_TARGET_UBOOT_CUSTOM_GIT=y
 BR2_TARGET_UBOOT_CUSTOM_REPO_URL="git://git.freescale.com/imx/uboot-imx.git"
 BR2_TARGET_UBOOT_CUSTOM_REPO_VERSION="rel_imx_3.10.17_1.0.0_ga"
+BR2_TARGET_UBOOT_CUSTOM_PATCH_DIR="board/freescale/imx6sabresd"
-- 
1.9.1

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Thomas Petazzoni]

Dear Luca Ceresoli,

On Fri, 19 Sep 2014 16:12:48 +0200, Luca Ceresoli wrote:

> Luca Ceresoli (3):
>   configs/freescale_imx6qsabresd: rename imx6 to imx6q
>   configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD
>   freescale/imx6-sabresd: document how to create a bootable SD
> 
> Vincent Stehl? (1):
>   imx6sabresd: boot to /init in mfgtools mode

I've applied your four patches. However, I'd like to see additional
patches to make the following improvements:

 * For the defconfigs, select a specific version of the kernel headers.
   Currently, your defconfigs are building a 3.10 kernel, but the
   kernel headers used in the toolchain are the latest ones (i.e 3.16
   currently), which is not good. This was already wrong in the
   existing defconfig, so I applied your patches nonetheless. But it
   should be fixed.

 * For the last patch adding a U-Boot patch, I believe we would prefer
   to use the global patch dir mechanism now, rather than the U-Boot
   specific option for patches.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Luca Ceresoli]

Dear Thomas,

thanks for your suggestions.

Thomas Petazzoni wrote:
> Dear Luca Ceresoli,
>
> On Fri, 19 Sep 2014 16:12:48 +0200, Luca Ceresoli wrote:
>
>> Luca Ceresoli (3):
>>    configs/freescale_imx6qsabresd: rename imx6 to imx6q
>>    configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD
>>    freescale/imx6-sabresd: document how to create a bootable SD
>>
>> Vincent Stehl? (1):
>>    imx6sabresd: boot to /init in mfgtools mode
>
> I've applied your four patches. However, I'd like to see additional
> patches to make the following improvements:
>
>   * For the defconfigs, select a specific version of the kernel headers.
>     Currently, your defconfigs are building a 3.10 kernel, but the
>     kernel headers used in the toolchain are the latest ones (i.e 3.16
>     currently), which is not good. This was already wrong in the
>     existing defconfig, so I applied your patches nonetheless. But it
>     should be fixed.

I'm working on this. I switched from Freescale's git repo to vanilla
Linux, since the Freescale releases do not provide any later version.

However I'm going to test this before sending a patch, to make sure it
works.

>
>   * For the last patch adding a U-Boot patch, I believe we would prefer
>     to use the global patch dir mechanism now, rather than the U-Boot
>     specific option for patches.

Yeah, fixed that. Will send together with the kernel version patch.

-- 
Luca

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Luca Ceresoli]

Dear Vincent,

Thomas Petazzoni wrote:
> Dear Luca Ceresoli,
>
> On Fri, 19 Sep 2014 16:12:48 +0200, Luca Ceresoli wrote:
>
>> Luca Ceresoli (3):
>>    configs/freescale_imx6qsabresd: rename imx6 to imx6q
>>    configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD
>>    freescale/imx6-sabresd: document how to create a bootable SD
>>
>> Vincent Stehl? (1):
>>    imx6sabresd: boot to /init in mfgtools mode
>
> I've applied your four patches. However, I'd like to see additional

Now that your patch for USB booting in mfgtools mode has been applied,
you may simplify the booting procedure documented in
https://community.freescale.com/docs/DOC-101842 to remove the tweaks
that are not needed anymore.

Also, the sentence "there are no files for the sabre sd in the boards
folder" is not true anymore.

-- 
Luca

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Thomas Petazzoni]

Dear Luca Ceresoli,

On Mon, 22 Sep 2014 15:31:31 +0200, Luca Ceresoli wrote:

> Now that your patch for USB booting in mfgtools mode has been applied,
> you may simplify the booting procedure documented in
> https://community.freescale.com/docs/DOC-101842 to remove the tweaks
> that are not needed anymore.
> 
> Also, the sentence "there are no files for the sabre sd in the boards
> folder" is not true anymore.

Also, why don't we have a package to build the host tool
imx-usb-loader? That would even further simply the instructions.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Luca Ceresoli]

Hi,

Luca Ceresoli wrote:
> Dear Thomas,
>
> thanks for your suggestions.
>
> Thomas Petazzoni wrote:
>> Dear Luca Ceresoli,
>>
>> On Fri, 19 Sep 2014 16:12:48 +0200, Luca Ceresoli wrote:
>>
>>> Luca Ceresoli (3):
>>>    configs/freescale_imx6qsabresd: rename imx6 to imx6q
>>>    configs/freescale_imx6dlsabresd: add defconfig for iMX6DL SabreSD
>>>    freescale/imx6-sabresd: document how to create a bootable SD
>>>
>>> Vincent Stehl? (1):
>>>    imx6sabresd: boot to /init in mfgtools mode
>>
>> I've applied your four patches. However, I'd like to see additional
>> patches to make the following improvements:
>>
>>   * For the defconfigs, select a specific version of the kernel headers.
>>     Currently, your defconfigs are building a 3.10 kernel, but the
>>     kernel headers used in the toolchain are the latest ones (i.e 3.16
>>     currently), which is not good. This was already wrong in the
>>     existing defconfig, so I applied your patches nonetheless. But it
>>     should be fixed.
>
> I'm working on this. I switched from Freescale's git repo to vanilla
> Linux, since the Freescale releases do not provide any later version.

...or maybe I'll do the other way around: use the kernel headers from
Freescale's custom git repo, since the defconfig we currently use
(imx_v7_defconfig) is not available in mainline, and the closest one
in mainline (imx_v6_v7_defconfig) has several differences, and I'd like
to keep things simple and working.

I wonder if there's a good reason to use a mainline kernel instead.

-- 
Luca

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Luca Ceresoli]

Dear Thomas,

Thomas Petazzoni wrote:
> Dear Luca Ceresoli,
>
> On Mon, 22 Sep 2014 15:31:31 +0200, Luca Ceresoli wrote:
>
>> Now that your patch for USB booting in mfgtools mode has been applied,
>> you may simplify the booting procedure documented in
>> https://community.freescale.com/docs/DOC-101842 to remove the tweaks
>> that are not needed anymore.
>>
>> Also, the sentence "there are no files for the sabre sd in the boards
>> folder" is not true anymore.
>
> Also, why don't we have a package to build the host tool
> imx-usb-loader? That would even further simply the instructions.

Sure it would. It's on my todo list, but I haven't started looking at
it yet. It may be be in the next patchset unless I hit any build issues.

-- 
Luca

[Buildroot] [PATCH v4 0/4] Improvements to i.MX6 SABRESD boards support

[Thomas Petazzoni]

Dear Luca Ceresoli,

On Mon, 22 Sep 2014 16:41:33 +0200, Luca Ceresoli wrote:

> ...or maybe I'll do the other way around: use the kernel headers from
> Freescale's custom git repo, since the defconfig we currently use
> (imx_v7_defconfig) is not available in mainline, and the closest one
> in mainline (imx_v6_v7_defconfig) has several differences, and I'd like
> to keep things simple and working.
> 
> I wonder if there's a good reason to use a mainline kernel instead.

Well, in general, mainline is nicer to look at. But from a practical
point of view, the Freescale kernel is probably the one they have
tested with all the multimedia features (OpenGL implementation,
hardware accelerated video encoding/decoding) and so on. So I'd say
that a Buildroot defconfig should probably use the Freescale kernel
preferably, so that most things work out of the box for users. (And it
*really* hurts me to say we should use vendor kernels instead of
mainline kernels, as I hate vendor kernels!).

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com