From patchwork Thu Jul 9 11:17:09 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jagan Teki X-Patchwork-Id: 1473 Return-Path: X-Original-To: linux-amarula@patchwork.amarulasolutions.com Delivered-To: linux-amarula@patchwork.amarulasolutions.com Received: from mail-pf1-f197.google.com (mail-pf1-f197.google.com [209.85.210.197]) by ganimede.amarulasolutions.com (Postfix) with ESMTPS id 347BA3F06D for ; Thu, 9 Jul 2020 13:19:58 +0200 (CEST) Received: by mail-pf1-f197.google.com with SMTP id u128sf1097441pfu.12 for ; Thu, 09 Jul 2020 04:19:58 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1594293597; cv=pass; d=google.com; s=arc-20160816; b=kvdxxqndUTEswvKFtS9++A33EPoOZ8Xy+eVW2MfcI5918dwSb+dddU+mIsBVVoVpkb QLW87/lJdQXhX//YEWnxJdWBEu8AzWn0JeY4p3fXiMHAvVy8TC63qyKJ67wXYBvSVeDG WbGQzJJ+jOoiQlhcITJ7djZu1tVKC/O45ftiK2Q47pUVQ3xWk4ZYdw+Fufd+3HLZ8QYM 5TiNg02rxh26FIo9+eJKNb3KWoTxbvBH9DlZ7NjF+UDO2NfICknLY7iBry1RCw4+FOCN jofWQF/haQRyiS87bpMTmrnVfCfRbkJQEZDhx7fze3ZUXMl4mxj8gmGMkwTKQq0vXqFg Paaw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-unsubscribe:list-archive:list-help:list-post:list-id :mailing-list:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=Ox9cq7axFm0rqUJfxfqZ8QNj+07ujMWhyL2ndB9utJc=; b=XWd6pluARwuEwJ1UQM5zajHnqU9OFqyx3n54gUdrKNWpIBwkWxA3QV/s6pI7MY7cNS nn8YE1QA5coSD8uQQqluVx9q3qs7c2ENRi/BcgLPZqeBsIbLYKSiWzr2ZuOkb6cImdj+ qdu/o69wEYtdS3fq0t7T0vvHqIsuf7JnOQp1/zeyihlC9d3bsaShewA7RLUBk1h1QWlz Kd9V1LVjXXhHaDzPp7L9byz1+fY5ADo8uaLTotu0HbjgMyjy1mo9Ja0/iMYiT09SJoCX GqKUtYTPbvsbIC1V33RWIrH827mpWheIlTYHMHkiC7LqsCRBqo0rTT3lFQ+ke6o1/xMz 0rsw== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@amarulasolutions.com header.s=google header.b=CqjXrZ7v; spf=pass (google.com: domain of jagan@amarulasolutions.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=jagan@amarulasolutions.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amarulasolutions.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:x-original-sender :x-original-authentication-results:precedence:mailing-list:list-id :list-post:list-help:list-archive:list-unsubscribe; bh=Ox9cq7axFm0rqUJfxfqZ8QNj+07ujMWhyL2ndB9utJc=; b=ANEfkd3SGrM9naVHvUoj8LcyBIPOKbGmhc3Y9OeHIgctwf4wHPAhMpGQaEtXeSAAOi 5wIszg6mfMWkM9N4Ed+eA9cE7vgFmhvuLJTxGum/SPA7p/sBkWIb68CgsuL4TPGds7B1 QMVaEtLB/NOvW7+JyB8mXlzvgW/XUiKmoHqqo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding:x-original-sender :x-original-authentication-results:precedence:mailing-list:list-id :x-spam-checked-in-group:list-post:list-help:list-archive :list-unsubscribe; bh=Ox9cq7axFm0rqUJfxfqZ8QNj+07ujMWhyL2ndB9utJc=; b=ujWhW7emFV4rvWYJnms4dyY4MVmBVHgVK2QM8NQKDmxDLQZr+RgmY639lxZFAebQ8o xvzHHj+yHz1IjhurbVjQQBH7mFgwGqU4wPICP7oGvAdaBKQjF9IHi3Yetm976GkQg5B6 x3nfzkpChY90bYqQ/p6gvomu2ISj/GKMfKf2RVsir46IoO+9heBmp8WK/nMegJPYV0w/ nfM/DrzPN2FfIL3C9MHVHczosFRtfoSLaXPW7501VsqblDGKaMmRNbXtWEOMKH4criPp qEKxx5NHNsg7n3IaAJntCsJXMjV6zvzRDodXj8rGcX8rjbmxJbzexLlbmgfoBPz7dbsi 8NWA== X-Gm-Message-State: AOAM532vpRtQbsV/+V87QBj6cIC1IeBufLcAb1fXSoAe3+d4UzvX/3KW RU3a96yPNcJkochmjpT0xYZBuNJm X-Google-Smtp-Source: ABdhPJzh+xq5uPc+r53H+EFbR+rW2Sn0bb31IeK7KRhh+4schXjmhbXjTlJVD2ycqirwaoqiElA3qw== X-Received: by 2002:a17:90b:46c1:: with SMTP id jx1mr15106798pjb.157.1594293596913; Thu, 09 Jul 2020 04:19:56 -0700 (PDT) X-BeenThere: linux-amarula@amarulasolutions.com Received: by 2002:a17:90b:380c:: with SMTP id mq12ls2491100pjb.1.gmail; Thu, 09 Jul 2020 04:19:56 -0700 (PDT) X-Received: by 2002:a17:90a:ba86:: with SMTP id t6mr14882111pjr.10.1594293596207; Thu, 09 Jul 2020 04:19:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1594293596; cv=none; d=google.com; s=arc-20160816; b=sQ7Yn/sye4ikYnlgDKzZ1HQ7YOP5h1veCWzIzZcCDqIHqsS0alIee4gqWKJZonP9Ou b3+NUmx3cr6u5LStqyO9/i2dzzu5eSiDuMgymtR1LAf9ncQLbAv2Cdsv6homEbggcsCA 0rwZ+eezmbkesv5DMphRMHNNb1z619/24MjcVmLRUSuFw3ZZqPvaJi3eQFiUwvWnUfn9 LvFamu4aTPWAvT7mJCnAFA14BMJFAumxKhqt1MIIdiTNRc1WDnYwU98TmGthL37k8u1o aora/TmUldcVAsRQ7zMe+BrwT8t/VbeBl4jkX7PxKlcs2aN5NbdqhIbraVu4qHTf0GJv zqXA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature; bh=O8LKNIwpFRsyT+02guzQQAUak77g9WaWu/0yafShqa8=; b=goMlt/IRWNWzVJFQCpFWoTbHunZaLwzUl5Ocj1AKdY3WvMkr/K8227SBdqGAhHaesu l9idusRpCi1LHJ60Egz1ZbPI7u7mRDiRk8D+2qcCc2BKq3COJdXmQ9zp6rXQ9MWrBso7 eoGWsMRuU/tvdTIsapN9qUxfpHIaIrCwiLasnIVh+8W5b9yP/De+ZnTnZYdKXtOXECty hRKm4djOhtO/cRN3PTPnvq0bumIq+jqmcyQusGnWcf9Y+nJ6srnx1Sg7MaDDST7SZl1H oMOZs4/tpUg3tqzVQmBzSDd/+9k0E9aFqSQ8+d+opCzchQO9v0K4slRvfnFx1iP8Hidw Sc6Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@amarulasolutions.com header.s=google header.b=CqjXrZ7v; spf=pass (google.com: domain of jagan@amarulasolutions.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=jagan@amarulasolutions.com Received: from mail-sor-f65.google.com (mail-sor-f65.google.com. [209.85.220.65]) by mx.google.com with SMTPS id q16sor3572907pjp.33.2020.07.09.04.19.56 for (Google Transport Security); Thu, 09 Jul 2020 04:19:56 -0700 (PDT) Received-SPF: pass (google.com: domain of jagan@amarulasolutions.com designates 209.85.220.65 as permitted sender) client-ip=209.85.220.65; X-Received: by 2002:a17:90a:db8a:: with SMTP id h10mr14736137pjv.58.1594293595681; Thu, 09 Jul 2020 04:19:55 -0700 (PDT) Received: from localhost.localdomain ([2405:201:c809:c7d5:24b9:69b0:e917:1d11]) by smtp.gmail.com with ESMTPSA id o23sm2608130pfd.126.2020.07.09.04.19.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2020 04:19:54 -0700 (PDT) From: Jagan Teki To: u-boot@lists.denx.de Cc: Simon Glass , Vignesh R , Tom Rini , Peng Fan , Daniel Schwierzeck , linux-amarula@amarulasolutions.com, Jagan Teki Subject: [PATCH 5/5] mtd: spi: Switch to MTD uclass (absolute UCLASS_SPI_FLASH) Date: Thu, 9 Jul 2020 16:47:09 +0530 Message-Id: <20200709111709.68904-6-jagan@amarulasolutions.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20200709111709.68904-1-jagan@amarulasolutions.com> References: <20200709111709.68904-1-jagan@amarulasolutions.com> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" X-Original-Sender: jagan@amarulasolutions.com X-Original-Authentication-Results: mx.google.com; dkim=pass header.i=@amarulasolutions.com header.s=google header.b=CqjXrZ7v; spf=pass (google.com: domain of jagan@amarulasolutions.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=jagan@amarulasolutions.com Precedence: list Mailing-list: list linux-amarula@amarulasolutions.com; contact linux-amarula+owners@amarulasolutions.com List-ID: X-Spam-Checked-In-Group: linux-amarula@amarulasolutions.com X-Google-Group-Id: 476853432473 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , Why MTD UCLASS switch? From a nutshell, all the memory-related flash devices like Parallel NOR, Raw NAND, SPI NOR/NAND stacks should be part of the MTD layer. The same theory applies to incase of MTD UCLASS in U-Boot. Unfortunately, no flash drivers are using the MTD UCLASS in the fully functional driver model. There are several reasons behind it like - MTD code always copies/sync from Linux to support the latest   flash features. - So no special interest to touch since flash code is much bigger   than usual and a lot of features are tested with existing code. - No real taker to implement it. So, this patch is trying to move further to use a common flash interface layer called MTD in the driver model way. As initial support it supporting spinor flash device drivers would be written as UCLASS_MTD. What happened to UCLASS_SPI_FLASH, why it is absolute now? Due to the effect of Linux MTD/SPI_NOR sync the existing spi_flash the structure has been affected such a way that it was dead and typedefs to spi_nor. What happened with this patch then? With this patch, the UCLASS_SPI_FLASH will be replaced with UCLASS_MTD by using dm-driven mtd_ops. absolute link: sf.c => sf uclass => sf driver => spi-nor-core - flash operations via dm_spi_flash_ops updated link: sf.c => mtd uclass => mtd driver => spi-nor-core - flash operations via mtd_ops (dm variant) Future work? - cleanup of spi_flash with spinor - renaming of CONFIG option changes - implement common mtd probe - implement mtd flash interface types so that the respective   commands like sf, mtd can make use of dm-mtd in the proper way. Signed-off-by: Jagan Teki --- arch/arm/mach-rockchip/spl-boot-order.c | 4 +- arch/x86/cpu/apollolake/spl.c | 8 +- arch/x86/lib/fsp2/fsp_init.c | 2 +- arch/x86/lib/fsp2/fsp_support.c | 7 +- arch/x86/lib/mrccache.c | 8 +- board/atmel/common/mac-spi-nor.c | 4 +- drivers/mtd/spi/Kconfig | 2 + drivers/mtd/spi/sandbox.c | 2 +- drivers/mtd/spi/sf-uclass.c | 106 ++++++------------------ drivers/mtd/spi/sf_dataflash.c | 25 +++--- include/dm/uclass-id.h | 1 - include/spi_flash.h | 53 +----------- test/dm/sf.c | 12 +-- 13 files changed, 71 insertions(+), 163 deletions(-) diff --git a/arch/arm/mach-rockchip/spl-boot-order.c b/arch/arm/mach-rockchip/spl-boot-order.c index 94673f34c9..64a745d189 100644 --- a/arch/arm/mach-rockchip/spl-boot-order.c +++ b/arch/arm/mach-rockchip/spl-boot-order.c @@ -62,7 +62,7 @@ static int spl_node_to_boot_device(int node) default: return -ENOSYS; } - } else if (!uclass_get_device_by_of_offset(UCLASS_SPI_FLASH, node, + } else if (!uclass_get_device_by_of_offset(UCLASS_MTD, node, &parent)) { return BOOT_DEVICE_SPI; } @@ -73,7 +73,7 @@ static int spl_node_to_boot_device(int node) * extended with awareness of the BLK layer (and matching OF_CONTROL) * soon. */ - if (!uclass_get_device_by_of_offset(UCLASS_SPI_FLASH, node, &parent)) + if (!uclass_get_device_by_of_offset(UCLASS_MTD, node, &parent)) return BOOT_DEVICE_SPI; return -1; diff --git a/arch/x86/cpu/apollolake/spl.c b/arch/x86/cpu/apollolake/spl.c index e1ee1e0624..9c80440bbb 100644 --- a/arch/x86/cpu/apollolake/spl.c +++ b/arch/x86/cpu/apollolake/spl.c @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -41,7 +42,7 @@ static int rom_load_image(struct spl_image_info *spl_image, debug("Reading from mapped SPI %lx, size %lx", spl_pos, spl_size); if (CONFIG_IS_ENABLED(SPI_FLASH_SUPPORT)) { - ret = uclass_find_first_device(UCLASS_SPI_FLASH, &dev); + ret = uclass_find_first_device(UCLASS_MTD, &dev); if (ret) return log_msg_ret("spi_flash", ret); if (!dev) @@ -77,7 +78,7 @@ static int spl_fast_spi_load_image(struct spl_image_info *spl_image, struct udevice *dev; int ret; - ret = uclass_first_device_err(UCLASS_SPI_FLASH, &dev); + ret = uclass_first_device_err(UCLASS_MTD, &dev); if (ret) return ret; @@ -89,8 +90,7 @@ static int spl_fast_spi_load_image(struct spl_image_info *spl_image, spl_image->name = "U-Boot"; spl_pos &= ~0xff000000; debug("Reading from flash %lx, size %lx\n", spl_pos, spl_size); - ret = spi_flash_read_dm(dev, spl_pos, spl_size, - (void *)spl_image->load_addr); + ret = mtd_dread(dev, spl_pos, spl_size, (void *)spl_image->load_addr); cpu_flush_l1d_to_l2(); if (ret) return ret; diff --git a/arch/x86/lib/fsp2/fsp_init.c b/arch/x86/lib/fsp2/fsp_init.c index 85cae54a0c..61af45d239 100644 --- a/arch/x86/lib/fsp2/fsp_init.c +++ b/arch/x86/lib/fsp2/fsp_init.c @@ -124,7 +124,7 @@ int fsp_locate_fsp(enum fsp_type_t type, struct binman_entry *entry, struct udevice *sf; /* Just use the SPI driver to get the memory map */ - ret = uclass_find_first_device(UCLASS_SPI_FLASH, &sf); + ret = uclass_find_first_device(UCLASS_MTD, &sf); if (ret) return log_msg_ret("Cannot get SPI flash", ret); ret = dm_spi_get_mmap(sf, &map_base, &map_size, &offset); diff --git a/arch/x86/lib/fsp2/fsp_support.c b/arch/x86/lib/fsp2/fsp_support.c index 3f2ca840dc..434bfdbc0e 100644 --- a/arch/x86/lib/fsp2/fsp_support.c +++ b/arch/x86/lib/fsp2/fsp_support.c @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -37,10 +38,10 @@ int fsp_get_header(ulong offset, ulong size, bool use_spi_flash, */ debug("offset=%x buf=%x\n", (uint)offset, (uint)buf); if (use_spi_flash) { - ret = uclass_first_device_err(UCLASS_SPI_FLASH, &dev); + ret = uclass_first_device_err(UCLASS_MTD, &dev); if (ret) return log_msg_ret("Cannot find flash device", ret); - ret = spi_flash_read_dm(dev, offset, PROBE_BUF_SIZE, buf); + ret = mtd_dread(dev, offset, PROBE_BUF_SIZE, buf); if (ret) return log_msg_ret("Cannot read flash", ret); } else { @@ -88,7 +89,7 @@ int fsp_get_header(ulong offset, ulong size, bool use_spi_flash, debug("Image base %x\n", (uint)base); debug("Image addr %x\n", (uint)fsp->fsp_mem_init); if (use_spi_flash) { - ret = spi_flash_read_dm(dev, offset, size, base); + ret = mtd_dread(dev, offset, size, base); if (ret) return log_msg_ret("Could not read FPS-M", ret); } else { diff --git a/arch/x86/lib/mrccache.c b/arch/x86/lib/mrccache.c index f181e8100c..30eb5cf669 100644 --- a/arch/x86/lib/mrccache.c +++ b/arch/x86/lib/mrccache.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -167,7 +168,7 @@ static int mrccache_update(struct udevice *sf, struct mrc_region *entry, debug("Erasing the MRC cache region of %x bytes at %x\n", entry->length, entry->offset); - ret = spi_flash_erase_dm(sf, entry->offset, entry->length); + ret = mtd_derase(sf, entry->offset, entry->length); if (ret) { debug("Failed to erase flash region\n"); return ret; @@ -178,8 +179,7 @@ static int mrccache_update(struct udevice *sf, struct mrc_region *entry, /* Write the data out */ offset = (ulong)cache - base_addr + entry->offset; debug("Write MRC cache update to flash at %lx\n", offset); - ret = spi_flash_write_dm(sf, offset, cur->data_size + sizeof(*cur), - cur); + ret = mtd_dwrite(sf, offset, cur->data_size + sizeof(*cur), cur); if (ret) { debug("Failed to write to SPI flash\n"); return log_msg_ret("Cannot update mrccache", ret); @@ -242,7 +242,7 @@ int mrccache_get_region(enum mrc_type_t type, struct udevice **devp, * the device here since it may put it into a strange state where the * memory map cannot be read. */ - ret = uclass_find_first_device(UCLASS_SPI_FLASH, &dev); + ret = uclass_find_first_device(UCLASS_MTD, &dev); if (ret || !dev) { /* * Fall back to searching the device tree since driver model diff --git a/board/atmel/common/mac-spi-nor.c b/board/atmel/common/mac-spi-nor.c index ced27b65e6..0b3f1b1340 100644 --- a/board/atmel/common/mac-spi-nor.c +++ b/board/atmel/common/mac-spi-nor.c @@ -107,10 +107,10 @@ void at91_spi_nor_set_ethaddr(void) if (env_get(ethaddr_name)) return; - if (uclass_first_device_err(UCLASS_SPI_FLASH, &dev)) + if (uclass_first_device_err(UCLASS_MTD, &dev)) return; - nor = dev_get_uclass_priv(dev); + nor = dev_get_priv(dev); if (!nor) return; diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig index 018e8c597e..9de0484b29 100644 --- a/drivers/mtd/spi/Kconfig +++ b/drivers/mtd/spi/Kconfig @@ -3,6 +3,8 @@ menu "SPI Flash Support" config DM_SPI_FLASH bool "Enable Driver Model for SPI flash" depends on DM && DM_SPI + select DM_MTD + imply SPL_DM_MTD imply SPI_FLASH help Enable driver model for SPI flash. This SPI flash interface diff --git a/drivers/mtd/spi/sandbox.c b/drivers/mtd/spi/sandbox.c index 8cbe97ee20..0b04f14b16 100644 --- a/drivers/mtd/spi/sandbox.c +++ b/drivers/mtd/spi/sandbox.c @@ -8,7 +8,7 @@ * Licensed under the GPL-2 or later. */ -#define LOG_CATEGORY UCLASS_SPI_FLASH +#define LOG_CATEGORY UCLASS_MTD #include #include diff --git a/drivers/mtd/spi/sf-uclass.c b/drivers/mtd/spi/sf-uclass.c index 44cdb3151d..9cab08f49b 100644 --- a/drivers/mtd/spi/sf-uclass.c +++ b/drivers/mtd/spi/sf-uclass.c @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -14,22 +15,6 @@ DECLARE_GLOBAL_DATA_PTR; -int spi_flash_read_dm(struct udevice *dev, u32 offset, size_t len, void *buf) -{ - return log_ret(sf_get_ops(dev)->read(dev, offset, len, buf)); -} - -int spi_flash_write_dm(struct udevice *dev, u32 offset, size_t len, - const void *buf) -{ - return log_ret(sf_get_ops(dev)->write(dev, offset, len, buf)); -} - -int spi_flash_erase_dm(struct udevice *dev, u32 offset, size_t len) -{ - return log_ret(sf_get_ops(dev)->erase(dev, offset, len)); -} - void spi_flash_free(struct spi_flash *flash) { device_remove(flash->spi->dev, DM_REMOVE_NORMAL); @@ -57,108 +42,78 @@ struct spi_flash *spi_flash_probe(unsigned int bus, unsigned int cs, str = strdup(name); #endif ret = spi_get_bus_and_cs(bus, cs, max_hz, spi_mode, - "spi_flash_std", str, &bus_dev, &slave); + "spi_flash_std", str, &bus_dev, &slave); if (ret) return NULL; - return dev_get_uclass_priv(slave->dev); + return dev_get_priv(slave->dev); } -static int spi_flash_post_bind(struct udevice *dev) +static int spi_flash_std_read(struct udevice *dev, loff_t from, size_t len, + u_char *buf) { -#if defined(CONFIG_NEEDS_MANUAL_RELOC) - struct dm_spi_flash_ops *ops = sf_get_ops(dev); - static int reloc_done; - - if (!reloc_done) { - if (ops->read) - ops->read += gd->reloc_off; - if (ops->write) - ops->write += gd->reloc_off; - if (ops->erase) - ops->erase += gd->reloc_off; - - reloc_done++; - } -#endif - return 0; -} - -static int spi_flash_std_read(struct udevice *dev, u32 offset, size_t len, - void *buf) -{ - struct spi_flash *flash = dev_get_uclass_priv(dev); + struct spi_flash *flash = dev_get_priv(dev); struct mtd_info *mtd = &flash->mtd; size_t retlen; - return log_ret(mtd->_read(mtd, offset, len, &retlen, buf)); + return log_ret(mtd->_read(mtd, from, len, &retlen, buf)); } -static int spi_flash_std_write(struct udevice *dev, u32 offset, size_t len, - const void *buf) +static int spi_flash_std_write(struct udevice *dev, loff_t to, size_t len, + const u_char *buf) { - struct spi_flash *flash = dev_get_uclass_priv(dev); + struct spi_flash *flash = dev_get_priv(dev); struct mtd_info *mtd = &flash->mtd; size_t retlen; - return mtd->_write(mtd, offset, len, &retlen, buf); + return mtd->_write(mtd, to, len, &retlen, buf); } -static int spi_flash_std_erase(struct udevice *dev, u32 offset, size_t len) +static int spi_flash_std_erase(struct udevice *dev, struct erase_info *instr) { - struct spi_flash *flash = dev_get_uclass_priv(dev); + struct spi_flash *flash = dev_get_priv(dev); struct mtd_info *mtd = &flash->mtd; - struct erase_info instr; - if (offset % mtd->erasesize || len % mtd->erasesize) { - printf("SF: Erase offset/length not multiple of erase size\n"); + if (instr->addr % mtd->erasesize || instr->len % mtd->erasesize) { + dev_err(dev, "Erase offset/length not multiple of erasesize\n"); return -EINVAL; } - memset(&instr, 0, sizeof(instr)); - instr.addr = offset; - instr.len = len; - - return mtd->_erase(mtd, &instr); + return mtd->_erase(mtd, instr); } static int spi_flash_std_probe(struct udevice *dev) { + struct spi_flash *flash = dev_get_priv(dev); struct spi_slave *slave = dev_get_parent_priv(dev); - struct spi_flash *flash; int ret; - if (!slave) { - printf("SF: Failed to set up slave\n"); - return -ENODEV; - } - - flash = dev_get_uclass_priv(dev); flash->dev = dev; flash->spi = slave; - /* Claim spi bus */ ret = spi_claim_bus(slave); if (ret) { - debug("SF: Failed to claim SPI bus: %d\n", ret); + dev_err(dev, "failed to claim bus (ret=%d)\n", ret); return ret; } ret = spi_nor_scan(flash); - if (ret) - goto err_read_id; + if (ret) { + dev_err(dev, "failed to scan spinor (ret=%d)\n", ret); + goto err_claim_bus; + } - if (CONFIG_IS_ENABLED(SPI_FLASH_MTD)) + if (IS_ENABLED(CONFIG_SPI_FLASH_MTD)) ret = spi_flash_mtd_register(flash); -err_read_id: +err_claim_bus: spi_release_bus(slave); return ret; } static int spi_flash_std_remove(struct udevice *dev) { - if (CONFIG_IS_ENABLED(SPI_FLASH_MTD)) + if (IS_ENABLED(CONFIG_SPI_FLASH_MTD)) spi_flash_mtd_unregister(); return 0; @@ -190,7 +145,7 @@ static int spi_flash_std_bind(struct udevice *dev) return 0; } -static const struct dm_spi_flash_ops spi_flash_std_ops = { +static const struct mtd_ops spi_flash_std_ops = { .read = spi_flash_std_read, .write = spi_flash_std_write, .erase = spi_flash_std_erase, @@ -203,7 +158,7 @@ static const struct udevice_id spi_flash_std_ids[] = { U_BOOT_DRIVER(spi_flash_std) = { .name = "spi_flash_std", - .id = UCLASS_SPI_FLASH, + .id = UCLASS_MTD, .of_match = spi_flash_std_ids, .probe = spi_flash_std_probe, .bind = spi_flash_std_bind, @@ -211,10 +166,3 @@ U_BOOT_DRIVER(spi_flash_std) = { .priv_auto_alloc_size = sizeof(struct spi_flash), .ops = &spi_flash_std_ops, }; - -UCLASS_DRIVER(spi_flash) = { - .id = UCLASS_SPI_FLASH, - .name = "spi_flash", - .post_bind = spi_flash_post_bind, - .per_device_auto_alloc_size = sizeof(struct spi_flash), -}; diff --git a/drivers/mtd/spi/sf_dataflash.c b/drivers/mtd/spi/sf_dataflash.c index 27d847d421..5c73b54cbc 100644 --- a/drivers/mtd/spi/sf_dataflash.c +++ b/drivers/mtd/spi/sf_dataflash.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -117,7 +118,7 @@ static int dataflash_waitready(struct spi_slave *spi) } /* Erase pages of flash */ -static int spi_dataflash_erase(struct udevice *dev, u32 offset, size_t len) +static int spi_dataflash_erase(struct udevice *dev, struct erase_info *instr) { struct dataflash *dataflash; struct spi_flash *spi_flash; @@ -125,6 +126,8 @@ static int spi_dataflash_erase(struct udevice *dev, u32 offset, size_t len) unsigned blocksize; uint8_t *command; uint32_t rem; + loff_t offset = instr->addr; + size_t len = instr->len; int status; dataflash = dev_get_priv(dev); @@ -136,7 +139,7 @@ static int spi_dataflash_erase(struct udevice *dev, u32 offset, size_t len) memset(dataflash->command, 0 , sizeof(dataflash->command)); command = dataflash->command; - debug("%s: erase addr=0x%x len 0x%x\n", dev->name, offset, len); + debug("%s: erase addr=0x%llx len 0x%x\n", dev->name, offset, len); div_u64_rem(len, spi_flash->page_size, &rem); if (rem) { @@ -146,7 +149,7 @@ static int spi_dataflash_erase(struct udevice *dev, u32 offset, size_t len) } div_u64_rem(offset, spi_flash->page_size, &rem); if (rem) { - printf("%s: offset(0x%x) isn't the multiple of page size(0x%x)\n", + printf("%s: offset(0x%llx) isn't the multiple of page size(0x%x)\n", dev->name, offset, spi_flash->page_size); return -EINVAL; } @@ -210,8 +213,8 @@ static int spi_dataflash_erase(struct udevice *dev, u32 offset, size_t len) * len : Amount to read * buf : Buffer containing the data */ -static int spi_dataflash_read(struct udevice *dev, u32 offset, size_t len, - void *buf) +static int spi_dataflash_read(struct udevice *dev, loff_t offset, size_t len, + u_char *buf) { struct dataflash *dataflash; struct spi_flash *spi_flash; @@ -227,7 +230,7 @@ static int spi_dataflash_read(struct udevice *dev, u32 offset, size_t len, memset(dataflash->command, 0 , sizeof(dataflash->command)); command = dataflash->command; - debug("%s: erase addr=0x%x len 0x%x\n", dev->name, offset, len); + debug("%s: erase addr=0x%llx len 0x%x\n", dev->name, offset, len); debug("READ: (%x) %x %x %x\n", command[0], command[1], command[2], command[3]); @@ -266,8 +269,8 @@ static int spi_dataflash_read(struct udevice *dev, u32 offset, size_t len, * len : Amount to write * buf : Buffer containing the data */ -int spi_dataflash_write(struct udevice *dev, u32 offset, size_t len, - const void *buf) +int spi_dataflash_write(struct udevice *dev, loff_t offset, size_t len, + const u_char *buf) { struct dataflash *dataflash; struct spi_flash *spi_flash; @@ -285,7 +288,7 @@ int spi_dataflash_write(struct udevice *dev, u32 offset, size_t len, memset(dataflash->command, 0 , sizeof(dataflash->command)); command = dataflash->command; - debug("%s: write 0x%x..0x%x\n", dev->name, offset, (offset + len)); + debug("%s: write 0x%llx..0x%llx\n", dev->name, offset, (offset + len)); pageaddr = ((unsigned)offset / spi_flash->page_size); to = ((unsigned)offset % spi_flash->page_size); @@ -676,7 +679,7 @@ err_jedec_probe: return status; } -static const struct dm_spi_flash_ops spi_dataflash_ops = { +static const struct mtd_ops spi_dataflash_ops = { .read = spi_dataflash_read, .write = spi_dataflash_write, .erase = spi_dataflash_erase, @@ -690,7 +693,7 @@ static const struct udevice_id spi_dataflash_ids[] = { U_BOOT_DRIVER(spi_dataflash) = { .name = "spi_dataflash", - .id = UCLASS_SPI_FLASH, + .id = UCLASS_MTD, .of_match = spi_dataflash_ids, .probe = spi_dataflash_probe, .priv_auto_alloc_size = sizeof(struct dataflash), diff --git a/include/dm/uclass-id.h b/include/dm/uclass-id.h index 7837d459f1..1f56f7f48a 100644 --- a/include/dm/uclass-id.h +++ b/include/dm/uclass-id.h @@ -99,7 +99,6 @@ enum uclass_id { UCLASS_SMEM, /* Shared memory interface */ UCLASS_SOUND, /* Playing simple sounds */ UCLASS_SPI, /* SPI bus */ - UCLASS_SPI_FLASH, /* SPI flash */ UCLASS_SPI_GENERIC, /* Generic SPI flash target */ UCLASS_SPMI, /* System Power Management Interface bus */ UCLASS_SYSCON, /* System configuration device */ diff --git a/include/spi_flash.h b/include/spi_flash.h index 24759944eb..f4852fc5c1 100644 --- a/include/spi_flash.h +++ b/include/spi_flash.h @@ -10,6 +10,7 @@ #define _SPI_FLASH_H_ #include /* Because we dereference struct udevice here */ +#include #include #include @@ -27,70 +28,24 @@ # define CONFIG_ENV_SPI_MODE CONFIG_SF_DEFAULT_MODE #endif -struct spi_slave; - -struct dm_spi_flash_ops { - int (*read)(struct udevice *dev, u32 offset, size_t len, void *buf); - int (*write)(struct udevice *dev, u32 offset, size_t len, - const void *buf); - int (*erase)(struct udevice *dev, u32 offset, size_t len); -}; - -/* Access the serial operations for a device */ -#define sf_get_ops(dev) ((struct dm_spi_flash_ops *)(dev)->driver->ops) - #if CONFIG_IS_ENABLED(DM_SPI_FLASH) -/** - * spi_flash_read_dm() - Read data from SPI flash - * - * @dev: SPI flash device - * @offset: Offset into device in bytes to read from - * @len: Number of bytes to read - * @buf: Buffer to put the data that is read - * @return 0 if OK, -ve on error - */ -int spi_flash_read_dm(struct udevice *dev, u32 offset, size_t len, void *buf); - -/** - * spi_flash_write_dm() - Write data to SPI flash - * - * @dev: SPI flash device - * @offset: Offset into device in bytes to write to - * @len: Number of bytes to write - * @buf: Buffer containing bytes to write - * @return 0 if OK, -ve on error - */ -int spi_flash_write_dm(struct udevice *dev, u32 offset, size_t len, - const void *buf); - -/** - * spi_flash_erase_dm() - Erase blocks of the SPI flash - * - * Note that @len must be a muiltiple of the flash sector size. - * - * @dev: SPI flash device - * @offset: Offset into device in bytes to start erasing - * @len: Number of bytes to erase - * @return 0 if OK, -ve on error - */ -int spi_flash_erase_dm(struct udevice *dev, u32 offset, size_t len); static inline int spi_flash_read(struct spi_flash *flash, u32 offset, size_t len, void *buf) { - return spi_flash_read_dm(flash->dev, offset, len, buf); + return mtd_dread(flash->dev, offset, len, buf); } static inline int spi_flash_write(struct spi_flash *flash, u32 offset, size_t len, const void *buf) { - return spi_flash_write_dm(flash->dev, offset, len, buf); + return mtd_dwrite(flash->dev, offset, len, buf); } static inline int spi_flash_erase(struct spi_flash *flash, u32 offset, size_t len) { - return spi_flash_erase_dm(flash->dev, offset, len); + return mtd_derase(flash->dev, offset, len); } struct sandbox_state; diff --git a/test/dm/sf.c b/test/dm/sf.c index 9e7dead684..a675923dc5 100644 --- a/test/dm/sf.c +++ b/test/dm/sf.c @@ -31,23 +31,23 @@ static int dm_test_spi_flash(struct unit_test_state *uts) src = map_sysmem(0x20000, full_size); ut_assertok(os_write_file("spi.bin", src, full_size)); - ut_assertok(uclass_first_device_err(UCLASS_SPI_FLASH, &dev)); + ut_assertok(uclass_first_device_err(UCLASS_MTD, &dev)); dst = map_sysmem(0x20000 + full_size, full_size); - ut_assertok(spi_flash_read_dm(dev, 0, size, dst)); + ut_assertok(mtd_dread(dev, 0, size, dst)); ut_asserteq_mem(src, dst, size); /* Erase */ - ut_assertok(spi_flash_erase_dm(dev, 0, size)); - ut_assertok(spi_flash_read_dm(dev, 0, size, dst)); + ut_assertok(mtd_derase(dev, 0, size)); + ut_assertok(mtd_dread(dev, 0, size, dst)); for (i = 0; i < size; i++) ut_asserteq(dst[i], 0xff); /* Write some new data */ for (i = 0; i < size; i++) src[i] = i; - ut_assertok(spi_flash_write_dm(dev, 0, size, src)); - ut_assertok(spi_flash_read_dm(dev, 0, size, dst)); + ut_assertok(mtd_dwrite(dev, 0, size, src)); + ut_assertok(mtd_dread(dev, 0, size, dst)); ut_asserteq_mem(src, dst, size); /* Check mapping */