Re: [U-Boot] [PATCH 1/6] fsl: Clean up printing of PCI boot info

2010-11-14 Thread Wolfgang Denk
Dear Peter Tyser,

In message <1288393169-9814-1-git-send-email-pty...@xes-inc.com> you wrote:
> Previously boards used a variety of indentations, newline styles, and
> colon styles for the PCI information that is printed on bootup.  This
> patch unifies the style to look like:
> 
> ...
> NAND:  1024 MiB
> PCIE1: connected as Root Complex
>Scanning PCI bus 01
> 04  01  8086  1010  0200  00
> 04  01  8086  1010  0200  00
> 03  00  10b5  8112  0604  00
> 02  01  10b5  8518  0604  00
> 02  02  10b5  8518  0604  00
> 08  00  1957  0040  0b20  00
> 07  00  10b5  8518  0604  00
> 09  00  10b5  8112  0604  00
> 07  01  10b5  8518  0604  00
> 07  02  10b5  8518  0604  00
> 06  00  10b5  8518  0604  00
> 02  03  10b5  8518  0604  00
> 01  00  10b5  8518  0604  00
> PCIE1: Bus 00 - 0b
> PCIE2: connected as Root Complex
>Scanning PCI bus 0d
> 0d  00  1957  0040  0b20  00
> PCIE2: Bus 0c - 0d
> In:serial
> ...
> 
> Signed-off-by: Peter Tyser 
> CC: w...@denx.de
> CC: s...@denx.de
> CC: ga...@kernel.crashing.org
> ---
> This series assumes "fsl_pci_init: Make fsl_pci_init_port() PCI/PCIe aware"
> has already been applied.
> 
>  board/atum8548/atum8548.c |   12 ++--
>  board/freescale/corenet_ds/pci.c  |   16 
>  board/freescale/mpc8536ds/mpc8536ds.c |   16 
>  board/freescale/mpc8540ads/mpc8540ads.c   |4 ++--
>  board/freescale/mpc8541cds/mpc8541cds.c   |6 +++---
>  board/freescale/mpc8544ds/mpc8544ds.c |   24 
>  board/freescale/mpc8548cds/mpc8548cds.c   |   16 
>  board/freescale/mpc8555cds/mpc8555cds.c   |6 +++---
>  board/freescale/mpc8560ads/mpc8560ads.c   |4 ++--
>  board/freescale/mpc8568mds/mpc8568mds.c   |8 
>  board/freescale/mpc8569mds/mpc8569mds.c   |8 
>  board/freescale/mpc8572ds/mpc8572ds.c |   20 ++--
>  board/freescale/mpc8610hpcd/mpc8610hpcd.c |   20 ++--
>  board/freescale/mpc8641hpcn/mpc8641hpcn.c |   18 +-
>  board/freescale/p1022ds/p1022ds.c |8 
>  board/freescale/p1_p2_rdb/pci.c   |   16 
>  board/freescale/p2020ds/p2020ds.c |   24 
>  board/pm854/pm854.c   |4 ++--
>  board/pm856/pm856.c   |4 ++--
>  board/sbc8548/sbc8548.c   |8 
>  board/sbc8641d/sbc8641d.c |   18 +-
>  board/tqc/tqm85xx/tqm85xx.c   |8 
>  board/xes/common/fsl_8xxx_pci.c   |   16 
>  drivers/pci/fsl_pci_init.c|   10 +-
>  24 files changed, 147 insertions(+), 147 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Gods don't like people not doing much work. People  who  aren't  busy
all the time might start to _think_.  - Terry Pratchett, _Small Gods_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH 1/6] fsl: Clean up printing of PCI boot info

2010-10-29 Thread Peter Tyser
Previously boards used a variety of indentations, newline styles, and
colon styles for the PCI information that is printed on bootup.  This
patch unifies the style to look like:

...
NAND:  1024 MiB
PCIE1: connected as Root Complex
   Scanning PCI bus 01
04  01  8086  1010  0200  00
04  01  8086  1010  0200  00
03  00  10b5  8112  0604  00
02  01  10b5  8518  0604  00
02  02  10b5  8518  0604  00
08  00  1957  0040  0b20  00
07  00  10b5  8518  0604  00
09  00  10b5  8112  0604  00
07  01  10b5  8518  0604  00
07  02  10b5  8518  0604  00
06  00  10b5  8518  0604  00
02  03  10b5  8518  0604  00
01  00  10b5  8518  0604  00
PCIE1: Bus 00 - 0b
PCIE2: connected as Root Complex
   Scanning PCI bus 0d
0d  00  1957  0040  0b20  00
PCIE2: Bus 0c - 0d
In:serial
...

Signed-off-by: Peter Tyser 
CC: w...@denx.de
CC: s...@denx.de
CC: ga...@kernel.crashing.org
---
This series assumes "fsl_pci_init: Make fsl_pci_init_port() PCI/PCIe aware"
has already been applied.

 board/atum8548/atum8548.c |   12 ++--
 board/freescale/corenet_ds/pci.c  |   16 
 board/freescale/mpc8536ds/mpc8536ds.c |   16 
 board/freescale/mpc8540ads/mpc8540ads.c   |4 ++--
 board/freescale/mpc8541cds/mpc8541cds.c   |6 +++---
 board/freescale/mpc8544ds/mpc8544ds.c |   24 
 board/freescale/mpc8548cds/mpc8548cds.c   |   16 
 board/freescale/mpc8555cds/mpc8555cds.c   |6 +++---
 board/freescale/mpc8560ads/mpc8560ads.c   |4 ++--
 board/freescale/mpc8568mds/mpc8568mds.c   |8 
 board/freescale/mpc8569mds/mpc8569mds.c   |8 
 board/freescale/mpc8572ds/mpc8572ds.c |   20 ++--
 board/freescale/mpc8610hpcd/mpc8610hpcd.c |   20 ++--
 board/freescale/mpc8641hpcn/mpc8641hpcn.c |   18 +-
 board/freescale/p1022ds/p1022ds.c |8 
 board/freescale/p1_p2_rdb/pci.c   |   16 
 board/freescale/p2020ds/p2020ds.c |   24 
 board/pm854/pm854.c   |4 ++--
 board/pm856/pm856.c   |4 ++--
 board/sbc8548/sbc8548.c   |8 
 board/sbc8641d/sbc8641d.c |   18 +-
 board/tqc/tqm85xx/tqm85xx.c   |8 
 board/xes/common/fsl_8xxx_pci.c   |   16 
 drivers/pci/fsl_pci_init.c|   10 +-
 24 files changed, 147 insertions(+), 147 deletions(-)

diff --git a/board/atum8548/atum8548.c b/board/atum8548/atum8548.c
index 671f9e9..4fcbb18 100644
--- a/board/atum8548/atum8548.c
+++ b/board/atum8548/atum8548.c
@@ -218,14 +218,14 @@ void pci_init_board(void)
 
pcie1_hose.region_count = 1;
 #endif
-   printf ("PCIE1 connected to Slot as %s (base addr %lx)\n",
+   printf ("PCIE1: connected to Slot as %s (base addr %lx)\n",
pcie_ep ? "Endpoint" : "Root Complex",
pci_info[num].regs);
 
first_free_busno = fsl_pci_init_port(&pci_info[num++],
&pcie1_hose, first_free_busno);
} else {
-   printf ("PCIE1: disabled\n");
+   printf("PCIE1: disabled\n");
}
 
puts("\n");
@@ -242,7 +242,7 @@ void pci_init_board(void)
if (!(devdisr & MPC85xx_DEVDISR_PCI1)) {
SET_STD_PCI_INFO(pci_info[num], 1);
pci_agent = fsl_setup_hose(&pci1_hose, pci_info[num].regs);
-   printf ("\nPCI1: %d bit, %s MHz, %s, %s, %s (base address 
%lx)\n",
+   printf("PCI1: %d bit, %s MHz, %s, %s, %s (base address %lx)\n",
(pci_32) ? 32 : 64,
(pci_speed == 3000) ? "33" :
(pci_speed == 6000) ? "66" : "unknown",
@@ -254,7 +254,7 @@ void pci_init_board(void)
first_free_busno = fsl_pci_init_port(&pci_info[num++],
&pci1_hose, first_free_busno);
} else {
-   printf ("PCI: disabled\n");
+   printf("PCI1: disabled\n");
}
 
puts("\n");
@@ -267,11 +267,11 @@ void pci_init_board(void)
SET_STD_PCI_INFO(pci_info[num], 2);
pci_agent = fsl_setup_hose(&pci2_hose, pci_info[num].regs);
 
-   puts ("PCI2\n");
+   puts("PCI2\n");
first_free_busno = fsl_pci_init_port(&pci_info[num++],
&pci1_hose, first_free_busno);
} else {
-   printf ("PCI2: disabled\n");
+   printf("PCI2: disabled\n");
}
puts("\n");
 #else
diff --git a/board/freescale/corenet_ds/pci.c b/board/freescale/cor