Added support for Frame rev8 PCBs with camera on LDO rail

Fixed extended VTS registers for manual shutter mode
Improved camera test scripts for better quality

Author: siliconwitch

source/application/lua_libraries/camera.c

@@ -711,6 +711,18 @@ static int lua_camera_set_shutter(lua_State *L)
         return luaL_error(L, "shutter must be between 4 and 16383");
     }
 
+    // If shutter is longer than frame length (VTS register)
+    if (shutter > 0x32A)
+    {
+        check_error(i2c_write(CAMERA, 0x380E, 0xFF, shutter >> 8).fail);
+        check_error(i2c_write(CAMERA, 0x380F, 0xFF, shutter).fail);
+    }
+    else
+    {
+        check_error(i2c_write(CAMERA, 0x380E, 0xFF, 0x03).fail);
+        check_error(i2c_write(CAMERA, 0x380F, 0xFF, 0x22).fail);
+    }
+
     check_error(i2c_write(CAMERA, 0x3500, 0x03, shutter >> 12).fail);
     check_error(i2c_write(CAMERA, 0x3501, 0xFF, shutter >> 4).fail);
     check_error(i2c_write(CAMERA, 0x3502, 0xF0, shutter << 4).fail);

source/application/main.c

@@ -58,15 +58,21 @@ static void set_power_rails(bool enable)
         // Turn on LDO0 (1.2V rail)
         check_error(i2c_write(PMIC, 0x39, 0x07, 0x06).fail);
 
-        // Turn on SBB2 (2.7V rail) with 333mA limit
+        // Turn on LDO1 (2.8V rail)
+        check_error(i2c_write(PMIC, 0x3B, 0x07, 0x06).fail);
+
+        // Turn on SBB2 (2.95V rail) with 333mA limit
         check_error(i2c_write(PMIC, 0x2E, 0x37, 0x36).fail);
 
         return;
     }
 
-    // Turn off SBB2 (2.7V rail) with active discharge resistor on
+    // Turn off SBB2 (2.95V rail) with active discharge resistor on
     check_error(i2c_write(PMIC, 0x2E, 0x0F, 0x0C).fail);
 
+    // Turn off LDO1 (2.8V rail)
+    check_error(i2c_write(PMIC, 0x3B, 0x0F, 0x0C).fail);
+
     // Turn off LDO0 (1.2V rail)
     check_error(i2c_write(PMIC, 0x39, 0x0F, 0x0C).fail);
 
@@ -216,12 +222,15 @@ static void hardware_setup(bool *factory_reset)
         // Set SBB0 to 1.0V
         check_error(i2c_write(PMIC, 0x29, 0x7F, 0x04).fail);
 
-        // Set SBB2 to 2.7V
-        check_error(i2c_write(PMIC, 0x2D, 0x7F, 0x26).fail);
+        // Set SBB2 to 2.95V
+        check_error(i2c_write(PMIC, 0x2D, 0x7F, 0x2B).fail);
 
         // Set LDO0 to 1.2V
         check_error(i2c_write(PMIC, 0x38, 0x7F, 0x10).fail);
 
+        // Set LDO1 to 2.8V
+        check_error(i2c_write(PMIC, 0x3A, 0x7F, 0x50).fail);
+
         // Turn/keep off FPGA before FPGA configuration
         set_power_rails(false);
 

tests/test_camera_and_audio.py

@@ -683,6 +683,9 @@ async def main():
     local state = 'CAPTURE'
     local state_time = 0
     
+    frame.display.power_save(true)
+    frame.camera.power_save(false)
+
     frame.microphone.stop()
     frame.microphone.start { }
 
@@ -716,7 +719,7 @@ async def main():
         end
 
         if frame.time.utc() - last_autoexp_time > 0.1 then
-            frame.camera.auto { }
+            frame.camera.auto { analog_gain_limit=40, shutter_limit=2500 }
             last_autoexp_time = frame.time.utc()
         end
 

tests/test_camera_fps.py

@@ -651,8 +651,8 @@ def receive_data(data):
 
     image_buffer += data[1:]
     print(
-        f"Received {str(len(image_buffer)-1)} bytes. FPS = {fps}. Press enter to finish      ",
-        end="\r",
+        f"\rReceived {str(len(image_buffer)-1)} bytes. FPS = {fps}. Press enter to finish      ",
+        end="",
     )
 
 
@@ -666,6 +666,9 @@ async def main():
     frame.display.power_save(true)
     frame.camera.power_save(false)
 
+    -- frame.camera.set_gain(40)
+    -- frame.camera.set_shutter(2500)
+
     while true do
         if state == 'CAPTURE' then
             frame.camera.capture { quality_factor = 50 }
@@ -698,10 +701,10 @@ async def main():
         end
 
         if frame.time.utc() - last_autoexp_time > 0.1 then
-            local stats = frame.camera.auto { analog_gain_limit=30,  shutter_limit=3200 }
-            -- if stats ~= nil then 
-            --     print('gain = '..stats['analog_gain']..', shutter = '..stats['shutter'])
-            -- end
+            local stats = frame.camera.auto { analog_gain_limit=40, shutter_limit=2500 }
+            if stats ~= nil then 
+                print('gain = '..stats['analog_gain']..', shutter = '..stats['shutter']..'        ')
+            end
             last_autoexp_time = frame.time.utc()
         end
     end
@@ -710,7 +713,8 @@ async def main():
     # Connect to bluetooth and upload file
     b = Bluetooth()
     await b.connect(
-        print_response_handler=lambda s: print(s), data_response_handler=receive_data
+        # print_response_handler=lambda s: print("\r" + s, end=""),
+        data_response_handler=receive_data,
     )
 
     print("Uploading script")