| Message ID | 1553859161-2628-1-git-send-email-yamada.masahiro@socionext.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [v2] drm: prefix header search paths with $(srctree)/ | expand |
On Fri, Mar 29, 2019 at 08:32:41PM +0900, Masahiro Yamada wrote: > Currently, the Kbuild core manipulates header search paths in a crazy > way [1]. > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > the search paths in the srctree. Some Makefiles are already written in > that way, but not all. The goal of this work is to make the notation > consistent, and finally get rid of the gross hacks. > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > ("kbuild: do not drop -I without parameter"). > > [1]: https://patchwork.kernel.org/patch/9632347/ > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > --- > > I put all gpu/drm changes into a single patch because > they are trivial conversion. > > If you are interested in the big picture of this work, > the full patch set is available at the following URL. > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git build-test > > > Changes in v2: > - fix up the new driver komeda > - Add Sam's Reviewed-by > > drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- > drivers/gpu/drm/amd/lib/Makefile | 2 +- > drivers/gpu/drm/arm/display/komeda/Makefile | 4 ++-- > drivers/gpu/drm/i915/gvt/Makefile | 2 +- > drivers/gpu/drm/msm/Makefile | 6 +++--- > drivers/gpu/drm/nouveau/Kbuild | 8 ++++---- > 6 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile > index 466da59..62bf9da 100644 > --- a/drivers/gpu/drm/amd/amdgpu/Makefile > +++ b/drivers/gpu/drm/amd/amdgpu/Makefile > @@ -23,7 +23,7 @@ > # Makefile for the drm device driver. This driver provides support for the > # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher. > > -FULL_AMD_PATH=$(src)/.. > +FULL_AMD_PATH=$(srctree)/$(src)/.. > DISPLAY_FOLDER_NAME=display > FULL_AMD_DISPLAY_PATH = $(FULL_AMD_PATH)/$(DISPLAY_FOLDER_NAME) > > diff --git a/drivers/gpu/drm/amd/lib/Makefile b/drivers/gpu/drm/amd/lib/Makefile > index 6902430..d534992 100644 > --- a/drivers/gpu/drm/amd/lib/Makefile > +++ b/drivers/gpu/drm/amd/lib/Makefile > @@ -27,6 +27,6 @@ > # driver components or later moved to kernel/lib for sharing with > # other drivers. > > -ccflags-y := -I$(src)/../include > +ccflags-y := -I $(srctree)/$(src)/../include > > obj-$(CONFIG_CHASH) += chash.o > diff --git a/drivers/gpu/drm/arm/display/komeda/Makefile b/drivers/gpu/drm/arm/display/komeda/Makefile > index 1b875e5..a72e30c 100644 > --- a/drivers/gpu/drm/arm/display/komeda/Makefile > +++ b/drivers/gpu/drm/arm/display/komeda/Makefile > @@ -1,8 +1,8 @@ > # SPDX-License-Identifier: GPL-2.0 > > ccflags-y := \ > - -I$(src)/../include \ > - -I$(src) > + -I $(srctree)/$(src)/../include \ > + -I $(srctree)/$(src) > > komeda-y := \ > komeda_drv.o \ > diff --git a/drivers/gpu/drm/i915/gvt/Makefile b/drivers/gpu/drm/i915/gvt/Makefile > index 271fb46..ea8324a 100644 > --- a/drivers/gpu/drm/i915/gvt/Makefile > +++ b/drivers/gpu/drm/i915/gvt/Makefile > @@ -5,5 +5,5 @@ GVT_SOURCE := gvt.o aperture_gm.o handlers.o vgpu.o trace_points.o firmware.o \ > execlist.o scheduler.o sched_policy.o mmio_context.o cmd_parser.o debugfs.o \ > fb_decoder.o dmabuf.o page_track.o > > -ccflags-y += -I$(src) -I$(src)/$(GVT_DIR) > +ccflags-y += -I $(srctree)/$(src) -I $(srctree)/$(src)/$(GVT_DIR)/ > i915-y += $(addprefix $(GVT_DIR)/, $(GVT_SOURCE)) > diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile > index 56a70c7..b7b1ebd 100644 > --- a/drivers/gpu/drm/msm/Makefile > +++ b/drivers/gpu/drm/msm/Makefile > @@ -1,7 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0 > -ccflags-y := -Idrivers/gpu/drm/msm > -ccflags-y += -Idrivers/gpu/drm/msm/disp/dpu1 > -ccflags-$(CONFIG_DRM_MSM_DSI) += -Idrivers/gpu/drm/msm/dsi > +ccflags-y := -I $(srctree)/$(src) > +ccflags-y += -I $(srctree)/$(src)/disp/dpu1 > +ccflags-$(CONFIG_DRM_MSM_DSI) += -I $(srctree)/$(src)/dsi > > msm-y := \ > adreno/adreno_device.o \ > diff --git a/drivers/gpu/drm/nouveau/Kbuild b/drivers/gpu/drm/nouveau/Kbuild > index ea3035e..4fae728 100644 > --- a/drivers/gpu/drm/nouveau/Kbuild > +++ b/drivers/gpu/drm/nouveau/Kbuild > @@ -1,7 +1,7 @@ > -ccflags-y += -I$(src)/include > -ccflags-y += -I$(src)/include/nvkm > -ccflags-y += -I$(src)/nvkm > -ccflags-y += -I$(src) > +ccflags-y += -I $(srctree)/$(src)/include > +ccflags-y += -I $(srctree)/$(src)/include/nvkm > +ccflags-y += -I $(srctree)/$(src)/nvkm > +ccflags-y += -I $(srctree)/$(src) > > # NVKM - HW resource manager > #- code also used by various userspace tools/tests > -- > 2.7.4
On Fri, Mar 29, 2019 at 08:32:41PM +0900, Masahiro Yamada wrote: > Currently, the Kbuild core manipulates header search paths in a crazy > way [1]. > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > the search paths in the srctree. Some Makefiles are already written in > that way, but not all. The goal of this work is to make the notation > consistent, and finally get rid of the gross hacks. > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > ("kbuild: do not drop -I without parameter"). > > [1]: https://patchwork.kernel.org/patch/9632347/ > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > --- > > I put all gpu/drm changes into a single patch because > they are trivial conversion. > > If you are interested in the big picture of this work, > the full patch set is available at the following URL. > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git build-test > > > Changes in v2: > - fix up the new driver komeda > - Add Sam's Reviewed-by > > drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- > drivers/gpu/drm/amd/lib/Makefile | 2 +- > drivers/gpu/drm/arm/display/komeda/Makefile | 4 ++-- > drivers/gpu/drm/i915/gvt/Makefile | 2 +- > drivers/gpu/drm/msm/Makefile | 6 +++--- > drivers/gpu/drm/nouveau/Kbuild | 8 ++++---- > 6 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile > index 466da59..62bf9da 100644 > --- a/drivers/gpu/drm/amd/amdgpu/Makefile > +++ b/drivers/gpu/drm/amd/amdgpu/Makefile > @@ -23,7 +23,7 @@ > # Makefile for the drm device driver. This driver provides support for the > # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher. > > -FULL_AMD_PATH=$(src)/.. > +FULL_AMD_PATH=$(srctree)/$(src)/.. > DISPLAY_FOLDER_NAME=display > FULL_AMD_DISPLAY_PATH = $(FULL_AMD_PATH)/$(DISPLAY_FOLDER_NAME) > > diff --git a/drivers/gpu/drm/amd/lib/Makefile b/drivers/gpu/drm/amd/lib/Makefile > index 6902430..d534992 100644 > --- a/drivers/gpu/drm/amd/lib/Makefile > +++ b/drivers/gpu/drm/amd/lib/Makefile > @@ -27,6 +27,6 @@ > # driver components or later moved to kernel/lib for sharing with > # other drivers. > > -ccflags-y := -I$(src)/../include > +ccflags-y := -I $(srctree)/$(src)/../include > > obj-$(CONFIG_CHASH) += chash.o > diff --git a/drivers/gpu/drm/arm/display/komeda/Makefile b/drivers/gpu/drm/arm/display/komeda/Makefile > index 1b875e5..a72e30c 100644 > --- a/drivers/gpu/drm/arm/display/komeda/Makefile > +++ b/drivers/gpu/drm/arm/display/komeda/Makefile > @@ -1,8 +1,8 @@ > # SPDX-License-Identifier: GPL-2.0 > > ccflags-y := \ > - -I$(src)/../include \ > - -I$(src) > + -I $(srctree)/$(src)/../include \ > + -I $(srctree)/$(src) > > komeda-y := \ > komeda_drv.o \ For komeda: Acked-by: Liviu Dudau <liviu.dudau@arm.com> I'm assuming the series in going to be merged into one go, so I don't have to take the individual patch in my tree, but if I'm wrong please let me know. Best regards, Liviu > diff --git a/drivers/gpu/drm/i915/gvt/Makefile b/drivers/gpu/drm/i915/gvt/Makefile > index 271fb46..ea8324a 100644 > --- a/drivers/gpu/drm/i915/gvt/Makefile > +++ b/drivers/gpu/drm/i915/gvt/Makefile > @@ -5,5 +5,5 @@ GVT_SOURCE := gvt.o aperture_gm.o handlers.o vgpu.o trace_points.o firmware.o \ > execlist.o scheduler.o sched_policy.o mmio_context.o cmd_parser.o debugfs.o \ > fb_decoder.o dmabuf.o page_track.o > > -ccflags-y += -I$(src) -I$(src)/$(GVT_DIR) > +ccflags-y += -I $(srctree)/$(src) -I $(srctree)/$(src)/$(GVT_DIR)/ > i915-y += $(addprefix $(GVT_DIR)/, $(GVT_SOURCE)) > diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile > index 56a70c7..b7b1ebd 100644 > --- a/drivers/gpu/drm/msm/Makefile > +++ b/drivers/gpu/drm/msm/Makefile > @@ -1,7 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0 > -ccflags-y := -Idrivers/gpu/drm/msm > -ccflags-y += -Idrivers/gpu/drm/msm/disp/dpu1 > -ccflags-$(CONFIG_DRM_MSM_DSI) += -Idrivers/gpu/drm/msm/dsi > +ccflags-y := -I $(srctree)/$(src) > +ccflags-y += -I $(srctree)/$(src)/disp/dpu1 > +ccflags-$(CONFIG_DRM_MSM_DSI) += -I $(srctree)/$(src)/dsi > > msm-y := \ > adreno/adreno_device.o \ > diff --git a/drivers/gpu/drm/nouveau/Kbuild b/drivers/gpu/drm/nouveau/Kbuild > index ea3035e..4fae728 100644 > --- a/drivers/gpu/drm/nouveau/Kbuild > +++ b/drivers/gpu/drm/nouveau/Kbuild > @@ -1,7 +1,7 @@ > -ccflags-y += -I$(src)/include > -ccflags-y += -I$(src)/include/nvkm > -ccflags-y += -I$(src)/nvkm > -ccflags-y += -I$(src) > +ccflags-y += -I $(srctree)/$(src)/include > +ccflags-y += -I $(srctree)/$(src)/include/nvkm > +ccflags-y += -I $(srctree)/$(src)/nvkm > +ccflags-y += -I $(srctree)/$(src) > > # NVKM - HW resource manager > #- code also used by various userspace tools/tests > -- > 2.7.4 >
Hi. On Fri, Mar 29, 2019 at 8:37 PM Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > > Currently, the Kbuild core manipulates header search paths in a crazy > way [1]. > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > the search paths in the srctree. Some Makefiles are already written in > that way, but not all. The goal of this work is to make the notation > consistent, and finally get rid of the gross hacks. > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > ("kbuild: do not drop -I without parameter"). > > [1]: https://patchwork.kernel.org/patch/9632347/ > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > --- > > I put all gpu/drm changes into a single patch because > they are trivial conversion. > > If you are interested in the big picture of this work, > the full patch set is available at the following URL. > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git build-test Is somebody taking care of this? > > Changes in v2: > - fix up the new driver komeda > - Add Sam's Reviewed-by > > drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- > drivers/gpu/drm/amd/lib/Makefile | 2 +- > drivers/gpu/drm/arm/display/komeda/Makefile | 4 ++-- > drivers/gpu/drm/i915/gvt/Makefile | 2 +- > drivers/gpu/drm/msm/Makefile | 6 +++--- > drivers/gpu/drm/nouveau/Kbuild | 8 ++++---- > 6 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile > index 466da59..62bf9da 100644 > --- a/drivers/gpu/drm/amd/amdgpu/Makefile > +++ b/drivers/gpu/drm/amd/amdgpu/Makefile > @@ -23,7 +23,7 @@ > # Makefile for the drm device driver. This driver provides support for the > # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher. > > -FULL_AMD_PATH=$(src)/.. > +FULL_AMD_PATH=$(srctree)/$(src)/.. > DISPLAY_FOLDER_NAME=display > FULL_AMD_DISPLAY_PATH = $(FULL_AMD_PATH)/$(DISPLAY_FOLDER_NAME) > > diff --git a/drivers/gpu/drm/amd/lib/Makefile b/drivers/gpu/drm/amd/lib/Makefile > index 6902430..d534992 100644 > --- a/drivers/gpu/drm/amd/lib/Makefile > +++ b/drivers/gpu/drm/amd/lib/Makefile > @@ -27,6 +27,6 @@ > # driver components or later moved to kernel/lib for sharing with > # other drivers. > > -ccflags-y := -I$(src)/../include > +ccflags-y := -I $(srctree)/$(src)/../include > > obj-$(CONFIG_CHASH) += chash.o > diff --git a/drivers/gpu/drm/arm/display/komeda/Makefile b/drivers/gpu/drm/arm/display/komeda/Makefile > index 1b875e5..a72e30c 100644 > --- a/drivers/gpu/drm/arm/display/komeda/Makefile > +++ b/drivers/gpu/drm/arm/display/komeda/Makefile > @@ -1,8 +1,8 @@ > # SPDX-License-Identifier: GPL-2.0 > > ccflags-y := \ > - -I$(src)/../include \ > - -I$(src) > + -I $(srctree)/$(src)/../include \ > + -I $(srctree)/$(src) > > komeda-y := \ > komeda_drv.o \ > diff --git a/drivers/gpu/drm/i915/gvt/Makefile b/drivers/gpu/drm/i915/gvt/Makefile > index 271fb46..ea8324a 100644 > --- a/drivers/gpu/drm/i915/gvt/Makefile > +++ b/drivers/gpu/drm/i915/gvt/Makefile > @@ -5,5 +5,5 @@ GVT_SOURCE := gvt.o aperture_gm.o handlers.o vgpu.o trace_points.o firmware.o \ > execlist.o scheduler.o sched_policy.o mmio_context.o cmd_parser.o debugfs.o \ > fb_decoder.o dmabuf.o page_track.o > > -ccflags-y += -I$(src) -I$(src)/$(GVT_DIR) > +ccflags-y += -I $(srctree)/$(src) -I $(srctree)/$(src)/$(GVT_DIR)/ > i915-y += $(addprefix $(GVT_DIR)/, $(GVT_SOURCE)) > diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile > index 56a70c7..b7b1ebd 100644 > --- a/drivers/gpu/drm/msm/Makefile > +++ b/drivers/gpu/drm/msm/Makefile > @@ -1,7 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0 > -ccflags-y := -Idrivers/gpu/drm/msm > -ccflags-y += -Idrivers/gpu/drm/msm/disp/dpu1 > -ccflags-$(CONFIG_DRM_MSM_DSI) += -Idrivers/gpu/drm/msm/dsi > +ccflags-y := -I $(srctree)/$(src) > +ccflags-y += -I $(srctree)/$(src)/disp/dpu1 > +ccflags-$(CONFIG_DRM_MSM_DSI) += -I $(srctree)/$(src)/dsi > > msm-y := \ > adreno/adreno_device.o \ > diff --git a/drivers/gpu/drm/nouveau/Kbuild b/drivers/gpu/drm/nouveau/Kbuild > index ea3035e..4fae728 100644 > --- a/drivers/gpu/drm/nouveau/Kbuild > +++ b/drivers/gpu/drm/nouveau/Kbuild > @@ -1,7 +1,7 @@ > -ccflags-y += -I$(src)/include > -ccflags-y += -I$(src)/include/nvkm > -ccflags-y += -I$(src)/nvkm > -ccflags-y += -I$(src) > +ccflags-y += -I $(srctree)/$(src)/include > +ccflags-y += -I $(srctree)/$(src)/include/nvkm > +ccflags-y += -I $(srctree)/$(src)/nvkm > +ccflags-y += -I $(srctree)/$(src) > > # NVKM - HW resource manager > #- code also used by various userspace tools/tests > -- > 2.7.4 >
On Fri, 26 Apr 2019 at 11:46, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > > Hi. > > > On Fri, Mar 29, 2019 at 8:37 PM Masahiro Yamada > <yamada.masahiro@socionext.com> wrote: > > > > Currently, the Kbuild core manipulates header search paths in a crazy > > way [1]. > > > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > > the search paths in the srctree. Some Makefiles are already written in > > that way, but not all. The goal of this work is to make the notation > > consistent, and finally get rid of the gross hacks. > > > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > > ("kbuild: do not drop -I without parameter"). > > > > [1]: https://patchwork.kernel.org/patch/9632347/ > > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > > --- > > > > I put all gpu/drm changes into a single patch because > > they are trivial conversion. > > > > If you are interested in the big picture of this work, > > the full patch set is available at the following URL. > > > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git build-test > > > Is somebody taking care of this? > Are you expecting this to be merged in the drm tree? if so please indicate that when posting. I'd assumed this would go via kbuild tree. If the later, Acked-by: Dave Airlie <airlied@redhat.com> Dave.
Hi Dave, > -----Original Message----- > From: Dave Airlie [mailto:airlied@gmail.com] > Sent: Friday, April 26, 2019 11:19 AM > To: Yamada, Masahiro/山田 真弘 <yamada.masahiro@socionext.com> > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; > dri-devel <dri-devel@lists.freedesktop.org>; nouveau > <nouveau@lists.freedesktop.org>; Sam Ravnborg <sam@ravnborg.org>; David > (ChunMing) Zhou <David1.Zhou@amd.com>; amd-gfx mailing list > <amd-gfx@lists.freedesktop.org>; James (Qian) Wang > <james.qian.wang@arm.com>; Ben Skeggs <bskeggs@redhat.com>; > linux-arm-msm <linux-arm-msm@vger.kernel.org>; Intel Graphics > Development <intel-gfx@lists.freedesktop.org>; > intel-gvt-dev@lists.freedesktop.org; Linux Kernel Mailing List > <linux-kernel@vger.kernel.org>; Christian König > <christian.koenig@amd.com>; Alex Deucher <alexander.deucher@amd.com>; > freedreno@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH v2] drm: prefix header search paths with > $(srctree)/ > > On Fri, 26 Apr 2019 at 11:46, Masahiro Yamada > <yamada.masahiro@socionext.com> wrote: > > > > Hi. > > > > > > On Fri, Mar 29, 2019 at 8:37 PM Masahiro Yamada > > <yamada.masahiro@socionext.com> wrote: > > > > > > Currently, the Kbuild core manipulates header search paths in a crazy > > > way [1]. > > > > > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > > > the search paths in the srctree. Some Makefiles are already written > in > > > that way, but not all. The goal of this work is to make the notation > > > consistent, and finally get rid of the gross hacks. > > > > > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > > > ("kbuild: do not drop -I without parameter"). > > > > > > [1]: https://patchwork.kernel.org/patch/9632347/ > > > > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > > > --- > > > > > > I put all gpu/drm changes into a single patch because > > > they are trivial conversion. > > > > > > If you are interested in the big picture of this work, > > > the full patch set is available at the following URL. > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.g > it build-test > > > > > > Is somebody taking care of this? > > > > Are you expecting this to be merged in the drm tree? if so please > indicate that when posting. Sorry for unclearness. Could you apply this to your drm tree? Thanks. > I'd assumed this would go via kbuild tree. > > If the later, > Acked-by: Dave Airlie <airlied@redhat.com> > Dave.
Daniel, drm-misc-next-fixes? Dave. On Fri, 26 Apr 2019 at 12:25, <yamada.masahiro@socionext.com> wrote: > > Hi Dave, > > > -----Original Message----- > > From: Dave Airlie [mailto:airlied@gmail.com] > > Sent: Friday, April 26, 2019 11:19 AM > > To: Yamada, Masahiro/山田 真弘 <yamada.masahiro@socionext.com> > > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; > > dri-devel <dri-devel@lists.freedesktop.org>; nouveau > > <nouveau@lists.freedesktop.org>; Sam Ravnborg <sam@ravnborg.org>; David > > (ChunMing) Zhou <David1.Zhou@amd.com>; amd-gfx mailing list > > <amd-gfx@lists.freedesktop.org>; James (Qian) Wang > > <james.qian.wang@arm.com>; Ben Skeggs <bskeggs@redhat.com>; > > linux-arm-msm <linux-arm-msm@vger.kernel.org>; Intel Graphics > > Development <intel-gfx@lists.freedesktop.org>; > > intel-gvt-dev@lists.freedesktop.org; Linux Kernel Mailing List > > <linux-kernel@vger.kernel.org>; Christian König > > <christian.koenig@amd.com>; Alex Deucher <alexander.deucher@amd.com>; > > freedreno@lists.freedesktop.org > > Subject: Re: [Intel-gfx] [PATCH v2] drm: prefix header search paths with > > $(srctree)/ > > > > On Fri, 26 Apr 2019 at 11:46, Masahiro Yamada > > <yamada.masahiro@socionext.com> wrote: > > > > > > Hi. > > > > > > > > > On Fri, Mar 29, 2019 at 8:37 PM Masahiro Yamada > > > <yamada.masahiro@socionext.com> wrote: > > > > > > > > Currently, the Kbuild core manipulates header search paths in a crazy > > > > way [1]. > > > > > > > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > > > > the search paths in the srctree. Some Makefiles are already written > > in > > > > that way, but not all. The goal of this work is to make the notation > > > > consistent, and finally get rid of the gross hacks. > > > > > > > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > > > > ("kbuild: do not drop -I without parameter"). > > > > > > > > [1]: https://patchwork.kernel.org/patch/9632347/ > > > > > > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > > > > --- > > > > > > > > I put all gpu/drm changes into a single patch because > > > > they are trivial conversion. > > > > > > > > If you are interested in the big picture of this work, > > > > the full patch set is available at the following URL. > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.g > > it build-test > > > > > > > > > Is somebody taking care of this? > > > > > > > Are you expecting this to be merged in the drm tree? if so please > > indicate that when posting. > > > Sorry for unclearness. > > Could you apply this to your drm tree? > > Thanks. > > > > > > I'd assumed this would go via kbuild tree. > > > > If the later, > > Acked-by: Dave Airlie <airlied@redhat.com> > > Dave.
On Fri, Apr 26, 2019 at 12:56:48PM +1000, Dave Airlie wrote: > Daniel, drm-misc-next-fixes? Makes sense. Pushed. Cheers, Daniel > > Dave. > > On Fri, 26 Apr 2019 at 12:25, <yamada.masahiro@socionext.com> wrote: > > > > Hi Dave, > > > > > -----Original Message----- > > > From: Dave Airlie [mailto:airlied@gmail.com] > > > Sent: Friday, April 26, 2019 11:19 AM > > > To: Yamada, Masahiro/山田 真弘 <yamada.masahiro@socionext.com> > > > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; > > > dri-devel <dri-devel@lists.freedesktop.org>; nouveau > > > <nouveau@lists.freedesktop.org>; Sam Ravnborg <sam@ravnborg.org>; David > > > (ChunMing) Zhou <David1.Zhou@amd.com>; amd-gfx mailing list > > > <amd-gfx@lists.freedesktop.org>; James (Qian) Wang > > > <james.qian.wang@arm.com>; Ben Skeggs <bskeggs@redhat.com>; > > > linux-arm-msm <linux-arm-msm@vger.kernel.org>; Intel Graphics > > > Development <intel-gfx@lists.freedesktop.org>; > > > intel-gvt-dev@lists.freedesktop.org; Linux Kernel Mailing List > > > <linux-kernel@vger.kernel.org>; Christian König > > > <christian.koenig@amd.com>; Alex Deucher <alexander.deucher@amd.com>; > > > freedreno@lists.freedesktop.org > > > Subject: Re: [Intel-gfx] [PATCH v2] drm: prefix header search paths with > > > $(srctree)/ > > > > > > On Fri, 26 Apr 2019 at 11:46, Masahiro Yamada > > > <yamada.masahiro@socionext.com> wrote: > > > > > > > > Hi. > > > > > > > > > > > > On Fri, Mar 29, 2019 at 8:37 PM Masahiro Yamada > > > > <yamada.masahiro@socionext.com> wrote: > > > > > > > > > > Currently, the Kbuild core manipulates header search paths in a crazy > > > > > way [1]. > > > > > > > > > > To fix this mess, I want all Makefiles to add explicit $(srctree)/ to > > > > > the search paths in the srctree. Some Makefiles are already written > > > in > > > > > that way, but not all. The goal of this work is to make the notation > > > > > consistent, and finally get rid of the gross hacks. > > > > > > > > > > Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 > > > > > ("kbuild: do not drop -I without parameter"). > > > > > > > > > > [1]: https://patchwork.kernel.org/patch/9632347/ > > > > > > > > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > > > > > --- > > > > > > > > > > I put all gpu/drm changes into a single patch because > > > > > they are trivial conversion. > > > > > > > > > > If you are interested in the big picture of this work, > > > > > the full patch set is available at the following URL. > > > > > > > > > > > > > git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.g > > > it build-test > > > > > > > > > > > > Is somebody taking care of this? > > > > > > > > > > Are you expecting this to be merged in the drm tree? if so please > > > indicate that when posting. > > > > > > Sorry for unclearness. > > > > Could you apply this to your drm tree? > > > > Thanks. > > > > > > > > > > > I'd assumed this would go via kbuild tree. > > > > > > If the later, > > > Acked-by: Dave Airlie <airlied@redhat.com> > > > Dave.
diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile index 466da59..62bf9da 100644 --- a/drivers/gpu/drm/amd/amdgpu/Makefile +++ b/drivers/gpu/drm/amd/amdgpu/Makefile @@ -23,7 +23,7 @@ # Makefile for the drm device driver. This driver provides support for the # Direct Rendering Infrastructure (DRI) in XFree86 4.1.0 and higher. -FULL_AMD_PATH=$(src)/.. +FULL_AMD_PATH=$(srctree)/$(src)/.. DISPLAY_FOLDER_NAME=display FULL_AMD_DISPLAY_PATH = $(FULL_AMD_PATH)/$(DISPLAY_FOLDER_NAME) diff --git a/drivers/gpu/drm/amd/lib/Makefile b/drivers/gpu/drm/amd/lib/Makefile index 6902430..d534992 100644 --- a/drivers/gpu/drm/amd/lib/Makefile +++ b/drivers/gpu/drm/amd/lib/Makefile @@ -27,6 +27,6 @@ # driver components or later moved to kernel/lib for sharing with # other drivers. -ccflags-y := -I$(src)/../include +ccflags-y := -I $(srctree)/$(src)/../include obj-$(CONFIG_CHASH) += chash.o diff --git a/drivers/gpu/drm/arm/display/komeda/Makefile b/drivers/gpu/drm/arm/display/komeda/Makefile index 1b875e5..a72e30c 100644 --- a/drivers/gpu/drm/arm/display/komeda/Makefile +++ b/drivers/gpu/drm/arm/display/komeda/Makefile @@ -1,8 +1,8 @@ # SPDX-License-Identifier: GPL-2.0 ccflags-y := \ - -I$(src)/../include \ - -I$(src) + -I $(srctree)/$(src)/../include \ + -I $(srctree)/$(src) komeda-y := \ komeda_drv.o \ diff --git a/drivers/gpu/drm/i915/gvt/Makefile b/drivers/gpu/drm/i915/gvt/Makefile index 271fb46..ea8324a 100644 --- a/drivers/gpu/drm/i915/gvt/Makefile +++ b/drivers/gpu/drm/i915/gvt/Makefile @@ -5,5 +5,5 @@ GVT_SOURCE := gvt.o aperture_gm.o handlers.o vgpu.o trace_points.o firmware.o \ execlist.o scheduler.o sched_policy.o mmio_context.o cmd_parser.o debugfs.o \ fb_decoder.o dmabuf.o page_track.o -ccflags-y += -I$(src) -I$(src)/$(GVT_DIR) +ccflags-y += -I $(srctree)/$(src) -I $(srctree)/$(src)/$(GVT_DIR)/ i915-y += $(addprefix $(GVT_DIR)/, $(GVT_SOURCE)) diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile index 56a70c7..b7b1ebd 100644 --- a/drivers/gpu/drm/msm/Makefile +++ b/drivers/gpu/drm/msm/Makefile @@ -1,7 +1,7 @@ # SPDX-License-Identifier: GPL-2.0 -ccflags-y := -Idrivers/gpu/drm/msm -ccflags-y += -Idrivers/gpu/drm/msm/disp/dpu1 -ccflags-$(CONFIG_DRM_MSM_DSI) += -Idrivers/gpu/drm/msm/dsi +ccflags-y := -I $(srctree)/$(src) +ccflags-y += -I $(srctree)/$(src)/disp/dpu1 +ccflags-$(CONFIG_DRM_MSM_DSI) += -I $(srctree)/$(src)/dsi msm-y := \ adreno/adreno_device.o \ diff --git a/drivers/gpu/drm/nouveau/Kbuild b/drivers/gpu/drm/nouveau/Kbuild index ea3035e..4fae728 100644 --- a/drivers/gpu/drm/nouveau/Kbuild +++ b/drivers/gpu/drm/nouveau/Kbuild @@ -1,7 +1,7 @@ -ccflags-y += -I$(src)/include -ccflags-y += -I$(src)/include/nvkm -ccflags-y += -I$(src)/nvkm -ccflags-y += -I$(src) +ccflags-y += -I $(srctree)/$(src)/include +ccflags-y += -I $(srctree)/$(src)/include/nvkm +ccflags-y += -I $(srctree)/$(src)/nvkm +ccflags-y += -I $(srctree)/$(src) # NVKM - HW resource manager #- code also used by various userspace tools/tests
Currently, the Kbuild core manipulates header search paths in a crazy way [1]. To fix this mess, I want all Makefiles to add explicit $(srctree)/ to the search paths in the srctree. Some Makefiles are already written in that way, but not all. The goal of this work is to make the notation consistent, and finally get rid of the gross hacks. Having whitespaces after -I does not matter since commit 48f6e3cf5bc6 ("kbuild: do not drop -I without parameter"). [1]: https://patchwork.kernel.org/patch/9632347/ Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> Reviewed-by: Sam Ravnborg <sam@ravnborg.org> --- I put all gpu/drm changes into a single patch because they are trivial conversion. If you are interested in the big picture of this work, the full patch set is available at the following URL. git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-kbuild.git build-test Changes in v2: - fix up the new driver komeda - Add Sam's Reviewed-by drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- drivers/gpu/drm/amd/lib/Makefile | 2 +- drivers/gpu/drm/arm/display/komeda/Makefile | 4 ++-- drivers/gpu/drm/i915/gvt/Makefile | 2 +- drivers/gpu/drm/msm/Makefile | 6 +++--- drivers/gpu/drm/nouveau/Kbuild | 8 ++++---- 6 files changed, 12 insertions(+), 12 deletions(-)