[U-Boot] [PATCH] mx6sabresd: Disable PCI driver

[U-Boot] [PATCH] mx6sabresd: Disable PCI driver

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

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

Disable it until a better solution is found.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 include/configs/mx6sabresd.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/configs/mx6sabresd.h b/include/configs/mx6sabresd.h
index e666ebb..d227f44 100644
--- a/include/configs/mx6sabresd.h
+++ b/include/configs/mx6sabresd.h
@@ -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
-- 
1.8.3.2

[U-Boot] [PATCH] mx6sabresd: Disable PCI driver

[Marek Vasut]

On Friday, June 20, 2014 at 07:00:28 PM, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Currently we have a kernel hang when U-boot PCI driver is enabled.
> 
> Disable it until a better solution is found.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

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

[U-Boot] [PATCH] mx6sabresd: Disable PCI driver

[Fabio Estevam]

On Fri, Jun 20, 2014 at 6:41 PM, Marek Vasut <marex@denx.de> wrote:
> On Friday, June 20, 2014 at 07:00:28 PM, Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam@freescale.com>
>>
>> Currently we have a kernel hang when U-boot PCI driver is enabled.
>>
>> Disable it until a better solution is found.
>>
>> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>
> 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.

[U-Boot] [PATCH] mx6sabresd: Disable PCI driver

[Jeroen Hofstee]

On za, 2014-06-21 at 11:00 -0300, Fabio Estevam wrote:
> On Fri, Jun 20, 2014 at 6:41 PM, Marek Vasut <marex@denx.de> wrote:
> > On Friday, June 20, 2014 at 07:00:28 PM, Fabio Estevam wrote:
> >> From: Fabio Estevam <fabio.estevam@freescale.com>
> >>
> >> Currently we have a kernel hang when U-boot PCI driver is enabled.
> >>
> >> Disable it until a better solution is found.
> >>
> >> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> >
> > 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