Fix Kindle model detection in some corner-cases (#5125)

* Fix Kindle model detection in some corner-cases

We were (currently) unaffected because we don't support the K1.
Still, this should be slightly more robust.

frontend/device/kindle/device.lua

@@ -824,8 +824,9 @@ local kindle_sn_fd = io.open("/proc/usid", "r")
 if not kindle_sn_fd then return end
 local kindle_sn = kindle_sn_fd:read()
 kindle_sn_fd:close()
-local kindle_devcode = string.sub(kindle_sn,3,4)
+-- NOTE: Attempt to sanely differentiate v1 from v2,
-local kindle_devcode_v2 = string.sub(kindle_sn,4,6)
+--       c.f., https://github.com/NiLuJe/FBInk/commit/8a1161734b3f5b4461247af461d26987f6f1632e
+local kindle_sn_lead = string.sub(kindle_sn,1,1)
 
 -- NOTE: Update me when new devices come out :)
 --       c.f., https://wiki.mobileread.com/wiki/Kindle_Serial_Numbers for identified variants
@@ -852,38 +853,47 @@ local pw4_set = Set { "0PP", "0T1", "0T2", "0T3", "0T4", "0T5", "0T6",
                   "16Q", "16R", "16S", "16T", "16U", "16V" }
 local kt4_set = Set { "10L", "0WF", "0WG", "0WH", "0WJ", "0VB" }
 
-if k2_set[kindle_devcode] then
+if kindle_sn_lead == "B" or kindle_sn_lead == "9" then
-    return Kindle2
+    local kindle_devcode = string.sub(kindle_sn,3,4)
-elseif dx_set[kindle_devcode] then
+
-    return Kindle2
+    if k2_set[kindle_devcode] then
-elseif dxg_set[kindle_devcode] then
+        return Kindle2
-    return KindleDXG
+    elseif dx_set[kindle_devcode] then
-elseif k3_set[kindle_devcode] then
+        return Kindle2
-    return Kindle3
+    elseif dxg_set[kindle_devcode] then
-elseif k4_set[kindle_devcode] then
+        return KindleDXG
-    return Kindle4
+    elseif k3_set[kindle_devcode] then
-elseif touch_set[kindle_devcode] then
+        return Kindle3
-    return KindleTouch
+    elseif k4_set[kindle_devcode] then
-elseif pw_set[kindle_devcode] then
+        return Kindle4
-    return KindlePaperWhite
+    elseif touch_set[kindle_devcode] then
-elseif pw2_set[kindle_devcode] then
+        return KindleTouch
-    return KindlePaperWhite2
+    elseif pw_set[kindle_devcode] then
-elseif kt2_set[kindle_devcode] then
+        return KindlePaperWhite
-    return KindleBasic
+    elseif pw2_set[kindle_devcode] then
-elseif kv_set[kindle_devcode] then
+        return KindlePaperWhite2
-    return KindleVoyage
+    elseif kt2_set[kindle_devcode] then
-elseif pw3_set[kindle_devcode_v2] then
+        return KindleBasic
-    return KindlePaperWhite3
+    elseif kv_set[kindle_devcode] then
-elseif koa_set[kindle_devcode_v2] then
+        return KindleVoyage
-    return KindleOasis
+    end
-elseif koa2_set[kindle_devcode_v2] then
+else
-    return KindleOasis2
+    local kindle_devcode_v2 = string.sub(kindle_sn,4,6)
-elseif kt3_set[kindle_devcode_v2] then
+
-    return KindleBasic2
+    if pw3_set[kindle_devcode_v2] then
-elseif pw4_set[kindle_devcode_v2] then
+        return KindlePaperWhite3
-    return KindlePaperWhite4
+    elseif koa_set[kindle_devcode_v2] then
-elseif kt4_set[kindle_devcode_v2] then
+        return KindleOasis
-    return KindleBasic3
+    elseif koa2_set[kindle_devcode_v2] then
+        return KindleOasis2
+    elseif kt3_set[kindle_devcode_v2] then
+        return KindleBasic2
+    elseif pw4_set[kindle_devcode_v2] then
+        return KindlePaperWhite4
+    elseif kt4_set[kindle_devcode_v2] then
+        return KindleBasic3
+    end
 end
 
-error("unknown Kindle model "..kindle_devcode.." ("..kindle_devcode_v2..")")
+local kindle_sn_prefix = string.sub(kindle_sn,1,6)
+error("unknown Kindle model: "..kindle_sn_prefix)

spec/unit/device_spec.lua

@@ -198,7 +198,7 @@ describe("device module", function()
             io.open = function(filename, mode)
                 if filename == "/proc/usid" then
                     return {
-                        read = function() return "XX13XX" end,
+                        read = function() return "B013XX" end,
                         close = function() end
                     }
                 else
@@ -219,7 +219,7 @@ describe("device module", function()
             io.open = function(filename, mode)
                 if filename == "/proc/usid" then
                     return {
-                        read = function() return "XX13XX" end,
+                        read = function() return "B013XX" end,
                         close = function() end
                     }
                 elseif filename == "/sys/class/backlight/max77696-bl/brightness" then
@@ -259,7 +259,7 @@ describe("device module", function()
                 if filename == "/proc/usid" then
                     return {
                         read = function()
-                            return "XXX0GCXXX"
+                            return "G0B0GCXXX"
                         end,
                         close = function() end
                     }