[U-Boot] [PATCH 01/10] x86: Return -1 when reading a PCI config register fails
Simon Glass
sjg at chromium.org
Fri Aug 14 18:49:55 CEST 2015
On 12 August 2015 at 20:44, Bin Meng <bmeng.cn at gmail.com> wrote:
> On Thu, Aug 13, 2015 at 10:09 AM, Simon Glass <sjg at chromium.org> wrote:
>> This can fail for internal reasons, so return a sensible value rather than
>> a random one.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>> arch/x86/cpu/pci.c | 9 ++++++---
>> 1 file changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/x86/cpu/pci.c b/arch/x86/cpu/pci.c
>> index f8da080..d2ec45a 100644
>> --- a/arch/x86/cpu/pci.c
>> +++ b/arch/x86/cpu/pci.c
>> @@ -76,7 +76,8 @@ unsigned int x86_pci_read_config8(pci_dev_t dev, unsigned where)
>> {
>> uint8_t value;
>>
>> - pci_hose_read_config_byte(get_hose(), dev, where, &value);
>> + if (pci_hose_read_config_byte(get_hose(), dev, where, &value))
>> + return -1U;
>>
>> return value;
>> }
>> @@ -85,7 +86,8 @@ unsigned int x86_pci_read_config16(pci_dev_t dev, unsigned where)
>> {
>> uint16_t value;
>>
>> - pci_hose_read_config_word(get_hose(), dev, where, &value);
>> + if (pci_hose_read_config_word(get_hose(), dev, where, &value))
>> + return -1U;
>>
>> return value;
>> }
>> @@ -94,7 +96,8 @@ unsigned int x86_pci_read_config32(pci_dev_t dev, unsigned where)
>> {
>> uint32_t value;
>>
>> - pci_hose_read_config_dword(get_hose(), dev, where, &value);
>> + if (pci_hose_read_config_dword(get_hose(), dev, where, &value))
>> + return -1U;
>>
>> return value;
>> }
>> --
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Applied to u-boot-x86.
More information about the U-Boot
mailing list