From nobody Tue Nov 18 15:28:28 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1611061447; cv=none; d=zohomail.com; s=zohoarc; b=WEl+pDM8XNHmOlSL6sYTa0WUPv5j9nJdCD8f1h4B+fjIA7H3kriyzupSN4TWInUuzOxJRCPU+6pqSjayVp/o50WIdS/SPL4JQaTCiXbBBeEdiTrX707S2HGI+IgQw3vv82CWR0MLa7v9C5WaQSY5rgkG6Zdtq6gxbeB9GRwfmvo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1611061447; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=eF3PdkZp88Dtx8+Gjn51Fph38Ji7ew7FQX7/6hEKv5w=; b=JgZLpUGUIyeb6jBwCYXAeZKM1A5pmyQsKjU1FMKzT28aeHO0SI+JgBYmXFCcaETTqbjFf2MFJibcJpdRSYhSe05aXDoSwj5XfMB++fsTefZX8amey8vzyjcOo7Bo5CJ7CBJ7vH0E6zWGM8V28XzDfP60kwKjh9JlSKaRsf+Zb6A= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1611061447527306.8235281953905; Tue, 19 Jan 2021 05:04:07 -0800 (PST) Received: from localhost ([::1]:47928 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1l1qg2-0007zX-9S for importer@patchew.org; Tue, 19 Jan 2021 08:04:06 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:44362) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1l1qeA-0006Tz-Rx for qemu-devel@nongnu.org; Tue, 19 Jan 2021 08:02:10 -0500 Received: from mail-lf1-x136.google.com ([2a00:1450:4864:20::136]:44249) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1l1qe4-0005vV-0K for qemu-devel@nongnu.org; Tue, 19 Jan 2021 08:02:10 -0500 Received: by mail-lf1-x136.google.com with SMTP id m25so28888905lfc.11 for ; Tue, 19 Jan 2021 05:02:01 -0800 (PST) Received: from localhost.localdomain (31-208-27-151.cust.bredband2.com. [31.208.27.151]) by smtp.gmail.com with ESMTPSA id e25sm2356745lfc.40.2021.01.19.05.01.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 Jan 2021 05:01:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=eF3PdkZp88Dtx8+Gjn51Fph38Ji7ew7FQX7/6hEKv5w=; b=hgogRVOcMOo2v1ThHrdeeUdouzs/HmoVUCToY+6k4NxxJfdVIvi8UyZX7vYUDjoF90 DaTwycskKkrV7oknCI7vH/7SO7cC6JENZc/KIHkjTdV4QfzsVZoYf1kPfxKOByfaO9NT 1glOQ7QVwAu+j+0VkDfAx6ti1WTSL0U27lhVwuVlQa8v4QA550vVzJ2bc4uqYNmMKMvZ DKgoMJM77E8We4o0m8+6HKM/J0RrEI4ILDin8WM2KYdp9gnMjWKnrgQUgAyr1HYEVUgX 59XUdjaYDzIR1yT2fXIoFkIHsAF1shlhjUBzvMBsFgw8A5ecf1uFccDv5MQ1XRAizKK/ 6TMA== 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; bh=eF3PdkZp88Dtx8+Gjn51Fph38Ji7ew7FQX7/6hEKv5w=; b=gCAIqXXpenGBRVZ9JBXiRnIvgjWobL5f6iy01ntUY7SpYsZWSglgpBB0al3o+Ccdnr ywj+RLdGxW+1HRvmWRR9ZCeWoWDuzZmAwomGJ6mZeWRyR77YarwaWFwlgNgsv6HUDkkb Z3V7zFu0A+WiYV89Ga/TVbuD90fQBzwJc4WY/tuBh95+968/NIOIIq9ppkZbZx4xHdRw sSTZEaCZFaj0PW9I1UzQLor/WBYaEtuGhM9C5bIj58gXnja/+tUvzKWIM59egYnvbSJ9 vveX7w4zv87aZtVJe2c1INzExZ/CwHrhTWgjqcFYLuyOLzJvisaE/4GB0x6y1oI/rbaE tnOw== X-Gm-Message-State: AOAM533VKLzWL9sqxH4UJ1Vd7LnByQHxvN0fmwVPoz3VeDxEoATpyp35 j6Lv5b6Gyw+/GEtzkvnLoukbe4Q1RLyvmA== X-Google-Smtp-Source: ABdhPJyWp1XqxgL91u7wVZE4Fbm0Ntzy4nSGIKpyiVrJeUvoV2p4NSXoJz2lHlNnGL0iqNHhomEQKw== X-Received: by 2002:a05:6512:338e:: with SMTP id h14mr1856046lfg.324.1611061319068; Tue, 19 Jan 2021 05:01:59 -0800 (PST) From: Francisco Iglesias To: qemu-devel@nongnu.org, frasse.iglesias@gmail.com Subject: [RFC PATCH v1 2/3] hw: block: m25p80: Support dummy byte accuracy Date: Tue, 19 Jan 2021 14:01:54 +0100 Message-Id: <20210119130155.30194-3-frasse.iglesias@gmail.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20210119130155.30194-1-frasse.iglesias@gmail.com> References: <20210119130155.30194-1-frasse.iglesias@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=2a00:1450:4864:20::136; envelope-from=frasse.iglesias@gmail.com; helo=mail-lf1-x136.google.com X-Spam_score_int: -1020 X-Spam_score: -102.1 X-Spam_bar: --------------------------------------------------- X-Spam_report: (-102.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, USER_IN_WELCOMELIST=-0.01, USER_IN_WHITELIST=-100 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, peter.maydell@linaro.org, alistair@alistair23.me, bin.meng@windriver.com, mreitz@redhat.com, alistair23@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" This patch adds support to toggle the accuracy in m25p80 between dummy clock cycle to dummy byte. By being able to do this the SPI controllers transfering through a txfifo will be able to support commands with dummy clock cycles. Signed-off-by: Francisco Iglesias --- hw/block/m25p80.c | 112 +++++++++++++++++++++++++++++++++++++--------- 1 file changed, 92 insertions(+), 20 deletions(-) diff --git a/hw/block/m25p80.c b/hw/block/m25p80.c index 1b3f2405a1..c9342e1842 100644 --- a/hw/block/m25p80.c +++ b/hw/block/m25p80.c @@ -462,6 +462,7 @@ struct Flash { =20 const FlashPartInfo *pi; =20 + bool dummy_byte_accuracy; }; =20 struct M25P80Class { @@ -862,26 +863,59 @@ static uint8_t numonyx_extract_cfg_num_dummies(Flash = *s) return num_dummies; } =20 +static uint8_t to_dummy_bytes(Flash *s, uint8_t dummy_clk_cycles) +{ + uint8_t lines =3D 1; + + switch (s->cmd_in_progress) { + case DOR: + case DOR4: + case DIOR: + case DIOR4: + lines =3D 2; + break; + case QOR: + case QOR4: + case QIOR: + case QIOR4: + lines =3D 4; + break; + default: + break; + } + + if ((dummy_clk_cycles * lines) % 8) { + qemu_log_mask(LOG_UNIMP, "M25P80: Non multiple of 8 number of dumm= y" + " clock cycles while operating with dummy byte" + " accuracy\n"); + } + + return (dummy_clk_cycles * lines) / 8; +} + static void decode_fast_read_cmd(Flash *s) { + uint8_t dummy_clk_cycles =3D 0; + uint8_t extra_bytes; + s->needed_bytes =3D get_addr_length(s); switch (get_man(s)) { /* Dummy cycles - modeled with bytes writes instead of bits */ case MAN_WINBOND: - s->needed_bytes +=3D 8; + dummy_clk_cycles =3D 8; break; case MAN_NUMONYX: - s->needed_bytes +=3D numonyx_extract_cfg_num_dummies(s); + dummy_clk_cycles =3D numonyx_extract_cfg_num_dummies(s); break; case MAN_MACRONIX: if (extract32(s->volatile_cfg, 6, 2) =3D=3D 1) { - s->needed_bytes +=3D 6; + dummy_clk_cycles =3D 6; } else { - s->needed_bytes +=3D 8; + dummy_clk_cycles =3D 8; } break; case MAN_SPANSION: - s->needed_bytes +=3D extract32(s->spansion_cr2v, + dummy_clk_cycles =3D extract32(s->spansion_cr2v, SPANSION_DUMMY_CLK_POS, SPANSION_DUMMY_CLK_LEN ); @@ -889,6 +923,14 @@ static void decode_fast_read_cmd(Flash *s) default: break; } + + if (s->dummy_byte_accuracy) { + extra_bytes =3D to_dummy_bytes(s, dummy_clk_cycles); + } else { + extra_bytes =3D dummy_clk_cycles; + } + + s->needed_bytes +=3D extra_bytes; s->pos =3D 0; s->len =3D 0; s->state =3D STATE_COLLECTING_DATA; @@ -896,38 +938,49 @@ static void decode_fast_read_cmd(Flash *s) =20 static void decode_dio_read_cmd(Flash *s) { + uint8_t dummy_clk_cycles =3D 0; + uint8_t extra_bytes; + s->needed_bytes =3D get_addr_length(s); /* Dummy cycles modeled with bytes writes instead of bits */ switch (get_man(s)) { case MAN_WINBOND: - s->needed_bytes +=3D WINBOND_CONTINUOUS_READ_MODE_CMD_LEN; + dummy_clk_cycles =3D WINBOND_CONTINUOUS_READ_MODE_CMD_LEN; break; case MAN_SPANSION: - s->needed_bytes +=3D SPANSION_CONTINUOUS_READ_MODE_CMD_LEN; - s->needed_bytes +=3D extract32(s->spansion_cr2v, + dummy_clk_cycles =3D SPANSION_CONTINUOUS_READ_MODE_CMD_LEN; + dummy_clk_cycles =3D extract32(s->spansion_cr2v, SPANSION_DUMMY_CLK_POS, SPANSION_DUMMY_CLK_LEN ); break; case MAN_NUMONYX: - s->needed_bytes +=3D numonyx_extract_cfg_num_dummies(s); + dummy_clk_cycles =3D numonyx_extract_cfg_num_dummies(s); break; case MAN_MACRONIX: switch (extract32(s->volatile_cfg, 6, 2)) { case 1: - s->needed_bytes +=3D 6; + dummy_clk_cycles =3D 6; break; case 2: - s->needed_bytes +=3D 8; + dummy_clk_cycles =3D 8; break; default: - s->needed_bytes +=3D 4; + dummy_clk_cycles =3D 4; break; } break; default: break; } + + if (s->dummy_byte_accuracy) { + extra_bytes =3D to_dummy_bytes(s, dummy_clk_cycles); + } else { + extra_bytes =3D dummy_clk_cycles; + } + + s->needed_bytes +=3D extra_bytes; s->pos =3D 0; s->len =3D 0; s->state =3D STATE_COLLECTING_DATA; @@ -935,39 +988,50 @@ static void decode_dio_read_cmd(Flash *s) =20 static void decode_qio_read_cmd(Flash *s) { + uint8_t dummy_clk_cycles =3D 0; + uint8_t extra_bytes; + s->needed_bytes =3D get_addr_length(s); /* Dummy cycles modeled with bytes writes instead of bits */ switch (get_man(s)) { case MAN_WINBOND: - s->needed_bytes +=3D WINBOND_CONTINUOUS_READ_MODE_CMD_LEN; - s->needed_bytes +=3D 4; + dummy_clk_cycles =3D WINBOND_CONTINUOUS_READ_MODE_CMD_LEN; + dummy_clk_cycles +=3D 4; break; case MAN_SPANSION: - s->needed_bytes +=3D SPANSION_CONTINUOUS_READ_MODE_CMD_LEN; - s->needed_bytes +=3D extract32(s->spansion_cr2v, + dummy_clk_cycles =3D SPANSION_CONTINUOUS_READ_MODE_CMD_LEN; + dummy_clk_cycles +=3D extract32(s->spansion_cr2v, SPANSION_DUMMY_CLK_POS, SPANSION_DUMMY_CLK_LEN ); break; case MAN_NUMONYX: - s->needed_bytes +=3D numonyx_extract_cfg_num_dummies(s); + dummy_clk_cycles =3D numonyx_extract_cfg_num_dummies(s); break; case MAN_MACRONIX: switch (extract32(s->volatile_cfg, 6, 2)) { case 1: - s->needed_bytes +=3D 4; + dummy_clk_cycles =3D 4; break; case 2: - s->needed_bytes +=3D 8; + dummy_clk_cycles =3D 8; break; default: - s->needed_bytes +=3D 6; + dummy_clk_cycles =3D 6; break; } break; default: break; } + + if (s->dummy_byte_accuracy) { + extra_bytes =3D to_dummy_bytes(s, dummy_clk_cycles); + } else { + extra_bytes =3D dummy_clk_cycles; + } + + s->needed_bytes +=3D extra_bytes; s->pos =3D 0; s->len =3D 0; s->state =3D STATE_COLLECTING_DATA; @@ -1291,6 +1355,13 @@ static int m25p80_cs(SSIPeripheral *ss, bool select) return 0; } =20 +static void m25p80_dummy_byte_accuracy(SSIPeripheral *ss, bool val) +{ + Flash *s =3D M25P80(ss); + + s->dummy_byte_accuracy =3D val; +} + static uint32_t m25p80_transfer8(SSIPeripheral *ss, uint32_t tx) { Flash *s =3D M25P80(ss); @@ -1494,6 +1565,7 @@ static void m25p80_class_init(ObjectClass *klass, voi= d *data) k->transfer =3D m25p80_transfer8; k->set_cs =3D m25p80_cs; k->cs_polarity =3D SSI_CS_LOW; + k->set_dummy_byte_accuracy =3D m25p80_dummy_byte_accuracy; dc->vmsd =3D &vmstate_m25p80; device_class_set_props(dc, m25p80_properties); dc->reset =3D m25p80_reset; --=20 2.20.1