You should probably add a higher-level explanation in the commit message that to make use of probes, the user needs to specific which buffers of the firmware topology they want to extract data from, and that debugfs is the configuration interface. The streaming part is handled via the compressed interface.
Define debugfs subdirectory delegated for IPC communitation with DSP.
communication.
Input format: uint,uint,(...) which are later translated into DWORDS sequence and further into instances of struct of interest given the IPC type.
we should probably add a documentation part that specifies the values expected, as you did some time back.
For Extractor probes, following have been enabled:
- PROBE_POINT_ADD (echo <..> probe_points)
- PROBE_POINT_REMOVE (echo <..> probe_points_remove)
- PROBE_POINT_INFO (cat probe_points)
Doesn't appear very intuitive to me, is this the same as in previous solutions or a new design of your own?
+static ssize_t ppoints_read(struct file *file,
char __user *to, size_t count, loff_t *ppos)
avoid ppoints acronym, use probe_points_read? same in the rest of the patch.