diff mbox

[2/2] freescale/imx6-sabresd: use up-to-date mainline Linux

Message ID 1411560193-9697-3-git-send-email-luca@lucaceresoli.net
State Rejected
Headers show

Commit Message

Luca Ceresoli Sept. 24, 2014, 12:03 p.m. UTC
The current defconfigs for iMX6 SABRESD a kernel from the Freescale git
repository and based on 3.10. This kernel is older than the toolchain kernel
headers used in Buildroot (currently 3.16.3).

Move to mainline Linux and upgrade the kernel version to the same version
used for the toolchain kernel headers.

Also switch the Linux defconfig from imx_v7 to imx_v6_v7 since the former is
not in mainline.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Cc: Vincent Stehlé <vincent.stehle@freescale.com>
Cc: Gilles Talis <gilles.talis@gmail.com>
Cc: Gary Bisson <bisson.gary@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 configs/freescale_imx6dlsabresd_defconfig | 6 ++----
 configs/freescale_imx6qsabresd_defconfig  | 6 ++----
 2 files changed, 4 insertions(+), 8 deletions(-)

Comments

Vincent Stehlé Sept. 24, 2014, 12:38 p.m. UTC | #1
On 09/24/2014 02:03 PM, Luca Ceresoli wrote:
> Move to mainline Linux and upgrade the kernel version to the same version
> used for the toolchain kernel headers.

Hi Luca,

I am not sure if abandoning the Freescale release kernel is the best
option for i.MX6. Indeed, if you want e.g. proper video
(package/freescale-imx/imx-vpu) and 3D
(package/freescale-imx/gpu-viv-bin-mx6q) support to still work as
expected you need to stay with the 3.10.17.

Wouldn't you prefer to align the headers instead?

Best regards,

V.
Luca Ceresoli Sept. 24, 2014, 1:47 p.m. UTC | #2
Dear Vincent,

Vincent Stehlé wrote:
> On 09/24/2014 02:03 PM, Luca Ceresoli wrote:
>> Move to mainline Linux and upgrade the kernel version to the same version
>> used for the toolchain kernel headers.
>
> Hi Luca,
>
> I am not sure if abandoning the Freescale release kernel is the best
> option for i.MX6. Indeed, if you want e.g. proper video
> (package/freescale-imx/imx-vpu) and 3D
> (package/freescale-imx/gpu-viv-bin-mx6q) support to still work as
> expected you need to stay with the 3.10.17.
>
> Wouldn't you prefer to align the headers instead?

I'm not sure what's the best way either.

As per discussion with Thomas, using mainline kernel is preferred, and I
tend to agree, probably because I've had some bad experiences with
other vendor's kernels in the past.

Still, having good hardware support is important of course, and I
probably will use Freescale's kernel in real products, not mainline,
for this reason.

The vendor kernel argument would be stronger for Buildroot if we could
point our users to some docs or examples that allow them to notice the
features working with the vendor kernel and not in mainline.
Something that allows a user to:
   1. build a defconfig for the board
   2. run the final rootfs on the board
   3. test the video or 3D accelerator (that doesn't work with mainline)

Anything on your blog, maybe?

Aah, this leads me to think again about the "demo defconfigs" (as
opposed to minimal defconfigs) that we spoke about in the past... but
this is getting quite OT probably.

Oh, look! Peter just committed a change to nitrogen6x_defconfig where
he's locking kernel headers to 3.10.17 to match the kernel. And guess
what, the kernel is not mainline (although it's from Boundary Devices,
not got.freescale.com, but that doesn't matter much).
Here it is: 
http://git.buildroot.org/buildroot/commit/?id=2b8937f238a3593c9267a68e0fdad83841a56657
Thomas Petazzoni Sept. 24, 2014, 1:48 p.m. UTC | #3
Dear Vincent Stehlé,

On Wed, 24 Sep 2014 14:38:35 +0200, Vincent Stehlé wrote:

> I am not sure if abandoning the Freescale release kernel is the best
> option for i.MX6. Indeed, if you want e.g. proper video
> (package/freescale-imx/imx-vpu) and 3D
> (package/freescale-imx/gpu-viv-bin-mx6q) support to still work as
> expected you need to stay with the 3.10.17.
> 
> Wouldn't you prefer to align the headers instead?

Yes, I agree. For such platforms, it makes more sense to use the vendor
kernel version.

Thomas
Luca Ceresoli Sept. 24, 2014, 2:29 p.m. UTC | #4
Dear Thomas,

Thomas Petazzoni wrote:
> Dear Vincent Stehlé,
>
> On Wed, 24 Sep 2014 14:38:35 +0200, Vincent Stehlé wrote:
>
>> I am not sure if abandoning the Freescale release kernel is the best
>> option for i.MX6. Indeed, if you want e.g. proper video
>> (package/freescale-imx/imx-vpu) and 3D
>> (package/freescale-imx/gpu-viv-bin-mx6q) support to still work as
>> expected you need to stay with the 3.10.17.
>>
>> Wouldn't you prefer to align the headers instead?
>
> Yes, I agree. For such platforms, it makes more sense to use the vendor
> kernel version.

Didn't you hate those vendor kernels? ;-)

Well, seems there's a common direction here, so I'll resurrect my other
patches and send them.
diff mbox

Patch

diff --git a/configs/freescale_imx6dlsabresd_defconfig b/configs/freescale_imx6dlsabresd_defconfig
index b8c4b22..4baa35a 100644
--- a/configs/freescale_imx6dlsabresd_defconfig
+++ b/configs/freescale_imx6dlsabresd_defconfig
@@ -10,10 +10,8 @@  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_SAME_AS_HEADERS=y
+BR2_LINUX_KERNEL_DEFCONFIG="imx_v6_v7"
 BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
 BR2_LINUX_KERNEL_DTS_SUPPORT=y
 BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6dl-sabresd"
diff --git a/configs/freescale_imx6qsabresd_defconfig b/configs/freescale_imx6qsabresd_defconfig
index 40a9384..eebf67b 100644
--- a/configs/freescale_imx6qsabresd_defconfig
+++ b/configs/freescale_imx6qsabresd_defconfig
@@ -10,10 +10,8 @@  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_SAME_AS_HEADERS=y
+BR2_LINUX_KERNEL_DEFCONFIG="imx_v6_v7"
 BR2_LINUX_KERNEL_UIMAGE_LOADADDR="0x10008000"
 BR2_LINUX_KERNEL_DTS_SUPPORT=y
 BR2_LINUX_KERNEL_INTREE_DTS_NAME="imx6q-sabresd"