Re: [U-Boot] [PATCH v2 55/55] x86: fdt: Drop the unused compatible strings in fdtdec

[Bin Meng]

Hi Simon,

On Mon, Jan 18, 2016 at 7:12 AM, Simon Glass  wrote:
> We have drivers for several more devices now, so drop the strings which are
> no-longer used.
>
> Signed-off-by: Simon Glass 
> Reviewed-by: Bin Meng 
> ---
>
> Changes in v2:
> - Drop bd82x6x_pci_init() function and associated DM-conversion patch
> - Drop LPC init method an 'Add an init() method to the LPC uclass' patch
>
>  arch/x86/cpu/ivybridge/lpc.c | 6 --
>  include/fdtdec.h | 5 -
>  lib/fdtdec.c | 5 -
>  3 files changed, 16 deletions(-)
>

This patch does not apply on existing u-boot-x86/master. I tried to
rebase u-boot-x86/master on top of latest u-boot/master, but it still
does not apply. Please rebase this patch on top of current
u-boot-x86/master and submit a new version. Thanks!

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v2 55/55] x86: fdt: Drop the unused compatible strings in fdtdec

[Simon Glass]

We have drivers for several more devices now, so drop the strings which are
no-longer used.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
---

Changes in v2:
- Drop bd82x6x_pci_init() function and associated DM-conversion patch
- Drop LPC init method an 'Add an init() method to the LPC uclass' patch

 arch/x86/cpu/ivybridge/lpc.c | 6 --
 include/fdtdec.h | 5 -
 lib/fdtdec.c | 5 -
 3 files changed, 16 deletions(-)

diff --git a/arch/x86/cpu/ivybridge/lpc.c b/arch/x86/cpu/ivybridge/lpc.c
index 9ab5ed3..deda2f5 100644
--- a/arch/x86/cpu/ivybridge/lpc.c
+++ b/arch/x86/cpu/ivybridge/lpc.c
@@ -491,8 +491,6 @@ static int lpc_early_init(struct udevice *dev)
 static int lpc_init_extra(struct udevice *dev)
 {
struct udevice *pch = dev->parent;
-   const void *blob = gd->fdt_blob;
-   int node;
 
debug("pch: lpc_init\n");
dm_pci_write_bar32(pch, 0, 0);
@@ -501,10 +499,6 @@ static int lpc_init_extra(struct udevice *dev)
dm_pci_write_bar32(pch, 3, 0x800);
dm_pci_write_bar32(pch, 4, 0x900);
 
-   node = fdtdec_next_compatible(blob, 0, COMPAT_INTEL_PCH);
-   if (node < 0)
-   return -ENOENT;
-
/* Set the value for PCI command register. */
dm_pci_write_config16(pch, PCI_COMMAND, 0x000f);
 
diff --git a/include/fdtdec.h b/include/fdtdec.h
index 27b350e..e545866 100644
--- a/include/fdtdec.h
+++ b/include/fdtdec.h
@@ -155,15 +155,10 @@ enum fdt_compat_id {
COMPAT_SAMSUNG_EXYNOS_SYSMMU,   /* Exynos sysmmu */
COMPAT_INTEL_MICROCODE, /* Intel microcode update */
COMPAT_MEMORY_SPD,  /* Memory SPD information */
-   COMPAT_INTEL_PANTHERPOINT_AHCI, /* Intel Pantherpoint AHCI */
-   COMPAT_INTEL_MODEL_206AX,   /* Intel Model 206AX CPU */
-   COMPAT_INTEL_GMA,   /* Intel Graphics Media Accelerator */
COMPAT_AMS_AS3722,  /* AMS AS3722 PMIC */
-   COMPAT_INTEL_ICH_SPI,   /* Intel ICH7/9 SPI controller */
COMPAT_INTEL_QRK_MRC,   /* Intel Quark MRC */
COMPAT_INTEL_X86_PINCTRL,   /* Intel ICH7/9 pin control */
COMPAT_SOCIONEXT_XHCI,  /* Socionext UniPhier xHCI */
-   COMPAT_INTEL_PCH,   /* Intel PCH */
COMPAT_INTEL_IRQ_ROUTER,/* Intel Interrupt Router */
COMPAT_ALTERA_SOCFPGA_DWMAC,/* SoCFPGA Ethernet controller */
COMPAT_ALTERA_SOCFPGA_DWMMC,/* SoCFPGA DWMMC controller */
diff --git a/lib/fdtdec.c b/lib/fdtdec.c
index b50d105..3c05559 100644
--- a/lib/fdtdec.c
+++ b/lib/fdtdec.c
@@ -60,15 +60,10 @@ static const char * const compat_names[COMPAT_COUNT] = {
COMPAT(SAMSUNG_EXYNOS_SYSMMU, "samsung,sysmmu-v3.3"),
COMPAT(INTEL_MICROCODE, "intel,microcode"),
COMPAT(MEMORY_SPD, "memory-spd"),
-   COMPAT(INTEL_PANTHERPOINT_AHCI, "intel,pantherpoint-ahci"),
-   COMPAT(INTEL_MODEL_206AX, "intel,model-206ax"),
-   COMPAT(INTEL_GMA, "intel,gma"),
COMPAT(AMS_AS3722, "ams,as3722"),
-   COMPAT(INTEL_ICH_SPI, "intel,ich-spi"),
COMPAT(INTEL_QRK_MRC, "intel,quark-mrc"),
COMPAT(INTEL_X86_PINCTRL, "intel,x86-pinctrl"),
COMPAT(SOCIONEXT_XHCI, "socionext,uniphier-xhci"),
-   COMPAT(COMPAT_INTEL_PCH, "intel,bd82x6x"),
COMPAT(COMPAT_INTEL_IRQ_ROUTER, "intel,irq-router"),
COMPAT(ALTERA_SOCFPGA_DWMAC, "altr,socfpga-stmmac"),
COMPAT(ALTERA_SOCFPGA_DWMMC, "altr,socfpga-dw-mshc"),
-- 
2.6.0.rc2.230.g3dd15c0

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot