[flashrom-gerrit] Change in flashrom[master]: realtek_mst_i2c_spi.c: Update write protection toggle function

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/46089 )

Change subject: realtek_mst_i2c_spi.c: Update write protection toggle function
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/flashrom/+/46089/6/realtek_mst_i2c_spi.c 
File realtek_mst_i2c_spi.c:

https://review.coreboot.org/c/flashrom/+/46089/6/realtek_mst_i2c_spi.c@157 
PS6, Line 157: ret |= realtek_mst_i2c_spi_write_register(fd, 0xF4, 0x9F);
             : 	ret |= realtek_mst_i2c_spi_write_register(fd, 0xF5, GPIO_CONFIG_ADDRESS >> 8);
             : 	ret |= realtek_mst_i2c_spi_write_register(fd, 0xF4, GPIO_CONFIG_ADDRESS & 0xFF);
> That's a good idea and thanks for identifying the pattern Angel. […]

I forgot to add, the func is almost correct but would be:

static int realtek_mst_i2c_spi_select_indexed_register(uint16_t address, uint8_t val)
 {
    int ret = 0;

ret |= realtek_mst_i2c_spi_write_register(fd, 0xF4, 0x9F);
    ret |= realtek_mst_i2c_spi_write_register(fd, 0xF5, address >> 8);
    ret |= realtek_mst_i2c_spi_write_register(fd, 0xF4, address & 0xFF);
    ret |= realtek_mst_i2c_spi_write_register(fd, 0xF5, val);

return ret;
 }