All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.