[U-Boot] [PATCH v2 1/5] pci: fix errant data types and corresponding access functions

Kumar Gala galak at kernel.crashing.org
Wed Sep 19 16:39:41 CEST 2012


On Aug 29, 2012, at 7:16 PM, Andrew Sharp wrote:

> In a couple of places, unsigned int and pci_config_*_dword were being
> used when u16 and _word should be used.  Unsigned int was also being
> used in a couple of places that should be pci_addr_t.
> 
> Signed-off-by: Andrew Sharp <andywyse6 at gmail.com>
> ---
> drivers/pci/pci.c      |    7 ++++---
> drivers/pci/pci_auto.c |   15 ++++++++-------
> 2 files changed, 12 insertions(+), 10 deletions(-)

some of these changes cause warnings now:

pci.c: In function 'pci_map_bar':
pci.c:124:2: warning: passing argument 3 of 'pci_read_config_dword' from incompatible pointer type
pci.c:69:1: note: expected 'u32 *' but argument is of type 'pci_addr_t *'
pci.c: In function 'pci_hose_config_device':
pci.c:406:3: warning: passing argument 4 of 'pci_hose_read_config_dword' from incompatible pointer type
pci.c:48:1: note: expected 'u32 *' but argument is of type 'pci_addr_t *'
pci_auto.c: In function 'pciauto_setup_device':
pci_auto.c:111:3: warning: passing argument 4 of 'pci_hose_read_config_dword' from incompatible pointer type


> 
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index 398542b..cd78312 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -118,11 +118,11 @@ PCI_WRITE_VIA_DWORD_OP(word, u16, 0x02, 0x0000ffff)
> void *pci_map_bar(pci_dev_t pdev, int bar, int flags)
> {
> 	pci_addr_t pci_bus_addr;
> -	u32 bar_response;
> +	pci_addr_t bar_response;

this should really be a u32 not pci_addr_t.

> 
> 	/* read BAR address */
> 	pci_read_config_dword(pdev, bar, &bar_response);
> -	pci_bus_addr = (pci_addr_t)(bar_response & ~0xf);
> +	pci_bus_addr = bar_response & ~0xf;
> 
> 	/*
> 	 * Pass "0" as the length argument to pci_bus_to_virt.  The arg
> @@ -385,7 +385,8 @@ int pci_hose_config_device(struct pci_controller *hose,
> 			   pci_addr_t mem,
> 			   unsigned long command)
> {
> -	unsigned int bar_response, old_command;
> +	pci_addr_t bar_response;
> +	unsigned int old_command;

Same comment here.  

> 	pci_addr_t bar_value;
> 	pci_size_t bar_size;
> 	unsigned char pin;
> diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto.c
> index 87ee2c2..2338706 100644
> --- a/drivers/pci/pci_auto.c
> +++ b/drivers/pci/pci_auto.c
> @@ -88,15 +88,15 @@ void pciauto_setup_device(struct pci_controller *hose,
> 			  struct pci_region *prefetch,
> 			  struct pci_region *io)
> {
> -	unsigned int bar_response;
> +	pci_addr_t bar_response;

again, this should not be pci_addr_t

> 	pci_addr_t bar_value;
> 	pci_size_t bar_size;
> -	unsigned int cmdstat = 0;
> +	u16 cmdstat = 0;
> 	struct pci_region *bar_res;
> 	int bar, bar_nr = 0;
> 	int found_mem64 = 0;
> 
> -	pci_hose_read_config_dword(hose, dev, PCI_COMMAND, &cmdstat);
> +	pci_hose_read_config_word(hose, dev, PCI_COMMAND, &cmdstat);
> 	cmdstat = (cmdstat & ~(PCI_COMMAND_IO | PCI_COMMAND_MEMORY)) | PCI_COMMAND_MASTER;
> 
> 	for (bar = PCI_BASE_ADDRESS_0; bar < PCI_BASE_ADDRESS_0 + (bars_num*4); bar += 4) {
> @@ -167,7 +167,7 @@ void pciauto_setup_device(struct pci_controller *hose,
> 		bar_nr++;
> 	}
> 
> -	pci_hose_write_config_dword(hose, dev, PCI_COMMAND, cmdstat);
> +	pci_hose_write_config_word(hose, dev, PCI_COMMAND, cmdstat);
> 	pci_hose_write_config_byte(hose, dev, PCI_CACHE_LINE_SIZE,
> 		CONFIG_SYS_PCI_CACHE_LINE_SIZE);
> 	pci_hose_write_config_byte(hose, dev, PCI_LATENCY_TIMER, 0x80);
> @@ -179,9 +179,9 @@ void pciauto_prescan_setup_bridge(struct pci_controller *hose,
> 	struct pci_region *pci_mem = hose->pci_mem;
> 	struct pci_region *pci_prefetch = hose->pci_prefetch;
> 	struct pci_region *pci_io = hose->pci_io;
> -	unsigned int cmdstat;
> +	u16 cmdstat;
> 
> -	pci_hose_read_config_dword(hose, dev, PCI_COMMAND, &cmdstat);
> +	pci_hose_read_config_word(hose, dev, PCI_COMMAND, &cmdstat);
> 
> 	/* Configure bus number registers */
> 	pci_hose_write_config_byte(hose, dev, PCI_PRIMARY_BUS,
> @@ -229,7 +229,8 @@ void pciauto_prescan_setup_bridge(struct pci_controller *hose,
> 	}
> 
> 	/* Enable memory and I/O accesses, enable bus master */
> -	pci_hose_write_config_dword(hose, dev, PCI_COMMAND, cmdstat | PCI_COMMAND_MASTER);
> +	pci_hose_write_config_word(hose, dev, PCI_COMMAND,
> +					cmdstat | PCI_COMMAND_MASTER);
> }
> 
> void pciauto_postscan_setup_bridge(struct pci_controller *hose,
> -- 
> 1.7.1
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot



More information about the U-Boot mailing list