Linaro / OpenCSD

CoreSight trace stream decoder developed openly
https://github.com/Linaro/opencsd/wiki
Other
141 stars 53 forks source link

skip over the header bits before checking cs_etmv*_magic #6

Closed sebpop closed 7 years ago

mathieupoirier commented 7 years ago

Sorry about the late reply.

I did not realise that rebasing to v4.9 and deleting perf-opencsd-4.9-rc1 would also close your pull request - apologies on that front.

Please resend, this time on perf-opencsd-4.9. While doing so add a better description of what the patch does along with a SoB. This was working before, thanks for fixing it.

Since this section of the code is not upstream yet we will have to merge your work in the official patch that does get sent to the mailing list. When that happens we will add you name as contributor in the patch description.

Thanks, Mathieu

sebpop commented 7 years ago

On Tue, Dec 13, 2016 at 11:57 AM, mathieupoirier notifications@github.com wrote:

Sorry about the late reply.

I did not realise that rebasing to v4.9 and deleting perf-opencsd-4.9-rc1 would also close your pull request - apologies on that front.

Please resend, this time on perf-opencsd-4.9. While doing so add a better description of what the patch does along with a SoB. This was working before, thanks for fixing it.

Since this section of the code is not upstream yet we will have to merge your work in the official patch that does get sent to the mailing list. When that happens we will add you name as contributor in the patch description.

Ok, I will resubmit for the new branch with a longer message about what was broken and how the patch is fixing it.

Thanks, Sebastian