[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