Re: linux-next: build failure after merge of the drm tree

.d2969f68dd64 100644 > --- a/drivers/gpu/drm/i915/gt/intel_gt_requests.h > +++ b/drivers/gpu/drm/i915/gt/intel_gt_requests.h > @@ -6,7 +6,7 @@ > #ifndef INTEL_GT_REQUESTS_H > #define INTEL_GT_REQUESTS_H > > -#include > +#include > > struct intel_engine_cs; > struct intel_gt; > -- > 2.32.0 > > -- > Cheers, > Stephen Rothwell -- Best Regards Masahiro Yamada

[PATCH] kbuild: use always-y instead of extra-y

As commit d0e628cd817f ("kbuild: doc: clarify the difference between extra-y and always-y") explained, extra-y should be used for listing the prerequsites of vmlinux. always-y is a better fix here. Signed-off-by: Masahiro Yamada --- Documentation/devicetree/bindings/Mak

Re: [PATCH] kbuild: use always-y instead of extra-y

On Thu, Jan 21, 2021 at 6:39 AM Rob Herring wrote: > > On Wed, Jan 20, 2021 at 03:23:51PM +0900, Masahiro Yamada wrote: > > As commit d0e628cd817f ("kbuild: doc: clarify the difference between > > extra-y and always-y") explained, extra-y should be used for listing >

[PATCH 1/3] printk: use CONFIG_CONSOLE_LOGLEVEL_* directly

on CHK include/generated/compile.h UPD include/generated/compile.h CC init/version.o AR init/built-in.a LD vmlinux.o ... For the same reason, do likewise for CONSOLE_LOGLEVEL_QUIET and MESSAGE_LOGLEVEL_DEFAULT. Signed-off-by: Masahiro Yamada ---

Re: [PATCH 1/3] printk: use CONFIG_CONSOLE_LOGLEVEL_* directly

On Tue, Feb 2, 2021 at 7:09 PM Sergey Senozhatsky wrote: > > On (21/02/02 16:02), Masahiro Yamada wrote: > > > > CONSOLE_LOGLEVEL_DEFAULT is nothing more than a shorthand of > > CONFIG_CONSOLE_LOGLEVEL_DEFAULT. > > > > When you change CONFIG_CONSOLE_LOGLEVEL_

Re: [PATCH 1/3] printk: use CONFIG_CONSOLE_LOGLEVEL_* directly

On Thu, Feb 4, 2021 at 12:23 AM Petr Mladek wrote: > > On Tue 2021-02-02 09:44:22, John Ogness wrote: > > On 2021-02-02, Masahiro Yamada wrote: > > > CONSOLE_LOGLEVEL_DEFAULT is nothing more than a shorthand of > > > CONFIG_CONSOLE_LOGLEVEL_DEFAUL

[PATCH] doc: use KCFLAGS instead of EXTRA_CFLAGS to pass flags from command line

g. Signed-off-by: Masahiro Yamada --- Documentation/process/4.Coding.rst| 2 +- Documentation/process/submit-checklist.rst| 2 +- Documentation/translations/it_IT/process/4.Coding.rst | 2 +- Documentation/translations/it_IT/proc

Re: [PATCH] [v2] Kbuild: move to -std=gnu11

building with -Wextra, ... > 'make W=1' option, as well as for three drivers in the kernel that always > enable -Werror, but it was only observed with the i915 driver so far. Same here. enable -Werror, but ... -> enable -Wextra, but ... Otherwise, Acked-by: Ma

Re: [PATCH] [v2] Kbuild: move to -std=gnu11

hanging the standard used, this patch explciitly enales the warnings with `-Wdeclaration-after-statement`, which takes effect regardless of which version of the C standard is in use." modpost is already built with -std=gnu89. If Wdeclaration-after-statement is implied by gnu89, why did nobody notice this before? -- Best Regards Masahiro Yamada

Re: linux-next: build failure after merge of the drm tree

On Mon, Sep 6, 2021 at 4:34 PM Daniel Vetter wrote: > > On Mon, Sep 6, 2021 at 12:49 AM Stephen Rothwell > wrote: > > Hi all, > > > > On Thu, 2 Sep 2021 07:50:38 +1000 Stephen Rothwell > > wrote: > > > > > > On Fri, 20 Aug 2021 15:23:34 +090

Re: [PATCH v2 3/3] Move IS_CONFIG_NONZERO() to kconfig.h

, even in the i915 driver. For a few sources of the warnings, replacing it with != 0 or > 0 is just fine. Of course, such an ugly macro is not worth being moved to -- Best Regards Masahiro Yamada

Re: [PATCH v2 3/3] Move IS_CONFIG_NONZERO() to kconfig.h

On Fri, Oct 1, 2021 at 12:55 AM Lucas De Marchi wrote: > > On Thu, Sep 30, 2021 at 11:01:36PM +0900, Masahiro Yamada wrote: > >On Thu, Sep 30, 2021 at 3:34 AM Lucas De Marchi > > wrote: > >> > >> The check for config value doesn't really belong to i915_

Re: [PATCH v3] drm/i915: remove IS_ACTIVE

counterpart. However the feedback received from Masahiro Yamada was that > it is too ugly, not providing much value. And just wrapping in a boolean > context is too dumb - we could simply open code it. > > As detailed in commit babaab2f4738 ("drm/i915: Encapsulate kconfig > c

Re: [PATCH v2 06/17] kbuild: enable building KUnit

+= kernel/ certs/ mm/ fs/ ipc/ security/ crypto/ block/ > +core-y += kernel/ certs/ mm/ fs/ ipc/ security/ crypto/ block/ kunit/ > > vmlinux-dirs := $(patsubst %/,%,$(filter %/, $(init-y) $(init-m) \ > $(core-y) $(core-m) $(drivers-y) $(drivers-m) \ > -

[PATCH] drm/i915: drop unneeded -Wall addition

The top level Makefile adds -Wall globally: KBUILD_CFLAGS := -Wall -Wundef -Werror=strict-prototypes -Wno-trigraphs \ I see two "-Wall" added for compiling under drivers/gpu/drm/i915/. Signed-off-by: Masahiro Yamada --- BTW, I have a question in the comment: "Note the

Re: [PATCH] drm/i915: drop unneeded -Wall addition

On Wed, May 15, 2019 at 3:25 PM Chris Wilson wrote: > > Quoting Masahiro Yamada (2019-05-15 05:37:53) > > The top level Makefile adds -Wall globally: > > > > KBUILD_CFLAGS := -Wall -Wundef -Werror=strict-prototypes -Wno-trigraphs > > \ > > > > I

Re: [PATCH v2] drm: prefix header search paths with $(srctree)/

Hi. On Fri, Mar 29, 2019 at 8:37 PM 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

[PATCH] drm: prefix header search paths with $(srctree)/

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 --- I put all gpu/drm changes into a si

Re: [PATCH] drm: prefix header search paths with $(srctree)/

On Thu, Jan 31, 2019 at 1:01 PM 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 wri

Re: mmotm 2019-07-04-15-01 uploaded (gpu/drm/i915/oa/)

27;drivers/gpu/drm/i915' failed > I checked next-20190704 tag. I see the empty file drivers/gpu/drm/i915/oa/Makefile Did someone delete it? -- Best Regards Masahiro Yamada

Re: mmotm 2019-07-04-15-01 uploaded (gpu/drm/i915/oa/)

On Fri, Jul 5, 2019 at 12:05 PM Masahiro Yamada wrote: > > On Fri, Jul 5, 2019 at 10:09 AM Randy Dunlap wrote: > > > > On 7/4/19 3:01 PM, a...@linux-foundation.org wrote: > > > The mm-of-the-moment snapshot 2019-07-04-15-01 has been uploaded to > > > > &

Re: mmotm 2019-07-04-15-01 uploaded (gpu/drm/i915/oa/)

On Fri, Jul 5, 2019 at 12:23 PM Randy Dunlap wrote: > > On 7/4/19 8:09 PM, Masahiro Yamada wrote: > > On Fri, Jul 5, 2019 at 12:05 PM Masahiro Yamada > > wrote: > >> > >> On Fri, Jul 5, 2019 at 10:09 AM Randy Dunlap wrote: > >>> > >

Re: [PATCH v7 06/18] kbuild: enable building KUnit

ndan Higgins > Cc: Masahiro Yamada > Cc: Michal Marek > Reviewed-by: Greg Kroah-Hartman > Reviewed-by: Logan Gunthorpe > --- > Kconfig | 2 ++ > Makefile | 2 +- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Kconfig b/Kconfig > index 48a80bea

Re: [PATCH v8 06/18] kbuild: enable building KUnit

ndan Higgins > Cc: Masahiro Yamada Please feel free to replace this with: Acked-by: Masahiro Yamada > Cc: Michal Marek > Reviewed-by: Greg Kroah-Hartman > Reviewed-by: Logan Gunthorpe -- Best Regards Masahiro Yamada

[PATCH v2] drm: prefix header search paths with $(srctree)/

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 Reviewed-by: Sam Ravnborg --- I put a

[PATCH] backlight: add include guards to platform_lcd.h and ili9320.h

Add header include guards just in case. Signed-off-by: Masahiro Yamada --- include/video/ili9320.h | 4 include/video/platform_lcd.h | 4 2 files changed, 8 insertions(+) diff --git a/include/video/ili9320.h b/include/video/ili9320.h index 62f424f0bc52..b76a0b8f16fc 100644

Re: [PATCH] drm/radeon: Add build directory to include path

On Wed, Jun 15, 2022 at 5:35 PM Michel Dänzer wrote: > > On 2022-04-14 18:57, Michel Dänzer wrote: > > On 2022-04-14 17:04, Masahiro Yamada wrote: > >> On Thu, Apr 14, 2022 at 10:50 PM Michel Dänzer > >> wrote: > >>> On 2022-04-14 15:34, Alex Deucher wro

Re: [PATCH] drm/radeon: Add build directory to include path

end it for in-tree drivers. > >> > >> Yes, exactly that. I'm like 95% sure that used to work, but I don't know > >> why exactly either. > >> > >>> FWIW, this is pretty common in the kernel according to git grep. > >> > >&g

Re: [PATCH v2 3/5] Makefile.compiler: replace cc-ifversion with compiler-specific macros

# gcc-11+, clang-14+ ifeq ($(call cc-min-version, 11, 14),y) dwarf-version-y := 5 else dwarf-version-y := 4 endif ... can be written in a more simpler way: dwarf-version-y := 4 dwarf-version-$(call gcc-min-version, 11) := 5 dwarf-version-$(call clang-min-version, 14) := 5 With $(call cc-min-version, 11, 14), you never know the meaning of 11, 14 until you see the definition of this macro. So, you feel like adding the comment "gcc-11+, clang-14+". The latter form, the code is self-documenting. > # ld-option > # Usage: KBUILD_LDFLAGS += $(call ld-option, -X, -Y) > -- > 2.37.2.672.g94769d06f0-goog > -- Best Regards Masahiro Yamada

[PATCH 1/2] drm/i915: change to_mock() to an inline function

Since this function is defined in a header file, it should be 'static inline' instead of 'static'. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/dr

Re: [PATCH 2/2] drm/i915: make more headers self-contained

On Fri, Nov 8, 2019 at 2:15 PM Masahiro Yamada wrote: > > The headers in the gem/selftests/, gt/selftests, gvt/, selftests/ > directories have never been compile-tested, but it would be possible > to make them self-contained. > > This commit only addresses missing an

[PATCH 2/2] drm/i915: make more headers self-contained

The headers in the gem/selftests/, gt/selftests, gvt/, selftests/ directories have never been compile-tested, but it would be possible to make them self-contained. This commit only addresses missing and forward struct declarations. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/i915/gem

[PATCH v2] drm/i915: make more headers self-contained

The headers in the gem/selftests/, gt/selftests, gvt/, selftests/ directories have never been compile-tested, but it would be possible to make them self-contained. This commit only addresses missing and forward struct declarations. Signed-off-by: Masahiro Yamada --- Rebase on git

[PATCH 2/4] drm/radeon: fix build rules of *_reg_safe.h

if_changed must have FORCE as a prerequisite, and the targets must be added to 'targets'. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/radeon/Makefile | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/radeon/Makefile

[PATCH 3/4] drm/radeon: use pattern rule to avoid code duplication in Makefile

This Makefile repeats similar build rules. Use a pattern rule. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/radeon/Makefile | 29 + 1 file changed, 1 insertion(+), 28 deletions(-) diff --git a/drivers/gpu/drm/radeon/Makefile b/drivers/gpu/drm/radeon/Makefile

[PATCH 4/4] drm/radeon: align short build log

/r300.o Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/radeon/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/radeon/Makefile b/drivers/gpu/drm/radeon/Makefile index 480a8d4a3c82..11c97edde54d 100644 --- a/drivers/gpu/drm/radeon/Makefile +++ b

Re: [PATCH] drm/amd/display: Fix compile error due to 'endif' missing

t; ifdef CONFIG_DRM_AMD_DC_DCN2_1 > CFLAGS_$(AMDDALPATH)/dc/dml/dcn21/display_mode_vba_21.o := $(dml_ccflags) > CFLAGS_$(AMDDALPATH)/dc/dml/dcn21/display_rq_dlg_calc_21.o := $(dml_ccflags) > -- > 2.6.2 > -- Best Regards Masahiro Yamada ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH -next] fbdev: c2p: Fix link failure on non-inlining

re...@ellerman.id.au > Signed-off-by: Geert Uytterhoeven Reviewed-by: Masahiro Yamada -- Best Regards Masahiro Yamada

Re: [PATCH -next] fbdev: c2p: Fix link failure on non-inlining

fig options are guarded by 'depends on ATARI' or 'depends on AMIGA'. The correct tag is: Fixes: 9012d011660e ("compiler: allow all arches to enable CONFIG_OPTIMIZE_INLINING") The commit id is stable. As an additional work, depends on (AMIGA || COMPILE_TEST) would be nice unless this driver contains m68k-specific code. -- Best Regards Masahiro Yamada

Re: [PATCH -next] fbdev: c2p: Fix link failure on non-inlining

anks. -- Best Regards Masahiro Yamada

[PATCH] drm/amd: remove meaningless descending into amd/amdkfd/

Since commit 04d5e2765802 ("drm/amdgpu: Merge amdkfd into amdgpu"), drivers/gpu/drm/amd/amdkfd/Makefile does not contain any syntax that is understood by the build system. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/d

Re: [RFC PATCH] gpu: no need to compile drm/ if CONFIG_DRM=n

th it, I plan to send a formal patch with > Signed-off-by. > CONFIG_DRM is tristate. Your patch requires everything under drivers/gpu/drm/ modular when CONFIG_DRM=m. At least, drivers/gpu/drm/drm_mipi_dsi.c is built-in only. I guess that is the reason why they use 'obj-y += drm/'. In other words, if you are able to make all the drm code modular, your patch is applicable. -- Best Regards Masahiro Yamada

[PATCH 5/5] drm: i915: hierachize Makefiles

You can use the -y syntax in sub-directory Makefiles of modules. Demonstrate how it works. PLEASE DO NOT APPLY FOR NOW: this is only for comments. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/i915/Makefile | 126 ++-- drivers/gpu/drm/i915/display/Makefile

[PATCH 0/5] kbuild: allow big modules to sub-divide Makefiles

y does not work. The single targets directly descend into the directory of that file resides. It missed subdir-ccflags-y if it is specifies in parent Makefiles. Perhaps, I will have to manage correct implementation of single targets. Masahiro Yamada (5): kbuild: treat an

[PATCH 3/4] video/logo: simplify cmd_logo

Shorten the code. It still works in the same way. Signed-off-by: Masahiro Yamada --- drivers/video/logo/Makefile | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile index 16f60c1e1766..7d672d40bf01 100644

[PATCH 1/4] video/logo: remove unneeded *.o pattern from clean-files

The pattern *.o is cleaned up globally by the top Makefile. Signed-off-by: Masahiro Yamada --- drivers/video/logo/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile index 228a89b9bdd1..10b75ce3ce09 100644 --- a

[PATCH 2/4] video/logo: fix unneeded generation of font C files

Currently, all the font C files are generated irrespective of CONFIG options. Adding them to extra-y is wrong. What we need to do here is to add them to 'targets' so that if_changed works properly. All files listed in 'targets' are cleaned, so clean-files is unneeded. Sig

[PATCH 0/4] video/logo: various fix and cleanups of drivers/video/logo/Makefile

Masahiro Yamada (4): video/logo: remove unneeded *.o pattern from clean-files video/logo: fix unneeded generation of font C files video/logo: simplify cmd_logo video/logo: move pnmtologo tool to drivers/video/logo/ from scripts/ drivers/video/logo/.gitignore | 1

Re: [PATCH 2/4] video/logo: fix unneeded generation of font C files

On Wed, Aug 21, 2019 at 12:56 PM Masahiro Yamada wrote: I will replace 'font' -> 'logo'. (My brain was corrupted.) > Currently, all the font C files are generated irrespective of CONFIG > options. Adding them to extra-y is wrong. What we need to do here is >

[PATCH v2 3/4] video/logo: simplify cmd_logo

Shorten the code. It still works in the same way. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/video/logo/Makefile | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile index 16f60c1e1766

[PATCH v2 4/4] video/logo: move pnmtologo tool to drivers/video/logo/ from scripts/

This tool is only used by drivers/video/logo/Makefile. No reason to keep it in scripts/. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/video/logo/.gitignore | 1 + drivers/video/logo/Makefile | 10 +- {scripts => drivers/video/l

[PATCH v2 1/4] video/logo: remove unneeded *.o pattern from clean-files

The pattern *.o is cleaned up globally by the top Makefile. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/video/logo/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile index 228a89b9bdd1

[PATCH v2 2/4] video/logo: do not generate unneeded logo C files

Currently, all the logo C files are generated irrespective of the CONFIG options. Adding them to extra-y is wrong. What we need to do here is to add them to 'targets' so that if_changed works properly. Files listed in 'targets' are cleaned, so clean-files is unneeded. Sig

[PATCH v2 0/4] video/logo: various fix and cleanups of drivers/video/logo/Makefile

Changes in v2: - Fix commit log Masahiro Yamada (4): video/logo: remove unneeded *.o pattern from clean-files video/logo: do not generate unneeded logo C files video/logo: simplify cmd_logo video/logo: move pnmtologo tool to drivers/video/logo/ from scripts/ drivers/video/logo

[PATCH 4/4] video/logo: move pnmtologo tool to drivers/video/logo/ from scripts/

This tool is only used by drivers/video/logo/Makefile. No reason to keep it in scripts/. Signed-off-by: Masahiro Yamada --- drivers/video/logo/.gitignore | 1 + drivers/video/logo/Makefile | 10 +- {scripts => drivers/video/logo}/pnmtologo.c | 0 scri

Re: [PATCH v2 00/14] Remove clang's -Qunused-arguments from KBUILD_CPPFLAGS

h/s390/kernel/vdso64/Makefile| 4 +-- > arch/s390/purgatory/Makefile| 2 +- > arch/x86/boot/compressed/Makefile | 2 +- > drivers/gpu/drm/amd/display/dc/dml/Makefile | 3 ++- > scripts/Kconfig.include | 2 +- > scripts/Makefile.clang | 2 ++ > scripts/Makefile.compiler | 8 +++--- > scripts/as-version.sh | 2 +- > 21 files changed, 74 insertions(+), 98 deletions(-) > --- > base-commit: 88603b6dc419445847923fcb7fe5080067a30f98 > change-id: 20221228-drop-qunused-arguments-0c5c7dae54fb > > Best regards, > -- > Nathan Chancellor > -- Best Regards Masahiro Yamada

Re: Linux 6.3-rc3

Kconfig time, so users do not need to give ARCH= option from the command line. https://github.com/u-boot/u-boot/blob/v2023.01/arch/Kconfig#L44 You may get a quick idea of what it will look like. I will take a look at this direction (the compiler choice in Kconfig first), but it will not happen soonish due to the limited time for upstream work. -- Best Regards Masahiro Yamada

[PATCH 1/2] drm/bridge: imx: fix mixed module-builtin object

odule-builtin objects"). Turn helpers in imx-ldb-helper.c into inline functions. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/bridge/imx/Makefile | 4 +- drivers/gpu/drm/bridge/imx/imx-ldb-helper.c | 221 drivers/gpu/drm/bridge/imx/imx-ldb-help

[PATCH 2/2] drm/bridge: imx: turn imx8{qm, qxp}-ldb into single-object modules

With the previous fix, these modules are built from a single C file. Rename the source files so they match the module names. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/bridge/imx/Makefile | 4 drivers/gpu/drm/bridge/imx/{imx8qm-ldb-drv.c => imx8qm-ld

[PATCH v2 1/2] drm/bridge: imx: fix mixed module-builtin object

odule-builtin objects"). Split imx-ldb-helper.c into a separate module. Signed-off-by: Masahiro Yamada --- Changes in v2: - Add a separate module instead of making the functions static inline drivers/gpu/drm/bridge/imx/Kconfig | 5 + drivers/gpu/drm/bridge/imx/Makefile

[PATCH v2 2/2] drm/bridge: imx: turn imx8{qm, qxp}-ldb into single-object modules

With the previous fix, these modules are built from a single C file. Rename the source files so they match the module names. Signed-off-by: Masahiro Yamada --- (no changes since v1) drivers/gpu/drm/bridge/imx/Makefile | 4 drivers/gpu/drm/bridge/imx/{imx8qm-ldb

Re: [PATCH 1/2] drm/bridge: imx: fix mixed module-builtin object

On Sun, Jun 4, 2023 at 1:52 PM Laurent Pinchart wrote: > > Hi Yamada-san, > > Thank you for the patch. > > On Sun, Jun 04, 2023 at 02:07:46AM +0900, Masahiro Yamada wrote: > > With CONFIG_DRM_IMX8QM_LDB=m and CONFIG_DRM_IMX8QXP_LDB=y (or vice > > versa), imx-ldb-help

Re: [PATCH v2 1/2] drm/bridge: imx: fix mixed module-builtin object

On Sun, Jun 4, 2023 at 10:26 PM Laurent Pinchart wrote: > > Hello Yamada-san, > > Thank you for the patch. > > On Sun, Jun 04, 2023 at 04:57:12PM +0900, Masahiro Yamada wrote: > > With CONFIG_DRM_IMX8QM_LDB=m and CONFIG_DRM_IMX8QXP_LDB=y (or vice > > versa), imx-ldb-

[PATCH v3 1/2] drm/bridge: imx: fix mixed module-builtin object

odule-builtin objects"). Split imx-ldb-helper.c into a separate module. Signed-off-by: Masahiro Yamada Reviewed-by: Laurent Pinchart Reviewed-by: Liu Ying --- Changes in v3: - Remove MODULE_ALIAS() Changes in v2: - Add a separate module instead of making the functions static inline d

[PATCH v3 2/2] drm/bridge: imx: turn imx8{qm, qxp}-ldb into single-object modules

With the previous fix, these modules are built from a single C file. Rename the source files so they match the module names. Signed-off-by: Masahiro Yamada Reviewed-by: Laurent Pinchart Reviewed-by: Liu Ying --- (no changes since v1) drivers/gpu/drm/bridge/imx/Makefile

Re: [PATCH] drm/amd/amdgpu: enable W=1 for amdgpu

nused-const-variable) \ > > + $(call cc-option, -Wstringop-truncation) \ > > + $(call cc-option, -Wpacked-not-aligned) > > +subdir-ccflags-y += $(cond-flags) > > subdir-ccflags-y += -Wno-unused-parameter > > subdir-ccflags-y += -Wno-type-limits > > subdir-ccflags-y += -Wno-sign-compare > > -- > > 2.40.1 > > -- Best Regards Masahiro Yamada

Re: [PATCH v5 00/17] Imagination Technologies PowerVR DRM driver

on/pvr_rogue_heap_config.h > create mode 100644 drivers/gpu/drm/imagination/pvr_rogue_meta.h > create mode 100644 drivers/gpu/drm/imagination/pvr_rogue_mips.h > create mode 100644 drivers/gpu/drm/imagination/pvr_rogue_mips_check.h > create mode 100644 drivers/gpu/drm/imagination/pvr_rogue_mmu_defs.h > create mode 100644 drivers/gpu/drm/imagination/pvr_stream.c > create mode 100644 drivers/gpu/drm/imagination/pvr_stream.h > create mode 100644 drivers/gpu/drm/imagination/pvr_stream_defs.c > create mode 100644 drivers/gpu/drm/imagination/pvr_stream_defs.h > create mode 100644 drivers/gpu/drm/imagination/pvr_sync.c > create mode 100644 drivers/gpu/drm/imagination/pvr_sync.h > create mode 100644 drivers/gpu/drm/imagination/pvr_vm.c > create mode 100644 drivers/gpu/drm/imagination/pvr_vm.h > create mode 100644 drivers/gpu/drm/imagination/pvr_vm_mips.c > create mode 100644 drivers/gpu/drm/imagination/pvr_vm_mips.h > create mode 100644 include/uapi/drm/pvr_drm.h > > -- > 2.41.0 > -- Best Regards Masahiro Yamada

[PATCH 09/28] drm/msm: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/msm/Makefile | 2 +- drivers/gpu/drm/msm/dsi/dsi.h| 6 +++--- drivers/gpu/drm/msm/edp/edp.h

[PATCH 25/28] drm/tdfx: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/tdfx/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tdfx/Makefile b/drivers/gpu/drm/tdfx/Makefile index 0379f29..74bd4ae3 100644

[PATCH 28/28] drm/via: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/via/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/via/Makefile b/drivers/gpu/drm/via/Makefile index d59e258..751fa8b 100644

[PATCH 24/28] drm/sis: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/sis/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/sis/Makefile b/drivers/gpu/drm/sis/Makefile index 441c061..7bf4c13 100644

[PATCH 27/28] drm/vgem: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/vgem/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vgem/Makefile b/drivers/gpu/drm/vgem/Makefile index bfcdea1..cb5d413 100644

[PATCH 23/28] drm/savage: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/savage/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/savage/Makefile b/drivers/gpu/drm/savage/Makefile index d8f84ac..cfd436b

[PATCH 22/28] drm/r128: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/r128/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/r128/Makefile b/drivers/gpu/drm/r128/Makefile index 1cc72ae..1a6700e 100644

[PATCH 19/28] drm/i2c: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/i2c/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/Makefile b/drivers/gpu/drm/i2c/Makefile index 43aa33b..a77acfc 100644

[PATCH 12/28] drm/radeon: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/radeon/Makefile | 2 +- drivers/gpu/drm/radeon/btc_dpm.c | 2 +- drivers/gpu/drm/radeon/ci_dpm.c | 2 +- drivers/gpu/drm/radeon

[PATCH 07/28] drm/hisilicon: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/hisilicon/hibmc/Makefile| 1 - drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 05/28] drm/bridge: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/bridge/Makefile| 2 -- drivers/gpu/drm/bridge/nxp-ptn3460.c | 10 +- drivers/gpu/drm/bridge/parade-ps8622.c | 8 3

[PATCH 10/28] drm/nouveau: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/msm/msm_fbdev.c | 2 +- drivers/gpu/drm/nouveau/Kbuild| 1 - drivers/gpu/drm/nouveau/nouveau_drm.c | 4 ++-- drivers/gpu/drm/nouveau

[PATCH 16/28] drm/vmwgfx: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/vmwgfx/Makefile | 3 --- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 2 +- drivers/gpu/drm/vmwgfx/vmwgfx_context.c | 2 +- drivers/gpu/drm

[PATCH 02/28] drm/amd: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 10 +- drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.c | 2

[PATCH 26/28] drm/udl: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/udl/Makefile | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/udl/Makefile b/drivers/gpu/drm/udl/Makefile index 195bcac..36f2e82 100644

[PATCH 11/28] drm/qxl: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/qxl/Makefile | 2 -- drivers/gpu/drm/qxl/qxl_debugfs.c | 2 +- drivers/gpu/drm/qxl/qxl_display.c | 2 +- drivers/gpu/drm/qxl/qxl_drv.c

[PATCH 00/28] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

Many Makefiles needed to add -Iinclude/drm as an include path, but the right thing to do is to include headers in the form #include This series fixes the source files, then rip off -Iinclude/drm flags. Masahiro Yamada (28): drm/ttm: fix include notation and remove -Iinclude/drm flag drm

[PATCH 06/28] drm/cirrus: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/cirrus/Makefile | 1 - drivers/gpu/drm/cirrus/cirrus_ttm.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu

[PATCH 04/28] drm/bochs: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/bochs/Makefile | 1 - drivers/gpu/drm/bochs/bochs.h | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm

[PATCH 18/28] drm/i810: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/i810/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i810/Makefile b/drivers/gpu/drm/i810/Makefile index 43844ec..639f859 100644

[PATCH 13/28] drm/tilcdc: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/tilcdc/Makefile | 1 - drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu

[PATCH 14/28] drm/vc4: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, use <...> instead of "..." for include/linux/*.h headers too. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/vc4/Makefile| 2 -- drivers/gpu/drm/vc4/

[PATCH 08/28] drm/mgag200: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/mgag200/Makefile | 1 - drivers/gpu/drm/mgag200/mgag200_ttm.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH 17/28] drm/gma500: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/gma500/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/Makefile b/drivers/gpu/drm/gma500/Makefile index 190e55f..c1c8dc1

[PATCH 01/28] drm/ttm: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. The fixes of include/drm/ttm/*.h will help driver Makefiles drop -Iinclude/drm flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/ttm/Makefile | 1 - drivers/gpu/drm/ttm/ttm_bo_vm.c

[PATCH 15/28] drm/virtio: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/virtio/Makefile | 2 -- drivers/gpu/drm/virtio/virtgpu_debugfs.c | 2 +- drivers/gpu/drm/virtio/virtgpu_drv.c | 4 ++-- drivers

[PATCH 03/28] drm/ast: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/ast/Makefile | 2 -- drivers/gpu/drm/ast/ast_ttm.c | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/ast

[PATCH 20/28] drm/mga: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/mga/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/mga/Makefile b/drivers/gpu/drm/mga/Makefile index 6068478..49e972c 100644

[PATCH 21/28] drm/omap: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/omapdrm/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/Makefile b/drivers/gpu/drm/omapdrm/Makefile index 48b7b75..b391be7

Re: [PATCH 05/28] drm/bridge: fix include notation and remove -Iinclude/drm flag

2017-04-21 14:51 GMT+09:00 Andrzej Hajda : > Hi Masahiro, > > On 20.04.2017 11:56, Masahiro Yamada wrote: >> Include instead of relative path from include/drm, then >> remove the -Iinclude/drm compiler flag. >> >> Signed-off-by: Masahiro Yamada >> ---

[PATCH v2 08/29] drm/hisilicon: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/hisilicon/hibmc/Makefile| 1 - drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c | 2 +- 2 files changed, 1 insertion(+), 2

[PATCH v2 11/29] drm/nouveau: fix include notation and remove -Iinclude/drm flag

Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/nouveau/Kbuild| 1 - drivers/gpu/drm/nouveau/nouveau_drm.c | 4 ++-- drivers/gpu/drm/nouveau/nouveau_drv.h | 2

[PATCH v2 25/29] drm/sis: remove unneeded -Iinclude/drm compiler flag

With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/sis/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/sis/Makefile b/drivers/gpu/drm/sis/Makefile index 441c061

  1   2   3   >