All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>,
	bhelgaas@google.com, r65037@freescale.com,
	shawn.guo@freescale.com, d.mueller@elsoft.ch,
	kernel@pengutronix.de, linux-arm-kernel@lists.infradead.org,
	linux-pci@vger.kernel.org, tharvey@gateworks.com,
	Fabio Estevam <fabio.estevam@freescale.com>
Subject: Re: [RFC] PCI: pci-imx6: Add delay to workaround kernel hang
Date: Thu, 17 Jul 2014 10:23:10 +0200	[thread overview]
Message-ID: <201407171023.10908.marex@denx.de> (raw)
In-Reply-To: <20140717065148.GT11056@pengutronix.de>

On Thursday, July 17, 2014 at 08:51:48 AM, Uwe Kleine-König wrote:
> Hello,
> 
> On Tue, Jun 24, 2014 at 04:18:27PM -0300, Fabio Estevam wrote:
> > From: Fabio Estevam <fabio.estevam@freescale.com>
> > 
> > When the mx6 PCI conctroller is initialized in the bootloader we see a
> > kernel hang inside imx6_add_pcie_port().
> > 
> > Adding a 30ms delay allows the kernel to boot.
> 
> Just my thought on how to debug that: I'd try to bisect the pci init
> routine in the boot loader. I.e. first only do the first half of the
> initialisation in U-Boot. Depending on Linux being able to boot or not
> initialize more or less on the next run.
> 
> Maybe there is a single register write that makes Linux fail?!

I am still hell-bent on thinking that the missing PCIe block reset is what makes 
the Linux fail. Missing block reset is always a problem. Or do we now have a 
mean to reset the PCIe block and it's PHY from software?

Best regards,
Marek Vasut

WARNING: multiple messages have this Message-ID (diff)
From: marex@denx.de (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC] PCI: pci-imx6: Add delay to workaround kernel hang
Date: Thu, 17 Jul 2014 10:23:10 +0200	[thread overview]
Message-ID: <201407171023.10908.marex@denx.de> (raw)
In-Reply-To: <20140717065148.GT11056@pengutronix.de>

On Thursday, July 17, 2014 at 08:51:48 AM, Uwe Kleine-K?nig wrote:
> Hello,
> 
> On Tue, Jun 24, 2014 at 04:18:27PM -0300, Fabio Estevam wrote:
> > From: Fabio Estevam <fabio.estevam@freescale.com>
> > 
> > When the mx6 PCI conctroller is initialized in the bootloader we see a
> > kernel hang inside imx6_add_pcie_port().
> > 
> > Adding a 30ms delay allows the kernel to boot.
> 
> Just my thought on how to debug that: I'd try to bisect the pci init
> routine in the boot loader. I.e. first only do the first half of the
> initialisation in U-Boot. Depending on Linux being able to boot or not
> initialize more or less on the next run.
> 
> Maybe there is a single register write that makes Linux fail?!

I am still hell-bent on thinking that the missing PCIe block reset is what makes 
the Linux fail. Missing block reset is always a problem. Or do we now have a 
mean to reset the PCIe block and it's PHY from software?

Best regards,
Marek Vasut

  reply	other threads:[~2014-07-17  8:32 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-24 19:18 [RFC] PCI: pci-imx6: Add delay to workaround kernel hang Fabio Estevam
2014-06-24 19:18 ` Fabio Estevam
2014-06-25 21:28 ` Marek Vasut
2014-06-25 21:28   ` Marek Vasut
2014-06-26  3:12 ` Shawn Guo
2014-06-26  3:12   ` Shawn Guo
2014-06-26  3:43   ` Fabio Estevam
2014-06-26  3:43     ` Fabio Estevam
2014-06-26  5:49     ` Shawn Guo
2014-06-26  5:49       ` Shawn Guo
2014-06-26  9:13       ` Marek Vasut
2014-06-26  9:13         ` Marek Vasut
2014-06-27  0:29       ` Tim Harvey
2014-06-27  0:29         ` Tim Harvey
2014-06-27 10:06         ` Hong-Xing.Zhu
2014-06-27 10:06           ` Hong-Xing.Zhu at freescale.com
2014-07-17  0:28           ` Tim Harvey
2014-07-17  0:28             ` Tim Harvey
2014-07-18  9:26         ` Lucas Stach
2014-07-18  9:26           ` Lucas Stach
2014-07-18  9:44           ` Marek Vasut
2014-07-18  9:44             ` Marek Vasut
2014-07-18 11:44           ` "David Müller (ELSOFT AG)"
2014-07-18 11:44             ` "David Müller (ELSOFT AG)"
2014-06-26  7:32     ` Juergen Borleis
2014-06-26  7:32       ` Juergen Borleis
2014-06-26  9:17       ` Marek Vasut
2014-06-26  9:17         ` Marek Vasut
2014-06-26  9:26         ` Juergen Borleis
2014-06-26  9:26           ` Juergen Borleis
2014-06-26  9:50           ` Marek Vasut
2014-06-26  9:50             ` Marek Vasut
2014-06-26 11:43       ` Fabio Estevam
2014-06-26 11:43         ` Fabio Estevam
2014-06-26  8:41 ` Lucas Stach
2014-06-26  8:41   ` Lucas Stach
2014-07-17  6:51 ` Uwe Kleine-König
2014-07-17  6:51   ` Uwe Kleine-König
2014-07-17  8:23   ` Marek Vasut [this message]
2014-07-17  8:23     ` Marek Vasut
2014-07-17 15:27     ` Shawn Guo
2014-07-17 15:27       ` Shawn Guo
2014-07-18 20:46       ` Marek Vasut
2014-07-18 20:46         ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201407171023.10908.marex@denx.de \
    --to=marex@denx.de \
    --cc=bhelgaas@google.com \
    --cc=d.mueller@elsoft.ch \
    --cc=fabio.estevam@freescale.com \
    --cc=festevam@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=r65037@freescale.com \
    --cc=shawn.guo@freescale.com \
    --cc=tharvey@gateworks.com \
    --cc=u.kleine-koenig@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.