[U-Boot] [PATCH 2/8] dm: pci: Only allow serial device to be bound before relocation
Bin Meng
bmeng.cn at gmail.com
Tue Aug 18 04:02:36 CEST 2015
Hi Simon,
On Tue, Aug 18, 2015 at 10:00 AM, Simon Glass <sjg at chromium.org> wrote:
> Hi Bin,
>
> On 15 August 2015 at 01:07, Bin Meng <bmeng.cn at gmail.com> wrote:
>> To further limit the memory space, we only allow serial device
>> to be bound before relocation.
>>
>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> ---
>>
>> drivers/pci/pci-uclass.c | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c
>> index 4160274..b7dca0f 100644
>> --- a/drivers/pci/pci-uclass.c
>> +++ b/drivers/pci/pci-uclass.c
>> @@ -487,9 +487,13 @@ static int pci_find_and_bind_driver(struct udevice *parent,
>> * whose driver has the DM_FLAG_PRE_RELOC set, to save
>> * precious memory space as on some platforms as that
>> * space is pretty limited (ie: using Cache As RAM).
>> + *
>> + * To further limit the memory space, we only allow
>> + * serial device to be bound.
>> */
>> if (!(gd->flags & GD_FLG_RELOC) &&
>> - !(drv->flags & DM_FLAG_PRE_RELOC))
>> + !(drv->flags & DM_FLAG_PRE_RELOC) &&
>> + (drv->id != UCLASS_SERIAL))
>> return 0;
>>
>> /*
>> --
>> 1.8.2.1
>>
>
> This seems to go too far. We may have other drivers that need to do
> pre-reloc init. Why not just rely on DM_FLAG_PRE_RELOC?
>
I was thinking the only usable pci driver before relocation is serial?
Regards,
Bin
More information about the U-Boot
mailing list