u-boot.git/common/cmd_pci.c, branch master

Move all command code into its own directory

2016-01-25T15:39:43+00:00

There are a lot of unrelated files in common, including all of the commands.
Moving them into their own directory makes them easier to find and is more
logical.

Some commands include non-command code, such as cmd_scsi.c. This should be
sorted out at some point so that the function can be enabled with or without
the associated command.

Unfortunately, with m68k I get this error:

m68k:  +   M5329AFEE
+arch/m68k/cpu/mcf532x/start.o: In function `_start':
+arch/m68k/cpu/mcf532x/start.S:159:(.text+0x452): relocation truncated to fit: R_68K_PC16 against symbol `board_init_f' defined in .text.board_init_f section in common/built-in.o

I hope someone can shed some light on what this means. I hope it isn't
depending on the position of code in the image.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Reviewed-by: Heiko Schocher 
Acked-by: Stefan Roese 
Acked-by: Przemyslaw Marczak

dm: pci: Add a dm_ prefix to pci_bus_find_bdf()

2016-01-12T17:19:09+00:00

Most driver model PCI functions have a dm_ prefix. At some point, when the
old code is converted to driver model and the old functions are removed, we
will drop that prefix.

For consistency, we should use the dm_ prefix for all driver model
functions. Update pci_bus_find_bdf() accordingly.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

dm: pci: Convert 'pci' command to driver model

2015-12-01T13:26:38+00:00

Adjust this command to use the correct PCI functions, instead of the
compatibility layer.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Tested-by: Bin Meng

pci: Move PCI header output code into its own function

2015-12-01T13:26:37+00:00

We want to share this code with the driver model version, so put it in a
separate function.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

pci: Use a separate 'dev' variable for the PCI device

2015-12-01T13:26:37+00:00

In the 'pci' command, add a separate variable to hold the PCI device. When
this code is converted to driver model, this variable will be used to hold a
struct udevice instead.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

pci: Use common functions to read/write config

2015-12-01T13:26:37+00:00

Currently we use switch() and access PCI configuration via several
functions, one for each data size. Adjust the code to use generic functions,
where the data size is a parameter.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Tested-by: Bin Meng

pci: Tidy up function comments in cmd_pci.c

2015-12-01T13:26:37+00:00

The function comments use an old style and some are incorrect. Update them.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

dm: pci: Reorder functions in cmd_pci.c

2015-12-01T13:26:37+00:00

Before converting this to driver model, reorder the code to avoid forward
function declarations.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

pci: Refactor the pciinfo() function

2015-12-01T13:26:37+00:00

This function uses macros to output data. It seems better to use a table of
registers rather than macro-based code generation. It also reduces the
code/data size by 2KB on ARM.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Tested-by: Bin Meng

pci: Use a separate variable for the bus number

2015-12-01T13:26:37+00:00

At present in do_pci(), bdf can either mean a bus number or a PCI bus number.
Use separate variables instead to reduce confusion.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng