Merge ~ethan.hsieh/+git/u-boot:devel into ~ethan.hsieh/+git/u-boot:ubuntu/bionic-proposed

Proposed by ethan.hsieh
Status: Needs review
Proposed branch: ~ethan.hsieh/+git/u-boot:devel
Merge into: ~ethan.hsieh/+git/u-boot:ubuntu/bionic-proposed
Diff against target: 42 lines (+7/-3)
2 files modified
configs/nitrogen6q2g_defconfig (+0/-2)
include/configs/nitrogen6x.h (+7/-1)
Reviewer Review Type Date Requested Status
Łukasz Zemczak (community) Approve
Dave Jones Pending
Shrirang Bagul Pending
ethan.hsieh Pending
Review via email: mp+377153@code.launchpad.net
To post a comment you must log in.
Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :

@Lukasz
I have no permission to send MP to https://code.launchpad.net/~usd-import-team/ubuntu/+source/u-boot/+git/u-boot/+ref/ubuntu/bionic-proposed
Could you help to review this patch?

Revision history for this message
ethan.hsieh (ethan.hsieh) wrote :
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

If I understand this correctly: since our u-boot isn't up-to-date, we're missing the 2 commits that migrate CONFIG_SYS_REDUNDAND_ENVIRONMENT (and others) to Kconfig, so those config changes were ignored on the defconfig.

This seems like a decent workaround for this, certainly feels safer than cherry-picking two rather big commits. Looks good to me.

review: Approve

Unmerged commits

56708cf... by ethan.hsieh

u-boot-imx: Fix bad CRC issue (LP: #1853395)
  - define env size
  - enable CONFIG_SYS_REDUNDAND_ENVIRONMENT

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/configs/nitrogen6q2g_defconfig b/configs/nitrogen6q2g_defconfig
2index 843cdab..346f793 100644
3--- a/configs/nitrogen6q2g_defconfig
4+++ b/configs/nitrogen6q2g_defconfig
5@@ -37,7 +37,6 @@ CONFIG_ENV_IS_IN_FAT=y
6 CONFIG_ENV_FAT_INTERFACE="mmc"
7 CONFIG_ENV_FAT_DEVICE_AND_PART="0:auto"
8 CONFIG_ENV_FAT_FILE="uboot.env"
9-CONFIG_ENV_SIZE=0x20000
10 CONFIG_DWC_AHSATA=y
11 CONFIG_USB_FUNCTION_FASTBOOT=y
12 CONFIG_FASTBOOT_BUF_ADDR=0x12000000
13@@ -66,4 +65,3 @@ CONFIG_USB_ETH_CDC=y
14 CONFIG_VIDEO=y
15 # CONFIG_VIDEO_SW_CURSOR is not set
16 CONFIG_OF_LIBFDT=y
17-CONFIG_SYS_REDUNDAND_ENVIRONMENT=y
18diff --git a/include/configs/nitrogen6x.h b/include/configs/nitrogen6x.h
19index cb5da38..5a8b627 100644
20--- a/include/configs/nitrogen6x.h
21+++ b/include/configs/nitrogen6x.h
22@@ -11,6 +11,10 @@
23
24 #include "mx6_common.h"
25
26+#if defined(CONFIG_TARGET_NITROGEN6X)
27+#define CONFIG_SYS_REDUNDAND_ENVIRONMENT
28+#endif
29+
30 #define CONFIG_MACH_TYPE 3769
31
32 /* Size of malloc() pool */
33@@ -160,7 +164,9 @@
34 (CONFIG_SYS_INIT_RAM_ADDR + CONFIG_SYS_INIT_SP_OFFSET)
35
36 /* Environment organization */
37-#ifndef CONFIG_ENV_SIZE
38+#if defined(CONFIG_TARGET_NITROGEN6X)
39+#define CONFIG_ENV_SIZE SZ_128K
40+#else
41 #define CONFIG_ENV_SIZE (8 * 1024)
42 #endif
43

Subscribers

People subscribed via source and target branches

to all changes: