Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Thanks to this change the linux kernel may use PEC for these registers
in much easier way.

In pytest, update query flags for TEMP_CURVE_POINTS and TEMP_MATRIX_ROW
registers.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Changes cover:
- fix broken handling of setting row/points (size of temp_matrix_row was too
  small, fan_n in set_tc_curve was read from a wrong type (uint16 not uint8))
- use structures for handling row/points
- add get functionality for row/points (before it was possible to read only
  the transfered data not the actual used)
- iterate over fan entries when getting row/points
- update pytests
  -- update query flags for TEMP_CURVE_POINTS and TEMP_MATRIX_ROW registers
  -- improve write testing of TEMP_CURVE_POINTS and TEMP_MATRIX_ROW
     registers
- update documentation

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

The line:
*cmds->cmds[cmd_index].data_len = tx_tmp;
triggered the write to memory, which is declared as read-only (const).
Such writes trigger the memory exception.

To make the code simpler and avoid problematic write,
support only writes that has a size of a register.

The fix was tested for PEC.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Without PEC the i2c write transaction is finished before the execution of a callback.
Make the same behavior when the PEC is used.
Triggering a reset over i2c returns write failure if the transaction is not finished.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

remove unused define as leftover of removal of extended commands

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Compare the written byte against magic number. If not match refuse
to change a firmware mode. The check is done for bootloader and
main mode.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Compare the written byte against magic number. If not match refuse
to change a firmware mode.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Calculate status_b when the content of this register is requested.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

For main_fw and bootloader

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Add "/" to make it similar to the main_fw value

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Chnging unused settings. Making setting to be similar to main_fw.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

If enabled there is wrong data returned via i2c, like:
$ i2cdump -y 9 0x12 s 0x9a
     0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f    0123456789abcdef
00: 22 44 88 49 92 4f 9e 54 a8 20 40 4d 9a 6f de 6e    "D?I?O?T? @M?o?n
10: dc

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Clocks (even these not used) are changed to be very similar to main_fw

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Clock is chaned to 8MHz for i2c to be the same as in main_fw

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Clock is chaned to 24MHz to be the same as in main_fw
The real fix is the use of the internal oscillator

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Without this Linux driver reports that the currently set fan speed is 0.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

According to PMBUS v1.3.1 part II specification the used prefix 0xff for extended
commands is reserved for future use:
"The PMBUS_COMMAND_EXT is reserved for future use to extend the PMBus command set
beyond the available 256 command codes."

As an alternative the standard points the MFR_SPECIFIC_COMMAND_EXT to be available
with 0xFE as the first byte of a command.
The same standard also points that standard commands from 0xc4 can be defined
by a manufacturer. To be compliant with the standard and to simplify the design
I have decided to move all used extended command to the standard range.
Be aware that this brings some non-backward compatible changes.

Changes include:
- main FW
- bootloader
- test_master
- documentation

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

These coefficients has to be fixed, there are just values to export values other than 0

Signed-off-by: Adam Wujek <dev_public@wujek.eu>

To be called at the read of FAN_COMMAND_n. For some reason it fixes
random restarts after the write of FAN_COMMAND_n.

Signed-off-by: Adam Wujek <dev_public@wujek.eu>