Re: [PATCH] ASoC: cs-amp-lib: Annotate struct cirrus_amp_efi_data with __counted_by()

From: Takashi Iwai
Date: Tue Apr 15 2025 - 08:07:28 EST


On Tue, 15 Apr 2025 14:03:18 +0200,
Richard Fitzgerald wrote:
>
> On 15/04/2025 12:51 pm, Thorsten Blum wrote:
> > On 15. Apr 2025, at 12:56, Mark Brown wrote:
> >> On Tue, Apr 15, 2025 at 11:03:55AM +0200, Thorsten Blum wrote:
> >>> Add the __counted_by() compiler attribute to the flexible array member
> >>> 'data' to improve access bounds-checking via CONFIG_UBSAN_BOUNDS and
> >>> CONFIG_FORTIFY_SOURCE.
> >>
> >> As documented in submitting-patches.rst please send patches to the
> >> maintainers for the code you would like to change. The normal kernel
> >> workflow is that people apply patches from their inboxes, if they aren't
> >> copied they are likely to not see the patch at all and it is much more
> >> difficult to apply patches.
> >
> > I just use whatever scripts/get_maintainer.pl outputs. Maybe the
> > MAINTAINERS file should be updated?
> >
> > Best,
> > Thorsten
> >
>
> MAINTAINERS only lists Takashi and Jaroslav as maintainers of
> include/sound/*.
>
> A separate section lists Mark as maintainer of a specific set of files
> within include/sound/
>
> I guess any files we put in include/sound that are for an ASoC driver
> have to also be added to Mark's ASoC section of MAINTAINERS?

Yes, it makes sense.


Takashi