From patchwork Mon Feb 12 14:51:33 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom Rini X-Patchwork-Id: 872100 X-Patchwork-Delegate: trini@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=lists.denx.de (client-ip=81.169.180.215; helo=lists.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 3zg7sw1NFqz9t2R for ; Tue, 13 Feb 2018 01:51:48 +1100 (AEDT) Received: by lists.denx.de (Postfix, from userid 105) id CC6B6C21EC8; Mon, 12 Feb 2018 14:51:44 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=-0.0 required=5.0 tests=RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id 4D22EC21D65; Mon, 12 Feb 2018 14:51:40 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 7B0FCC21D65; Mon, 12 Feb 2018 14:51:39 +0000 (UTC) Received: from mail-yw0-f195.google.com (mail-yw0-f195.google.com [209.85.161.195]) by lists.denx.de (Postfix) with ESMTPS id C1A17C21C51 for ; Mon, 12 Feb 2018 14:51:38 +0000 (UTC) Received: by mail-yw0-f195.google.com with SMTP id l11so6848380ywh.2 for ; Mon, 12 Feb 2018 06:51:38 -0800 (PST) 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; bh=ItYuQ4LOxWCPirAUXju4zSRPcZj+j53MlBNHkSNOt7k=; b=GhTjZQRYzMnW+ceirGEznK2kPSY6g+KSaIENYw0mMoWlk74IzPdU5UCKX+xKXrRnX7 osSuWYLZ242dMFWhAaXRB4rtG5JRddUFyC1QMrNjZzDVN5CkWrs0PsI+YPFVbc06fN21 hg/s5YGNLBl9CZn84vkKI1iW2PH7xYGAnmXBqRjGecqnSkeQ4Ygh/cUgB0SCqYK2GqyW HwvvtDUkVielakXSI73G5C2ug6f5e+3XBt1bgqrKW+ZCvQMS/kJmLjXUU3u9VTl1AAHy NXYhFzCWJdDFaaQDAq2eHPmPPxvnqBXWgxO4ZnzXh7JCLKthJEqMEJlghYgdRgfzGQzB 6W2g== X-Gm-Message-State: APf1xPAS+O1jXVXwE48JPdcfxxpU1MX+/NGiRh6jSRSlkEp3o7FhJQQ6 btisXGv+rOLZbARpu+rB19Ni X-Google-Smtp-Source: AH8x225Z92seQm5QAFU2jf7eIbGEOvvtn6+mQgE+/OOOdUeQIWzFuNooneA6Y5Bz4icpwO2S5y2Oqw== X-Received: by 10.129.92.66 with SMTP id q63mr7763818ywb.138.1518447097136; Mon, 12 Feb 2018 06:51:37 -0800 (PST) Received: from localhost.localdomain (cpe-65-184-142-68.ec.res.rr.com. [65.184.142.68]) by smtp.gmail.com with ESMTPSA id q16sm3333152ywa.103.2018.02.12.06.51.36 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 12 Feb 2018 06:51:36 -0800 (PST) From: Tom Rini To: u-boot@lists.denx.de Date: Mon, 12 Feb 2018 09:51:33 -0500 Message-Id: <1518447093-7154-1-git-send-email-trini@konsulko.com> X-Mailer: git-send-email 2.7.4 Cc: Michal Simek Subject: [U-Boot] [PATCH] SystemACE: Remove X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" This driver is no longer used on any supported platform in U-Boot and there is no interest in maintaining it further from people that have used it historically. Cc: Simon Glass Cc: Michal Simek c: Alexey Brodkin Signed-off-by: Tom Rini Acked-by: Michal Simek --- arch/sandbox/include/asm/io.h | 4 - configs/sandbox_defconfig | 3 - drivers/block/Kconfig | 19 --- drivers/block/Makefile | 1 - drivers/block/blk-uclass.c | 2 - drivers/block/systemace.c | 303 ------------------------------------------ include/blk.h | 1 - 7 files changed, 333 deletions(-) delete mode 100644 drivers/block/systemace.c diff --git a/arch/sandbox/include/asm/io.h b/arch/sandbox/include/asm/io.h index 59729f5635e8..d3e8b969d63c 100644 --- a/arch/sandbox/include/asm/io.h +++ b/arch/sandbox/include/asm/io.h @@ -171,10 +171,6 @@ static inline void _outsw(volatile u16 *port, const void *buf, int ns) #define insw(port, buf, ns) _insw((u16 *)port, buf, ns) #define outsw(port, buf, ns) _outsw((u16 *)port, buf, ns) -/* For systemace.c */ -#define out16(addr, val) -#define in16(addr) 0 - #include #include diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig index c0f4c0a89f12..063828333d58 100644 --- a/configs/sandbox_defconfig +++ b/configs/sandbox_defconfig @@ -81,9 +81,6 @@ CONFIG_DEVRES=y CONFIG_DEBUG_DEVRES=y CONFIG_ADC=y CONFIG_ADC_SANDBOX=y -CONFIG_SYSTEMACE=y -CONFIG_SYS_SYSTEMACE_BASE=0x0 -CONFIG_SYS_SYSTEMACE_WIDTH=16 CONFIG_CLK=y CONFIG_CPU=y CONFIG_DM_DEMO=y diff --git a/drivers/block/Kconfig b/drivers/block/Kconfig index 73836ada094b..15fd1bcb2b7e 100644 --- a/drivers/block/Kconfig +++ b/drivers/block/Kconfig @@ -44,22 +44,3 @@ config IDE This allows access to raw blocks and filesystems on an IDE drive from U-Boot. See also CMD_IDE which provides an 'ide' command for performing various IDE operations. - -config SYSTEMACE - bool "Xilinx SystemACE support" - select HAVE_BLOCK_DEVICE - help - Adding this option adds support for Xilinx SystemACE chips attached - via some sort of local bus. The address of the chip must also be - defined in the CONFIG_SYS_SYSTEMACE_BASE macro. - - When SystemACE support is added, the "ace" device type becomes - available to the fat commands, i.e. fatls. - -config SYS_SYSTEMACE_BASE - hex "Base address of SystemACE chip" - depends on SYSTEMACE - -config SYS_SYSTEMACE_WIDTH - int "Word size of access to the of SystemACE chip" - depends on SYSTEMACE diff --git a/drivers/block/Makefile b/drivers/block/Makefile index d06a598f1409..3d52bd8f4639 100644 --- a/drivers/block/Makefile +++ b/drivers/block/Makefile @@ -13,5 +13,4 @@ endif obj-$(CONFIG_IDE) += ide.o obj-$(CONFIG_SANDBOX) += sandbox.o -obj-$(CONFIG_SYSTEMACE) += systemace.o obj-$(CONFIG_BLOCK_CACHE) += blkcache.o diff --git a/drivers/block/blk-uclass.c b/drivers/block/blk-uclass.c index bfda2211f0e8..c32aee67605a 100644 --- a/drivers/block/blk-uclass.c +++ b/drivers/block/blk-uclass.c @@ -22,7 +22,6 @@ static const char *if_typename_str[IF_TYPE_COUNT] = { [IF_TYPE_SD] = "sd", [IF_TYPE_SATA] = "sata", [IF_TYPE_HOST] = "host", - [IF_TYPE_SYSTEMACE] = "ace", [IF_TYPE_NVME] = "nvme", [IF_TYPE_EFI] = "efi", }; @@ -37,7 +36,6 @@ static enum uclass_id if_type_uclass_id[IF_TYPE_COUNT] = { [IF_TYPE_SD] = UCLASS_INVALID, [IF_TYPE_SATA] = UCLASS_AHCI, [IF_TYPE_HOST] = UCLASS_ROOT, - [IF_TYPE_SYSTEMACE] = UCLASS_INVALID, [IF_TYPE_NVME] = UCLASS_NVME, [IF_TYPE_EFI] = UCLASS_EFI, }; diff --git a/drivers/block/systemace.c b/drivers/block/systemace.c deleted file mode 100644 index 9392beaf052e..000000000000 --- a/drivers/block/systemace.c +++ /dev/null @@ -1,303 +0,0 @@ -/* - * Copyright (c) 2004 Picture Elements, Inc. - * Stephen Williams (XXXXXXXXXXXXXXXX) - * - * SPDX-License-Identifier: GPL-2.0+ - */ - -/* - * The Xilinx SystemACE chip support is activated by defining - * CONFIG_SYSTEMACE to turn on support, and CONFIG_SYS_SYSTEMACE_BASE - * to set the base address of the device. This code currently - * assumes that the chip is connected via a byte-wide bus. - * - * The CONFIG_SYSTEMACE also adds to fat support the device class - * "ace" that allows the user to execute "fatls ace 0" and the - * like. This works by making the systemace_get_dev function - * available to cmd_fat.c:get_dev and filling in a block device - * description that has all the bits needed for FAT support to - * read sectors. - * - * According to Xilinx technical support, before accessing the - * SystemACE CF you need to set the following control bits: - * FORCECFGMODE : 1 - * CFGMODE : 0 - * CFGSTART : 0 - */ - -#include -#include -#include -#include -#include - -/* - * The ace_readw and writew functions read/write 16bit words, but the - * offset value is the BYTE offset as most used in the Xilinx - * datasheet for the SystemACE chip. The CONFIG_SYS_SYSTEMACE_BASE is defined - * to be the base address for the chip, usually in the local - * peripheral bus. - */ - -static u32 base = CONFIG_SYS_SYSTEMACE_BASE; -static u32 width = CONFIG_SYS_SYSTEMACE_WIDTH; - -static void ace_writew(u16 val, unsigned off) -{ - if (width == 8) { -#if !defined(__BIG_ENDIAN) - writeb(val >> 8, base + off); - writeb(val, base + off + 1); -#else - writeb(val, base + off); - writeb(val >> 8, base + off + 1); -#endif - } else - out16(base + off, val); -} - -static u16 ace_readw(unsigned off) -{ - if (width == 8) { -#if !defined(__BIG_ENDIAN) - return (readb(base + off) << 8) | readb(base + off + 1); -#else - return readb(base + off) | (readb(base + off + 1) << 8); -#endif - } - - return in16(base + off); -} - -#ifndef CONFIG_BLK -static struct blk_desc systemace_dev = { 0 }; -#endif - -static int get_cf_lock(void) -{ - int retry = 10; - - /* CONTROLREG = LOCKREG */ - unsigned val = ace_readw(0x18); - val |= 0x0002; - ace_writew((val & 0xffff), 0x18); - - /* Wait for MPULOCK in STATUSREG[15:0] */ - while (!(ace_readw(0x04) & 0x0002)) { - - if (retry < 0) - return -1; - - udelay(100000); - retry -= 1; - } - - return 0; -} - -static void release_cf_lock(void) -{ - unsigned val = ace_readw(0x18); - val &= ~(0x0002); - ace_writew((val & 0xffff), 0x18); -} - -/* - * This function is called (by dereferencing the block_read pointer in - * the dev_desc) to read blocks of data. The return value is the - * number of blocks read. A zero return indicates an error. - */ -#ifdef CONFIG_BLK -static unsigned long systemace_read(struct udevice *dev, unsigned long start, - lbaint_t blkcnt, void *buffer) -#else -static unsigned long systemace_read(struct blk_desc *block_dev, - unsigned long start, lbaint_t blkcnt, - void *buffer) -#endif -{ - int retry; - unsigned blk_countdown; - unsigned char *dp = buffer; - unsigned val; - - if (get_cf_lock() < 0) { - unsigned status = ace_readw(0x04); - - /* If CFDETECT is false, card is missing. */ - if (!(status & 0x0010)) { - printf("** CompactFlash card not present. **\n"); - return 0; - } - - printf("**** ACE locked away from me (STATUSREG=%04x)\n", - status); - return 0; - } -#ifdef DEBUG_SYSTEMACE - printf("... systemace read %lu sectors at %lu\n", blkcnt, start); -#endif - - retry = 2000; - for (;;) { - val = ace_readw(0x04); - - /* If CFDETECT is false, card is missing. */ - if (!(val & 0x0010)) { - printf("**** ACE CompactFlash not found.\n"); - release_cf_lock(); - return 0; - } - - /* If RDYFORCMD, then we are ready to go. */ - if (val & 0x0100) - break; - - if (retry < 0) { - printf("**** SystemACE not ready.\n"); - release_cf_lock(); - return 0; - } - - udelay(1000); - retry -= 1; - } - - /* The SystemACE can only transfer 256 sectors at a time, so - limit the current chunk of sectors. The blk_countdown - variable is the number of sectors left to transfer. */ - - blk_countdown = blkcnt; - while (blk_countdown > 0) { - unsigned trans = blk_countdown; - - if (trans > 256) - trans = 256; - -#ifdef DEBUG_SYSTEMACE - printf("... transfer %lu sector in a chunk\n", trans); -#endif - /* Write LBA block address */ - ace_writew((start >> 0) & 0xffff, 0x10); - ace_writew((start >> 16) & 0x0fff, 0x12); - - /* NOTE: in the Write Sector count below, a count of 0 - causes a transfer of 256, so &0xff gives the right - value for whatever transfer count we want. */ - - /* Write sector count | ReadMemCardData. */ - ace_writew((trans & 0xff) | 0x0300, 0x14); - -/* - * For FPGA configuration via SystemACE is reset unacceptable - * CFGDONE bit in STATUSREG is not set to 1. - */ -#ifndef SYSTEMACE_CONFIG_FPGA - /* Reset the configruation controller */ - val = ace_readw(0x18); - val |= 0x0080; - ace_writew(val, 0x18); -#endif - - retry = trans * 16; - while (retry > 0) { - int idx; - - /* Wait for buffer to become ready. */ - while (!(ace_readw(0x04) & 0x0020)) { - udelay(100); - } - - /* Read 16 words of 2bytes from the sector buffer. */ - for (idx = 0; idx < 16; idx += 1) { - unsigned short val = ace_readw(0x40); - *dp++ = val & 0xff; - *dp++ = (val >> 8) & 0xff; - } - - retry -= 1; - } - - /* Clear the configruation controller reset */ - val = ace_readw(0x18); - val &= ~0x0080; - ace_writew(val, 0x18); - - /* Count the blocks we transfer this time. */ - start += trans; - blk_countdown -= trans; - } - - release_cf_lock(); - - return blkcnt; -} - -#ifdef CONFIG_BLK -static int systemace_bind(struct udevice *dev) -{ - struct blk_desc *bdesc; - struct udevice *bdev; - int ret; - - ret = blk_create_devicef(dev, "systemace_blk", "blk", IF_TYPE_SYSTEMACE, - -1, 512, 0, &bdev); - if (ret) { - debug("Cannot create block device\n"); - return ret; - } - bdesc = dev_get_uclass_platdata(bdev); - bdesc->removable = 1; - bdesc->part_type = PART_TYPE_UNKNOWN; - bdesc->log2blksz = LOG2(bdesc->blksz); - - /* Ensure the correct bus mode (8/16 bits) gets enabled */ - ace_writew(width == 8 ? 0 : 0x0001, 0); - - return 0; -} - -static const struct blk_ops systemace_blk_ops = { - .read = systemace_read, -}; - -U_BOOT_DRIVER(systemace_blk) = { - .name = "systemace_blk", - .id = UCLASS_BLK, - .ops = &systemace_blk_ops, - .bind = systemace_bind, -}; -#else -static int systemace_get_dev(int dev, struct blk_desc **descp) -{ - /* The first time through this, the systemace_dev object is - not yet initialized. In that case, fill it in. */ - if (systemace_dev.blksz == 0) { - systemace_dev.if_type = IF_TYPE_UNKNOWN; - systemace_dev.devnum = 0; - systemace_dev.part_type = PART_TYPE_UNKNOWN; - systemace_dev.type = DEV_TYPE_HARDDISK; - systemace_dev.blksz = 512; - systemace_dev.log2blksz = LOG2(systemace_dev.blksz); - systemace_dev.removable = 1; - systemace_dev.block_read = systemace_read; - - /* - * Ensure the correct bus mode (8/16 bits) gets enabled - */ - ace_writew(width == 8 ? 0 : 0x0001, 0); - - part_init(&systemace_dev); - } - *descp = &systemace_dev; - - return 0; -} - -U_BOOT_LEGACY_BLK(systemace) = { - .if_typename = "ace", - .if_type = IF_TYPE_SYSTEMACE, - .max_devs = 1, - .get_dev = systemace_get_dev, -}; -#endif diff --git a/include/blk.h b/include/blk.h index 69b5a98e5673..636eed9de912 100644 --- a/include/blk.h +++ b/include/blk.h @@ -32,7 +32,6 @@ enum if_type { IF_TYPE_SD, IF_TYPE_SATA, IF_TYPE_HOST, - IF_TYPE_SYSTEMACE, IF_TYPE_NVME, IF_TYPE_EFI,