Change in flashrom[master]: mstarddc_spi.c: Drop mstarddc_ prefix for spi data struct members

[Edward O'Callaghan (Code Review)]

Edward O'Callaghan has submitted this change. ( https://review.coreboot.org/c/flashrom/+/54034 )

Change subject: mstarddc_spi.c: Drop mstarddc_ prefix for spi data struct members
......................................................................

mstarddc_spi.c: Drop mstarddc_ prefix for spi data struct members

The name of the struct already contains mstarddc_ prefix, so prefix
doesn't need to be repeated in members names.

BUG=b:185191942
TEST=builds

Change-Id: Ic04b14ce502917ae3b959cf2acf23d58b8752d47
Signed-off-by: Anastasia Klimchuk 
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54034
Tested-by: build bot (Jenkins) 
Reviewed-by: Edward O'Callaghan 
---
M mstarddc_spi.c
1 file changed, 16 insertions(+), 15 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Edward O'Callaghan: Looks good to me, approved

diff --git a/mstarddc_spi.c b/mstarddc_spi.c
index da9f152..0f638df 100644
--- a/mstarddc_spi.c
+++ b/mstarddc_spi.c
@@ -32,9 +32,9 @@
 #include "spi.h"

struct mstarddc_spi_data {
-	int mstarddc_fd;
-	int mstarddc_addr;
-	int mstarddc_doreset;
+	int fd;
+	int addr;
+	int doreset;
 };

// MSTAR DDC Commands
@@ -49,9 +49,9 @@
    struct mstarddc_spi_data *mstarddc_data = data;

// Reset, disables ISP mode
-	if (mstarddc_data->mstarddc_doreset == 1) {
+	if (mstarddc_data->doreset == 1) {
    	uint8_t cmd = MSTARDDC_SPI_RESET;
-		if (write(mstarddc_data->mstarddc_fd, &cmd, 1) < 0) {
+		if (write(mstarddc_data->fd, &cmd, 1) < 0) {
    		msg_perr("Error sending reset command: errno %d.\n",
    			 errno);
    		return -1;
@@ -62,7 +62,7 @@
    		  "or an error occurred.\n");
    }

-	if (close(mstarddc_data->mstarddc_fd) < 0) {
+	if (close(mstarddc_data->fd) < 0) {
    	msg_perr("Error closing device: errno %d.\n", errno);
    	return -1;
    }
@@ -89,7 +89,7 @@
    if (!ret && writecnt) {
    	cmd[0] = MSTARDDC_SPI_WRITE;
    	memcpy(cmd + 1, writearr, writecnt);
-		if (write(mstarddc_data->mstarddc_fd, cmd, writecnt + 1) < 0) {
+		if (write(mstarddc_data->fd, cmd, writecnt + 1) < 0) {
    		msg_perr("Error sending write command: errno %d.\n",
    			 errno);
    		ret = -1;
@@ -103,16 +103,16 @@
    	cmd[0] = MSTARDDC_SPI_READ;
    	i2c_data.nmsgs = 2;
    	i2c_data.msgs = msg;
-		i2c_data.msgs[0].addr = mstarddc_data->mstarddc_addr;
+		i2c_data.msgs[0].addr = mstarddc_data->addr;
    	i2c_data.msgs[0].len = 1;
    	i2c_data.msgs[0].flags = 0;
    	i2c_data.msgs[0].buf = cmd;
-		i2c_data.msgs[1].addr = mstarddc_data->mstarddc_addr;
+		i2c_data.msgs[1].addr = mstarddc_data->addr;
    	i2c_data.msgs[1].len = readcnt;
    	i2c_data.msgs[1].flags = I2C_M_RD;
    	i2c_data.msgs[1].buf = readarr;

-		if (ioctl(mstarddc_data->mstarddc_fd, I2C_RDWR, &i2c_data) < 0) {
+		if (ioctl(mstarddc_data->fd, I2C_RDWR, &i2c_data) < 0) {
    		msg_perr("Error sending read command: errno %d.\n",
    			 errno);
    		ret = -1;
@@ -121,7 +121,7 @@

if (!ret && (writecnt || readcnt)) {
    	cmd[0] = MSTARDDC_SPI_END;
-		if (write(mstarddc_data->mstarddc_fd, cmd, 1) < 0) {
+		if (write(mstarddc_data->fd, cmd, 1) < 0) {
    		msg_perr("Error sending end command: errno %d.\n",
    			 errno);
    		ret = -1;
@@ -131,7 +131,7 @@
    /* Do not reset if something went wrong, as it might prevent from
     * retrying flashing. */
    if (ret != 0)
-		mstarddc_data->mstarddc_doreset = 0;
+		mstarddc_data->doreset = 0;

if (cmd)
    	free(cmd);
@@ -235,9 +235,10 @@
    	ret = -1;
    	goto out;
    }
-	mstarddc_data->mstarddc_fd = mstarddc_fd;
-	mstarddc_data->mstarddc_addr = mstarddc_addr;
-	mstarddc_data->mstarddc_doreset = mstarddc_doreset;
+
+	mstarddc_data->fd = mstarddc_fd;
+	mstarddc_data->addr = mstarddc_addr;
+	mstarddc_data->doreset = mstarddc_doreset;

// Register shutdown function
    register_shutdown(mstarddc_spi_shutdown, mstarddc_data);