[U-Boot] [PATCH] x86: baytrail: Add 2nd eMMC controller to the PCI probe list

Bin Meng bmeng.cn at gmail.com
Mon Sep 26 08:33:11 CEST 2016


Hi Stefan,

On Fri, Sep 16, 2016 at 8:37 PM, Stefan Roese <sr at denx.de> wrote:
> With this addition, the eMMC device available on the congatec and DFI
> BayTrail SoM is detected correctly.
>
> Signed-off-by: Stefan Roese <sr at denx.de>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Bin Meng <bmeng.cn at gmail.com>
> ---
>  arch/x86/cpu/baytrail/valleyview.c | 1 +
>  include/pci_ids.h                  | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/arch/x86/cpu/baytrail/valleyview.c b/arch/x86/cpu/baytrail/valleyview.c
> index b31f24e..9868de0 100644
> --- a/arch/x86/cpu/baytrail/valleyview.c
> +++ b/arch/x86/cpu/baytrail/valleyview.c
> @@ -14,6 +14,7 @@
>  static struct pci_device_id mmc_supported[] = {
>         { PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_VALLEYVIEW_SDIO },
>         { PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_VALLEYVIEW_SDCARD },
> +       { PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BYT_EMMC2 },

pure nits, should we use VALLEYVIEW_EMMC2? or should we change
previous VALLEYVIEW_SDIO/SDCARD to BYT_SDIO/SDCARD, for consistency?

>         {},
>  };
>
> diff --git a/include/pci_ids.h b/include/pci_ids.h
> index 17a01a0..e63b94f 100644
> --- a/include/pci_ids.h
> +++ b/include/pci_ids.h
> @@ -2606,6 +2606,7 @@
>  #define PCI_DEVICE_ID_INTEL_VALLEYVIEW_IDE_ALT 0x0f21
>  #define PCI_DEVICE_ID_INTEL_VALLEYVIEW_SATA    0x0f22
>  #define PCI_DEVICE_ID_INTEL_VALLEYVIEW_SATA_ALT        0x0f23
> +#define PCI_DEVICE_ID_INTEL_BYT_EMMC2  0x0f50
>  #define PCI_DEVICE_ID_INTEL_82541ER    0x1078
>  #define PCI_DEVICE_ID_INTEL_82541GI_LF 0x107c
>  #define PCI_DEVICE_ID_INTEL_82542      0x1000
> --

Regards,
Bin


More information about the U-Boot mailing list