Skip to contentSkip to conversation

tegra: p2771: Add CONFIG_EFI_LOADER_BOUNCE_BUFFER

PATCH New Peter Robinson 2 participants 3 messages Jul 6, 2018 – Jul 9, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 tegra: p2771: Add CONFIG_EFI_LOADER_BOUNCE_BUFFER

The Jetson TX2 needs EFI loader bounce buffer enabled otherwise grub doesn't see the storage when it loads.

---
 configs/p2771-0000-000_defconfig | 1 +
 configs/p2771-0000-500_defconfig | 1 +
 2 files changed, 2 insertions(+)
configs/p2771-0000-000_defconfig +1 -0
index 29e1edcd68f..7ed036dc4ce 100644
@@ -4,6 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x80080000
 CONFIG_TEGRA186=y
 CONFIG_DEFAULT_DEVICE_TREE="tegra186-p2771-0000-000"
 CONFIG_OF_SYSTEM_SETUP=y
+CONFIG_EFI_LOADER_BOUNCE_BUFFER=y
 CONFIG_CONSOLE_MUX=y
 CONFIG_SYS_STDIO_DEREGISTER=y
 CONFIG_SYS_PROMPT="Tegra186 (P2771-0000-000) # "
configs/p2771-0000-500_defconfig +1 -0
index cc10669608f..9ae61263b5d 100644
@@ -4,6 +4,7 @@ CONFIG_SYS_TEXT_BASE=0x80080000
 CONFIG_TEGRA186=y
 CONFIG_DEFAULT_DEVICE_TREE="tegra186-p2771-0000-500"
 CONFIG_OF_SYSTEM_SETUP=y
+CONFIG_EFI_LOADER_BOUNCE_BUFFER=y
 CONFIG_CONSOLE_MUX=y
 CONFIG_SYS_STDIO_DEREGISTER=y
 CONFIG_SYS_PROMPT="Tegra186 (P2771-0000-500) # "
Signature · 1 line
2.17.1
#1 lore raw

PATCH 1/1 tegra: p2771: Add CONFIG_EFI_LOADER_BOUNCE_BUFFER

Quoted 2 lines · Peter Robinson

The Jetson TX2 needs EFI loader bounce buffer enabled otherwise grub doesn't see the storage when it loads.

I suspect the need for this option is imposed by the SoC HW not the board design? If so, shouldn't the SoC Kconfig symbol select this EFI bounce buffer Kconfig symbol?

#2 lore raw

PATCH 1/1 tegra: p2771: Add CONFIG_EFI_LOADER_BOUNCE_BUFFER

Quoted 10 lines · Stephen Warren
Quoted 4 lines · Peter Robinson

The Jetson TX2 needs EFI loader bounce buffer enabled otherwise grub doesn't see
the storage when it loads.

I suspect the need for this option is imposed by the SoC HW not the board design? If so, shouldn't the SoC Kconfig symbol select this EFI bounce buffer Kconfig symbol?

Possibly but all the other devices using this option are setting it in their configs AFAICT so that's why I went this route, I suspect also the Jetson X1 might need it too but I've not had time to test that yet.

Peter