PCI config read functions

[Andrew Goodbody]

Hi Tom,

Smatch reports that the callers of the functions 
(dm_)pci_config_read(8|16|32) from drivers/pci/pci-uclass.c do not check 
the return value for an error before using the variable that should have 
been assigned with the value read. However in the error path that 
variable is not assigned to and so the code can use an uninitialised 
variable in that case.
eg line 1396 in drivers/pci/pci-uclass.c addr may not have been assigned 
to in the error case.

This also seems to be the case with callers from other files as well.

Would a patch that adds a default assignment before returning an error 
to the affected functions be accepted?

Thanks,
Andrew