- 30 Sep, 2022 3 commits
-
-
Vaibhav Gupta authored
In "main()", the "strncat()" is given the size equal to the destination size. Thus, in worst case scenario, the string which needs to be suffixed, will be truncated. Fix the warning. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
In "main()", the "strncat()" is given the size equal to the destination size. Thus, in worst case scenario, the string which needs to be suffixed, will be truncated. Fix the warning. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
Variable "basepath" is defined as "/sys/bus/zio/devices/" and variable "attribute"'s array element are in the form of "/xyz". Thus, when the two are concatenated, the result is: "/sys/bus/zio/devices//xyz". Thus remove extra forward-slash. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
- 29 Sep, 2022 2 commits
-
-
Vaibhav Gupta authored
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
"strncat" needs the size of buffer that has to be concatenated, as its 3rd argument. In function, "int fau_write_attribute(enum fau_attribute attr, uint32_t val)" "strncat" when called for the second time is being overflowed. Fix that. The goal that this code is trying to achieve can be done in one line with "snprintf". Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
- 26 Jul, 2022 5 commits
-
-
Vaibhav Gupta authored
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
In function 'fa_calib_gain_update()', the first argument ('fa') passed to 'from_timer()' is not declared anywhere, and thus the build throws compilation error. Fix this by declaring the variable and then defining it by the value produced by 'from_timer()'. Then we can use the same variable for the next steps. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
Variable name 'LINUX' is very ambiguous for its purpose in the makefile. Also, this project builds as a part of COHT project which uses another variable name 'KERNELSRC' for the same purpose. Hence, this change makes this project uniform with others. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
The value of this variable is initialized when these drivers are build as a part of entire COHT project. But if these drivers are build independently, but inside COHT source tree, there must be a default value. Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
Vaibhav Gupta authored
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
-
- 11 Jan, 2022 5 commits
-
-
Federico Vaga authored
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
-
Tristan Gingold authored
-
Tristan Gingold authored
-
- 26 Oct, 2021 1 commit
-
-
Federico Vaga authored
On SVEC carriers we do need to handle shots one by one to be able to reconfigure the VME DMA window. To handle it we make use of the completion mechanism: we transfer a shot and we wait its completion, and then we do the next one. The completion goes toghether with the ZIO block, therefore the completion variable is stored togheter with the block. However, in the fa_dma_complete() code we were cleaning up the block too soon, so the completion variable is unusable. Anticipating the completion fixes the problem. Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
- 10 Sep, 2021 4 commits
-
-
Federico Vaga authored
6.0.0 - 2021-09-10 ================== Added ----- - hdl: configurable auto byte swap in hardware, useful for SVEC to reduce software complexity - hdl,sw: DMA data is always little-endian - sw: software version validation against FPGA version - bld: flawfinder check on software tools Changed ------- - sw: offsets are not anymore in uV but they are just raw values Fixed ----- - sw: security fixes detected by flawfinder - sw: fixes detected by checkpatch.pl - sw: style fixes detected by checkpatch.pl - sw: improve compatibility with newer ( > 3.10) Linux kernel versions
-
Federico Vaga authored
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
This reverts commit e2d435fb. Apparently the temperature issue has nothing to do with the signal stabilization in FPGA
-
- 08 Sep, 2021 2 commits
-
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Tristan Gingold authored
-
- 07 Sep, 2021 3 commits
-
-
Federico Vaga authored
6.0.0 - 2021-09-07 ================== Added ----- - hdl: configurable auto byte swap in hardware, useful for SVEC to reduce software complexity - hdl,sw: DMA data is always little-endian - sw: software version validation against FPGA version - bld: flawfinder check on software tools Changed ------- - sw: offsets are not anymore in uV but they are just raw values Fixed ----- - sw: security fixes detected by flawfinder - sw: fixes detected by checkpatch.pl - sw: style fixes detected by checkpatch.pl - sw: improve compatibility with newer ( > 3.10) Linux kernel versions
-
Federico Vaga authored
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
- 06 Sep, 2021 2 commits
-
-
Federico Vaga authored
The serdes does not lock if the ADC chip is not configured before. We did not need this before because we were not changing the ADC chip operational mode, but now we ask the chip to use a different number of bits. Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
-
- 03 Sep, 2021 1 commit
-
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
- 31 Aug, 2021 12 commits
-
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
This goes against what described in the ZIO framework, but it is actually a source of problems because the framework is largely unmaintained. With this patch the driver play only with raw values. It will be part of the library duty to propose a different convention. Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
The data coming out from the FMC-ADC-110M FPGA is always using the little-endian byte order. This means that we need to fix the byte order only on big-endian CPUs. This allows to improve performances on SVEC-based designs since there is no need anymore to fix the endianness in software becuase it is already done in hardware. Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Federico Vaga authored
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Mathis MARION authored
`get_seconds` disappeared in kernel 5.11 and onwards, it is now required to call `ktime_get_real_seconds`. Signed-off-by: Gwenhael GOAVEC <gwenhael.goavec@femto-st.fr> Signed-off-by: Mathis MARION <mathis.marion@grenoble-inp.org>
-
Mathis MARION authored
`setup_timer` is no longer available in kernel 4.15 and onwards, in favor of `timer_setup`. Before, we could pass an argument to the callback function as an unsigned int (generally used with a pointer cast), but now the timer itself is passed and we retrieve the container struct with a new macro `from_timer` which is uses `container_of`. The flags parameter can be set as 0 for the same behavior. Signed-off-by: Gwenhael GOAVEC <gwenhael.goavec@femto-st.fr> Signed-off-by: Mathis MARION <mathis.marion@grenoble-inp.org> Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
-
Mathis MARION authored
The signature for `__sg_alloc_table_from_pages` changed in kernel 5.10 and onwards. Two agruments are added which can be passed as NULL and 0 for the same behavior as before, except for error checking, which was addressed in this patch. Signed-off-by: Gwenhael GOAVEC <gwenhael.goavec@femto-st.fr> Signed-off-by: Mathis MARION <mathis.marion@grenoble-inp.org>
-