Skip to contentSkip to conversation

mx6sabresd: Disable PCI driver

PATCH New Fabio Estevam 3 participants 4 messages Jun 20, 2014 – Jun 21, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 mx6sabresd: Disable PCI driver

From: Fabio Estevam <[email protected]>

Currently we have a kernel hang when U-boot PCI driver is enabled.

Disable it until a better solution is found.

---
 include/configs/mx6sabresd.h | 1 -
 1 file changed, 1 deletion(-)
include/configs/mx6sabresd.h +0 -1
index e666ebb..d227f44 100644
@@ -49,7 +49,6 @@
 #define CONFIG_IMX_HDMI
 #define CONFIG_IMX_VIDEO_SKIP
 
-#define CONFIG_CMD_PCI
 #ifdef CONFIG_CMD_PCI
 #define CONFIG_PCI
 #define CONFIG_PCI_PNP
Signature · 1 line
1.8.3.2
#1 lore raw

PATCH 1/1 mx6sabresd: Disable PCI driver

Quoted 7 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Currently we have a kernel hang when U-boot PCI driver is enabled.

Disable it until a better solution is found.

This looks like solving a bug on a wrong end. It is the kernel that needs to be fixed, not U-Boot.

Best regards,
Marek Vasut

#2 lore raw

PATCH 1/1 mx6sabresd: Disable PCI driver

Quoted 11 lines · Marek Vasut
Quoted 7 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Currently we have a kernel hang when U-boot PCI driver is enabled.

Disable it until a better solution is found.

This looks like solving a bug on a wrong end. It is the kernel that needs to be fixed, not U-Boot.

Since there is no fix in the kernel I prefer to remove this from U-boot from the time being to avoid the kernel hang.

#3 lore raw

PATCH 1/1 mx6sabresd: Disable PCI driver

Quoted 8 lines · Fabio Estevam
Quoted 7 lines · Marek Vasut
Quoted 6 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

Currently we have a kernel hang when U-boot PCI driver is enabled.

Disable it until a better solution is found.

Quoted 7 lines
Quoted 4 lines
Quoted 1 line

This looks like solving a bug on a wrong end. It is the kernel that needs to be fixed, not U-Boot.

Since there is no fix in the kernel I prefer to remove this from U-boot from the time being to avoid the kernel hang.

this sounds evil, just fix/workaround it in your linux version.

Regards,
Jeroen