Received: by 2002:a05:7412:8521:b0:e2:908c:2ebd with SMTP id t33csp450035rdf; Fri, 3 Nov 2023 05:47:45 -0700 (PDT) X-Google-Smtp-Source: AGHT+IFAFF5ASK7y/zwlu0syrAPFP65pjOeNOkFjhhJYtIHUTYZw46cqKYknGXqK5aFJMmYKrfil X-Received: by 2002:a05:6808:1986:b0:3b5:6533:d2cb with SMTP id bj6-20020a056808198600b003b56533d2cbmr12232793oib.46.1699015665307; Fri, 03 Nov 2023 05:47:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1699015665; cv=none; d=google.com; s=arc-20160816; b=utUoqn8CsR7ufeoPpdECMcQXSZULe4VfUoMRamLCJBHSIzKp9r1HqzSl9jS2wao8pG uhtlmF/lpwTmo4bjuy/7t8Q3xUVf94Ttlyj+4Z65VxJjSUueDSqoMSpw/9XNMwErNEZz zkQRkPMm+FZ1shuF+hFsnlutIYLJ5bpYPut2ZbZ5qIvOhCMsHoGX3pywF8YGuIu8MFE4 89Jli0YBNMzWYpYaqsGgekYtL1riCybYiVsaiOB/23K77F5S4Suxj2NDql6BBsMicO8s /EDLP0QPrbYsGOl2SRHRJ+OXsS91+5Lf5wU4T8ATSekblB5ABt4n3wUhS1PojL0eJqBp SXsw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=98kfUYAbNIz3Da0sfQzDbhJiGYz1OZgDqFtol2F0EvY=; fh=8WutLte5sqsIp0QNGNr3bVo1nt3XY1qC5lOLXvo5AZg=; b=yhQTtID0uXYYoDu8mOl5wNOLGs+HVYRV3h/iWnDoEm3tkzTBdHY10NAb7VWNZCIS30 EH8rbsrU+5Gss6mraWyTjLs6JLYIBGiizKz1mKy/Nnqw90SpljhPPGSa6TLdvBM9MpLF dPoXiuF4FUa6CLgJwasbkZfmBbWswQIPwapY2P71bQTjAorN9PEDJkeEVrKmf8/5/00z aDyynEKdqaKWcjnvWKUFftqpoaZx+3pDOu7RfTdZjMq/pbgxYDFXqlfO6Aw6KoK73hKx kz4zX+MskjanxjcOBx5TTLcxiFC/Ilv3YqPLcMxHMoyPZLDv/qVQQyYiks3wR1qUx/pv plxw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=D3+lSdzO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from fry.vger.email (fry.vger.email. [23.128.96.38]) by mx.google.com with ESMTPS id w63-20020a638242000000b005b8aef54248si1469534pgd.411.2023.11.03.05.47.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Nov 2023 05:47:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=D3+lSdzO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id A13BC82E1E29; Fri, 3 Nov 2023 05:47:42 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229379AbjKCMrg (ORCPT + 99 others); Fri, 3 Nov 2023 08:47:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54786 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229940AbjKCMrf (ORCPT ); Fri, 3 Nov 2023 08:47:35 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A7AF4CE; Fri, 3 Nov 2023 05:47:32 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D0C26C433C7; Fri, 3 Nov 2023 12:47:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1699015652; bh=YO66OvhP/VVHPNNy7Ei4mXR0Rw9gNOGEAcE5AmCI8wY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=D3+lSdzOQRk80PA8gPgTD8xYIKG2lrOnNFtRDeVdlBfG63Aers/mg7I4t8mFTPi9Q CW1HnBwAVo9YkCAyWfuj/1gSvy5Xb/zEBmaJzP6A6SIjAvsnT+EbIILp50AteGyo+J x7Kjygt9tYFgMSXS0lLD3fzQ9mHYTi1SNtuZIuUCs2OP3DCiJjG+mPcYG9rpLITvil hg5/Q+j/yEl+kcDXT3r9WGsxD9dr5KHWeMNcgxx16JsodFE5M1EQF3B3sSZLXJXFQv 3O40cmAJcRVf+aArbfvdPfkqLBpWosdJOdPeiABcl8HwXMDHfEFAFTA0DRnv0batA5 Q2uePOM2ZK9bA== Date: Fri, 3 Nov 2023 12:47:25 +0000 From: Mark Brown To: Md Sadre Alam Cc: agross@kernel.org, andersson@kernel.org, konrad.dybcio@linaro.org, robh+dt@kernel.org, conor+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, miquel.raynal@bootlin.com, richard@nod.at, vigneshr@ti.com, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, linux-spi@vger.kernel.org, quic_srichara@quicinc.com, qpic_varada@quicinc.com Subject: Re: [RFC PATCH 4/5] spi: qpic: Add support for qpic spi nand driver Message-ID: References: <20231031120307.1600689-1-quic_mdalam@quicinc.com> <20231031120307.1600689-5-quic_mdalam@quicinc.com> <2b9e943a-198e-7999-b898-e7b2498a9ffa@quicinc.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="Op0au9/l1MW2X+EM" Content-Disposition: inline In-Reply-To: <2b9e943a-198e-7999-b898-e7b2498a9ffa@quicinc.com> X-Cookie: He's dead, Jim. X-Spam-Status: No, score=-1.7 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Fri, 03 Nov 2023 05:47:42 -0700 (PDT) --Op0au9/l1MW2X+EM Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Nov 03, 2023 at 04:50:54PM +0530, Md Sadre Alam wrote: > On 10/31/2023 7:53 PM, Mark Brown wrote: > > On Tue, Oct 31, 2023 at 05:33:06PM +0530, Md Sadre Alam wrote: > > > + ret = submit_descs(snandc); > > > + if (ret) > > > + dev_err(snandc->dev, "failure in sbumitting spiinit descriptor\n"); > > > + > > > + free_descs(snandc); > > This seems to be doing a bit more than I would expect an init function > > to, and it's very surprising to see the descriptors freed immediately > > after something called a submit (which suggests that the descriptors are > > still in flight). > Our controller supports only bam mode , that means for writing/reading even > single register we have to use bam. > submit_descs() is synchronous so I/O is complete when it returns. > Hence freeing the descriptor after it. That seems like the BAM API is very confusing and error prone. --Op0au9/l1MW2X+EM Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmVE69wACgkQJNaLcl1U h9Dkugf/VsaUo4k4Z3jLIBAgzXGrzjlRrTxTxJyGu9jDAXZeMeTn2A2E+PgfGf5D 5ZxgLrmrSJLMsRGWIRe8MqKiyThyN2rOC/er/oJE8n4SZ+FicSBXPCdR6TPUKCsf M5Rl4Vhq+rBRkArWp/Y5luoG9/pDCs5mhGbY1SKkaR06RvftMV7MHgWodxyn7U5G iexZwY9I3IzkpkB85edqNl0Vh89vvsjPKptH8WZOuc+mF1dRZGc5dkXwgHAScxKr I5dAQky8hSXYoMsialzHArWpIWjdmuMELn9CKMTLpQvwc7pUWkqIi4GGB1TuYWMB LiPTinFMPGc2XD8XqtWCHNbTvA9BfQ== =xTC+ -----END PGP SIGNATURE----- --Op0au9/l1MW2X+EM--