From: Vignesh Raghavendra <vigneshr@ti.com>
To: <Tudor.Ambarus@microchip.com>, <ashish.kumar@nxp.com>,
<marek.vasut@gmail.com>, <dwmw2@infradead.org>,
<computersforpeace@gmail.com>, <miquel.raynal@bootlin.com>,
<richard@nod.at>, <linux-mtd@lists.infradead.org>
Cc: kuldeep.singh@nxp.com, linux-kernel@vger.kernel.org
Subject: Re: [EXT] Re: [Patch v3] drivers: mtd: spi-nor: Add flash property for mt25qu512a and mt35xu02g
Date: Thu, 29 Aug 2019 14:46:48 +0530 [thread overview]
Message-ID: <33252600-3823-c41e-8818-68222ab42d62@ti.com> (raw)
In-Reply-To: <a5049dca-e00e-ca28-7853-526ec7eac281@microchip.com>
Hi,
On 29/08/19 1:39 PM, Tudor.Ambarus@microchip.com wrote:
[...]
>
>>>> +
>>>> + /* Micron */
>>>> + { "mt25qu512a", INFO6(0x20bb20, 0x104400, 64 * 1024, 1024, SECT_4K |
>>>> + USE_FSR | SPI_NOR_DUAL_READ |
>>>> + SPI_NOR_QUAD_READ | SPI_NOR_4B_OPCODES)
>>>> + },
>>>
>>> I'm looking at the following datasheets: mt25qu512a [1] and n25q512a [2].
>>> Both flashes have the same Extended Device ID data. What will happen, is
>>> that you'll always hit the first valid entry, so "mt25qu512a", and you'll indicate
>>> a 'wrong' flash name for n25q512a. If there is nothing that differentiate
>>> between the two, maybe you can add a comment in the code that says that
>>> "n25q512a" was re-branded to "mt25qu512a" after the STM spin-off.
>>> Whatever solution will be, it will be better if you do it in a separate patch.
>> Hi Tudor,
>> Considering both are same, should I rename to mt25qu51a, and add SPI_NOR_4B_OPCODES or
>> Keep n25q512a, and comment about mt25qu51a and add SPI_NOR_4B_OPCODES.
>
> I see two options:
> 1/ either rename "n25q512a" to "mt25qu512a (n25q512a)" and add the
> SPI_NOR_4B_OPCODES
I would go with first option so as not to alarm the users who are used
to seeing n25q512a in kernel log. This option shows both old and new
brand names.
Regards
Vignesh
> 2/ or keep "n25q512a", add SPI_NOR_4B_OPCODES, and add a comment about
> re-branding to mt25qu512a.
>
> Which one do you like better? What about you, Vignesh?
>
>>
>> For separate patch comment you mean split mt25qu512a and mt35xu02g into 2 patch.
>
> yes, send a separate patch for mt35xu02g, as the changes are not related.
>>
>>>
>
> cut
>
>>>> + { "mt35xu02g", INFO(0x2c5b1c, 0, 128 * 1024, 2048,
>>>> + SECT_4K | USE_FSR | SPI_NOR_OCTAL_READ |
>>>> + SPI_NOR_4B_OPCODES) },
>>>
>>> Is there a public datasheet for this flash?
>> No, data sheet in under NDA, I have asked micron FAE for public data sheet, will resend after the same is published.
>>
>
> No need to wait, I'll trust you. It was better if I could verify the info, but
> if we can't, that's it. Just send a different patch for this change.
>
> Cheers,
> ta
>
--
Regards
Vignesh
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
prev parent reply other threads:[~2019-08-29 9:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-13 10:38 [Patch v3] drivers: mtd: spi-nor: Add flash property for mt25qu512a and mt35xu02g Ashish Kumar
2019-08-26 9:15 ` Ashish Kumar
2019-08-27 7:50 ` Vignesh Raghavendra
2019-08-27 10:59 ` Vignesh Raghavendra
2019-08-27 12:04 ` Tudor.Ambarus
2019-08-29 7:47 ` [EXT] " Ashish Kumar
2019-08-29 8:09 ` Tudor.Ambarus
2019-08-29 9:16 ` Vignesh Raghavendra [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=33252600-3823-c41e-8818-68222ab42d62@ti.com \
--to=vigneshr@ti.com \
--cc=Tudor.Ambarus@microchip.com \
--cc=ashish.kumar@nxp.com \
--cc=computersforpeace@gmail.com \
--cc=dwmw2@infradead.org \
--cc=kuldeep.singh@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=marek.vasut@gmail.com \
--cc=miquel.raynal@bootlin.com \
--cc=richard@nod.at \
/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).