Resolve "Implement hardware monitoring API"

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!12

In the case of error, the current API just reports via dev_err(), and
return a safe value. This is fine for callibration tools. But other
places, like sysfs, care about the eroor and the error code.

Thus, the API should return the error code in case of error, and the
tools which want a safe value, should do it in their own space.
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>

Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>

Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>

Instead of manually calculating the size, allocating the memory, and
writing data to it, use devm_kasprintf(). It will do all the job in one
go.
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>

Information from sensors like temperature, which are useful for
user-space, can be exposed via sysfs.

Earlier, the driver developers had to take care of defining the attributes,
initiating them, and perform other related jobs.

But since a long time, the HWMON framework in Linux has developed
significantly. And, utilizing it will reduce lot of maintainence work
for the developers.

All the sysfs related part is handled by the framework which also allows
user-space tools, like lm-sensors, to provide the data to the user in a
simplified manner.
Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>

Resolve "Do a 6.0.4 release"

Closes #17

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!6

And use it to avoid hard-coding the latest release version in the introduction.

Resolve "Migrate to common CI and publish bitstreams"

Closes #27

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!11

Resolve "Update to latest dependency releases"

Closes #21

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!8

GN4124: 3.1.1
DDR: 2.0.1
VME64x: 2.2
Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Resolve "Build FPGA bitstreams in CI"

Closes #25

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!10

Resolve "fix broken CI for centos 7"

Closes #23

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!9

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Resolve "Add support for WRTD based designs"

Closes wrtd-reference-designs#11 and #16

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!5

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Resolve "Review Vaibhav Patches"

Closes #20

See merge request be-cem-edl/fec/hardware-modules/fmc-adc-100m14b4cha!7

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

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>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

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>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

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>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

"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>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

Signed-off-by: Vaibhav Gupta <vaibhav.gupta@cern.ch>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>

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>
Reviewed-by: Federico Vaga <federico.vaga@cern.ch>