arch: arm: meson: support Amlogic chip_id v1 and v2
authorEvgeny Bachinin <EABachinin@salutedevices.com>
Mon, 10 Feb 2025 17:50:16 +0000 (20:50 +0300)
committerNeil Armstrong <neil.armstrong@linaro.org>
Tue, 18 Mar 2025 14:27:25 +0000 (15:27 +0100)
Patch introduces:
* chip_id API - useful for various things, but used now for
  device_id (did) generation as mentioned in [1] on our private board
  code. Our device_id is calculated by means of permutations of
  chip_id value.
* new SoCs (a1, s4, etc) are usually coming with the support of chip_id
  v2 right away, whereas secure monitors on old SoCs (like axg, g12b,
  g12a, etc) may support only chip_id v1. Chip_id API handles both
  cases
* meson_sm_get_serial() is described via chip_id API.

Links:
[1] https://lore.kernel.org/linux-arm-kernel/202311242104.RjBPI3uI-lkp@intel.com/T/#m630fbeea6a6e7d531290b5c0af205af4fb979757

Signed-off-by: Viacheslav Bocharov <adeep@lexina.in>
Co-developed-by: Arseniy Krasnov <avkrasnov@salutedevices.com>
Signed-off-by: Arseniy Krasnov <avkrasnov@salutedevices.com>
Signed-off-by: Evgeny Bachinin <EABachinin@salutedevices.com>
Link: https://lore.kernel.org/r/20250210-meson_chip_id_all_vers-v1-3-b98f8b6880b8@salutedevices.com
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
arch/arm/include/asm/arch-meson/boot.h
arch/arm/include/asm/arch-meson/sm.h
arch/arm/mach-meson/board-info.c
arch/arm/mach-meson/sm.c

index c67d12d..a11dfde 100644 (file)
@@ -21,4 +21,18 @@ int meson_get_boot_device(void);
 
 int meson_get_soc_rev(char *buff, size_t buff_len);
 
+/**
+ * meson_get_socinfo - retrieve cpu_id of the Amlogic SoC
+ *
+ * The value in the following format is read from register:
+ *   +-----------+------------+------------+------------+
+ *   | family_id | package_id |  chip_rev  | layout_rev |
+ *   +-----------+------------+------------+------------+
+ *   | 31     24 | 23      16 | 15       8 | 7        0 |
+ *   +-----------+------------+------------+------------+
+ *
+ * Return: 4 bytes value of cpu_id on success or 0 on failure.
+ */
+u32 meson_get_socinfo(void);
+
 #endif /* __MESON_BOOT_H__ */
index 4b1d564..4d61495 100644 (file)
@@ -6,6 +6,8 @@
 #ifndef __MESON_SM_H__
 #define __MESON_SM_H__
 
+#include <asm/types.h>
+
 /**
  * meson_sm_read_efuse - read efuse memory into buffer
  *
@@ -27,16 +29,60 @@ ssize_t meson_sm_read_efuse(uintptr_t offset, void *buffer, size_t size);
 ssize_t meson_sm_write_efuse(uintptr_t offset, void *buffer, size_t size);
 
 #define SM_SERIAL_SIZE 12
+#define MESON_CPU_ID_SZ        4
+#define MESON_CHIP_ID_SZ 16
+
+/**
+ * union meson_cpu_id - Amlogic cpu_id.
+ * @raw: buffer to hold the cpu_id value as sequential bytes.
+ * @val: cpu_id represented as 32 bit value.
+ */
+union meson_cpu_id {
+       u8 raw[MESON_CPU_ID_SZ];
+       u32 val;
+};
+
+/**
+ * struct meson_sm_chip_id - Amlogic chip_id.
+ * @cpu_id: cpu_id value, which is distinct from socinfo in that the order of
+ *          PACK & MINOR bytes are swapped according to Amlogic chip_id format.
+ * @serial: 12 byte unique SoC number, identifying particular die, read
+ *          usually from efuse OTP storage. Serial comes in little-endian
+ *          order.
+ */
+struct meson_sm_chip_id {
+       union meson_cpu_id cpu_id;
+       u8 serial[SM_SERIAL_SIZE];
+};
 
 /**
- * meson_sm_get_serial - read chip unique id into buffer
+ * meson_sm_get_serial - read chip unique serial (OTP data) into buffer
  *
  * @buffer: pointer to buffer
  * @size: buffer size.
+ *
+ * Serial is returned in big-endian order.
+ *
  * @return: zero on success or -errno on failure
  */
 int meson_sm_get_serial(void *buffer, size_t size);
 
+/**
+ * meson_sm_get_chip_id - read Amlogic chip_id
+ *
+ * @chip_id: pointer to buffer capable to hold the struct meson_sm_chip_id
+ *
+ * Amlogic SoCs support 2 versions of chip_id. Function requests the newest
+ * one (v2), but if chip_id v2 is not supported, then secure monitor returns
+ * v1. All differences between v1 and v2 versions are handled by this function
+ * and chip_id is returned in unified format.
+ *
+ * chip_id contains serial, which is returned here in little-endian order.
+ *
+ * @return: 0 on success or -errno on failure
+ */
+int meson_sm_get_chip_id(struct meson_sm_chip_id *chip_id);
+
 enum {
        REBOOT_REASON_COLD = 0,
        REBOOT_REASON_NORMAL = 1,
index 94b0661..d3807dc 100644 (file)
@@ -126,7 +126,7 @@ static const char *socinfo_to_soc_id(u32 socinfo)
        return "Unknown";
 }
 
-static u32 get_socinfo(void)
+u32 meson_get_socinfo(void)
 {
        struct regmap *regmap;
        int nodeoffset, ret;
@@ -165,7 +165,7 @@ int checkboard(void)
 {
        u32 socinfo;
 
-       socinfo = get_socinfo();
+       socinfo = meson_get_socinfo();
        if (!socinfo)
                return 0;
 
@@ -184,7 +184,7 @@ int meson_get_soc_rev(char *buff, size_t buff_len)
 {
        u32 socinfo;
 
-       socinfo = get_socinfo();
+       socinfo = meson_get_socinfo();
        if (!socinfo)
                return -1;
 
index 43a259d..b1f91ca 100644 (file)
 #include <regmap.h>
 #include <sm.h>
 #include <syscon.h>
+#include <asm/arch/boot.h>
 #include <asm/arch/sm.h>
 #include <asm/cache.h>
 #include <asm/global_data.h>
 #include <asm/ptrace.h>
 #include <linux/bitops.h>
+#include <linux/compiler_attributes.h>
 #include <linux/err.h>
 #include <linux/kernel.h>
 #include <linux/bitfield.h>
@@ -76,30 +78,131 @@ ssize_t meson_sm_write_efuse(uintptr_t offset, void *buffer, size_t size)
        return err;
 }
 
-#define SM_CHIP_ID_LENGTH      119
-#define SM_CHIP_ID_OFFSET      4
+/*
+ * Helps to handle two flavors of cpu_id layouts:
+ *
+ * - in-register view (value read from cpu_id reg, a.k.a. socinfo):
+ *   +-----------+------------+------------+------------+
+ *   | family_id | package_id |  chip_rev  | layout_rev |
+ *   +-----------+------------+------------+------------+
+ *   | 31     24 | 23      16 | 15       8 | 7        0 |
+ *   +-----------+------------+------------+------------+
+ *
+ * - in-efuse view (value, residing inside efuse/shmem data usually for
+ *   chip_id v2. Chip_id v1 does not contain cpu_id value inside efuse
+ *   data (i.e. in chip_id_efuse)):
+ *   +-----------+------------+------------+------------+
+ *   | family_id |  chip_rev  | package_id | layout_rev |
+ *   +-----------+------------+------------+------------+
+ *   | 31     24 | 23      16 | 15       8 | 7        0 |
+ *   +-----------+------------+------------+------------+
+ */
+enum {
+       /* In-register view of cpu_id */
+       CPU_ID_REG_MAJOR,       /* 31-24 bits */
+       CPU_ID_REG_PACK,        /* 23-16 bits */
+       CPU_ID_REG_MINOR,       /* 15-8 bits */
+       CPU_ID_REG_MISC,        /* 7-0 bits */
 
-int meson_sm_get_serial(void *buffer, size_t size)
+       /* In-efuse view of cpu_id */
+       CPU_ID_MAJOR = CPU_ID_REG_MAJOR,
+       CPU_ID_PACK  = CPU_ID_REG_MINOR,
+       CPU_ID_MINOR = CPU_ID_REG_PACK,
+       CPU_ID_MISC  = CPU_ID_REG_MISC,
+};
+
+/*
+ * This is a beginning chunk of the whole efuse storage area, containing
+ * data related to chip_id only
+ */
+struct chip_id_efuse {
+       u32 version;
+       u8 raw[MESON_CHIP_ID_SZ]; /* payload */
+} __packed;
+
+static void meson_sm_serial_reverse(u8 serial[SM_SERIAL_SIZE])
+{
+       for (int i = 0; i < SM_SERIAL_SIZE / 2; i++) {
+               int k = SM_SERIAL_SIZE - 1 - i;
+
+               swap(serial[i], serial[k]);
+       }
+}
+
+int meson_sm_get_chip_id(struct meson_sm_chip_id *chip_id)
 {
        struct udevice *dev;
+       union meson_cpu_id socinfo;
        struct pt_regs regs = { 0 };
-       u8 id_buffer[SM_CHIP_ID_LENGTH];
+       struct chip_id_efuse chip_id_efuse;
        int err;
 
        dev = meson_get_sm_device();
        if (IS_ERR(dev))
                return PTR_ERR(dev);
 
-       err = sm_call_read(dev, id_buffer, SM_CHIP_ID_LENGTH,
+       /*
+        * Request v2. If not supported by secure monitor, then v1 should be
+        * returned.
+        */
+       regs.regs[1] = 2;
+
+       err = sm_call_read(dev, &chip_id_efuse, sizeof(chip_id_efuse),
                           MESON_SMC_CMD_CHIP_ID_GET, &regs);
-       if (err < 0)
-               pr_err("Failed to read serial number (%d)\n", err);
+       if (err < 0) {
+               pr_err("Failed to read chip_id (%d)\n", err);
+               return err;
+       }
+
+       if (chip_id_efuse.version == 2) {
+               memcpy((u8 *)chip_id, chip_id_efuse.raw,
+                      sizeof(struct meson_sm_chip_id));
+               return 0;
+       }
 
-       memcpy(buffer, id_buffer + SM_CHIP_ID_OFFSET, size);
+       /*
+        * Legacy chip_id (v1) read out, transform data
+        * to expected order format (little-endian)
+        */
+       memcpy(chip_id->serial, chip_id_efuse.raw, sizeof(chip_id->serial));
+       meson_sm_serial_reverse(chip_id->serial);
+
+       socinfo.val = meson_get_socinfo();
+       if (!socinfo.val)
+               return -ENODEV;
+
+       chip_id->cpu_id = (union meson_cpu_id){
+               .raw[CPU_ID_MAJOR] = socinfo.raw[CPU_ID_REG_MAJOR],
+               .raw[CPU_ID_PACK]  = socinfo.raw[CPU_ID_REG_PACK],
+               .raw[CPU_ID_MINOR] = socinfo.raw[CPU_ID_REG_MINOR],
+               .raw[CPU_ID_MISC]  = socinfo.raw[CPU_ID_REG_MISC],
+       };
 
        return 0;
 }
 
+int meson_sm_get_serial(void *buffer, size_t size)
+{
+       struct meson_sm_chip_id chip_id;
+       int ret;
+
+       if (size < SM_SERIAL_SIZE)
+               return -EINVAL;
+
+       ret = meson_sm_get_chip_id(&chip_id);
+       if (ret)
+               return ret;
+
+       /*
+        * The order of serial inside chip_id and serial which function must
+        * return does not match: stick here to big-endian for backward
+        * compatibility.
+        */
+       meson_sm_serial_reverse(chip_id.serial);
+       memcpy(buffer, chip_id.serial, sizeof(chip_id.serial));
+       return ret;
+}
+
 #define AO_SEC_SD_CFG15                0xfc
 #define REBOOT_REASON_MASK     GENMASK(15, 12)