Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Reply-To: christian.koenig@amd.com
Subject: Re: [PATCH] drm/sched: remove unneeded -Iinclude/drm compiler flag
To:     Nayan Deshmukh <nayan26deshmukh@gmail.com>,
        yamada.masahiro@socionext.com
Cc:     airlied@linux.ie, Alex Deucher <alexander.deucher@amd.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Maling list - DRI developers 
        <dri-devel@lists.freedesktop.org>,
        =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>
References: <1530853931-12061-1-git-send-email-yamada.masahiro@socionext.com>
 <CAK7LNAS_XrYKuv7OgTEt9RSGLbCe-XumV4QNKUtS2YT9HWMk3Q@mail.gmail.com>
 <CAFd4ddxsBsg0gKwW3iCidFRSC+p0VaQFQb=GyAZrgoVVGnDscg@mail.gmail.com>
From:   =?UTF-8?Q?Christian_K=c3=b6nig?= <ckoenig.leichtzumerken@gmail.com>
Message-ID: <e5fbc3a3-1b79-703d-966a-df7b0e07f655@gmail.com>
Date:   Mon, 30 Jul 2018 15:09:30 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.9.1
MIME-Version: 1.0
In-Reply-To: <CAFd4ddxsBsg0gKwW3iCidFRSC+p0VaQFQb=GyAZrgoVVGnDscg@mail.gmail.com>
Content-Type: multipart/alternative;
 boundary="------------267EE5EC39C8D4305ECB5EFF"
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

This is a multi-part message in MIME format.
--------------267EE5EC39C8D4305ECB5EFF
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit

Am 30.07.2018 um 14:49 schrieb Nayan Deshmukh:
>
>
> On Sat, Jul 28, 2018 at 6:54 PM Masahiro Yamada 
> <yamada.masahiro@socionext.com <mailto:yamada.masahiro@socionext.com>> 
> wrote:
>
>     Hi.
>
>     2018-07-06 14:12 GMT+09:00 Masahiro Yamada
>     <yamada.masahiro@socionext.com
>     <mailto:yamada.masahiro@socionext.com>>:
>     > I refactored the include directives under include/drm/ some time
>     ago.
>     > This flag is unneeded.
>     >
>
> A minor nit-pick in the commit message, something like "With the 
> refactored include directives under include/drm, this flag is 
> uneeded." might be better.
>
>     > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com
>     <mailto:yamada.masahiro@socionext.com>>
>
> With or without that the patch is Acked-by: Nayan Deshmukh 
> <nayan26deshmukh@gmail.com <mailto:nayan26deshmukh@gmail.com>>

I've pushed that patch into our internal branch for scheduler work.

Thanks,
Christian.

>
> Regards,
> Nayan Deshmukh
>
>
>     Ping?
>
>
>     > ---
>     >
>     >  drivers/gpu/drm/scheduler/Makefile | 1 -
>     >  1 file changed, 1 deletion(-)
>     >
>     > diff --git a/drivers/gpu/drm/scheduler/Makefile
>     b/drivers/gpu/drm/scheduler/Makefile
>     > index bd0377c..7665883 100644
>     > --- a/drivers/gpu/drm/scheduler/Makefile
>     > +++ b/drivers/gpu/drm/scheduler/Makefile
>     > @@ -20,7 +20,6 @@
>     >  # OTHER DEALINGS IN THE SOFTWARE.
>     >  #
>     >  #
>     > -ccflags-y := -Iinclude/drm
>     >  gpu-sched-y := gpu_scheduler.o sched_fence.o
>     >
>     >  obj-$(CONFIG_DRM_SCHED) += gpu-sched.o
>     > --
>     > 2.7.4
>     >
>
>
>
>     -- 
>     Best Regards
>     Masahiro Yamada
>     _______________________________________________
>     dri-devel mailing list
>     dri-devel@lists.freedesktop.org
>     <mailto:dri-devel@lists.freedesktop.org>
>     https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel


--------------267EE5EC39C8D4305ECB5EFF
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: 8bit

<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Am 30.07.2018 um 14:49 schrieb Nayan
      Deshmukh:<br>
    </div>
    <blockquote type="cite"
cite="mid:CAFd4ddxsBsg0gKwW3iCidFRSC+p0VaQFQb=GyAZrgoVVGnDscg@mail.gmail.com">
      <meta http-equiv="content-type" content="text/html; charset=utf-8">
      <div dir="ltr"><br>
        <br>
        <div class="gmail_quote">
          <div dir="ltr">On Sat, Jul 28, 2018 at 6:54 PM Masahiro Yamada
            &lt;<a href="mailto:yamada.masahiro@socionext.com"
              moz-do-not-send="true">yamada.masahiro@socionext.com</a>&gt;
            wrote:<br>
          </div>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
            <br>
            2018-07-06 14:12 GMT+09:00 Masahiro Yamada &lt;<a
              href="mailto:yamada.masahiro@socionext.com"
              target="_blank" moz-do-not-send="true">yamada.masahiro@socionext.com</a>&gt;:<br>
            &gt; I refactored the include directives under include/drm/
            some time ago.<br>
            &gt; This flag is unneeded.<br>
            &gt;<br>
          </blockquote>
          <div>A minor nit-pick in the commit message, something like
            "With the refactored include directives under include/drm,
            this flag is uneeded." might be better. </div>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">
            &gt; Signed-off-by: Masahiro Yamada &lt;<a
              href="mailto:yamada.masahiro@socionext.com"
              target="_blank" moz-do-not-send="true">yamada.masahiro@socionext.com</a>&gt;<br>
          </blockquote>
          <div>
            <div>With or without that the patch is Acked-by: Nayan
              Deshmukh &lt;<a href="mailto:nayan26deshmukh@gmail.com"
                target="_blank" moz-do-not-send="true">nayan26deshmukh@gmail.com</a>&gt;<br>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    I've pushed that patch into our internal branch for scheduler work.<br>
    <br>
    Thanks,<br>
    Christian.<br>
    <br>
    <blockquote type="cite"
cite="mid:CAFd4ddxsBsg0gKwW3iCidFRSC+p0VaQFQb=GyAZrgoVVGnDscg@mail.gmail.com">
      <div dir="ltr">
        <div class="gmail_quote">
          <div>
            <div><br>
            </div>
            <div>Regards,<br>
            </div>
            <div>Nayan Deshmukh <br>
            </div>
          </div>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <br>
            Ping?<br>
            <br>
            <br>
            &gt; ---<br>
            &gt;<br>
            &gt;  drivers/gpu/drm/scheduler/Makefile | 1 -<br>
            &gt;  1 file changed, 1 deletion(-)<br>
            &gt;<br>
            &gt; diff --git a/drivers/gpu/drm/scheduler/Makefile
            b/drivers/gpu/drm/scheduler/Makefile<br>
            &gt; index bd0377c..7665883 100644<br>
            &gt; --- a/drivers/gpu/drm/scheduler/Makefile<br>
            &gt; +++ b/drivers/gpu/drm/scheduler/Makefile<br>
            &gt; @@ -20,7 +20,6 @@<br>
            &gt;  # OTHER DEALINGS IN THE SOFTWARE.<br>
            &gt;  #<br>
            &gt;  #<br>
            &gt; -ccflags-y := -Iinclude/drm<br>
            &gt;  gpu-sched-y := gpu_scheduler.o sched_fence.o<br>
            &gt;<br>
            &gt;  obj-$(CONFIG_DRM_SCHED) += gpu-sched.o<br>
            &gt; --<br>
            &gt; 2.7.4<br>
            &gt;<br>
            <br>
            <br>
            <br>
            -- <br>
            Best Regards<br>
            Masahiro Yamada<br>
            _______________________________________________<br>
            dri-devel mailing list<br>
            <a href="mailto:dri-devel@lists.freedesktop.org"
              target="_blank" moz-do-not-send="true">dri-devel@lists.freedesktop.org</a><br>
            <a
              href="https://lists.freedesktop.org/mailman/listinfo/dri-devel"
              rel="noreferrer" target="_blank" moz-do-not-send="true">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a><br>
          </blockquote>
        </div>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
dri-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a>
<a class="moz-txt-link-freetext" href="https://lists.freedesktop.org/mailman/listinfo/dri-devel">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>

--------------267EE5EC39C8D4305ECB5EFF--