All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Santhosh Kumar K <s-k6@ti.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>,
	u-boot@lists.denx.de, Prasanth Babu Mantena <p-mantena@ti.com>
Subject: Re: Problem with "Cleanup dma device in spl and move dma channel[0]" series and am64
Date: Tue, 31 Dec 2024 07:53:43 -0600	[thread overview]
Message-ID: <20241231135343.GH3476@bill-the-cat> (raw)
In-Reply-To: <228b234e-9483-4102-8cf3-5cf8c893381d@ti.com>

[-- Attachment #1: Type: text/plain, Size: 1493 bytes --]

On Tue, Dec 31, 2024 at 11:58:29AM +0530, Santhosh Kumar K wrote:
> Hi, Tom,
> 
> On 30/12/24 23:12, Tom Rini wrote:
> > Hey all,
> > 
> > As part of testing the SPI fixes, I tracked down a problem on my AM64X
> > SR1.0 GP to the above series. With the series applied, I get:
> > => sf probe
> > jedec_spi_nor flash@0: non-uniform erase sector maps are not supported yet.
> > SF: Detected s28hs512t with page size 256 Bytes, erase size 256 KiB, total 64 MiB
> > => sf read $loadaddr 0 4000
> > device 0 offset 0x0, size 0x4000
> > ti-udma dma-controller@485c0000: k3_dmaring Ring probed rings:288,
> > sci-dev-id:30
> > ti-udma dma-controller@485c0000: dma-ring-reset-quirk: disabled
> > ............................... (never ends).
> > 
> > With the series reverted:
> > => sf probe
> > SF: Detected s28hs512t with page size 256 Bytes, erase size 256 KiB, total 64 MiB
> > => sf read $loadaddr 0 4000
> > device 0 offset 0x0, size 0x4000
> > SF: 16384 bytes @ 0x0 Read: OK
> > =>
> 
> A fix for this issue has already been provided (not merged yet).
> 
> Patch:
> https://lore.kernel.org/u-boot/20241218130045.2070314-1-p-mantena@ti.com/
> 
> With this fix:
> => sf probe
> SF: Detected s28hs512t with page size 256 Bytes, erase size 256 KiB, total
> 64 MiB
> => sf read $loadaddr 0 4000
> device 0 offset 0x0, size 0x4000
> SF: 16384 bytes @ 0x0 Read: OK
> =>
> 

Thanks. And that is the kind of change you can remind me I've missed :)

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

      reply	other threads:[~2024-12-31 13:53 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-30 17:42 Problem with "Cleanup dma device in spl and move dma channel[0]" series and am64 Tom Rini
2024-12-31  6:28 ` Santhosh Kumar K
2024-12-31 13:53   ` Tom Rini [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=20241231135343.GH3476@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=p-mantena@ti.com \
    --cc=s-k6@ti.com \
    --cc=u-boot@lists.denx.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.