diff mbox series

[v3,1/8] CI: Exclude devicetree-rebasing subtree for CONFIG checks

Message ID 20231228115804.3626579-2-sumit.garg@linaro.org
State Superseded
Headers show
Series An effort to bring DT bindings compliance within U-Boot | expand

Commit Message

Sumit Garg Dec. 28, 2023, 11:57 a.m. UTC
Since devicetree-rebasing is an external repo with its own coding style,
exclude it from Azure and gitlab CI CONFIG checks.

Reviewed-by: Tom Rini <trini@konsulko.com>
Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
---

Changes in v3:
--------------
- Picked up review tag

Changes in v2:
--------------
- Excluded gitab CI config check and added commit description.

 .azure-pipelines.yml | 3 ++-
 .gitlab-ci.yml       | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

Comments

Ilias Apalodimas Dec. 28, 2023, 12:02 p.m. UTC | #1
On Thu, 28 Dec 2023 at 13:58, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> Since devicetree-rebasing is an external repo with its own coding style,
> exclude it from Azure and gitlab CI CONFIG checks.
>
> Reviewed-by: Tom Rini <trini@konsulko.com>
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> ---
>
> Changes in v3:
> --------------
> - Picked up review tag
>
> Changes in v2:
> --------------
> - Excluded gitab CI config check and added commit description.
>
>  .azure-pipelines.yml | 3 ++-
>  .gitlab-ci.yml       | 3 ++-
>  2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
> index d6f3fa423c6..f100c4493e6 100644
> --- a/.azure-pipelines.yml
> +++ b/.azure-pipelines.yml
> @@ -65,7 +65,8 @@ stages:
>        # have no matches.
>        - script: git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
>                    :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
> -                  :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
> +                  :^include/linux/kconfig.h :^tools/ :^devicetree-rebasing/ &&
> +                  exit 1 || exit 0
>
>    - job: docs
>      displayName: 'Build documentation'
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index fee165198ae..96ca9f72bbc 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -161,7 +161,8 @@ check for new CONFIG symbols outside Kconfig:
>      # have no matches.
>      - git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
>          :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
> -        :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
> +        :^include/linux/kconfig.h :^tools/ :^devicetree-rebasing/ &&
> +        exit 1 || exit 0
>
>  # build documentation
>  docs:
> --
> 2.34.1
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
Simon Glass Dec. 28, 2023, 1:37 p.m. UTC | #2
On Thu, Dec 28, 2023 at 12:03 PM Ilias Apalodimas
<ilias.apalodimas@linaro.org> wrote:
>
> On Thu, 28 Dec 2023 at 13:58, Sumit Garg <sumit.garg@linaro.org> wrote:
> >
> > Since devicetree-rebasing is an external repo with its own coding style,
> > exclude it from Azure and gitlab CI CONFIG checks.
> >
> > Reviewed-by: Tom Rini <trini@konsulko.com>
> > Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> > ---
> >
> > Changes in v3:
> > --------------
> > - Picked up review tag
> >
> > Changes in v2:
> > --------------
> > - Excluded gitab CI config check and added commit description.
> >
> >  .azure-pipelines.yml | 3 ++-
> >  .gitlab-ci.yml       | 3 ++-
> >  2 files changed, 4 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
index d6f3fa423c6..f100c4493e6 100644
--- a/.azure-pipelines.yml
+++ b/.azure-pipelines.yml
@@ -65,7 +65,8 @@  stages:
       # have no matches.
       - script: git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
                   :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
-                  :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
+                  :^include/linux/kconfig.h :^tools/ :^devicetree-rebasing/ &&
+                  exit 1 || exit 0
 
   - job: docs
     displayName: 'Build documentation'
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index fee165198ae..96ca9f72bbc 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -161,7 +161,8 @@  check for new CONFIG symbols outside Kconfig:
     # have no matches.
     - git grep -E '^#[[:blank:]]*(define|undef)[[:blank:]]*CONFIG_'
         :^doc/ :^arch/arm/dts/ :^scripts/kconfig/lkc.h
-        :^include/linux/kconfig.h :^tools/ && exit 1 || exit 0
+        :^include/linux/kconfig.h :^tools/ :^devicetree-rebasing/ &&
+        exit 1 || exit 0
 
 # build documentation
 docs: