Skip to content

Commit ce74520

Browse files
committed
applications: sdp: mspi: change opcode cast to uint32
Change cast of opcode in ep_recv function to uint32_t instead of uint8_t. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
1 parent c2daec3 commit ce74520

File tree

3 files changed

+12
-10
lines changed

3 files changed

+12
-10
lines changed

CODEOWNERS

+1
Original file line numberDiff line numberDiff line change
@@ -346,6 +346,7 @@
346346
/include/drivers/flash/flash_ipuc.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-charon
347347
/include/drivers/gpio/ @nrfconnect/ncs-co-drivers @nrfconnect/ncs-ll-ursus
348348
/include/drivers/bme68x_iaq.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-cia
349+
/include/drivers/mspi/nrf_mspi.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-ll-ursus
349350
/include/drivers/mspi/nrfe_mspi.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-ll-ursus
350351
/include/drivers/sensor_sim.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-cia
351352
/include/drivers/sensor_stub.h @nrfconnect/ncs-co-drivers @nrfconnect/ncs-cia

applications/sdp/mspi/src/main.c

+5-3
Original file line numberDiff line numberDiff line change
@@ -423,8 +423,9 @@ static void ep_recv(const void *data, size_t len, void *priv)
423423

424424
(void)priv;
425425
(void)len;
426-
uint8_t opcode = *(uint8_t *)data;
426+
uint32_t opcode = *(uint32_t *)data;
427427
uint32_t num_bytes = 0;
428+
uint32_t *resp_buff = (uint32_t *)response_buffer;
428429

429430
#if defined(CONFIG_SDP_MSPI_FAULT_TIMER)
430431
if (fault_timer != NULL) {
@@ -483,7 +484,8 @@ static void ep_recv(const void *data, size_t len, void *priv)
483484
}
484485

485486
/* If there is reset defined on any data line, assume it is no longer valid and
486-
* clear it. */
487+
* clear it.
488+
*/
487489
if (dev_config->dev_config.io_mode != MSPI_IO_MODE_SINGLE &&
488490
reset_configs[dev_config->device_index].pin_number != PIN_UNUSED) {
489491
nrf_vpr_csr_vio_out_clear_set(
@@ -545,7 +547,7 @@ static void ep_recv(const void *data, size_t len, void *priv)
545547
break;
546548
}
547549

548-
response_buffer[0] = opcode;
550+
resp_buff[0] = opcode;
549551
ipc_service_send(&ep, (const void *)response_buffer,
550552
sizeof(nrfe_mspi_opcode_t) + num_bytes);
551553

drivers/mspi/mspi_nrfe.c

+6-7
Original file line numberDiff line numberDiff line change
@@ -1063,13 +1063,12 @@ static int nrfe_mspi_init(const struct device *dev)
10631063
}
10641064

10651065
static const struct nrf_mspi_driver_api drv_api = {
1066-
.std_api =
1067-
{
1068-
.config = api_config,
1069-
.dev_config = api_dev_config,
1070-
.get_channel_status = api_get_channel_status,
1071-
.transceive = api_transceive,
1072-
},
1066+
.std_api = {
1067+
.config = api_config,
1068+
.dev_config = api_dev_config,
1069+
.get_channel_status = api_get_channel_status,
1070+
.transceive = api_transceive,
1071+
},
10731072
.reset_pin_config = api_reset_config,
10741073
.reset_pin_set = api_reset_set,
10751074
};

0 commit comments

Comments
 (0)