diff mbox series

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

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

Commit Message

Sumit Garg Dec. 14, 2023, 1:50 p.m. UTC
Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
---
 .azure-pipelines.yml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Tom Rini Dec. 14, 2023, 2:27 p.m. UTC | #1
On Thu, Dec 14, 2023 at 07:20:56PM +0530, Sumit Garg wrote:
> Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> ---
>  .azure-pipelines.yml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> 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'

The GitLab tests needs an equivalent change.
Sumit Garg Dec. 14, 2023, 2:46 p.m. UTC | #2
On Thu, 14 Dec 2023 at 19:57, Tom Rini <trini@konsulko.com> wrote:
>
> On Thu, Dec 14, 2023 at 07:20:56PM +0530, Sumit Garg wrote:
> > Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> > ---
> >  .azure-pipelines.yml | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > 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'
>
> The GitLab tests needs an equivalent change.
>

Sure I will do that in the next spin of this patch-set, thanks for catching it.

-Sumit

> --
> Tom
Simon Glass Dec. 20, 2023, 4:47 a.m. UTC | #3
Hi Sumit,

On Thu, 14 Dec 2023 at 07:46, Sumit Garg <sumit.garg@linaro.org> wrote:
>
> On Thu, 14 Dec 2023 at 19:57, Tom Rini <trini@konsulko.com> wrote:
> >
> > On Thu, Dec 14, 2023 at 07:20:56PM +0530, Sumit Garg wrote:
> > > Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> > > ---
> > >  .azure-pipelines.yml | 3 ++-
> > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> > > 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'
> >
> > The GitLab tests needs an equivalent change.
> >
>
> Sure I will do that in the next spin of this patch-set, thanks for catching it.

Also, please add a commit message with that :-)

Regards,
Simon
Sumit Garg Dec. 20, 2023, 11:26 a.m. UTC | #4
On Wed, 20 Dec 2023 at 10:17, Simon Glass <sjg@chromium.org> wrote:
>
> Hi Sumit,
>
> On Thu, 14 Dec 2023 at 07:46, Sumit Garg <sumit.garg@linaro.org> wrote:
> >
> > On Thu, 14 Dec 2023 at 19:57, Tom Rini <trini@konsulko.com> wrote:
> > >
> > > On Thu, Dec 14, 2023 at 07:20:56PM +0530, Sumit Garg wrote:
> > > > Signed-off-by: Sumit Garg <sumit.garg@linaro.org>
> > > > ---
> > > >  .azure-pipelines.yml | 3 ++-
> > > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > > >
> > > > 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'
> > >
> > > The GitLab tests needs an equivalent change.
> > >
> >
> > Sure I will do that in the next spin of this patch-set, thanks for catching it.
>
> Also, please add a commit message with that :-)
>

Ack.

-Sumit

> Regards,
> Simon
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'