[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#839595: flash-kernel: Please add support for SolidRun Clearfog



On Mon, Oct 03, 2016 at 01:32:22PM +0200, Christoph Egger wrote:
> Karsten Merker <merker@debian.org> writes:
> > Well, that is a _very_ minimalistic default environment ;-).
> > So there isn't any boot command handling at all in the default
> > environment of mainline u-boot for this system, which poses a
> > problem for supporting this setup in flash-kernel.
> 
> I supposed this is CONFIG_DISTRO_DEFAULTS? I'll rebuild u-boot and see
> (also for the plain initrd)

Yes and no :-).

Setting CONFIG_DISTRO_DEFAULTS is required to enable the commands
that are used by the common boot environment, but it doesn't
automatically include the actual environment.

The attached patch enables CONFIG_DISTRO_DEFAULTS and includes
the necessary headers to define the common environment (only
build-tested due to lack of appropriate hardware).  One thing
that is still missing in the patch is the definition of various
default load addresses (fdt_addr_r, ramdisk_addr_r,
kernel_addr_r, pxefile_addr_r, scriptaddr).  Is there an
"official" list of the standard load addresses for this platform
so that I could add them to the patch?

Could you please build u-boot with the attached patch and
send the output of "env default -f -a; printenv"?

Regards,
Karsten
-- 
Gem. Par. 28 Abs. 4 Bundesdatenschutzgesetz widerspreche ich der Nutzung
sowie der Weitergabe meiner personenbezogenen Daten für Zwecke der
Werbung sowie der Markt- oder Meinungsforschung.
>From aa338add3f60e85bf083cedd4fac92f7e5750bc3 Mon Sep 17 00:00:00 2001
From: Karsten Merker <merker@debian.org>
Date: Mon, 3 Oct 2016 14:31:28 +0200
Subject: [PATCH] Enable the common distro boot environment for the clearfog
 platform.

Signed-off-by: Karsten Merker <merker@debian.org>
---
 configs/clearfog_defconfig | 10 +---------
 include/configs/clearfog.h |  6 ++++++
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/configs/clearfog_defconfig b/configs/clearfog_defconfig
index 588dfb2..b398238 100644
--- a/configs/clearfog_defconfig
+++ b/configs/clearfog_defconfig
@@ -9,10 +9,9 @@ CONFIG_SPL_LIBDISK_SUPPORT=y
 CONFIG_SPL_MMC_SUPPORT=y
 CONFIG_SPL_SERIAL_SUPPORT=y
 CONFIG_DEFAULT_DEVICE_TREE="armada-388-clearfog"
+CONFIG_DISTRO_DEFAULTS=y
 CONFIG_BOOTDELAY=3
 CONFIG_SPL=y
-CONFIG_HUSH_PARSER=y
-CONFIG_CMD_BOOTZ=y
 # CONFIG_CMD_IMLS is not set
 # CONFIG_CMD_FLASH is not set
 CONFIG_CMD_MMC=y
@@ -22,15 +21,8 @@ CONFIG_CMD_I2C=y
 CONFIG_CMD_USB=y
 # CONFIG_CMD_SETEXPR is not set
 CONFIG_CMD_TFTPPUT=y
-CONFIG_CMD_DHCP=y
-CONFIG_CMD_MII=y
-CONFIG_CMD_PING=y
 CONFIG_CMD_CACHE=y
 CONFIG_CMD_TIME=y
-CONFIG_CMD_EXT2=y
-CONFIG_CMD_EXT4=y
-CONFIG_CMD_FAT=y
-CONFIG_CMD_FS_GENERIC=y
 CONFIG_SPL_OF_TRANSLATE=y
 CONFIG_SPI_FLASH=y
 CONFIG_DEBUG_UART=y
diff --git a/include/configs/clearfog.h b/include/configs/clearfog.h
index 3990900..8affb96 100644
--- a/include/configs/clearfog.h
+++ b/include/configs/clearfog.h
@@ -7,6 +7,12 @@
 #ifndef _CONFIG_CLEARFOG_H
 #define _CONFIG_CLEARFOG_H
 
+/* Include the common distro boot environment */
+#ifndef CONFIG_SPL_BUILD
+#include <config_distro_defaults.h>
+#include <config_distro_bootcmd.h>
+#endif
+
 /*
  * High Level Configuration Options (easy to change)
  */
-- 
2.1.4


Reply to: