This is an automated email from Gerrit.

Spencer Oliver ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/948

-- gerrit

commit 20c3f63513018bc94f47941f3bef8fca3a84dde6
Author: Spencer Oliver <[email protected]>
Date:   Thu Nov 1 11:19:29 2012 +0000

    hla: add ability to configure read/write buffer size
    
    Other adapters (TI ICDI) that use this driver can use a larger
    read/write buffer size than the original stlink could.
    
    Change-Id: I9beb7748049097cbe29a2340799c450bd74e199d
    Signed-off-by: Spencer Oliver <[email protected]>

diff --git a/src/jtag/drivers/stlink_usb.c b/src/jtag/drivers/stlink_usb.c
index a5e5a6b..7ba0900 100644
--- a/src/jtag/drivers/stlink_usb.c
+++ b/src/jtag/drivers/stlink_usb.c
@@ -1160,6 +1160,9 @@ static int stlink_usb_open(struct hl_interface_param_s 
*param, void **fd)
 
        h->transport = param->transport;
 
+       /* set max read/write buffer size in bytes */
+       param->max_buffer = 512;
+
        const uint16_t vids[] = { param->vid, 0 };
        const uint16_t pids[] = { param->pid, 0 };
 
diff --git a/src/jtag/hla/hla_interface.c b/src/jtag/hla/hla_interface.c
index 1693529..fe6d20c 100644
--- a/src/jtag/hla/hla_interface.c
+++ b/src/jtag/hla/hla_interface.c
@@ -37,7 +37,7 @@
 
 #include <target/target.h>
 
-static struct hl_interface_s hl_if = { {0, 0, 0, 0, 0, 0}, 0, 0 };
+static struct hl_interface_s hl_if = { {0, 0, 0, 0, 0, 0, 0}, 0, 0 };
 
 int hl_interface_open(enum hl_transports tr)
 {
diff --git a/src/jtag/hla/hla_interface.h b/src/jtag/hla/hla_interface.h
index 84a7d41..40405b3 100644
--- a/src/jtag/hla/hla_interface.h
+++ b/src/jtag/hla/hla_interface.h
@@ -44,6 +44,8 @@ struct hl_interface_param_s {
        unsigned api;
        /** */
        enum hl_transports transport;
+       /** */
+       int max_buffer;
 };
 
 struct hl_interface_s {
diff --git a/src/jtag/hla/hla_layout.c b/src/jtag/hla/hla_layout.c
index 04b50d7..5565729 100644
--- a/src/jtag/hla/hla_layout.c
+++ b/src/jtag/hla/hla_layout.c
@@ -50,6 +50,12 @@ static int hl_layout_open(struct hl_interface_s *adapter)
                return res;
        }
 
+       /* make sure adapter has set the buffer size */
+       if (!adapter->param.max_buffer) {
+               LOG_ERROR("buffer size not set");
+               return ERROR_FAIL;
+       }
+
        return ERROR_OK;
 }
 
diff --git a/src/target/hla_target.c b/src/target/hla_target.c
index 3c3b841..8d2a429 100644
--- a/src/target/hla_target.c
+++ b/src/target/hla_target.c
@@ -671,11 +671,11 @@ static int adapter_read_memory(struct target *target, 
uint32_t address,
                uint32_t size, uint32_t count,
                uint8_t *buffer)
 {
+       struct hl_interface_s *adapter = target_to_adapter(target);
        int res;
-       uint32_t buffer_threshold = 128;
+       uint32_t buffer_threshold = (adapter->param.max_buffer / 4);
        uint32_t addr_increment = 4;
        uint32_t c;
-       struct hl_interface_s *adapter = target_to_adapter(target);
 
        if (!count || !buffer)
                return ERROR_COMMAND_SYNTAX_ERROR;
@@ -687,7 +687,7 @@ static int adapter_read_memory(struct target *target, 
uint32_t address,
         */
        if (size != 4) {
                count *= size;
-               buffer_threshold = 64;
+               buffer_threshold = (adapter->param.max_buffer / 4) / 2;
                addr_increment = 1;
        }
 
@@ -719,11 +719,11 @@ static int adapter_write_memory(struct target *target, 
uint32_t address,
                uint32_t size, uint32_t count,
                const uint8_t *buffer)
 {
+       struct hl_interface_s *adapter = target_to_adapter(target);
        int res;
-       uint32_t buffer_threshold = 128;
+       uint32_t buffer_threshold = (adapter->param.max_buffer / 4);
        uint32_t addr_increment = 4;
        uint32_t c;
-       struct hl_interface_s *adapter = target_to_adapter(target);
 
        if (!count || !buffer)
                return ERROR_COMMAND_SYNTAX_ERROR;
@@ -735,7 +735,7 @@ static int adapter_write_memory(struct target *target, 
uint32_t address,
         */
        if (size != 4) {
                count *= size;
-               buffer_threshold = 64;
+               buffer_threshold = (adapter->param.max_buffer / 4) / 2;
                addr_increment = 1;
        }
 

-- 

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to