Resolve L2SDP-532
Closes L2SDP-532
Merge request reports
Activity
requested review from @kooistra
assigned to @walle
63 68 CONSTANT c_mem_reg_temp_nof_data : NATURAL := 1; 64 69 CONSTANT c_mem_reg_temp_data : t_c_mem := (c_mem_reg_rd_latency, c_mem_reg_temp_adr_w , c_mem_reg_temp_dat_w , c_mem_reg_temp_nof_data, 'X'); 65 70 71 -- temp = (693 * adc)/1024 - 265 => adc = (temp + 265)*1024/693 Formule hier op regel 27 en 71 heeft 693 en 265, op regel 120 (en in ICD) staat 708 en 273, welke is correct?
DIt is ook van belang voor het FPGA_temperature_R punt in ICD https://support.astron.nl/confluence/display/L2M/L3+SDP+Decision%3A+SDPTR+-+SDPFW+register+map+interface, kun je daar zonodig een nieuw ticket (Fix FPGA_temperature_R calculation in SDPTR) voor maken?
Volgens de datasheet op pagina 10 (link op regel 25) zou het dus moeten zijn zoals het hier staat. Ik weet alleen niet waar de berekening vandaan komt zoals hij er eerder in stond.
Het is uiteindelijk maar een klein verschil tussen de twee.
Edited by Reinier van der Walle
173 181 BEGIN 174 182 IF mm_rst = '1' THEN 175 183 controller_csr_write <= '0'; 184 start_sense_mm <= '0'; 185 start_sense_mm_d1 <= '0'; 186 start_sense_mm_d2 <= '0'; 176 187 ELSIF rising_edge(mm_clk) THEN 177 188 start_sense_mm <= start_sense; 178 189 start_sense_mm_d1 <= start_sense_mm; 179 190 start_sense_mm_d2 <= start_sense_mm_d1; 180 191 if start_sense_mm_d1 = '1' and start_sense_mm_d2 = '0' then 181 192 controller_csr_write <= '1'; 182 else 193 else changed this line in version 2 of the diff
84 92 BEGIN 85 93 86 94 -- temperature sensor 87 95 temp_alarm <= '1' WHEN (UNSIGNED(temp_data) > UNSIGNED(c_temp_high_raw)) ELSE '0'; mentioned in commit ca3e313c
unassigned @walle