<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
        {margin-top:0;
        margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:14pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Dear friends,</p>
<p><br>
</p>
<p>who are the other users of the 3316 module? As I understand, most other groups are using the faster 250 MHz, 14-bit version, while we are (the only) ones who are using the 125 MHz, 16-bit version.</p>
<p><br>
</p>
<p>If there is an interest to use the software version that was bugfixed as a common standard for the future, we should not be the only ones testing it. <span>😊</span></p>
<p><span>(But if we are the only user of the software for now, we could have a look if we want to adjust the output of the module better to our needs. For example, at the moment the temperature of the module is written to the data of every individual channel.
 Even though it is a global variable, whose more suitable place would be in the header(s) information.)</span></p>
<p><span><br>
</span></p>
<p><span><br>
</span></p>
<p><span>Best greetings</span></p>
<p><span>GĂŒnter</span></p>
<p><br>
</p>
<div id="x_Signature">
<div style="font-family:Tahoma; font-size:13px"></div>
</div>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von HĂ„kan T Johansson <f96hajo@chalmers.se><br>
<b>Gesendet:</b> Samstag, 27. April 2024 15:58:13<br>
<b>An:</b> Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
<b>Betreff:</b> Re: [subexp-daq] SIS3316 implementation in NURDLIB fixed</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText"><br>
I think this was the most recent mail regarding the SIS3316.<br>
<br>
To not let this drop, I rebased 'rebasing_sis3316_dma_align' on top of <br>
current master (no conflicts).  In the hope that this gets tested and <br>
then merged.<br>
<br>
Cheers,<br>
HĂ„kan<br>
<br>
<br>
<br>
On Sat, 23 Mar 2024, HĂ„kan T Johansson wrote:<br>
<br>
><br>
> Dear GĂŒnter,<br>
><br>
> I just pushed a new branch 'rebasing_sis3316_dma_align' to gitlab which <br>
> changes to use that method instead.  It also moved the calculation of <br>
> bytes_to_read up, as otherwise warnings for possibly uninitialised variable <br>
> use were generated.<br>
><br>
> I am not sure if the check for buffer overflow is taking into account other <br>
> data already added to the output.  This I think Hans is better to figure out.<br>
><br>
> Perhaps too many variables at once, but this new branch has been rebased on <br>
> top of current nurdlib master.<br>
><br>
> Cheers,<br>
> HĂ„kan<br>
><br>
><br>
><br>
><br>
> On Fri, 22 Mar 2024, Weber, Guenter Dr. wrote:<br>
><br>
>> <br>
>> Yes, I think you are right.<br>
>> <br>
>> <br>
>> ____________________________________________________________________________<br>
>> Von: subexp-daq <subexp-daq-bounces@lists.chalmers.se> im Auftrag von HĂ„kan<br>
>> T Johansson <f96hajo@chalmers.se><br>
>> Gesendet: Freitag, 22. MĂ€rz 2024 17:37:42<br>
>> An: Discuss use of Nurdlib, TRLO II, drasi and UCESB.<br>
>> Betreff: Re: [subexp-daq] SIS3316 implementation in NURDLIB fixed  <br>
>> <br>
>> Dear GĂŒnter,<br>
>> <br>
>> just had a quick look:<br>
>> <br>
>> This old code with +1 is surely not good:<br>
>> <br>
>>                 bytes_to_read =<br>
>>                     ((a_words_to_read * sizeof(uint32_t)) + 1) & ~0x7;<br>
>> <br>
>> Since sizeof(uint32_t) is 4, the addition of 1 would not have any effect.<br>
>> <br>
>> However, I'm a bit wondering about addint 8 (and 16 in the case of ~0xf).<br>
>> <br>
>> How about the following:<br>
>> <br>
>>                 bytes_to_read =<br>
>>                     ((a_words_to_read * sizeof(uint32_t)) + 0x7) & ~0x7;<br>
>> <br>
>> (and '+ 0xf' with '& ~0xf') ?<br>
>> <br>
>> That ought to bring it up to the next boundary if the read count was<br>
>> unaligned.  And the if-statements would not be needed.<br>
>> <br>
>> Cheers,<br>
>> HĂ„kan<br>
>> <br>
>> <br>
>> <br>
>> <br>
>> On Fri, 22 Mar 2024, Weber, Guenter Dr. wrote:<br>
>> <br>
>> ><br>
>> > Dear friends,<br>
>> ><br>
>> ><br>
>> > we just pushed a fixed version of the SIS3316 implementation. Compared to<br>
>> > the original version of REBASING_SIS3316 a lot of glitches in the code<br>
>> were<br>
>> > fixed in a first run a few days ago and now we finally also fixed a<br>
>> > long-standing problem with the readout of the averaged traces.<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> > Best greetings and have a nice weekend everybody<br>
>> ><br>
>> > GĂŒnter<br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> ><br>
>> <br>
>> <br>
></div>
</span></font>
</body>
</html>