[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 02:44:10PM +0200, Karsten Merker wrote:

> 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"?

Hello,

I've just found that I made a mistake in the patch - as a result
it builds, but probably won't work properly.  The additional
include statement in include/configs/clearfog.h must be moved
from the beginning of the file to the end of the file, otherwise
some definitions on which the header relies aren't defined
properly.

Attached is a new version of the patch (again build-tested only
due to lack of appropriate hardware).

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 1f6998d1084ef8c203a36b3ef15495add85f792b 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..e372fb5 100644
--- a/include/configs/clearfog.h
+++ b/include/configs/clearfog.h
@@ -148,4 +148,10 @@
  */
 #include "mv-common.h"
 
+/* Include the common distro boot environment */
+#ifndef CONFIG_SPL_BUILD
+#include <config_distro_defaults.h>
+#include <config_distro_bootcmd.h>
+#endif
+
 #endif /* _CONFIG_CLEARFOG_H */
-- 
2.1.4


Reply to: