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.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
