Refactor OnyxSdkLightsController to improve handling of brightness types and streamline SDK bridging

Move ONYX_GO7 to use the SDK

Author: Tukks

app/src/main/java/org/koreader/launcher/device/LightsFactory.kt

@@ -10,14 +10,14 @@ object LightsFactory {
         get() {
             return when (DeviceInfo.ID) {
                 DeviceInfo.Id.BOYUE_S62,
-                -> {
+                    -> {
                     logController("Boyue S62")
                     BoyueS62RootController()
                 }
+
                 DeviceInfo.Id.ONYX_GALILEO2,
                 DeviceInfo.Id.ONYX_GO_COLOR7,
                 DeviceInfo.Id.ONYX_GO6,
-                DeviceInfo.Id.ONYX_GO7,
                 DeviceInfo.Id.ONYX_GO7GEN2,
                 DeviceInfo.Id.ONYX_NOTE_AIR_3C,
                 DeviceInfo.Id.ONYX_NOTE_AIR_4C,
@@ -31,24 +31,27 @@ object LightsFactory {
                 DeviceInfo.Id.ONYX_POKE6,
                 DeviceInfo.Id.ONYX_TAB_ULTRA_C,
                 DeviceInfo.Id.ONYX_TAB_ULTRA_C_PRO,
-                -> {
+                    -> {
                     logController("Onyx Adb")
                     OnyxAdbLightsController()
                 }
+
                 DeviceInfo.Id.ONYX_C67,
                 DeviceInfo.Id.ONYX_MAGICBOOK,
                 DeviceInfo.Id.ONYX_MONTECRISTO3,
-                -> {
+                    -> {
                     logController("Onyx C67")
                     OnyxC67Controller()
                 }
+
                 DeviceInfo.Id.ONYX_JDREAD,
                 DeviceInfo.Id.ONYX_NOVA3_COLOR,
                 DeviceInfo.Id.TAGUS_GEA,
-                -> {
+                    -> {
                     logController("Onyx Color")
                     OnyxColorController()
                 }
+
                 DeviceInfo.Id.ONYX_DARWIN7,
                 DeviceInfo.Id.ONYX_EDISON,
                 DeviceInfo.Id.ONYX_FAUST3,
@@ -63,10 +66,11 @@ object LightsFactory {
                 DeviceInfo.Id.ONYX_POKE2,
                 DeviceInfo.Id.ONYX_POKE3,
                 DeviceInfo.Id.ONYX_POKE_PRO,
-                -> {
+                    -> {
                     logController("Onyx/Qualcomm")
                     OnyxWarmthController()
                 }
+
                 DeviceInfo.Id.ONYX_DARWIN5,
                 DeviceInfo.Id.ONYX_DARWIN9,
                 DeviceInfo.Id.ONYX_LEAF2,
@@ -82,10 +86,12 @@ object LightsFactory {
                 DeviceInfo.Id.ONYX_POKE4LITE,
                 DeviceInfo.Id.ONYX_TAB_ULTRA,
                 DeviceInfo.Id.STORYTEL_READER2,
-                -> {
+                DeviceInfo.Id.ONYX_GO7,
+                    -> {
                     logController("Onyx/Sdk")
                     OnyxSdkLightsController()
                 }
+
                 DeviceInfo.Id.CREMA_0710C,
                 DeviceInfo.Id.CREMA_CARTA_G,
                 DeviceInfo.Id.MEEBOOK_P6,
@@ -95,35 +101,40 @@ object LightsFactory {
                 DeviceInfo.Id.TOLINO_SHINE3,
                 DeviceInfo.Id.TOLINO_VISION4,
                 DeviceInfo.Id.TOLINO_VISION5,
-                -> {
+                    -> {
                     logController("TolinoNTX")
                     TolinoNtxController()
                 }
+
                 DeviceInfo.Id.CREMA_0660L,
                 DeviceInfo.Id.HISENSE_TOUCH_LITE,
                 DeviceInfo.Id.TOLINO_PAGE2,
-                -> {
+                    -> {
                     logController("TolinoNTXNoWarmth")
                     TolinoNtxNoWarmthController()
                 }
+
                 DeviceInfo.Id.NOOK_GL4,
                 DeviceInfo.Id.TOLINO_EPOS2,
-                -> {
+                    -> {
                     logController("TolinoRoot")
                     TolinoRootController()
                 }
+
                 DeviceInfo.Id.TOLINO_EPOS3,
                 DeviceInfo.Id.TOLINO_VISION6,
                 DeviceInfo.Id.TOLINO_SHINE4,
-                -> {
+                    -> {
                     logController("TolinoB300Controller")
                     TolinoB300Controller()
                 }
+
                 DeviceInfo.Id.ONYX_PALMA2_PRO,
-                -> {
+                    -> {
                     logController("OnyxPalma2Pro")
                     OnyxPalma2ProController()
                 }
+
                 else -> {
                     logController("Generic")
                     GenericController()
@@ -132,7 +143,11 @@ object LightsFactory {
         }
 
     private fun logController(name: String?) {
-        Log.i(TAG, String.format(Locale.US,
-            "Using %s driver", name))
+        Log.i(
+            TAG, String.format(
+                Locale.US,
+                "Using %s driver", name
+            )
+        )
     }
 }