public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Miquel Raynal <miquel.raynal@bootlin.com>, quic_mdalam@quicinc.com
Cc: vigneshr@ti.com, richard@nod.at, linux-mtd@lists.infradead.org,
	linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
	quic_srichara@quicinc.com, dan.carpenter@linaro.org
Subject: Re: [PATCH 00/10] mtd: rawnand: qcom: Bunch of fixes and cleanups
Date: Mon, 7 Aug 2023 17:20:21 +0530	[thread overview]
Message-ID: <20230807115021.GB18257@thinkpad> (raw)
In-Reply-To: <20230806155309.25427927@xps-13>

On Sun, Aug 06, 2023 at 03:53:09PM +0200, Miquel Raynal wrote:
> Hi Manivannan,
> 
> manivannan.sadhasivam@linaro.org wrote on Sat,  5 Aug 2023 23:11:36
> +0530:
> 
> > Hi Miquel,
> > 
> > This series has fixes for the smatch warnings reported by Kbuild bot [1]
> > and also several cleanup patches based on my code observation.
> > 
> > I've only compile tested this series. So let's wait for Sadre/Sricharan to
> > give a tested-by tag to make sure I didn't mess up anything.
> 
> I reviewed all the patches, they look good to me. I'm waiting for the
> tests. Please provide the output of nandbiterrs -i.
> 

I tested the series on Qcom SDX55 based dev board, but it was broken already due
to commit <3fc92384b654> ("mtd: rawnand: qcom: Implement exec_op()").

I'm seeing a string of this error while the partitions were being enumerated:
qcom-nandc 1b30000.nand-controller: failed to copy last codeword

It just goes on...

Sadre, on what platform did you test 3fc92384b654? From a quick look, the BAM
DMA transactions are timing out.

- Mani

> Thanks,
> Miquèl

-- 
மணிவண்ணன் சதாசிவம்

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

      reply	other threads:[~2023-08-07 11:50 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-08-05 17:41 [PATCH 00/10] mtd: rawnand: qcom: Bunch of fixes and cleanups Manivannan Sadhasivam
2023-08-05 17:41 ` [PATCH 01/10] mtd: rawnand: qcom: Remove superfluous initialization of "ret" Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 02/10] mtd: rawnand: qcom: Rename variables in qcom_op_cmd_mapping() Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 03/10] mtd: rawnand: qcom: Handle unsupported opcode " Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 04/10] mtd: rawnand: qcom: Fix the opcode check in qcom_check_op() Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 05/10] mtd: rawnand: qcom: Use EOPNOTSUPP instead of ENOTSUPP Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 06/10] mtd: rawnand: qcom: Wrap qcom_nand_exec_op() to 80 columns Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 07/10] mtd: rawnand: qcom: Unmap sg_list and free desc within submic_descs() Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 08/10] mtd: rawnand: qcom: Simplify the call to nand_prog_page_end_op() Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 09/10] mtd: rawnand: qcom: Do not override the error no of submit_descs() Manivannan Sadhasivam
2023-08-18 14:41   ` Miquel Raynal
2023-08-05 17:41 ` [PATCH 10/10] mtd: rawnand: qcom: Sort includes alphabetically Manivannan Sadhasivam
2023-08-06 13:53 ` [PATCH 00/10] mtd: rawnand: qcom: Bunch of fixes and cleanups Miquel Raynal
2023-08-07 11:50   ` Manivannan Sadhasivam [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230807115021.GB18257@thinkpad \
    --to=manivannan.sadhasivam@linaro.org \
    --cc=dan.carpenter@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=miquel.raynal@bootlin.com \
    --cc=quic_mdalam@quicinc.com \
    --cc=quic_srichara@quicinc.com \
    --cc=richard@nod.at \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox