From patchwork Tue Jul 15 00:56:29 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 369810 X-Patchwork-Delegate: jagannadh.teki@gmail.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 9CE15140171 for ; Tue, 15 Jul 2014 11:01:48 +1000 (EST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id DCEE1A75A7; Tue, 15 Jul 2014 03:01:12 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Ts+8raXJXhi0; Tue, 15 Jul 2014 03:01:12 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 611BEA75F3; Tue, 15 Jul 2014 02:58:47 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 6CAF2A7582 for ; Tue, 15 Jul 2014 02:58:37 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id IDMAVNyUIMRr for ; Tue, 15 Jul 2014 02:58:34 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-pd0-f202.google.com (mail-pd0-f202.google.com [209.85.192.202]) by theia.denx.de (Postfix) with ESMTPS id 71FDEA7585 for ; Tue, 15 Jul 2014 02:58:09 +0200 (CEST) Received: by mail-pd0-f202.google.com with SMTP id r10so1089469pdi.5 for ; Mon, 14 Jul 2014 17:58:08 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=K91MOdyufV+IAqL/eGy1wBhGyMxUFiR9epP4wlOJAwA=; b=d2vEPKjq6gwwl/vRZG+4npvcQmueeKbMvymPSRiWgMP2VFbpBR21y7YtTOIoy3AdxF SMdkzGe93KOPzUbqBmBa6v8eKui18roK5KSkBVAS9GiqrR64sZTXoexAOKQZJOpe94vg mv7OzAmigbAHVm2pnucEvQ6BhDFxMwKa2BjIGjaC+7pboiFwrW3+3HwAW7ZoOykXIDB/ COiVtoQSMMghVcP5kbI+sdNE8XUX64bD0cCPWQKPi5fKUmqU/klXDzXoSXL+Vo6kO0fG D2lAklMertAfxpuNiBNkBTt+Ka/s0VWFTLpRLEnIZpp/+SQDuRLPCDJUAl9hh0fE1sSX oS6Q== X-Gm-Message-State: ALoCoQlcFlEHC4PIhWlkjwRSuIydueXcY3aVqfk8gNmXIcLm1NKQJo7bVX5jYsA+kjk6SOFbOM1f X-Received: by 10.70.43.77 with SMTP id u13mr9526744pdl.0.1405385888516; Mon, 14 Jul 2014 17:58:08 -0700 (PDT) Received: from corp2gmr1-1.hot.corp.google.com (corp2gmr1-1.hot.corp.google.com [172.24.189.92]) by gmr-mx.google.com with ESMTPS id v44si749690yhv.0.2014.07.14.17.58.08 for (version=TLSv1.1 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 14 Jul 2014 17:58:08 -0700 (PDT) Received: from kaki.bld.corp.google.com (kaki.bld.corp.google.com [172.29.216.32]) by corp2gmr1-1.hot.corp.google.com (Postfix) with ESMTP id 5BA3131C3AC; Mon, 14 Jul 2014 17:58:08 -0700 (PDT) Received: by kaki.bld.corp.google.com (Postfix, from userid 121222) id 2B7EC220CF3; Mon, 14 Jul 2014 18:58:08 -0600 (MDT) From: Simon Glass To: U-Boot Mailing List Date: Mon, 14 Jul 2014 18:56:29 -0600 Message-Id: <1405385792-4469-23-git-send-email-sjg@chromium.org> X-Mailer: git-send-email 2.0.0.526.g5318336 In-Reply-To: <1405385792-4469-1-git-send-email-sjg@chromium.org> References: <1405385792-4469-1-git-send-email-sjg@chromium.org> Cc: Jagannadha Sutradharudu Teki Subject: [U-Boot] [PATCH 22/25] dm: sf: Add tests for SPI flash X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.11 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de Add a simple test for SPI that uses SPI flash. It operates by creating a SPI flash file, and using the 'sf test' command to test that all operations work correctly. Signed-off-by: Simon Glass --- test/dm/Makefile | 1 + test/dm/sf.c | 43 +++++++++++++++++++++++++++++++++++++++++++ test/dm/test.dts | 17 ++++++++++++++++- 3 files changed, 60 insertions(+), 1 deletion(-) create mode 100644 test/dm/sf.c diff --git a/test/dm/Makefile b/test/dm/Makefile index d1b9c9a..75d3d41 100644 --- a/test/dm/Makefile +++ b/test/dm/Makefile @@ -19,4 +19,5 @@ obj-$(CONFIG_DM_TEST) += ut.o ifneq ($(CONFIG_SANDBOX),) obj-$(CONFIG_DM_GPIO) += gpio.o obj-$(CONFIG_DM_SPI) += spi.o +obj-$(CONFIG_DM_SPI_FLASH) += sf.o endif diff --git a/test/dm/sf.c b/test/dm/sf.c new file mode 100644 index 0000000..57dd134 --- /dev/null +++ b/test/dm/sf.c @@ -0,0 +1,43 @@ +/* + * Copyright (C) 2013 Google, Inc + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include + +/* Test that sandbox SPI flash works correctly */ +static int dm_test_spi_flash(struct dm_test_state *dms) +{ + /* + * Create an empty test file and run the SPI flash tests. This is a + * long way from being a unit test, but it does test SPI device and + * emulator binding, probing, the SPI flash emulator including + * device tree decoding, plus the file-based backing store of SPI. + * + * More targeted tests could be created to perform the above steps + * one at a time. This might not increase test coverage much, but + * it would make bugs easier to find. It's not clear whether the + * benefit is worth the extra complexity. + */ + ut_asserteq(0, run_command_list( + "sb save hostfs - spi.bin 0 200000;" + "sf probe;" + "sf test 0 10000", -1, 0)); + /* + * Since we are about to destroy all devices, we must tell sandbox + * to forget the emulation device + */ + sandbox_sf_unbind_emul(state_get_current(), 0, 0); + + return 0; +} +DM_TEST(dm_test_spi_flash, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT); diff --git a/test/dm/test.dts b/test/dm/test.dts index 141d82c..db88160 100644 --- a/test/dm/test.dts +++ b/test/dm/test.dts @@ -82,7 +82,7 @@ compatible = "google,another-fdt-test"; }; - base-gpios { + gpio_a: base-gpios { compatible = "sandbox,gpio"; gpio-bank-name = "a"; num-gpios = <20>; @@ -93,4 +93,19 @@ gpio-bank-name = "b"; num-gpios = <10>; }; + + spi@0 { + #address-cells = <1>; + #size-cells = <0>; + reg = <0>; + compatible = "sandbox,spi"; + cs-gpios = <0>, <&gpio_a 0>; + spi.bin@0 { + reg = <0>; + compatible = "spansion,m25p16", "spi-flash"; + spi-max-frequency = <40000000>; + sandbox,filename = "spi.bin"; + }; + }; + };