aasorokiin commented on a change in pull request #32:
URL: https://github.com/apache/tvm-vta/pull/32#discussion_r692641676



##########
File path: src/dpi/module.cc
##########
@@ -118,16 +120,22 @@ class HostDevice {
 
 class MemDevice {
  public:
-  void SetRequest(uint8_t opcode, uint64_t addr, uint32_t len);
-  MemResponse ReadData(uint8_t ready);
-  void WriteData(uint64_t value);
+  void  SetRequest(uint8_t rd_req_valid,uint64_t rd_req_addr, uint32_t 
rd_req_len, uint32_t rd_req_id, uint64_t wr_req_addr, uint32_t wr_req_len, 
uint8_t wr_req_valid);

Review comment:
       If it is not run by TVM make, than it is not. Fixed long lines in a new 
PR




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to