sof icon indicating copy to clipboard operation
sof copied to clipboard

Published 20 hours ago • verified publisher icon thesofproject

Remove sof logging ctx

Open btian1 opened this issue 1 year ago • 6 comments

btian1 avatar Jan 11 '24 12:01 btian1

@dbaluta @iuliana-prodan , could you help check from IPC3 side? as this change only have impact on IPC3 and I don't have a way to check.

btian1 avatar Jan 15 '24 02:01 btian1

Will try to have a look this week.

dbaluta avatar Jan 15 '24 09:01 dbaluta

Will try to have a look this week.

Any progress? @dbaluta , this DRAFT patch try to remove each module's ctx dependency for further one logging interface. And since ctx is only used by IPC3 logging, request help from you here for double confirm, if it is acceptable:

  1. All DECLARE_TR_CTX will be removed.
  2. Zephyr and non-zephyr logging will have same logging interface.

btian1 avatar Jan 19 '24 06:01 btian1

@btian1 tested this on imx8mp. Logging works fine after your patch.

dbaluta avatar Jan 25 '24 13:01 dbaluta

@btian1 draft ?

lgirdwood avatar Feb 23 '24 13:02 lgirdwood

If i remember correctly this is only for testing purposes.

dbaluta avatar Feb 23 '24 14:02 dbaluta