Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp350876ybk; Wed, 20 May 2020 01:01:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJynMDrBw5BAxVMA8HuYNwuJugqQKk+Tgb4Jp0DbVB+MzES0S/fIp19pvJwrqwnS95vQwk2O X-Received: by 2002:a17:906:2ec6:: with SMTP id s6mr2941829eji.532.1589961698926; Wed, 20 May 2020 01:01:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589961698; cv=none; d=google.com; s=arc-20160816; b=H4psStWdN8hlO2wDcRm6oZp0PuCePmRgdz1yqsxoqRHig7+zpnFvbAEheVT9qWW/ol 0uAVChbTaJyL+c86RkRalhK9FWfOoMFW67gCcRg6765NAGuSV2YIrfHsXwNS0wRvTsgF P3C3l4plBBxkkYt840AhOKM3w4Fk5/TMcLWDe7XdfEIy1/McJ4owyUnTvr7wuZEyWcX0 hS89Ex6844zl1gmUlgbDGfLYASe/ZFUyEAB8UUf6aZ5nPXPrtjxbpwy71lukU1uTJzHH F14XCWaUKW8m1zjLR141nvE+Jy+Zf6z1n9SVLxc0gjZaNYvWovMw6oP8JLS6rl1ig0Qi VV9w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:from:message-id:mime-version:subject :cc:to:references:in-reply-to; bh=fyItfs33iMD3ltaHmejkHkzNOKYaYZBbURyBGCS7Nc4=; b=kEfMpjqpB7Rd+Y/HEsN1B2NAT6+gbrFQ1EZXl5w3yPujiCTyoLveTcQhWyP+N5Cq7P cr6f95n4oVrd3bUqunjCGfTrGtX6JQWpKLEXSmbT99SKBwlccZKOYb4vFiD/RiPhD6Fx xplVJA98gGtQk6csnouKf/2/7RolTfV6/Tai39QH/DMoHJxoidZp0ipQvZCiUjMl587f wBVpQiVIUzlFWr83VWLfVM143H363mIrBZRTbAnWCQSaZfnnwsknHTLlFGkN9khn/WjY qBAkjXM1DFe9LmCJT+MccX4i8tPAf0eVYvN7U3KF3lPDvYbc+Os3P+1SvEAQdxSWlQvm V+fg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ov4si1406634ejb.237.2020.05.20.01.01.15; Wed, 20 May 2020 01:01:38 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726560AbgETH7j (ORCPT + 99 others); Wed, 20 May 2020 03:59:39 -0400 Received: from twhmllg3.macronix.com ([122.147.135.201]:37467 "EHLO TWHMLLG3.macronix.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726224AbgETH7j (ORCPT ); Wed, 20 May 2020 03:59:39 -0400 Received: from twhfmlp1.macronix.com (twhfmlp1.macronix.com [172.17.20.91]) by TWHMLLG3.macronix.com with ESMTP id 04K7xUdE096136; Wed, 20 May 2020 15:59:30 +0800 (GMT-8) (envelope-from masonccyang@mxic.com.tw) Received: from MXML06C.mxic.com.tw (mxml06c.mxic.com.tw [172.17.14.55]) by Forcepoint Email with ESMTP id B9D79C5271EE9FAE1410; Wed, 20 May 2020 15:59:30 +0800 (CST) In-Reply-To: <20200519142642.24131-10-p.yadav@ti.com> References: <20200519142642.24131-1-p.yadav@ti.com> <20200519142642.24131-10-p.yadav@ti.com> To: "Pratyush Yadav" Cc: "Alexandre Belloni" , "Boris Brezillon" , "Mark Brown" , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org, "Ludovic Desroches" , "Matthias Brugger" , "Miquel Raynal" , "Nicolas Ferre" , "Sekhar Nori" , "Pratyush Yadav" , "Richard Weinberger" , "Tudor Ambarus" , "Vignesh Raghavendra" , juliensu@mxic.com.tw Subject: Re: [PATCH v5 09/19] mtd: spi-nor: sfdp: parse xSPI Profile 1.0 table MIME-Version: 1.0 X-KeepSent: 83616464:480FA751-4825856E:002A4483; type=4; name=$KeepSent X-Mailer: Lotus Notes Release 8.5.3FP4 SHF90 June 10, 2013 Message-ID: From: masonccyang@mxic.com.tw Date: Wed, 20 May 2020 15:59:31 +0800 X-MIMETrack: Serialize by Router on MXML06C/TAIWAN/MXIC(Release 9.0.1FP10 HF265|July 25, 2018) at 2020/05/20 PM 03:59:30, Serialize complete at 2020/05/20 PM 03:59:30 Content-Type: text/plain; charset="US-ASCII" X-MAIL: TWHMLLG3.macronix.com 04K7xUdE096136 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Pratyush, > +/** > + * spi_nor_parse_profile1() - parse the xSPI Profile 1.0 table > + * @nor: pointer to a 'struct spi_nor' > + * @param_header: pointer to the 'struct sfdp_parameter_header' describing > + * the 4-Byte Address Instruction Table length and version. > + * @params: pointer to the 'struct spi_nor_flash_parameter' to be. > + * > + * Return: 0 on success, -errno otherwise. > + */ > +static int spi_nor_parse_profile1(struct spi_nor *nor, > + const struct sfdp_parameter_header *profile1_header, > + struct spi_nor_flash_parameter *params) > +{ > + u32 *table, opcode, addr; > + size_t len; > + int ret, i; > + > + len = profile1_header->length * sizeof(*table); > + table = kmalloc(len, GFP_KERNEL); > + if (!table) > + return -ENOMEM; > + > + addr = SFDP_PARAM_HEADER_PTP(profile1_header); > + ret = spi_nor_read_sfdp(nor, addr, len, table); > + if (ret) > + goto out; > + > + /* Fix endianness of the table DWORDs. */ > + for (i = 0; i < profile1_header->length; i++) > + table[i] = le32_to_cpu(table[i]); > + > + /* Get 8D-8D-8D fast read opcode and dummy cycles. */ > + opcode = FIELD_GET(PROFILE1_DWORD1_RD_FAST_CMD, table[0]); > + > + /* > + * Update the fast read settings. We set the default dummy cycles to 20 > + * here. Flashes can change this value if they need to when enabling > + * octal mode. > + */ > + spi_nor_set_read_settings(¶ms->reads[SNOR_CMD_READ_8_8_8_DTR], > + 0, 20, opcode, > + SNOR_PROTO_8_8_8_DTR); > + I thought we have a agreement that only do parse here, no other read parameters setting. Driver should get dummy cycles used for various frequencies from 4th and 5th DWORD of xSPI table.[1] [1] https://patchwork.ozlabs.org/project/linux-mtd/patch/1587451187-6889-3-git-send-email-masonccyang@mxic.com.tw/ In addition, 20 dummy cycles is for 200MHz but not for 100MHz, 133MHz and 166MHz in case of read performance concern. Given a correct dummy cycles for a specific device. [2] [2] https://patchwork.ozlabs.org/project/linux-mtd/patch/1587451187-6889-5-git-send-email-masonccyang@mxic.com.tw/ > + /* > + * Set the Read Status Register dummy cycles and dummy address bytes. > + */ > + if (table[0] & PROFILE1_DWORD1_RDSR_DUMMY) > + params->rdsr_dummy = 8; > + else > + params->rdsr_dummy = 4; > + > + if (table[0] & PROFILE1_DWORD1_RDSR_ADDR_BYTES) > + params->rdsr_addr_nbytes = 4; > + else > + params->rdsr_addr_nbytes = 0; > + > +out: > + kfree(table); > + return ret; > +} > + thanks & best regards, Mason CONFIDENTIALITY NOTE: This e-mail and any attachments may contain confidential information and/or personal data, which is protected by applicable laws. Please be reminded that duplication, disclosure, distribution, or use of this e-mail (and/or its attachments) or any part thereof is prohibited. If you receive this e-mail in error, please notify us immediately and delete this mail as well as its attachment(s) from your system. In addition, please be informed that collection, processing, and/or use of personal data is prohibited unless expressly permitted by personal data protection laws. Thank you for your attention and cooperation. Macronix International Co., Ltd. ===================================================================== ============================================================================ CONFIDENTIALITY NOTE: This e-mail and any attachments may contain confidential information and/or personal data, which is protected by applicable laws. Please be reminded that duplication, disclosure, distribution, or use of this e-mail (and/or its attachments) or any part thereof is prohibited. If you receive this e-mail in error, please notify us immediately and delete this mail as well as its attachment(s) from your system. In addition, please be informed that collection, processing, and/or use of personal data is prohibited unless expressly permitted by personal data protection laws. Thank you for your attention and cooperation. Macronix International Co., Ltd. =====================================================================