diff --git a/CODEOWNERS b/CODEOWNERS index 3a5a8b899053..365405b49e7a 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -2,45 +2,55 @@ # Each line is a file pattern followed by one or more owners. # These owners will be the default owners for everything in the repo. -* @lgirdwood @plbossart @mmaka1 @lbetlej @dbaluta +* @lgirdwood @plbossart @mmaka1 @lbetlej @dbaluta @kv2019i # Order is important. The last matching pattern has the most precedence. -# So if a pull request only touches javascript files, only these owners -# will be requested to review. +# File patterns work mostly like .gitignore. Try to keep this file +# simple because it's literally impossible to test. +# https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners # include files src/include/sof/drivers/dmic.h @singalsu src/include/ipc/** @thesofproject/steering-committee -src/include/ipc/** @randerwang @marcinszkudlinski +src/include/ipc/** @randerwang @marcinszkudlinski @pblaszko +src/include/ipc4/** @randerwang @marcinszkudlinski @pblaszko src/include/kernel/** @thesofproject/steering-committee src/include/user/** @thesofproject/steering-committee -src/include/sof/debug/gdb/* @mrajwa -src/include/sof/audio/kpb.h @mrajwa -src/include/sof/audio/mux.h @akloniex -src/include/sof/audio/codec_adapter/* @cujomalainey @mrajwa @dbaluta +src/include/sof/debug/gdb/* @abonislawski +src/include/sof/audio/aria/* @abonislawski +src/include/sof/audio/kpb.h @fkwasowi @iganakov +src/include/sof/audio/mux.h @fkwasowi +src/include/sof/audio/up_down_mixer/* @abonislawski +src/include/sof/audio/codec_adapter/* @cujomalainey @abonislawski @dbaluta +src/include/sof/audio/module_adapter/* @ranj063 @jxstelter src/include/sof/drivers/acp_dai_dma.h @bhiregoudar @sunilkumardommati src/include/sof/drivers/afe* @yaochunhung # audio component -src/audio/src* @singalsu +src/audio/src* @singalsu @abonislawski +# src/audio/chain_dma* TODO +src/audio/copier/* @randerwang @abonislawski @pblaszko src/audio/eq* @singalsu src/audio/eq_fir* @singalsu src/audio/eq_iir* @singalsu src/audio/tone.c @singalsu -src/audio/kpb.c @mrajwa -src/audio/mux/* @akloniex -src/audio/dcblock* @cujomalainey @chiang831 @johnylin76 -src/audio/crossover* @cujomalainey @chiang831 @johnylin76 +src/audio/kpb.c @fkwasowi @iganakov +src/audio/mux/* @fkwasowi +src/audio/dcblock* @thesofproject/google +src/audio/crossover* @thesofproject/google src/audio/tdfb* @singalsu -src/audio/drc/* @johnylin76 @cujomalainey @chiang831 -src/audio/multiband_drc/* @johnylin76 @cujomalainey @chiang831 -src/audio/codec_adapter/* @cujomalainey @mrajwa @dbaluta -src/audio/google_hotword_detect.c @bzhg @cujomalainey @chiang831 @johnylin76 +src/audio/drc/* @thesofproject/google +src/audio/selector/* @tlissows +src/audio/up_down_mixer/* @abonislawski +src/audio/multiband_drc/* @thesofproject/google +src/audio/codec_adapter/* @cujomalainey @abonislawski @dbaluta +src/audio/module_adapter/* @ranj063 @jxstelter +src/audio/google/* @thesofproject/google # platforms src/platform/haswell/* @randerwang src/platform/suecreek/* @lyakh -src/arch/xtensa/debug/gdb/* @mrajwa +src/arch/xtensa/debug/gdb/* @abonislawski src/platform/imx8/** @dbaluta src/platform/amd/** @bhiregoudar @sunilkumardommati src/platform/mt8195/** @yaochunhung @kuanhsuncheng @@ -56,39 +66,59 @@ src/drivers/mediatek/mt8195/* @yaochunhung @kuanhsuncheng # other libs src/math/* @singalsu -src/ipc/* @bardliao @marcinszkudlinski +src/ipc/* @bardliao @marcinszkudlinski @pblaszko src/drivers/intel/cavs/sue-ipc.c @lyakh -src/lib/* @libinyang -src/debug/gdb/* @mrajwa -src/schedule @mrajwa +# src/lib/* TODO +src/debug/gdb/* @abonislawski +src/schedule @pblaszko @marcinszkudlinski @dbaluta @LaurentiuM1234 + +# samples +samples/audio/detect_test.c @iganakov # other helpers -# Mostly overridden by *.(ba)sh below -scripts/* @marc-hb @aborisovich +# Many files overridden by *.(ba)sh pattern below +# scripts/* TODO # tools(old 'soft' repo) tools/logger/* @bkokoszx @akloniex -tools/topology/* @ranj063 +tools/topology/** @ranj063 +tools/topology/topology2/** @ranj063 @jsarha tools/testbench/* @ranj063 tools/test/audio/* @singalsu tools/ctl/* @singalsu tools/tune/* @singalsu -tools/tune/crossover/* @cujomalainey @johnylin76 @chiang831 -tools/tune/dcblock/* @cujomalainey @johnylin76 @chiang831 -tools/tune/drc/* @cujomalainey @johnylin76 @chiang831 -tools/oss-fuzz/* @cujomalainey - -# build system -**/CMakeLists.txt @marc-hb @aborisovich -# **/Kconfig -# scripts/cmake/** -# scripts/kconfig/** - -*.sh @marc-hb -*.bash @marc-hb +tools/tune/crossover/* @thesofproject/google +tools/tune/dcblock/* @thesofproject/google +tools/tune/drc/* @thesofproject/google +tools/oss-fuzz/* @thesofproject/google + +zephyr/** @kv2019i @lyakh @iuliana-prodan @dbaluta @abonislawski +zephyr/lib/cpu.c @tmleman +zephyr/lib/alloc.c @dabekjakub +zephyr/lib/regions_mm.c @dabekjakub + +# installer +# installer/** TODO + +# CMake + +# Include only "top-level" CMakeLists.txt files; the other ones are just +# dumb list of source files and generate too much noise. +# /CMakeLists.txt TODO +#/*/CMakeLists.txt TODO +# /test/cmocka/CMakeLists.txt TODO + +# There's a small enough number of files in tools/ and little +# churn: keep it simple and take them all. +# FIXME: some topology CMakeLists.txt files are configuration files +# in disguise. Move them to actual configuration files. +# /tools/**/CMakeLists.txt TODO + +# *.sh TODO +# *.bash TODO *trace.* @akloniex -/.github/ @dbaluta @cujomalainey @lgirdwood @marc-hb @aborisovich +/.github/ @dbaluta @cujomalainey @lgirdwood # You can also use email addresses if you prefer. #docs/* docs@example.com