From 064ea51863a4d873efef2cf9bfc557233a7057fb Mon Sep 17 00:00:00 2001
From: Selene ToyKeeper
Date: Tue, 28 Nov 2023 12:40:16 -0700
Subject: fixed FW3X thermal regulation

It had no minimum thermal stepdown defined, so it used MAX_7135 by default,
which was level 149/150 ... thus virtually no regulation.

I also made it easier to switch between internal and external temperature
sensors by just commenting out a few lines in the hwdef.h.
---
 hw/lumintop/fw3x-lume1/hwdef.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'hw/lumintop/fw3x-lume1/hwdef.c')

diff --git a/hw/lumintop/fw3x-lume1/hwdef.c b/hw/lumintop/fw3x-lume1/hwdef.c
index 791a080..306a58c 100644
--- a/hw/lumintop/fw3x-lume1/hwdef.c
+++ b/hw/lumintop/fw3x-lume1/hwdef.c
@@ -83,10 +83,8 @@ bool gradual_tick_main(uint8_t gt) {
     // 150/150 is full FET + zero regulated,
     // 149/150 is zero FET + full regulated,
     // so don't try to gradually adjust between
-    if ((RAMP_SIZE == actual_level) || (gt >= RAMP_SIZE-1)) {
-        set_level(gt + 1);
-        return true;
-    }
+    // if target is in the top 2 levels, just let the parent handle it
+    if (gt >= RAMP_SIZE-2) return true;
 
     PWM1_DATATYPE ch1 = PWM1_GET(gt);
 
@@ -110,6 +108,8 @@ bool gradual_tick_main(uint8_t gt) {
 
 ////////// external temperature sensor //////////
 
+#ifdef ADMUX_THERM_EXTERNAL_SENSOR
+
 void hwdef_set_admux_therm() {
     // put the ADC in temperature mode
     // ADCSRB: [VDEN, VDPD, -, -, ADLAR, ADTS2, ADTS1, ADTS0]
@@ -140,3 +140,5 @@ uint16_t temp_raw2cooked(uint16_t measurement) {
     return k6;
 }
 
+#endif
+
-- 
cgit v1.2.3