)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1001982,"name":"Ian Thompson","email":"ianst@cadence.com","username":"ianstcdns"},"change_message_id":"65083d4866c8b8b4aaf1a424ce520f1947019cec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"41eabe31_911dfb6e","updated":"2022-09-28 20:45:23.000000000","message":"Good catch, @erhankur!  Did you see this manifest as a bad memory read while attempting to use LDDR32.P?  I think this must have been uncovered by my recent 53d17e...  I can see missing this by testing with an initial memory read that had 4-byte multiple + 4-byte alignment; this wouldn\u0027t show up on unaligned and/or byte/short reads after the probe.  Thanks!","commit_id":"35716d0f5720b10ad1220ee3d387eb2507285fda"},{"author":{"_account_id":1001964,"name":"Erhan Kurubas","display_name":"Erhan Kurubas","email":"erhan.kurubas@espressif.com","username":"erhankur"},"change_message_id":"ce19ab2a8f3830408343c6fbf5856414e27eb510","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"adf752d0_cb7ea39d","in_reply_to":"41eabe31_911dfb6e","updated":"2022-09-28 21:18:05.000000000","message":"Yes. It was ok before 53d17e. \nIn my case, at the first 8 bytes read attempt from the aligned address, DSR returns ExecOverrun error. I see `Disabling LDDR32.P/SDDR32.P` msg then 2nd attempt is OK but buffer is wrong.\nDo you think, printing this msg as INFO level is useful? DEBUG might be better to hide the detail from the users.","commit_id":"35716d0f5720b10ad1220ee3d387eb2507285fda"},{"author":{"_account_id":1001964,"name":"Erhan Kurubas","display_name":"Erhan Kurubas","email":"erhan.kurubas@espressif.com","username":"erhankur"},"change_message_id":"28f0a4b5c948bc9c0fbab87c8c0d7415bcca68b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e6eead97_9632571c","in_reply_to":"541bc1c4_533ca51f","updated":"2022-09-28 22:14:54.000000000","message":"Done. Thanks.","commit_id":"35716d0f5720b10ad1220ee3d387eb2507285fda"},{"author":{"_account_id":1001982,"name":"Ian Thompson","email":"ianst@cadence.com","username":"ianstcdns"},"change_message_id":"566bbb48df01746d894f2c44275e461909f8d357","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"541bc1c4_533ca51f","in_reply_to":"adf752d0_cb7ea39d","updated":"2022-09-28 22:11:01.000000000","message":"The \"disabling ...\" message is really a way for the target layer to probe whether the fast load/store instructions are present or not in the config.  at some point, it might be better to do this during attach instead of on the first read attempt, but regardless of where it\u0027s done, this message is likely something we want to suppress since it could be confusing to users who don\u0027t know what that instruction does.  So probably better to move it to a debug message as you suggest.","commit_id":"35716d0f5720b10ad1220ee3d387eb2507285fda"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"8cea8bd87448a6bbeedd80815f9bbc0a864f4094","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5bca9fab_3768e0e0","updated":"2022-09-30 21:15:57.000000000","message":"thanks","commit_id":"0773196c2f9ad04553b8ba569a79f8456acb60a3"}]}