This is an automated email from the ASF dual-hosted git repository.

jerzy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git


The following commit(s) were added to refs/heads/master by this push:
     new 3e70b8258  hw/drivers/adc_nrf52: Fix non-blocking read
3e70b8258 is described below

commit 3e70b8258e71f7fd6b8f37c36f0eaa3a42c67c17
Author: Jerzy Kasenberg <jerzy.kasenb...@codecoup.pl>
AuthorDate: Fri May 12 15:00:03 2023 +0200

     hw/drivers/adc_nrf52: Fix non-blocking read
    
     buffer size passed to nrf_adc_set_buffer() was sometimes used
     as if it was in bytes and sometimes as if it was in 16 bits words.
    
     Now it always mean buffer size in bytes.
    
    Signed-off-by: Jerzy Kasenberg <jerzy.kasenb...@codecoup.pl>
---
 hw/drivers/adc/adc_nrf52/src/adc_nrf52.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/hw/drivers/adc/adc_nrf52/src/adc_nrf52.c 
b/hw/drivers/adc/adc_nrf52/src/adc_nrf52.c
index 9dce3dec9..e3712995c 100644
--- a/hw/drivers/adc/adc_nrf52/src/adc_nrf52.c
+++ b/hw/drivers/adc/adc_nrf52/src/adc_nrf52.c
@@ -411,7 +411,6 @@ nrf52_adc_set_buffer(struct adc_dev *dev, void *buf1, void 
*buf2, int buf_len)
         return OS_EINVAL;
     }
 
-    buf_len /= sizeof(nrf_saadc_value_t);
     g_drv_instance.primary_buf = buf1;
     g_drv_instance.primary_size = buf_len;
     if (buf2) {
@@ -432,7 +431,6 @@ nrf52_adc_release_buffer(struct adc_dev *dev, void *buf, 
int buf_len)
         return OS_EINVAL;
     }
 
-    buf_len /= sizeof(nrf_saadc_value_t);
     if (!g_drv_instance.primary_buf) {
         g_drv_instance.primary_buf = buf;
         g_drv_instance.primary_size = buf_len;
@@ -622,7 +620,7 @@ nrf52_saadc_irq_handler(void)
         nrf_saadc_event_clear(NRF_SAADC, NRF_SAADC_EVENT_END);
 
         buf = g_drv_instance.primary_buf;
-        bufsize = g_drv_instance.primary_size * sizeof(nrf_saadc_value_t);
+        bufsize = g_drv_instance.primary_size;
         if (g_drv_instance.secondary_buf) {
             g_drv_instance.primary_buf = g_drv_instance.secondary_buf;
             g_drv_instance.primary_size = g_drv_instance.secondary_size;

Reply via email to