Re: [PATCH v2] ASoC: Intel: Remove unused intel-sst.h trace header

[Steven Rostedt <rostedt@goodmis.org>]

On Thu, 12 Nov 2026 10:53:04 +0100
Cezary Rojewski <cezary.rojewski@intel.com> wrote:

> On 11/11/2026 11:31 PM, Temitope obabi wrote:
> > Commit 970d299b0a0a ("ASoC: Intel: Remove unused code") removed
> > sound/soc/intel/common/sst-dsp.c, the only file that defined
> > CREATE_TRACE_POINTS and included trace/events/intel-sst.h.
> > 
> > Nothing includes this header anymore, so none of its tracepoints are
> > built. Remove the unused header.
> > 
> > Signed-off-by: Temitope obabi <obabitemitope765@gmail.com>
> > ---
> > Changes in v2:
> > - Remove the whole header instead of only sst_ipc_inbox_rdata, since
> >   nothing includes it (reported by the Sashiko review of v1).
> > - Drop the Fixes tag and the claim about memory used per tracepoint.
> > - This also replaces the earlier sst_ipc_msg_tx patch for the same header.  
> Next time, please CC sound/soc/intel maintainers. The very commit starts
> with "ASoC: Intel:".
> 
> Nitpick: if trace-subsystem maintainers permit ASoC-scoping for
> include/trace, please update the title to: "ASoC: Intel: atom:" to
> highlight which driver it does relate to.

I can make these updates.

> 
> The nitpick alone is not a reason for re-send.
> 
> Acked-by: Cezary Rojewski <cezary.rojewski@intel.com>

Thanks for the Ack.

Temitope,

Files in include/trace/events/ are special and unfortunately, the
get-maintainers.pl script doesn't usually work for them. That's because
the headers in that directory really belong to the subsystems that
define tracepoints for.

The best way to find out who to Cc is to find a file where the
tracepoint is used and run get-maintainers on that. Unfortunately, the
file that had them was removed so that wouldn't work either. The next
best thing is to look at the commit that removed the file, and Cc those
that were involved in the removal. In general, you should always Cc
those involved with anything in the commit of the fixes tag (which this
commit should have had for 970d299b0a0a).

 $ git show 970d299b0a0a
[..]
    Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com>
    Signed-off-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
    Link: https://patch.msgid.link/20241009083419.319038-1-amadeuszx.slawinski@linux.intel.com
    Signed-off-by: Mark Brown <broonie@kernel.org>

All the emails in the above should be Cc'd on changes like this.

Thanks,

-- Steve