storagecells for jungfrauvchip1.1: cannot set addl, storagecelldealy and storagecell start is only from 0-3

python/slsdet/detector.py

@@ -2040,7 +2040,7 @@ class Detector(CppDetectorApi):
         [Jungfrau] Number of additional storage cells. 
         Note
         ----
-        For advanced users only. \n
+        Only for chip v1.0. For advanced users only. \n
         Options: 0 - 15. Default is 0.
         The #images = #frames x #triggers x (#storagecells + 1)
         """
@@ -2059,7 +2059,7 @@ class Detector(CppDetectorApi):
         Note
         ----
         For advanced users only.
-        Options 0-15. Default is 15. \n
+        Options 0-max. max is 15 (default) for chipv1.0 and 3 (default) for chipv1.1. \n
         """
         return self.getStorageCellStart()
 
@@ -2073,7 +2073,7 @@ class Detector(CppDetectorApi):
         [Jungfrau] Additional time delay between 2 consecutive exposures in burst mode, accepts either a value in seconds or datetime.timedelta
         Note
         -----
-        For advanced users only \n
+        Only applicable for chipv1.0. For advanced users only \n
         Value: 0-1638375 ns (resolution of 25ns) \n
         :getter: always returns in seconds. To get in datetime.delta, use getStorageCellDelay
 

slsDetectorServers/jungfrauDetectorServer/slsDetectorFunctionList.c

@@ -458,8 +458,13 @@ void setupDetector() {
     setPeriod(DEFAULT_PERIOD);
     setDelayAfterTrigger(DEFAULT_DELAY);
     setNumAdditionalStorageCells(DEFAULT_NUM_STRG_CLLS);
-    setStorageCellDelay(DEFAULT_STRG_CLL_DLY);
+    if (getChipVersion() == 11) {
-    selectStoragecellStart(DEFAULT_STRG_CLL_STRT);
+        selectStoragecellStart(DEFAULT_STRG_CLL_STRT_CHIP11);
+    } else {
+        selectStoragecellStart(DEFAULT_STRG_CLL_STRT);
+        // not applicable for chipv1.1
+        setStorageCellDelay(DEFAULT_STRG_CLL_DLY);
+    }
     /*setClockDivider(RUN_CLK, HALF_SPEED); depends if all the previous stuff
      * works*/
     setTiming(DEFAULT_TIMING_MODE);
@@ -773,14 +778,30 @@ uint32_t getADCInvertRegister() {
 
 /* parameters - timer */
 int selectStoragecellStart(int pos) {
+    int value = pos;
+    uint32_t addr = DAQ_REG;
+    uint32_t mask = DAQ_STRG_CELL_SLCT_MSK;
+    int offset = DAQ_STRG_CELL_SLCT_OFST;
+    if (getChipVersion() == 11) {
+        value = 1 << pos;
+        addr = CONFIG_V11_REG;
+        mask = CONFIG_V11_STRG_CLL_MSK;
+        offset = CONFIG_V11_STRG_CLL_OFST;
+    }
     if (pos >= 0) {
         LOG(logINFO, ("Setting storage cell start: %d\n", pos));
-        bus_w(DAQ_REG, bus_r(DAQ_REG) & ~DAQ_STRG_CELL_SLCT_MSK);
+        bus_w(addr, bus_r(addr) & ~mask);
-        bus_w(DAQ_REG, bus_r(DAQ_REG) | ((pos << DAQ_STRG_CELL_SLCT_OFST) &
+        bus_w(addr, bus_r(addr) | ((value << offset) & mask));
-                                         DAQ_STRG_CELL_SLCT_MSK));
+    }
+    return ((bus_r(addr) & mask) >> offset);
+}
+
+int getMaxStoragecellStart() {
+    if (getChipVersion() == 11) {
+        return MAX_STORAGE_CELL_CHIP11_VAL;
+    } else {
+        return MAX_STORAGE_CELL_VAL;
     }
-    return ((bus_r(DAQ_REG) & DAQ_STRG_CELL_SLCT_MSK) >>
-            DAQ_STRG_CELL_SLCT_OFST);
 }
 
 int setNextFrameNumber(uint64_t value) {

slsDetectorServers/jungfrauDetectorServer/slsDetectorServer_defs.h

@@ -102,6 +102,7 @@ enum CLKINDEX { RUN_CLK, ADC_CLK, DBIT_CLK, NUM_CLOCKS };
 #define DEFAULT_TMP_THRSHLD           (65 * 1000) // milli degree Celsius
 #define DEFAULT_NUM_STRG_CLLS         (0)
 #define DEFAULT_STRG_CLL_STRT         (0xf)
+#define DEFAULT_STRG_CLL_STRT_CHIP11  (0x3)
 #define DEFAULT_STRG_CLL_DLY          (0)
 
 #define HIGHVOLTAGE_MIN (60)
@@ -113,6 +114,7 @@ enum CLKINDEX { RUN_CLK, ADC_CLK, DBIT_CLK, NUM_CLOCKS };
 #define MAX_TIMESLOT_VAL            (0x1F)
 #define MAX_THRESHOLD_TEMP_VAL      (127999) // millidegrees
 #define MAX_STORAGE_CELL_VAL        (15)     // 0xF
+#define MAX_STORAGE_CELL_CHIP11_VAL (3)     
 #define MAX_STORAGE_CELL_DLY_NS_VAL (ASIC_CTRL_EXPSRE_TMR_MAX_VAL)
 #define ACQ_TIME_MIN_CLOCK          (2)
 

slsDetectorServers/slsDetectorServer/include/slsDetectorFunctionList.h

@@ -202,6 +202,7 @@ int getReadoutMode();
 // parameters - timer
 #ifdef JUNGFRAUD
 int selectStoragecellStart(int pos);
+int getMaxStoragecellStart();
 #endif
 #if defined(JUNGFRAUD) || defined(EIGERD)
 int setNextFrameNumber(uint64_t value);

slsDetectorServers/slsDetectorServer/src/slsDetectorServer_funcs.c

@@ -2036,10 +2036,14 @@ int set_num_additional_storage_cells(int file_des) {
 #else
     // only set
     if (Server_VerifyLock() == OK) {
-        if (arg > MAX_STORAGE_CELL_VAL) {
+        if (getChipVersion() == 11) {
+            ret = FAIL;
+            sprintf(mess, "Cannot set addl. number of storage cells for chip v1.1\n");
+            LOG(logERROR, (mess));
+        } else if (arg > getMaxStoragecellStart()) {
             ret = FAIL;
             sprintf(mess, "Max Storage cell number should not exceed %d\n",
-                    MAX_STORAGE_CELL_VAL);
+                    getMaxStoragecellStart());
             LOG(logERROR, (mess));
         } else {
             setNumAdditionalStorageCells(arg);
@@ -2483,9 +2487,15 @@ int get_storage_cell_delay(int file_des) {
     functionNotImplemented();
 #else
     // get only
-    retval = getStorageCellDelay();
+    if (getChipVersion() == 11) {
-    LOG(logDEBUG1,
+        ret = FAIL;
+        strcpy(mess, "Storage cell delay is not applicable for chipv 1.1\n");
+        LOG(logERROR, (mess));
+    } else {
+        retval = getStorageCellDelay();
+        LOG(logDEBUG1,
         ("retval storage cell delay %lld ns\n", (long long int)retval));
+    }
 #endif
     return Server_SendResult(file_des, INT64, &retval, sizeof(retval));
 }
@@ -2505,7 +2515,11 @@ int set_storage_cell_delay(int file_des) {
 #else
     // only set
     if (Server_VerifyLock() == OK) {
-        if (arg > MAX_STORAGE_CELL_DLY_NS_VAL) {
+        if (getChipVersion() == 11) {
+            ret = FAIL;
+            strcpy(mess, "Storage cell delay is not applicable for chipv 1.1\n");
+            LOG(logERROR, (mess));
+        } else if (arg > MAX_STORAGE_CELL_DLY_NS_VAL) {
             ret = FAIL;
             sprintf(mess,
                     "Max Storage cell delay value should not exceed %lld ns\n",
@@ -4007,9 +4021,9 @@ int storage_cell_start(int file_des) {
 #else
     // set & get
     if ((arg == GET_FLAG) || (Server_VerifyLock() == OK)) {
-        if (arg > MAX_STORAGE_CELL_VAL) {
+        if (arg > getMaxStoragecellStart()) {
             ret = FAIL;
-            strcpy(mess, "Max Storage cell number should not exceed 15\n");
+            strcpy(mess, "Max Storage cell number should not exceed %d\n", getMaxStoragecellStart());
             LOG(logERROR, (mess));
         } else {
             retval = selectStoragecellStart(arg);

slsDetectorSoftware/include/sls/Detector.h

@@ -1144,7 +1144,7 @@ class Detector {
     Result<int> getNumberOfAdditionalStorageCells(Positions pos = {}) const;
 
     /** [Jungfrau] Advanced \n
-     * Options: 0 - 15. Default: 0. \n
+     * Only for chipv1.0. Options: 0 - 15. Default: 0. \n
      * The #images = #frames x #triggers x (#storagecells + 1) */
     void setNumberOfAdditionalStorageCells(int value);
 
@@ -1152,7 +1152,7 @@ class Detector {
     Result<int> getStorageCellStart(Positions pos = {}) const;
 
     /** [Jungfrau] Advanced. Sets the storage cell storing the first acquisition
-     * of the series. Options: 0-15. Default: 15.
+     * of the series. Options: 0-max. max is 15 (default) for chipv1.0 and 3 (default) for chipv1.1.
      */
     void setStorageCellStart(int cell, Positions pos = {});
 
@@ -1160,7 +1160,8 @@ class Detector {
     Result<ns> getStorageCellDelay(Positions pos = {}) const;
 
     /** [Jungfrau] Advanced \n Additional time delay between 2 consecutive
-     * exposures in burst mode. \n Options: (0-1638375 ns (resolution of 25ns)
+     * exposures in burst mode. \n Options: (0-1638375 ns (resolution of 25ns)\n
+     * Only applicable for chipv1.0. 
      */
     void setStorageCellDelay(ns value, Positions pos = {});
     ///@{

slsDetectorSoftware/src/CmdProxy.h

@@ -1848,21 +1848,21 @@ class CmdProxy {
     INTEGER_COMMAND_SET_NOID_GET_ID(
         storagecells, getNumberOfAdditionalStorageCells,
         setNumberOfAdditionalStorageCells, StringTo<int>,
-        "[0-15]\n\t[Jungfrau] Number of additional storage cells. Default is "
+        "[0-15]\n\t[Jungfrau] Only for chipv1.0. Number of additional storage cells. Default is "
         "0. For advanced users only. \n\tThe #images = #frames x #triggers x "
         "(#storagecells + 1).");
 
     INTEGER_COMMAND_VEC_ID(
         storagecell_start, getStorageCellStart, setStorageCellStart,
         StringTo<int>,
-        "[0-15]\n\t[Jungfrau] Storage cell that stores the first acquisition "
+        "[0-max]\n\t[Jungfrau] Storage cell that stores the first acquisition "
-        "of the series. Default is 15. For advanced users only.");
+        "of the series. max is 15 (default) for chipv1.0 and 3 (default) for chipv1.1. For advanced users only.");
 
     TIME_COMMAND(
         storagecell_delay, getStorageCellDelay, setStorageCellDelay,
         "[duration (0-1638375 ns)] [(optional unit) ns|us|ms|s]\n\t[Jungfrau] "
         "Additional time delay between 2 consecutive exposures in burst mode "
-        "(resolution of 25ns). For advanced users only.");
+        "(resolution of 25ns). Only applicable for chipv1.0. For advanced users only.");
 
     /* Gotthard Specific */
     TIME_GET_COMMAND(exptimel, getExptimeLeft,