[PATCH v3 2/3] mtd: nand: add new parsing rule for Hynix's H27UBG8T2CTR

[Brian Norris]

On Fri, Mar 07, 2014 at 04:00:13PM +0800, Huang Shijie wrote:
> 于 2014年03月07日 15:46, Brian Norris 写道:
> >Are you sure you need to use the device ID (0xD7)? It's best if we don't
> >have to constrain ourselves to particular flash sizes here... Are there
> >Hynix MLC that don't follow the id_data[5] pattern for differentiating
> >technology types (>20nm vs<=20nm)? I'll have to check my datasheets
> >when I get back in the office.
> >
> The H27UCG8T2ATR is also a 20nm chip whose device ID is 0xDE. Its
> id_data[5] is 0xc4.
> But it also use the old parsing rule: the page size is the multiple of 2K.
> 
> that's why i checked the device ID.

Ugh, what a pain :( What's the point of a decode table, if we don't know
what chips to use it for?

Perhaps could it better to check for dev_id != 0xDE instead? Hopefully
there would be fewer 20nm chips that use the old table than those that
use the new.

I'll try to remember to take a look tomorrow to see how this matches
with the Hynix parts I've seen.

Brian