linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <Tudor.Ambarus@microchip.com>
To: <michael@walle.cc>, <code@reto-schneider.ch>
Cc: <linux-mtd@lists.infradead.org>, <sr@denx.de>,
	<reto.schneider@husqvarnagroup.com>, <miquel.raynal@bootlin.com>,
	<p.yadav@ti.com>, <richard@nod.at>, <vigneshr@ti.com>,
	<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v1] mtd: spi-nor: Add support for XM25QH64C
Date: Mon, 28 Jun 2021 05:32:51 +0000	[thread overview]
Message-ID: <e3f4aa39-b70c-4684-a1d4-2f1930d54bb0@microchip.com> (raw)
In-Reply-To: <1ba367f93650cb65122acd32fb4a4159@walle.cc>

Hi, Reto,

On 6/14/21 9:56 AM, Michael Walle wrote:
> 
> NB. XMC ignores the continuation codes and this particular device will
> collide with M25PE64/M45PE64. Although I couldn't find any datasheet,
> so I don't know if these devices actually exist.

Would you please dump more bytes of ID when reading it? Maybe they put the
manufacturer continuation codes, after the manufacturer ID and the flash ID.
Or maybe they defined an extended ID, let's see what's there.

Also, does this flash has a vendor specific table? Check the JEDEC Standard No. 216D.01,
section "6.3.3 Definition of Parameter ID Field". Parameter IDS with "An MSB of 01h though
7Fh indicates a Vendor Owned table and provides the bank number of a JEDEC JEP106 assigned
Manufacturer ID." We may get the bank number from there.

Cheers,
ta

      parent reply	other threads:[~2021-06-28  5:33 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-13 12:12 [PATCH v1] mtd: spi-nor: Add support for XM25QH64C Reto Schneider
2021-06-14  6:56 ` Michael Walle
2021-06-17 11:59   ` Reto Schneider
2021-06-17 14:19     ` Michael Walle
2021-06-17 15:06       ` Reto Schneider
2021-06-28  5:17         ` Tudor.Ambarus
2021-06-28  4:55   ` Tudor.Ambarus
2021-06-30 14:15     ` Michael Walle
2021-06-30 18:28       ` Tudor.Ambarus
2021-07-01  5:25         ` Tudor.Ambarus
2021-07-01  6:15         ` Michael Walle
2021-06-28  5:32   ` Tudor.Ambarus [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=e3f4aa39-b70c-4684-a1d4-2f1930d54bb0@microchip.com \
    --to=tudor.ambarus@microchip.com \
    --cc=code@reto-schneider.ch \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=michael@walle.cc \
    --cc=miquel.raynal@bootlin.com \
    --cc=p.yadav@ti.com \
    --cc=reto.schneider@husqvarnagroup.com \
    --cc=richard@nod.at \
    --cc=sr@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).