Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SKIP SOF-TEST] [stable-v2.2] sync CODEOWNERS with main branch #9497

Merged
merged 15 commits into from
Sep 20, 2024

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Sep 19, 2024

People typically don't git branch like software does. It may actually happen in some projects but not in SOF AFAIK so let's stop spamming people who left - especially myself :-)

This makes the CODEOWNERS file almost identical to the one in the main branch, see below.

This means a few patterns that don't match anything in this branch are getting added - who cares.

This was performed with the usual "brute force" forklift method...

git rev-list  origin/main ^HEAD -- CODEOWNERS | tac | git cherry-pick -x --stdin

... + skipping platform removals and fixing a couple git conflicts.


git diff origin/main -- CODEOWNERS

--- a/CODEOWNERS
+++ b/CODEOWNERS
@@ -48,6 +48,8 @@ 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/*            @abonislawski
 src/platform/imx8/**                   @dbaluta
 src/platform/amd/**                    @bhiregoudar @sunilkumardommati
@@ -55,6 +57,8 @@ src/platform/mt8195/**                @yaochunhung @kuanhsuncheng

 # drivers
 src/drivers/intel/dmic.c               @singalsu
+src/drivers/intel/cavs/sue-iomux.c     @lyakh
+src/drivers/intel/haswell/*            @randerwang
 src/drivers/imx/**                     @dbaluta
 src/drivers/dw/*                       @lyakh
 src/drivers/amd/*                      @bhiregoudar @sunilkumardommati
@@ -63,6 +67,7 @@ src/drivers/mediatek/mt8195/*         @yaochunhung @kuanhsuncheng
 # other libs
 src/math/*                             @singalsu
 src/ipc/*                              @bardliao @marcinszkudlinski @pblaszko
+src/drivers/intel/cavs/sue-ipc.c       @lyakh
 # src/lib/*                            TODO
 src/debug/gdb/*                                @abonislawski
 src/schedule                           @pblaszko @marcinszkudlinski @dbaluta @LaurentiuM1234

cujomalainey and others added 15 commits September 19, 2024 15:16
Now we won't need code changes to update reviewers when we have
additions or departures.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
(cherry picked from commit df36d25)
Forgot to update this after moving our components to a folder

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
(cherry picked from commit 3b8ef3a)
Dumb search/replace per @mwasko's request in thesofproject#6933.

Urgently required because the @mrawja account does not exist anymore and
Github complains.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit cb4cb1d)
I don't want to be spammed any more every time someone adds or renames a
source file.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit 235dd98)
Mark Ranjani Sridharan for full topology folder. Add Jyri
Sarha for topology2 folders.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit 9353c2d)
Add Kai Vehmanen, Guennadi Liakhovetski, Iulia Prodan and Daniel
Baluta as owners for the 'zephyr/' subfolder.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit 441bd03)
Add Marc Herbert as code owner for installer subfolder.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit 94a128d)
Add a link to online github documentation with regard to
file syntax and how it can be tested.

Suggested-by: Marc Herbert <marc.herbert@intel.com>
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit 430893f)
Fix minor misunderstanding in review of git commit 430893f
("CODEOWNERS: refer to github documentation on file syntax")

Signed-off-by: Marc Herbert <marc.herbert@gmail.com>
(cherry picked from commit 7975c26)
Intel code owners added based on latest contribution
and expertise.

Signed-off-by: Michal Wasko <michal.wasko@intel.com>
(cherry picked from commit e40cf6d)
With the work done for zephyr DMA domain we got a lot of insights
about how scheduler works.

So add myself and Laurentiu as codeowners.

Signed-off-by: Daniel Baluta <daniel.baluta@nxp.com>
(cherry picked from commit aa91bed)
Andrey is not at Intel any more.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit ed8dd91)
Add myself to the list of default owners that get added to
all pull requests if no other entries match.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit 7c013b2)
New assignment.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit 077187d)
Both are reported by Github as "Unknown owner on line 31: make sure
@makarukp exists and has write access to the repository"

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
(cherry picked from commit 945af87)
@kv2019i
Copy link
Collaborator

kv2019i commented Sep 20, 2024

Thank you @marc-hb !

@kv2019i kv2019i merged commit ce8d8c6 into thesofproject:stable-v2.2 Sep 20, 2024
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants