Daniel Baluta

Results 78 comments of Daniel Baluta

Looks good to me. There are some sparse errors, looks like are not related to this patch. Anyhow, can you please doublecheck? https://github.com/thesofproject/linux/actions/runs/8617799215/job/23618632734?pr=4913

@tobonex is there a Linux corresponding PR? Can you please add a brief overview of what information is this interface is supposed to gather and add it into corresponding commit...

@softwarecki what is `native loadable modules`? Is the support from Zephyr for loading modules or the one from SOF?

@singalsu AFAIK the simulated Host will self-generate the data right? Can we call it "Virtual"? @lgirdwood @singalsu we are trying to decouple SOF firmware from Linux kernel to have it...

Can you please try to address at least some of the checkpatch.pl errors: for example this: ``` WARNING: macros should not use a trailing semicolon #243: FILE: src/include/sof/trace/trace_nonzephyr.h:137: +#define trace_check_size_uint32(a)...

@btian1 Please let me know when the patches are in a final form and will try to give them a test

@LaurentiuM1234 can you please try to see if logging still works after this PR? @btian1 can you please confirm the testing conditions? Is it with XTOS right?

@LaurentiuM1234 can you post a snippet of the log w/o the patch. It looks like now the component is shown as "Unknown". I wonder if w/o the patch the real...

@btian1 looks like with your patch we lose ability to get the component for which the log was generated. Is this intended right? If so then maybe we can remove...

I think we are fine with this, but it won't compile on arm64. Cc: @LaurentiuM1234 I wonder why we havent added a build for imx93 here?