Merge branch '2021-03-19-assorted-fixes'

- NVMe cache fixes, API bugfix with !CONFIG_SYS_64BIT_LBA, ahci mediatek
  fix when debugging, ti-sysc uclass fix, sl28 MMC boot fix

diff --git a/configs/kontron_sl28_defconfig b/configs/kontron_sl28_defconfig
index 1c781e0..0c6c191 100644 (file)
--- a/configs/kontron_sl28_defconfig
+++ b/configs/kontron_sl28_defconfig
@@ -70,7 +70,7 @@ CONFIG_I2C_SET_DEFAULT_BUS_NUM=y
 CONFIG_I2C_DEFAULT_BUS_NUMBER=0
 CONFIG_I2C_MUX=y
 CONFIG_DM_MMC=y
-CONFIG_MMC_HS400_SUPPORT=y
+CONFIG_MMC_HS200_SUPPORT=y
 CONFIG_FSL_ESDHC=y
 CONFIG_FSL_ESDHC_SUPPORT_ADMA2=y
 CONFIG_DM_SPI_FLASH=y

diff --git a/drivers/ata/mtk_ahci.c b/drivers/ata/mtk_ahci.c
index 554175b..2c5227d 100644 (file)
--- a/drivers/ata/mtk_ahci.c
+++ b/drivers/ata/mtk_ahci.c
@@ -21,6 +21,7 @@
 #include <sata.h>
 #include <scsi.h>
 #include <syscon.h>
+#include <dm/device_compat.h>
 
 #define SYS_CFG                        0x14
 #define SYS_CFG_SATA_MSK       GENMASK(31, 30)

diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c
index 4e3d610..778c065 100644 (file)
--- a/drivers/bus/ti-sysc.c
+++ b/drivers/bus/ti-sysc.c
@@ -148,12 +148,6 @@ clocks_err:
        return err;
 }
 
-UCLASS_DRIVER(ti_sysc) = {
-       .id = UCLASS_SIMPLE_BUS,
-       .name = "ti_sysc",
-       .post_bind = dm_scan_fdt_dev
-};
-
 U_BOOT_DRIVER(ti_sysc) = {
        .name = "ti_sysc",
        .id = UCLASS_SIMPLE_BUS,

diff --git a/drivers/nvme/nvme.c b/drivers/nvme/nvme.c
index 5d6331a..c61dab2 100644 (file)
--- a/drivers/nvme/nvme.c
+++ b/drivers/nvme/nvme.c
@@ -22,6 +22,8 @@
 #define NVME_AQ_DEPTH          2
 #define NVME_SQ_SIZE(depth)    (depth * sizeof(struct nvme_command))
 #define NVME_CQ_SIZE(depth)    (depth * sizeof(struct nvme_completion))
+#define NVME_CQ_ALLOCATION     ALIGN(NVME_CQ_SIZE(NVME_Q_DEPTH), \
+                                     ARCH_DMA_MINALIGN)
 #define ADMIN_TIMEOUT          60
 #define IO_TIMEOUT             30
 #define MAX_PRP_POOL           512
@@ -144,8 +146,14 @@ static __le16 nvme_get_cmd_id(void)
 
 static u16 nvme_read_completion_status(struct nvme_queue *nvmeq, u16 index)
 {
-       u64 start = (ulong)&nvmeq->cqes[index];
-       u64 stop = start + sizeof(struct nvme_completion);
+       /*
+        * Single CQ entries are always smaller than a cache line, so we
+        * can't invalidate them individually. However CQ entries are
+        * read only by the CPU, so it's safe to always invalidate all of them,
+        * as the cache line should never become dirty.
+        */
+       ulong start = (ulong)&nvmeq->cqes[0];
+       ulong stop = start + NVME_CQ_ALLOCATION;
 
        invalidate_dcache_range(start, stop);
 
@@ -241,7 +249,7 @@ static struct nvme_queue *nvme_alloc_queue(struct nvme_dev *dev,
                return NULL;
        memset(nvmeq, 0, sizeof(*nvmeq));
 
-       nvmeq->cqes = (void *)memalign(4096, NVME_CQ_SIZE(depth));
+       nvmeq->cqes = (void *)memalign(4096, NVME_CQ_ALLOCATION);
        if (!nvmeq->cqes)
                goto free_nvmeq;
        memset((void *)nvmeq->cqes, 0, NVME_CQ_SIZE(depth));
@@ -339,7 +347,7 @@ static void nvme_init_queue(struct nvme_queue *nvmeq, u16 qid)
        nvmeq->q_db = &dev->dbs[qid * 2 * dev->db_stride];
        memset((void *)nvmeq->cqes, 0, NVME_CQ_SIZE(nvmeq->q_depth));
        flush_dcache_range((ulong)nvmeq->cqes,
-                          (ulong)nvmeq->cqes + NVME_CQ_SIZE(nvmeq->q_depth));
+                          (ulong)nvmeq->cqes + NVME_CQ_ALLOCATION);
        dev->online_queues++;
 }
 
@@ -481,6 +489,7 @@ int nvme_get_features(struct nvme_dev *dev, unsigned fid, unsigned nsid,
                      dma_addr_t dma_addr, u32 *result)
 {
        struct nvme_command c;
+       int ret;
 
        memset(&c, 0, sizeof(c));
        c.features.opcode = nvme_admin_get_features;
@@ -488,12 +497,20 @@ int nvme_get_features(struct nvme_dev *dev, unsigned fid, unsigned nsid,
        c.features.prp1 = cpu_to_le64(dma_addr);
        c.features.fid = cpu_to_le32(fid);
 
+       ret = nvme_submit_admin_cmd(dev, &c, result);
+
        /*
-        * TODO: add cache invalidate operation when the size of
-        * the DMA buffer is known
+        * TODO: Add some cache invalidation when a DMA buffer is involved
+        * in the request, here and before the command gets submitted. The
+        * buffer size varies by feature, also some features use a different
+        * field in the command packet to hold the buffer address.
+        * Section 5.21.1 (Set Features command) in the NVMe specification
+        * details the buffer requirements for each feature.
+        *
+        * At the moment there is no user of this function.
         */
 
-       return nvme_submit_admin_cmd(dev, &c, result);
+       return ret;
 }
 
 int nvme_set_features(struct nvme_dev *dev, unsigned fid, unsigned dword11,
@@ -508,8 +525,14 @@ int nvme_set_features(struct nvme_dev *dev, unsigned fid, unsigned dword11,
        c.features.dword11 = cpu_to_le32(dword11);
 
        /*
-        * TODO: add cache flush operation when the size of
-        * the DMA buffer is known
+        * TODO: Add a cache clean (aka flush) operation when a DMA buffer is
+        * involved in the request. The buffer size varies by feature, also
+        * some features use a different field in the command packet to hold
+        * the buffer address. Section 5.21.1 (Set Features command) in the
+        * NVMe specification details the buffer requirements for each
+        * feature.
+        * At the moment the only user of this function is not using
+        * any DMA buffer at all.
         */
 
        return nvme_submit_admin_cmd(dev, &c, result);

diff --git a/include/api_public.h b/include/api_public.h
index def103c..5a4465e 100644 (file)
--- a/include/api_public.h
+++ b/include/api_public.h
@@ -70,12 +70,25 @@ struct sys_info {
        int                     mr_no;  /* number of memory regions */
 };
 
-#undef CONFIG_SYS_64BIT_LBA
-#ifdef CONFIG_SYS_64BIT_LBA
-typedef        u_int64_t lbasize_t;
-#else
+/*
+ * FIXME: Previously this code was:
+ *
+ *   #undef CONFIG_SYS_64BIT_LBA
+ *   #ifdef CONFIG_SYS_64BIT_LBA
+ *   typedef u_int64_t lbasize_t;
+ *   #else
+ *   typedef unsigned long lbasize_t;
+ *   #endif
+ *
+ * But we cannot just undefine CONFIG_SYS_64BIT_LBA, because then in
+ * api/api_storage.c the type signature of lbaint_t will be different if
+ * CONFIG_SYS_64BIT_LBA is enabled for the board, which can result in various
+ * bugs.
+ * So simply define lbasize_t as an unsigned long, since this was what was done
+ * anyway for at least 13 years, but don't undefine CONFIG_SYS_64BIT_LBA.
+ */
 typedef unsigned long lbasize_t;
-#endif
+
 typedef unsigned long lbastart_t;
 
 #define DEV_TYP_NONE   0x0000