Patchwork [mtd-www:] nand-data: add four Toshiba nand chips

login
register
mail settings
Submitter Huang Shijie
Date Aug. 13, 2013, 9:09 a.m.
Message ID <1376384964-740-1-git-send-email-b32955@freescale.com>
Download mbox | patch
Permalink /patch/266755/
State New
Headers show

Comments

Huang Shijie - Aug. 13, 2013, 9:09 a.m.
Add the items for four Toshiba nand chips.
All the four nand chips are supported by the kernel now.

Signed-off-by: Huang Shijie <b32955@freescale.com>
---
 nand-data/nanddata.csv |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
Artem Bityutskiy - Aug. 16, 2013, 2:41 p.m.
On Tue, 2013-08-13 at 17:09 +0800, Huang Shijie wrote:
> Add the items for four Toshiba nand chips.
> All the four nand chips are supported by the kernel now.
> 
> Signed-off-by: Huang Shijie <b32955@freescale.com>

I can take it if Brian is OK with this, thanks! I guess he can ask dwmw2
for mtd-www rights too.
Brian Norris - Aug. 20, 2013, 3:39 a.m.
On Tue, Aug 13, 2013 at 05:09:24PM +0800, Huang Shijie wrote:
> Add the items for four Toshiba nand chips.
> All the four nand chips are supported by the kernel now.
G> 
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
>  nand-data/nanddata.csv |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/nand-data/nanddata.csv b/nand-data/nanddata.csv
> index 20a6f86..408f0df 100644
> --- a/nand-data/nanddata.csv
> +++ b/nand-data/nanddata.csv
> @@ -192,6 +192,10 @@
>  "Toshiba","TC58NVG0S3E","0x98","0xD1","0x98D1...",128,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
>  "Toshiba","TC58NVG1S3E","0x98","0xDA","0x98DA...",256,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
>  "Toshiba","TC58NVG2S3E","0x98","0xDC","0x98DC...",512,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
> +"Toshiba","TC58NVG2S0F","0x98","0xDC","0x98dc902676150108",512,4096,224,"256 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"4-bit/512-byte","1",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
> +"Toshiba","TC58NVG3S0F","0x98","0xD3","0x98d3902676150208",1024,4096,232,"256 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"4-bit/512-byte","1",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
> +"Toshiba","TC58NVG5D2","0x98","0xD7","0x98d7943276560900",4096,8192,640,"1 MB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"40-bit/1024-byte","2",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
> +"Toshiba","TC58NVG6D2","0x98","0xDE","0x98de948276560420",8192,8192,640,"2 MB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"40-bit/1024-byte","2",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"

I would prefer the "Full ID" entries follow the capitalization style of
the other chips (all-caps). Also the "Correct OOB" entry is missing
(should be TRUE). But otherwise:

Acked-by: Brian Norris <computersforpeace@gmail.com>

I made the edits and pushed to mtd-www.git. Thanks!

(I'm not sure how to update the linux-mtd website.)

Brian

Patch

diff --git a/nand-data/nanddata.csv b/nand-data/nanddata.csv
index 20a6f86..408f0df 100644
--- a/nand-data/nanddata.csv
+++ b/nand-data/nanddata.csv
@@ -192,6 +192,10 @@ 
 "Toshiba","TC58NVG0S3E","0x98","0xD1","0x98D1...",128,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
 "Toshiba","TC58NVG1S3E","0x98","0xDA","0x98DA...",256,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
 "Toshiba","TC58NVG2S3E","0x98","0xDC","0x98DC...",512,2048,64,"128 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"1-bit/512-byte","1?",5,"Extended ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table"
+"Toshiba","TC58NVG2S0F","0x98","0xDC","0x98dc902676150108",512,4096,224,"256 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"4-bit/512-byte","1",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
+"Toshiba","TC58NVG3S0F","0x98","0xD3","0x98d3902676150208",1024,4096,232,"256 KB","x8","N/A","1st or 2nd page (1st byte in OOB = non FFh)",TRUE,TRUE,FALSE,FALSE,TRUE,FALSE,"4-bit/512-byte","1",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
+"Toshiba","TC58NVG5D2","0x98","0xD7","0x98d7943276560900",4096,8192,640,"1 MB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"40-bit/1024-byte","2",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
+"Toshiba","TC58NVG6D2","0x98","0xDE","0x98de948276560420",8192,8192,640,"2 MB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"40-bit/1024-byte","2",8,"Full ID decode",TRUE,FALSE,TRUE,TRUE,TRUE,,TRUE,TRUE,TRUE,1,1,1,"Single",,"Datasheet doesn't list full ID, only shows ID decoder table, get the full ID by testing"
 "Hynix","H27UAG8T2M","0xAD","0xD5","0xADD514B644",2048,4096,128,"512 KB","x8","N/A","Last or (last-2)th page (1st byte in OOB = non FFh)",FALSE,FALSE,TRUE,TRUE,TRUE,FALSE,"4-bit/528-byte",2,5,"Extended ID decode",FALSE,FALSE,FALSE,TRUE,TRUE,TRUE,TRUE,TRUE,FALSE,1,1,1,"Single",,
 "Samsung","K9GAG08U0E","0xEC","0xD5","0xECD584725042",2048,8192,436,"1024 KB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"24-bit/1078.5-byte",2,6,"Extended ID decode (Samsung)",FALSE,FALSE,FALSE,TRUE,TRUE,TRUE,TRUE,TRUE,FALSE,1,1,1,"Single",,"Weird 1st AND last page scan?"
 "Samsung","K9LBG08U0E","0xEC","0xD7","0xECD7C5725442",4096,8192,436,"1024 KB","x8","N/A","1st or last page (1st byte in OOB = non FFh)",TRUE,FALSE,TRUE,FALSE,TRUE,FALSE,"24-bit/1078.5-byte",2,6,"Extended ID decode (Samsung)",FALSE,FALSE,FALSE,TRUE,TRUE,TRUE,TRUE,TRUE,FALSE,1,2,1,"Single","K9GAG08U0E","Weird 1st AND last page scan?"