[U-Boot] [PATCH 38/57] x86: ivybridge: Drop the special PCI driver

[Simon Glass]

Hi Bin,

On 13 December 2015 at 05:56, Bin Meng <bmeng.cn at gmail.com> wrote:
> Hi Simon,
>
> On Tue, Dec 8, 2015 at 11:38 AM, Simon Glass <sjg at chromium.org> wrote:
>> There is nothing special about the ivybridge pci driver now, so just use
>> the generic one.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>>  arch/x86/cpu/ivybridge/Makefile |  1 -
>>  arch/x86/cpu/ivybridge/pci.c    | 46 -----------------------------------------
>>  2 files changed, 47 deletions(-)
>>  delete mode 100644 arch/x86/cpu/ivybridge/pci.c
>>
>> diff --git a/arch/x86/cpu/ivybridge/Makefile b/arch/x86/cpu/ivybridge/Makefile
>> index bdbd3fa..259a5df 100644
>> --- a/arch/x86/cpu/ivybridge/Makefile
>> +++ b/arch/x86/cpu/ivybridge/Makefile
>> @@ -15,7 +15,6 @@ obj-y += model_206ax.o
>>  obj-y += microcode_intel.o
>>  obj-y += northbridge.o
>>  obj-y += pch.o
>> -obj-y += pci.o
>>  obj-y += report_platform.o
>>  obj-y += sata.o
>>  obj-y += sdram.o
>> diff --git a/arch/x86/cpu/ivybridge/pci.c b/arch/x86/cpu/ivybridge/pci.c
>> deleted file mode 100644
>> index 5195002..0000000
>> --- a/arch/x86/cpu/ivybridge/pci.c
>> +++ /dev/null
>> @@ -1,46 +0,0 @@
>> -/*
>> - * Copyright (c) 2011 The Chromium OS Authors.
>> - * (C) Copyright 2008,2009
>> - * Graeme Russ, <graeme.russ at gmail.com>
>> - *
>> - * (C) Copyright 2002
>> - * Daniel Engström, Omicron Ceti AB, <daniel at omicron.se>
>> - *
>> - * SPDX-License-Identifier:    GPL-2.0+
>> - */
>> -
>> -#include <common.h>
>> -#include <dm.h>
>> -#include <pci.h>
>> -#include <asm/pci.h>
>> -#include <asm/post.h>
>> -#include <asm/arch/bd82x6x.h>
>> -#include <asm/arch/pch.h>
>> -
>> -static int pci_ivybridge_probe(struct udevice *bus)
>> -{
>> -       if (!(gd->flags & GD_FLG_RELOC))
>> -               return 0;
>> -       post_code(0x50);
>> -       post_code(0x51);
>> -
>> -       return 0;
>> -}
>> -
>> -static const struct dm_pci_ops pci_ivybridge_ops = {
>> -       .read_config    = pci_x86_read_config,
>> -       .write_config   = pci_x86_write_config,
>> -};
>> -
>> -static const struct udevice_id pci_ivybridge_ids[] = {
>> -       { .compatible = "intel,pci-ivybridge" },
>> -       { }
>> -};
>> -
>> -U_BOOT_DRIVER(pci_ivybridge_drv) = {
>> -       .name           = "pci_ivybridge",
>> -       .id             = UCLASS_PCI,
>> -       .of_match       = pci_ivybridge_ids,
>> -       .ops            = &pci_ivybridge_ops,
>> -       .probe          = pci_ivybridge_probe,
>> -};
>> --
>
> We don't need change chromebook_link.dts to update pci node compatible string?
>
> Regards,
> Bin

It isn't necessary, but I can remove the 'intel,pci-ivybridge' string.

Regards,
Simon