* [U-Boot] [PATCH] pci: imx: display message if no pcie link
@ 2015-05-08 22:17 Tim Harvey
2015-05-22 20:50 ` [U-Boot] " Scott Wood
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Tim Harvey @ 2015-05-08 22:17 UTC (permalink / raw)
To: u-boot
If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
detected.
Signed-off-by: Tim Harvey <tharvey@gateworks.com>
---
drivers/pci/pcie_imx.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/pci/pcie_imx.c b/drivers/pci/pcie_imx.c
index fd7e4d4..ca485ba 100644
--- a/drivers/pci/pcie_imx.c
+++ b/drivers/pci/pcie_imx.c
@@ -588,7 +588,9 @@ static int imx_pcie_link_up(void)
udelay(10);
count++;
if (count >= 2000) {
- debug("phy link never came up\n");
+#ifdef CONFIG_PCI_SCAN_SHOW
+ puts("PCI: pcie phy link never came up\n");
+#endif
debug("DEBUG_R0: 0x%08x, DEBUG_R1: 0x%08x\n",
readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R0),
readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R1));
--
1.9.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] pci: imx: display message if no pcie link
2015-05-08 22:17 [U-Boot] [PATCH] pci: imx: display message if no pcie link Tim Harvey
@ 2015-05-22 20:50 ` Scott Wood
2015-05-22 21:17 ` Tim Harvey
2015-05-22 21:26 ` Tom Rini
2015-05-27 13:21 ` [U-Boot] [PATCH] " Tim Harvey
2015-05-28 9:45 ` Stefano Babic
2 siblings, 2 replies; 8+ messages in thread
From: Scott Wood @ 2015-05-22 20:50 UTC (permalink / raw)
To: u-boot
On Fri, May 08, 2015 at 03:17:10PM -0700, Tim Harvey wrote:
> If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
> detected.
>
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> ---
> drivers/pci/pcie_imx.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
Why was this and another PCI patch assigned to me in patchwork (I've
since cleared the delegation)?
-Scott
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] pci: imx: display message if no pcie link
2015-05-22 20:50 ` [U-Boot] " Scott Wood
@ 2015-05-22 21:17 ` Tim Harvey
2015-05-22 21:26 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Tim Harvey @ 2015-05-22 21:17 UTC (permalink / raw)
To: u-boot
On Fri, May 22, 2015 at 1:50 PM, Scott Wood <scottwood@freescale.com> wrote:
> On Fri, May 08, 2015 at 03:17:10PM -0700, Tim Harvey wrote:
>> If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
>> detected.
>>
>> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
>> ---
>> drivers/pci/pcie_imx.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> Why was this and another PCI patch assigned to me in patchwork (I've
> since cleared the delegation)?
>
> -Scott
Scott,
Just lucky I guess ;P
Who is the right person to send this to? If a subsection doesn't have
a clear maintainer like drivers/pci would it be Tom?
Tim
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] pci: imx: display message if no pcie link
2015-05-22 20:50 ` [U-Boot] " Scott Wood
2015-05-22 21:17 ` Tim Harvey
@ 2015-05-22 21:26 ` Tom Rini
2015-05-22 21:38 ` Scott Wood
1 sibling, 1 reply; 8+ messages in thread
From: Tom Rini @ 2015-05-22 21:26 UTC (permalink / raw)
To: u-boot
On Fri, May 22, 2015 at 03:50:39PM -0500, Scott Wood wrote:
> On Fri, May 08, 2015 at 03:17:10PM -0700, Tim Harvey wrote:
> > If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
> > detected.
> >
> > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > ---
> > drivers/pci/pcie_imx.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
>
> Why was this and another PCI patch assigned to me in patchwork (I've
> since cleared the delegation)?
I probably typo'd "sb" as "sc" when doing my triage. Did you have any
other imx related patches by chance?
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150522/3da0ff0c/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] pci: imx: display message if no pcie link
2015-05-22 21:26 ` Tom Rini
@ 2015-05-22 21:38 ` Scott Wood
0 siblings, 0 replies; 8+ messages in thread
From: Scott Wood @ 2015-05-22 21:38 UTC (permalink / raw)
To: u-boot
On Fri, 2015-05-22 at 17:26 -0400, Tom Rini wrote:
> On Fri, May 22, 2015 at 03:50:39PM -0500, Scott Wood wrote:
> > On Fri, May 08, 2015 at 03:17:10PM -0700, Tim Harvey wrote:
> > > If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
> > > detected.
> > >
> > > Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> > > ---
> > > drivers/pci/pcie_imx.c | 4 +++-
> > > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > Why was this and another PCI patch assigned to me in patchwork (I've
> > since cleared the delegation)?
>
> I probably typo'd "sb" as "sc" when doing my triage. Did you have any
> other imx related patches by chance?
Just the two PCI patches and some SPI MTD patches.
-Scott
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] pci: imx: display message if no pcie link
2015-05-08 22:17 [U-Boot] [PATCH] pci: imx: display message if no pcie link Tim Harvey
2015-05-22 20:50 ` [U-Boot] " Scott Wood
@ 2015-05-27 13:21 ` Tim Harvey
2015-05-27 13:30 ` Stefano Babic
2015-05-28 9:45 ` Stefano Babic
2 siblings, 1 reply; 8+ messages in thread
From: Tim Harvey @ 2015-05-27 13:21 UTC (permalink / raw)
To: u-boot
On Fri, May 8, 2015 at 3:17 PM, Tim Harvey <tharvey@gateworks.com> wrote:
> If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
> detected.
>
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> ---
> drivers/pci/pcie_imx.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pcie_imx.c b/drivers/pci/pcie_imx.c
> index fd7e4d4..ca485ba 100644
> --- a/drivers/pci/pcie_imx.c
> +++ b/drivers/pci/pcie_imx.c
> @@ -588,7 +588,9 @@ static int imx_pcie_link_up(void)
> udelay(10);
> count++;
> if (count >= 2000) {
> - debug("phy link never came up\n");
> +#ifdef CONFIG_PCI_SCAN_SHOW
> + puts("PCI: pcie phy link never came up\n");
> +#endif
> debug("DEBUG_R0: 0x%08x, DEBUG_R1: 0x%08x\n",
> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R0),
> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R1));
> --
> 1.9.1
>
Stefano,
I probably should have sent this one to you directly being an imx driver.
Regards,
Tim
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] pci: imx: display message if no pcie link
2015-05-27 13:21 ` [U-Boot] [PATCH] " Tim Harvey
@ 2015-05-27 13:30 ` Stefano Babic
0 siblings, 0 replies; 8+ messages in thread
From: Stefano Babic @ 2015-05-27 13:30 UTC (permalink / raw)
To: u-boot
Hi Tim,
On 27/05/2015 15:21, Tim Harvey wrote:
> On Fri, May 8, 2015 at 3:17 PM, Tim Harvey <tharvey@gateworks.com> wrote:
>> If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
>> detected.
>>
>> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
>> ---
>> drivers/pci/pcie_imx.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/pci/pcie_imx.c b/drivers/pci/pcie_imx.c
>> index fd7e4d4..ca485ba 100644
>> --- a/drivers/pci/pcie_imx.c
>> +++ b/drivers/pci/pcie_imx.c
>> @@ -588,7 +588,9 @@ static int imx_pcie_link_up(void)
>> udelay(10);
>> count++;
>> if (count >= 2000) {
>> - debug("phy link never came up\n");
>> +#ifdef CONFIG_PCI_SCAN_SHOW
>> + puts("PCI: pcie phy link never came up\n");
>> +#endif
>> debug("DEBUG_R0: 0x%08x, DEBUG_R1: 0x%08x\n",
>> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R0),
>> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R1));
>> --
>> 1.9.1
>>
>
> Stefano,
>
> I probably should have sent this one to you directly being an imx driver.
>
I have seen the patch and the related discussion who should take care of
it ;-)
Do not worry - I set myself as delegate for this and I will merge it soon.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] pci: imx: display message if no pcie link
2015-05-08 22:17 [U-Boot] [PATCH] pci: imx: display message if no pcie link Tim Harvey
2015-05-22 20:50 ` [U-Boot] " Scott Wood
2015-05-27 13:21 ` [U-Boot] [PATCH] " Tim Harvey
@ 2015-05-28 9:45 ` Stefano Babic
2 siblings, 0 replies; 8+ messages in thread
From: Stefano Babic @ 2015-05-28 9:45 UTC (permalink / raw)
To: u-boot
On 09/05/2015 00:17, Tim Harvey wrote:
> If CONFIG_PCI_SCAN_SHOW enabled then lets print a message of no link was
> detected.
>
> Signed-off-by: Tim Harvey <tharvey@gateworks.com>
> ---
> drivers/pci/pcie_imx.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pcie_imx.c b/drivers/pci/pcie_imx.c
> index fd7e4d4..ca485ba 100644
> --- a/drivers/pci/pcie_imx.c
> +++ b/drivers/pci/pcie_imx.c
> @@ -588,7 +588,9 @@ static int imx_pcie_link_up(void)
> udelay(10);
> count++;
> if (count >= 2000) {
> - debug("phy link never came up\n");
> +#ifdef CONFIG_PCI_SCAN_SHOW
> + puts("PCI: pcie phy link never came up\n");
> +#endif
> debug("DEBUG_R0: 0x%08x, DEBUG_R1: 0x%08x\n",
> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R0),
> readl(MX6_DBI_ADDR + PCIE_PHY_DEBUG_R1));
>
Applied to u-boot-imx, thanks !
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2015-05-28 9:45 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-08 22:17 [U-Boot] [PATCH] pci: imx: display message if no pcie link Tim Harvey
2015-05-22 20:50 ` [U-Boot] " Scott Wood
2015-05-22 21:17 ` Tim Harvey
2015-05-22 21:26 ` Tom Rini
2015-05-22 21:38 ` Scott Wood
2015-05-27 13:21 ` [U-Boot] [PATCH] " Tim Harvey
2015-05-27 13:30 ` Stefano Babic
2015-05-28 9:45 ` Stefano Babic
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.