Anastasia Klimchuk has uploaded this change for review.

View Change

raiden_debug_spi: Use spi data in configure_protocol

Spi data for this programmer now contains pointer to spi_master
struct, and can be used in configure_protocol.

This patch is making init code of raiden_debug_spi a bit more
consistent with other programmers.

BUG=b:185191942
TEST=test in CB:57918

Change-Id: I5df0609efcd1fdb274b8cc84536253d5dbae5270
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
---
M raiden_debug_spi.c
1 file changed, 6 insertions(+), 6 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/73/57973/1

diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c
index 9cba2a8..d783fef 100644
--- a/raiden_debug_spi.c
+++ b/raiden_debug_spi.c
@@ -1317,15 +1317,16 @@
  * is being used by the device USB SPI interface and if needed query the
  * device for its capabilities.
  *
- * @param spi_config    Raiden SPI config which will be modified.
+ * @param data	Raiden SPI data, data contains pointer to config which will be modified.
  *
- * @returns             Returns status code with 0 on success.
+ * @returns	Returns status code with 0 on success.
  */
-static int configure_protocol(struct spi_master *spi_config)
+static int configure_protocol(struct raiden_debug_spi_data *data)
 {
 	int status = 0;
 	struct raiden_debug_spi_data *ctx_data =
-		(struct raiden_debug_spi_data *)spi_config->data;
+		(struct raiden_debug_spi_data *)data;
+	struct spi_master *spi_config = ctx_data->spi_config;
 
 	ctx_data->protocol_version =
 		ctx_data->dev->interface_descriptor->bInterfaceProtocol;
@@ -1596,13 +1597,12 @@
 	data->out_ep = out_endpoint;
 	data->spi_config = spi_config;
 
-	spi_config->data = data; /* data is needed to configure protocol below */
 	/*
 	 * The SPI master needs to be configured based on the device connected.
 	 * Using the device protocol interrogation, we will set the limits on
 	 * the write and read sizes and switch command functions.
 	 */
-	ret = configure_protocol(spi_config);
+	ret = configure_protocol(data);
 	if (ret) {
 		msg_perr("Raiden: Error configuring protocol\n"
 			 "    protocol       = %u\n"

To view, visit change 57973. To unsubscribe, or for help writing mail filters, visit settings.