aboutsummaryrefslogtreecommitdiffstats
path: root/raiden_debug_spi.c
diff options
context:
space:
mode:
authorEdward O'Callaghan <quasisec@google.com>2022-10-17 12:31:59 +1100
committerEdward O'Callaghan <quasisec@chromium.org>2022-12-12 00:02:26 +0000
commit1e01eefcba573c436d55b863c7972b3029564215 (patch)
treeaa8850783eb557f4921020e8171fbd7c3ea1ade3 /raiden_debug_spi.c
parent78e421bdf73d5739daa25a9f71b2c323dc98e840 (diff)
downloadflashrom-1e01eefcba573c436d55b863c7972b3029564215.tar.gz
flashrom-1e01eefcba573c436d55b863c7972b3029564215.tar.bz2
flashrom-1e01eefcba573c436d55b863c7972b3029564215.zip
tree/: Replace NULL-case of programmer_delay() with internal_delay
Replace `programmer_delay(NULL, [..])` calls with direct `internal_delay([..])` dispatches explicitly. Custom driver delays remain hooked as well as core flashrom logic. The NULL base case of 'programmer_delay()' then becomes a condition to validate for layering violations or invalid flash contexts. Change-Id: I1da230804d5e8f47a6e281feb66f381514dc6861 Signed-off-by: Edward O'Callaghan <quasisec@google.com> Reviewed-on: https://review.coreboot.org/c/flashrom/+/68434 Reviewed-by: Anastasia Klimchuk <aklm@chromium.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'raiden_debug_spi.c')
-rw-r--r--raiden_debug_spi.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c
index 517b1738..24781334 100644
--- a/raiden_debug_spi.c
+++ b/raiden_debug_spi.c
@@ -892,7 +892,7 @@ static int send_command_v1(const struct flashctx *flash,
/* Reattempting will not result in a recovery. */
return status;
}
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
continue;
}
@@ -927,7 +927,7 @@ static int send_command_v1(const struct flashctx *flash,
/* Reattempting will not result in a recovery. */
return status;
}
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
}
}
@@ -962,7 +962,7 @@ static int get_spi_config_v2(struct raiden_debug_spi_data *ctx_data)
" config attempt = %d\n"
" status = 0x%05x\n",
config_attempt + 1, status);
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
continue;
}
@@ -972,7 +972,7 @@ static int get_spi_config_v2(struct raiden_debug_spi_data *ctx_data)
" config attempt = %d\n"
" status = 0x%05x\n",
config_attempt + 1, status);
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
continue;
}
@@ -1016,7 +1016,7 @@ static int get_spi_config_v2(struct raiden_debug_spi_data *ctx_data)
config_attempt + 1,
rsp_config.packet_v2.packet_id,
rsp_config.packet_size);
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
}
return USB_SPI_HOST_INIT_FAILURE;
}
@@ -1240,7 +1240,7 @@ static int send_command_v2(const struct flashctx *flash,
/* Reattempting will not result in a recovery. */
return status;
}
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
continue;
}
for (read_attempt = 0; read_attempt < READ_RETRY_ATTEMPTS;
@@ -1277,7 +1277,7 @@ static int send_command_v2(const struct flashctx *flash,
}
/* Device needs to reset its transmit index. */
restart_response_v2(ctx_data);
- programmer_delay(NULL, RETRY_INTERVAL_US);
+ internal_delay(RETRY_INTERVAL_US);
}
}
}