Received: by 2002:a17:90a:bc8d:0:0:0:0 with SMTP id x13csp1166012pjr; Thu, 21 May 2020 01:30:14 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz6nbOwWKXa9mGLWGSkt7wb+RgpxJGWJj/lRir7X2kvbniuov5vyPT3+mbfJsIEwx3iNsos X-Received: by 2002:aa7:c5c8:: with SMTP id h8mr6710568eds.222.1590049814708; Thu, 21 May 2020 01:30:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590049814; cv=none; d=google.com; s=arc-20160816; b=FkOB/NszPpG4kLqQc4xYpPaPgxuxXXKKmcm8oRHcpGYAAPGQqp1NEZesnX8Fj5t4Es bpus2wRvGd48VFCEstxxe+gV73OE+xwfIKdTDx1eXy/T6LKqcTLCZSE38rQe74x5Su8e /xKlISsCP+QvExTWslakZjPaaCHwTPeFkPdlPWLnQ9/mJpctHEyKnQ2WEHhsagaahMiC iFmHULRC3ySGRezO0W/cEebG/qxZw2mPthzv8W3JQV8h4b/0tKtTC8rwlejeW39smvDZ CE91ToCjguhyWNJ1nVVeZagEwF3BkTzOkznI+Cw99YRj3a1AlJCtTPrVvwJ0oRyNkeIa Z3cA== 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=2xUEZ+CllEChvn7ZSzSEsSqLGCihYFtzcWBDYMsUA1U=; b=N6zNOByAkEZoOujadhIXDS1/JWyhSROY8bV5V1hLnw2bxXbF4qw+efSCTJrhCGtb2r VARTicYb6OixhRkU6pmc46v9uJybRwTOiaVuPSIdj9cca3oioTdGQrVGn/ptEgU4U9Ye CthSozYDBNWscagUwf86zDUC1MSd9Kj+2AldG9PDCXeLexkbGhM9CWURUgwLHFrUa/UJ WxeDL0utt4CKge+F2N198oDrqr3dX41nmOy6mCYdaszX9WUqvEN2yeKdzDWvkNxPUVmC mb8C5/H//t7oh/q2SQx33f9caLDPfrGb9+DJcr1FA0LINCPvwBb5Vt+TEo01xx82Oc+7 oMhw== 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 dc4si2679156edb.565.2020.05.21.01.29.51; Thu, 21 May 2020 01:30:14 -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 S1728587AbgEUI14 (ORCPT + 99 others); Thu, 21 May 2020 04:27:56 -0400 Received: from twhmllg3.macronix.com ([211.75.127.131]:45575 "EHLO TWHMLLG3.macronix.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728481AbgEUI14 (ORCPT ); Thu, 21 May 2020 04:27:56 -0400 Received: from twhfmlp1.macronix.com (twhfmlp1.macronix.com [172.17.20.91]) by TWHMLLG3.macronix.com with ESMTP id 04L8RljY069236; Thu, 21 May 2020 16:27:47 +0800 (GMT-8) (envelope-from masonccyang@mxic.com.tw) Received: from MXML06C.mxic.com.tw (mxml06c.macronix.com [172.17.14.55]) by Forcepoint Email with ESMTP id 1B985808C8C345E414D3; Thu, 21 May 2020 16:27:48 +0800 (CST) In-Reply-To: <20200519142642.24131-2-p.yadav@ti.com> References: <20200519142642.24131-1-p.yadav@ti.com> <20200519142642.24131-2-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 01/19] spi: spi-mem: allow specifying whether an op is DTR or not MIME-Version: 1.0 X-KeepSent: 1FE36FB9:9FBEFCD6-4825856F:002D767F; type=4; name=$KeepSent X-Mailer: Lotus Notes Release 8.5.3FP4 SHF90 June 10, 2013 Message-ID: From: masonccyang@mxic.com.tw Date: Thu, 21 May 2020 16:27:47 +0800 X-MIMETrack: Serialize by Router on MXML06C/TAIWAN/MXIC(Release 9.0.1FP10 HF265|July 25, 2018) at 2020/05/21 PM 04:27:48, Serialize complete at 2020/05/21 PM 04:27:48 Content-Type: text/plain; charset="US-ASCII" X-MAIL: TWHMLLG3.macronix.com 04L8RljY069236 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Pratyush, Given cmd.nbytes a initial value & check it ! > > [PATCH v5 01/19] spi: spi-mem: allow specifying whether an op is DTR or not > > Each phase is given a separate 'dtr' field so mixed protocols like > 4S-4D-4D can be supported. > > Signed-off-by: Pratyush Yadav > --- > drivers/spi/spi-mem.c | 3 +++ > include/linux/spi/spi-mem.h | 8 ++++++++ > 2 files changed, 11 insertions(+) > > diff --git a/drivers/spi/spi-mem.c b/drivers/spi/spi-mem.c > index 9a86cc27fcc0..93e255287ab9 100644 > --- a/drivers/spi/spi-mem.c > +++ b/drivers/spi/spi-mem.c > @@ -156,6 +156,9 @@ bool spi_mem_default_supports_op(struct spi_mem *mem, > op->data.dir == SPI_MEM_DATA_OUT)) > return false; > > + if (op->cmd.dtr || op->addr.dtr || op->dummy.dtr || op->data.dtr) > + return false; > + + if (op->cmd.nbytes != 1) + return false; > return true; > } > EXPORT_SYMBOL_GPL(spi_mem_default_supports_op); static int spi_mem_check_op(const struct spi_mem_op *op) { - if (!op->cmd.buswidth) + if (!op->cmd.buswidth || op->cmd.nbytes < 1 || op->cmd.nbytes > 2) return -EINVAL; > diff --git a/include/linux/spi/spi-mem.h b/include/linux/spi/spi-mem.h > index af9ff2f0f1b2..e3dcb956bf61 100644 > --- a/include/linux/spi/spi-mem.h > +++ b/include/linux/spi/spi-mem.h #define SPI_MEM_OP_CMD(__opcode, __buswidth) \ { \ .buswidth = __buswidth, \ .opcode = __opcode, \ + .nbytes = 1, \ } > @@ -71,9 +71,11 @@ enum spi_mem_data_dir { > * struct spi_mem_op - describes a SPI memory operation > * @cmd.buswidth: number of IO lines used to transmit the command > * @cmd.opcode: operation opcode > + * @cmd.dtr: whether the command opcode should be sent in DTR mode or not > * @addr.nbytes: number of address bytes to send. Can be zero if the operation > * does not need to send an address > * @addr.buswidth: number of IO lines used to transmit the address cycles > + * @addr.dtr: whether the address should be sent in DTR mode or not > * @addr.val: address value. This value is always sent MSB first on the bus. > * Note that only @addr.nbytes are taken into account in this > * address value, so users should make sure the value fits in the > @@ -81,7 +83,9 @@ enum spi_mem_data_dir { > * @dummy.nbytes: number of dummy bytes to send after an opcode or address. Can > * be zero if the operation does not require dummy bytes > * @dummy.buswidth: number of IO lanes used to transmit the dummy bytes > + * @dummy.dtr: whether the dummy bytes should be sent in DTR mode or not > * @data.buswidth: number of IO lanes used to send/receive the data > + * @data.dtr: whether the data should be sent in DTR mode or not > * @data.dir: direction of the transfer > * @data.nbytes: number of data bytes to send/receive. Can be zero if the > * operation does not involve transferring data > @@ -91,22 +95,26 @@ enum spi_mem_data_dir { > struct spi_mem_op { > struct { > u8 buswidth; > + u8 dtr : 1; > u8 opcode; > } cmd; > > struct { > u8 nbytes; > u8 buswidth; > + u8 dtr : 1; > u64 val; > } addr; > > struct { > u8 nbytes; > u8 buswidth; > + u8 dtr : 1; > } dummy; > > struct { > u8 buswidth; > + u8 dtr : 1; > enum spi_mem_data_dir dir; > unsigned int nbytes; > union { > -- > 2.26.2 > 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. =====================================================================