linux icon indicating copy to clipboard operation
linux copied to clipboard

Create tesla_mcu_defconfig

Open jnuyens opened this issue 6 years ago • 5 comments

Added the current kernel config for easier hacking and reference.

jnuyens avatar Jul 06 '18 23:07 jnuyens

@jnuyens your kernel config is from a rooted car, they do actually provide kernel configs for the cid and ic in /arch/arm/configs/ts001_defconfig and /arch/arm/configs/ts001_ic_defconfig...

There are a number of unspecified options, so maybe you're supposed to do:

make ts001_defconfig ARCH=arm
yes "" | make oldconfig
make -j4

You need tesla.cpio, which I assume is the product of a successful buildroot... have you had any luck building either?

EDIT: Hmm... looks like I need ARCH=arm every time I invoke make...

deftdawg avatar Nov 29 '18 18:11 deftdawg

From taking a look at the build doc, I'm guessing that they compile natively on an arm system, since they don't mention arch or any cross compile stuff.

deftdawg avatar Nov 29 '18 19:11 deftdawg

I think this should work, once a proper build root "tesla.cpio" can be produced:

apt install gcc-arm-linux-gnueabi libncurses5-dev
alias make='ARCH=arm make'
make ts001_defconfig
make -j4 CROSS_COMPILE=arm-linux-gnueabi- Image zImage tegra30-tesla-cid.dtb

deftdawg avatar Nov 29 '18 21:11 deftdawg

It's a correct answer, but I prefer not to advise to make an alias like that. Such an alias only remains active in the current session, and you don't want to have a persistent make alias like that because it will mean that on your host system things could go very wrong, also it hides what is actually being done. Just specify ARCH=arm with any make command you issue:

apt install gcc-arm-linux-gnueabi libncurses5-dev make ARCH=arm ts001_defconfig make ARCH=arm -j4 CROSS_COMPILE=arm-linux-gnueabi- Image zImage tegra30-tesla-cid.dtb

jnuyens avatar Nov 30 '18 10:11 jnuyens

I would guess there is no further need for this patch as the kernel config already exists in the source tree... through the documentation could use an update to indicate that it's already there.

deftdawg avatar Nov 30 '18 18:11 deftdawg