[U-Boot] [PATCH] GE Bx50v3 boards: fix fdt file variable

The CONFIG_DEFAULT_FDT_FILE expects just the file not a full path as that might vary depending on the variant of Linux distro you might be using.
Cc: Martin Donnelly martin.donnelly@ge.com Signed-off-by: Peter Robinson pbrobinson@gmail.com --- configs/ge_b650v3_defconfig | 2 +- configs/ge_b850v3_defconfig | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/configs/ge_b650v3_defconfig b/configs/ge_b650v3_defconfig index dd44663dde..78187051eb 100644 --- a/configs/ge_b650v3_defconfig +++ b/configs/ge_b650v3_defconfig @@ -5,7 +5,7 @@ CONFIG_TARGET_GE_B650V3=y CONFIG_BOOTDELAY=1 CONFIG_SYS_CONSOLE_IS_IN_ENV=y CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y -CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b650v3.dtb" +CONFIG_DEFAULT_FDT_FILE="imx6q-b650v3.dtb" CONFIG_BOARD_EARLY_INIT_F=y CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y diff --git a/configs/ge_b850v3_defconfig b/configs/ge_b850v3_defconfig index 45e5fb1ff4..967ddba13a 100644 --- a/configs/ge_b850v3_defconfig +++ b/configs/ge_b850v3_defconfig @@ -5,7 +5,7 @@ CONFIG_TARGET_GE_B850V3=y CONFIG_BOOTDELAY=1 CONFIG_SYS_CONSOLE_IS_IN_ENV=y CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE=y -CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b850v3.dtb" +CONFIG_DEFAULT_FDT_FILE="imx6q-b850v3.dtb" CONFIG_BOARD_EARLY_INIT_F=y CONFIG_HUSH_PARSER=y CONFIG_CMD_BOOTZ=y

Hi Peter,
On Thu, Jun 29, 2017 at 8:44 AM, Peter Robinson pbrobinson@gmail.com wrote:
The CONFIG_DEFAULT_FDT_FILE expects just the file not a full path as that might vary depending on the variant of Linux distro you might be using.
Cc: Martin Donnelly martin.donnelly@ge.com Signed-off-by: Peter Robinson pbrobinson@gmail.com
configs/ge_b650v3_defconfig | 2 +- configs/ge_b850v3_defconfig | 2 +-
Patch looks good, but there is another one too:
configs/ge_b450v3_defconfig:CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b450v3.dtb"
Could you also include the change for configs/ge_b450v3_defconfig and send a v2?
Thanks

On Thu, Jun 29, 2017 at 12:51 PM, Fabio Estevam festevam@gmail.com wrote:
Hi Peter,
On Thu, Jun 29, 2017 at 8:44 AM, Peter Robinson pbrobinson@gmail.com wrote:
The CONFIG_DEFAULT_FDT_FILE expects just the file not a full path as that might vary depending on the variant of Linux distro you might be using.
Cc: Martin Donnelly martin.donnelly@ge.com Signed-off-by: Peter Robinson pbrobinson@gmail.com
configs/ge_b650v3_defconfig | 2 +- configs/ge_b850v3_defconfig | 2 +-
Patch looks good, but there is another one too:
configs/ge_b450v3_defconfig:CONFIG_DEFAULT_FDT_FILE="/boot/imx6q-b450v3.dtb"
Could you also include the change for configs/ge_b450v3_defconfig and send a v2?
Sure, clearly I was slightly blind in not seeing that :)
participants (2)
-
Fabio Estevam
-
Peter Robinson