On 9/25/23 16:28, Tom Rini wrote:
On Mon, Sep 25, 2023 at 08:01:41AM -0600, Simon Glass wrote:
Hi Michal,
On Mon, 25 Sept 2023 at 07:38, Michal Simek <michal.si...@amd.com> wrote:
On 9/25/23 15:10, Simon Glass wrote:
Hi Michal,
On Mon, 25 Sept 2023 at 00:06, Michal Simek <michal.si...@amd.com> wrote:
Hi Simon,
On 9/23/23 20:13, Simon Glass wrote:
Current alignment which is using 16 bytes is not correct in connection to
trace_clocks description and it's length.
That's why use start_addr variable and record proper size based on used
entries.
Fixes: be16fc81b2ed ("trace: Update proftool to use new binary format").
Signed-off-by: Michal Simek <michal.si...@amd.com>
Reviewed-by: Simon Glass <s...@chromium.org>
---
Changes in v2:
- s/start_addr/start_ofs/g'
tools/proftool.c | 31 +++++++++++++++++++++++++++++--
1 file changed, 29 insertions(+), 2 deletions(-)
Applied to u-boot-dm, thanks!
FYI: I have merged it to my tree and already sent pull request to Tom.
Without it I couldn't pass CI loop to get all reviewed features in.
https://lore.kernel.org/all/ab72c480-e9f8-416e-adf5-726f7d40c...@amd.com/
Ah OK, well that's fine. It was in my patchwork queue still, which
suggests that the patches were not set to 'applied'?
I am not using patchwork. But I expect my reply to cover letter was recorded
there.
Probably. If you reply to each patch, it shows up in the patch, but
the cover letter is hidden somewhere else.
Patchwork doesn't, but b4 does, handle tags sent to the cover letter
rather than individually.
For tags yes it works nicely with b4. Pretty much just reply to cover letter and
b4 copy it to all patches.
But this was more about that you can't see my Applied reply there.
I don't think that b4 will be able to catch it but I would love to be wrong.
M
