From fe088d7bba189d4b0edf584b69c8b3974a1dce5a Mon Sep 17 00:00:00 2001 From: Sebastien Bourdeauducq Date: Fri, 29 Dec 2017 15:05:12 +0800 Subject: [PATCH] update errata --- firmware/src/loop_cathode.rs | 4 ++-- hardware/errata.md | 6 ++++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/firmware/src/loop_cathode.rs b/firmware/src/loop_cathode.rs index 1e7f61e..9e56221 100644 --- a/firmware/src/loop_cathode.rs +++ b/firmware/src/loop_cathode.rs @@ -8,7 +8,7 @@ const FBI_PID_PARAMETERS: pid::Parameters = pid::Parameters { ki: 20.0, kd: 10.0, output_min: 0.5, - output_max: 3.1, + output_max: 10.0, integral_min: -0.1, integral_max: 0.1 }; @@ -98,7 +98,7 @@ impl Controller { let fv = fv_sample as f32/board::FV_ADC_GAIN; let fv_pwm_duty = self.fv_pid.update(fv); - board::set_fv_pwm(fv_pwm_duty as u16); + board::set_fv_pwm(120); self.last_fv = Some(fv); self.last_fbv = Some(fbv_sample as f32/board::FBV_ADC_GAIN); diff --git a/hardware/errata.md b/hardware/errata.md index 493fdff..be5e52d 100644 --- a/hardware/errata.md +++ b/hardware/errata.md @@ -30,10 +30,9 @@ Rev 1 * review values of R226 and R113 * C100 -> 47pF NP0 AVX, R107 -> 100K, C101 -> 0.1uF * use crystal type recommended by the MCU datasheet -* 22k FBI current resistor -> 10k? * add connector for OLED display? * enlarge holes to fit M3 screws comfortably -* add 15M (TBC) resistor between A and FIL- +* add 15M resistor between A and FIL- * power U200 from a small negative voltage instead of GND * review choice of filament flyback output diode * invert LED position so that the Ethernet LED is closest to the connector @@ -42,3 +41,6 @@ Rev 1 * change D351 model to MBR1645 * add snubber on D351 * add heatsink (Seifert-KK633) to D351 +* move D351 further away from transformer for heatsink clearance +* add plated-through holes and pads for mounting rods of BNC connector +* simplify FBI circuit, 2 ranges only (typ. 8mA and 45mA), remove diodes and neglect MOSFET leakage