x86: Use struct mrc_region to describe a mrc region

Currently struct fmap_entry is used to describe a mrc region.
However this structure contains some other fields that are not
related to mrc cache and causes confusion. Besides, it does not
include a base address field to store SPI flash's base address.
Instead in the mrccache.c it tries to use CONFIG_ROM_SIZE to
calculate the SPI flash base address, which unfortunately is
not 100% correct as CONFIG_ROM_SIZE may not match the whole
SPI flash size.

Define a new struct mrc_region and use it instead.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Simon Glass <sjg@chromium.org>

diff --git a/arch/x86/cpu/ivybridge/sdram.c b/arch/x86/cpu/ivybridge/sdram.c
index 20c2e6891087e356912b05ec2c6571d4c86d212e..9121426cae3ed2abe8e21ce6e919a7ee85b96a1b 100644 (file)
--- a/arch/x86/cpu/ivybridge/sdram.c
+++ b/arch/x86/cpu/ivybridge/sdram.c
@@ -138,7 +138,7 @@ static int read_seed_from_cmos(struct pei_data *pei_data)
 static int prepare_mrc_cache(struct pei_data *pei_data)
 {
        struct mrc_data_container *mrc_cache;
-       struct fmap_entry entry;
+       struct mrc_region entry;
        int ret;
 
        ret = read_seed_from_cmos(pei_data);

diff --git a/arch/x86/include/asm/mrccache.h b/arch/x86/include/asm/mrccache.h
index bcf7117f09e404b25d32d53a7ca979247b506283..e35b5ed771030c847f0f95bcd6e92cfe1833e8f9 100644 (file)
--- a/arch/x86/include/asm/mrccache.h
+++ b/arch/x86/include/asm/mrccache.h
@@ -22,7 +22,12 @@ struct __packed mrc_data_container {
        u8      data[0];        /* Variable size, platform/run time dependent */
 };
 
-struct fmap_entry;
+struct mrc_region {
+       u32     base;
+       u32     offset;
+       u32     length;
+};
+
 struct udevice;
 
 /**
@@ -34,7 +39,7 @@ struct udevice;
  * @entry:     Position and size of MRC cache in SPI flash
  * @return pointer to latest record, or NULL if none
  */
-struct mrc_data_container *mrccache_find_current(struct fmap_entry *entry);
+struct mrc_data_container *mrccache_find_current(struct mrc_region *entry);
 
 /**
  * mrccache_update() - update the MRC cache with a new record
@@ -48,7 +53,7 @@ struct mrc_data_container *mrccache_find_current(struct fmap_entry *entry);
  * @return 0 if updated, -EEXIST if the record is the same as the latest
  * record, -EINVAL if the record is not valid, other error if SPI write failed
  */
-int mrccache_update(struct udevice *sf, struct fmap_entry *entry,
+int mrccache_update(struct udevice *sf, struct mrc_region *entry,
                    struct mrc_data_container *cur);
 
 /**
@@ -87,7 +92,7 @@ int mrccache_reserve(void);
  * tree, -EINVAL if MRC region properties format is incorrect, other error
  * if SPI flash probe failed.
  */
-int mrccache_get_region(struct udevice **devp, struct fmap_entry *entry);
+int mrccache_get_region(struct udevice **devp, struct mrc_region *entry);
 
 /**
  * mrccache_save() - save MRC data to the SPI flash

diff --git a/arch/x86/lib/mrccache.c b/arch/x86/lib/mrccache.c
index 0efae6dbdf24231ab80d8ec181f0db4f31008e05..53a1259d095369aad533eef8ec63435b1bff6d95 100644 (file)
--- a/arch/x86/lib/mrccache.c
+++ b/arch/x86/lib/mrccache.c
@@ -40,13 +40,13 @@ static int is_mrc_cache(struct mrc_data_container *cache)
        return cache && (cache->signature == MRC_DATA_SIGNATURE);
 }
 
-struct mrc_data_container *mrccache_find_current(struct fmap_entry *entry)
+struct mrc_data_container *mrccache_find_current(struct mrc_region *entry)
 {
        struct mrc_data_container *cache, *next;
        ulong base_addr, end_addr;
        uint id;
 
-       base_addr = (1ULL << 32) - CONFIG_ROM_SIZE + entry->offset;
+       base_addr = entry->base + entry->offset;
        end_addr = base_addr + entry->length;
        cache = NULL;
 
@@ -85,12 +85,12 @@ struct mrc_data_container *mrccache_find_current(struct fmap_entry *entry)
  *
  * @return next cache entry if found, NULL if we got to the end
  */
-static struct mrc_data_container *find_next_mrc_cache(struct fmap_entry *entry,
+static struct mrc_data_container *find_next_mrc_cache(struct mrc_region *entry,
                struct mrc_data_container *cache)
 {
        ulong base_addr, end_addr;
 
-       base_addr = (1ULL << 32) - CONFIG_ROM_SIZE + entry->offset;
+       base_addr = entry->base + entry->offset;
        end_addr = base_addr + entry->length;
 
        cache = next_mrc_block(cache);
@@ -106,7 +106,7 @@ static struct mrc_data_container *find_next_mrc_cache(struct fmap_entry *entry,
        return cache;
 }
 
-int mrccache_update(struct udevice *sf, struct fmap_entry *entry,
+int mrccache_update(struct udevice *sf, struct mrc_region *entry,
                    struct mrc_data_container *cur)
 {
        struct mrc_data_container *cache;
@@ -118,7 +118,7 @@ int mrccache_update(struct udevice *sf, struct fmap_entry *entry,
                return -EINVAL;
 
        /* Find the last used block */
-       base_addr = (1ULL << 32) - CONFIG_ROM_SIZE + entry->offset;
+       base_addr = entry->base + entry->offset;
        debug("Updating MRC cache data\n");
        cache = mrccache_find_current(entry);
        if (cache && (cache->data_size == cur->data_size) &&
@@ -189,10 +189,11 @@ int mrccache_reserve(void)
        return 0;
 }
 
-int mrccache_get_region(struct udevice **devp, struct fmap_entry *entry)
+int mrccache_get_region(struct udevice **devp, struct mrc_region *entry)
 {
        const void *blob = gd->fdt_blob;
        int node, mrc_node;
+       u32 reg[2];
        int ret;
 
        /* Find the flash chip within the SPI controller node */
@@ -200,13 +201,19 @@ int mrccache_get_region(struct udevice **devp, struct fmap_entry *entry)
        if (node < 0)
                return -ENOENT;
 
+       if (fdtdec_get_int_array(blob, node, "memory-map", reg, 2))
+               return -FDT_ERR_NOTFOUND;
+       entry->base = reg[0];
+
        /* Find the place where we put the MRC cache */
        mrc_node = fdt_subnode_offset(blob, node, "rw-mrc-cache");
        if (mrc_node < 0)
                return -EPERM;
 
-       if (fdtdec_read_fmap_entry(blob, mrc_node, "rm-mrc-cache", entry))
-               return -EINVAL;
+       if (fdtdec_get_int_array(blob, mrc_node, "reg", reg, 2))
+               return -FDT_ERR_NOTFOUND;
+       entry->offset = reg[0];
+       entry->length = reg[1];
 
        if (devp) {
                ret = uclass_get_device_by_of_offset(UCLASS_SPI_FLASH, node,
@@ -222,7 +229,7 @@ int mrccache_get_region(struct udevice **devp, struct fmap_entry *entry)
 int mrccache_save(void)
 {
        struct mrc_data_container *data;
-       struct fmap_entry entry;
+       struct mrc_region entry;
        struct udevice *sf;
        int ret;