New WDC Red Pro drives: WDC WD6002FFWX

[Kai Harrekilde-Petersen]

I found that my new 6TB WDC Red Pro drive wasn't recognized by drivedb.h so I created a patch so all Red Pro's can be detected.
I extracted model names from ​https://www.wdc.com/content/dam/wdc/website/downloadable_assets/eng/spec_data_sheet/2879-800022.pdf

--- drivedb.h.old  2016-10-23 17:17:15.000000000 +0200
+++ drivedb.h   2017-01-01 14:06:13.042894951 +0100
@@ -3406,7 +3406,11 @@
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
   { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+    "WDC WD([234]001|2002)FFSX-.*",
+    "", "", ""
+  },
+  { "Western Digital Red Pro", // tested with WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([568]001|[46]002)FFWX-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

[Kai Harrekilde-Petersen]

Output from "smartctl -q noserial -x" as requested

[Christian Franke]

Thanks for this patch. IMO an extra entry is not needed. I would suggest this simplified version:

--- drivedb.h	(revision 4367)
+++ drivedb.h	(working copy)
@@ -3405,8 +3405,9 @@
     "", "",
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
-  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81,
+      // WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([2-68]00[12])FF[SW]X-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

[Kai Harrekilde-Petersen]

I'm fine with either - I just wanted to avoid matching too much ;-)

Replying to chrfranke:

Thanks for this patch. IMO an extra entry is not needed. I would suggest this simplified version:

--- drivedb.h	(revision 4367)
+++ drivedb.h	(working copy)
@@ -3405,8 +3405,9 @@
     "", "",
     "-v 22,raw48,Helium_Level" // WD80EFZX
   },
-  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81
-    "WDC WD[234]001FFSX-.*",
+  { "Western Digital Red Pro", // tested with WDC WD2001FFSX-68JNUN0/81.00A81,
+      // WDC WD6002FFWX-68TZ4N0/83.H0A83
+    "WDC WD([2-68]00[12])FF[SW]X-.*",
     "", "", ""
   },
   { "Western Digital Purple", // tested with WDC WD40PURX-64GVNY0/80.00A80

[Christian Franke]

r4570, r4571.