Changes in Makefile

9b926bcf - drm/radeon: Only build fbdev if DRM_FBDEV_EMULATION is set

Thu, 16 Mar 2023 09:37:38 +0000

drm/radeon: Only build fbdev if DRM_FBDEV_EMULATION is setMake building fbdev emulation depend on DRM_FBDEV_EMULATION. Alsorename the source file to radeon_fbdev.c to align with other fbdevfiles.Reviewed-by: Alex Deucher Signed-off-by: Thomas Zimmermann Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

01ad1d9c - drm/radeon: Drop legacy MST support

Wed, 17 Aug 2022 19:38:45 +0000

drm/radeon: Drop legacy MST supportRight now, radeon is technically the only non-atomic driver still makinguse of the MST helpers - and thus the final user of all of the legacy MSThelpers. Originally I was going to look into seeing if we could move legacyMST into the radeon driver itself, however:* SI and CIK both can use amdgpu, which still supports MST* It currently doesn't work according to my own testing. I'm sure with some troubleshooting we could likely fix it, but that brings me to point #2:* It was never actually enabled by default, and is still marked as experimental in the module parameter description* If people were using it, someone probably would have probably seen a bug report about how it is currently not functional by now. That certainly doesn't appear to be the case, since before getting access to my own hardware I had to go out of my way to try finding someone to help test whether this legacy MST code even works - even amongst AMD employees.* Getting rid of this code and only having atomic versions of the MST helpers to maintain is likely going to be a lot easier in the long run, and will make it a lot easier for others contributing to this code to follow along with what's happening.FWIW - if anyone still wants this code to be in the tree and has a goodidea of how to support this without needing to maintain the legacy MSThelpers (trying to move them would probably be acceptable), I'm happy tosuggestions. But my hope is that we can just drop this code and forgetabout it. I've already run this idea by Harry Wentland and Alex Deucher afew times as well.Signed-off-by: Lyude Paul Cc: Wayne Lin Cc: Ville Syrjälä Cc: Fangzhi Zuo Cc: Jani Nikula Cc: Imre Deak Cc: Daniel Vetter Cc: Sean Paul Acked-by: Alex Deucher Acked-by: Jani Nikula Link: https://patchwork.freedesktop.org/patch/msgid/20220817193847.557945-17-lyude@redhat.com List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

1f43b890 - drm/radeon: fix incorrrect SPDX-License-Identifiers

Wed, 15 Jun 2022 16:02:08 +0000

drm/radeon: fix incorrrect SPDX-License-Identifiersradeon is MIT. This were incorrectly changed incommit b24413180f56 ("License cleanup: add SPDX GPL-2.0 license identifier to files with no license")andcommit d198b34f3855 (".gitignore: add SPDX License Identifier")and:commit ec8f24b7faaf ("treewide: Add SPDX license identifier - Makefile/Kconfig")Fixes: d198b34f3855 (".gitignore: add SPDX License Identifier")Fixes: ec8f24b7faaf ("treewide: Add SPDX license identifier - Makefile/Kconfig")Fixes: b24413180f56 ("License cleanup: add SPDX GPL-2.0 license identifier to files with no license")Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/2053Reviewed-by: Christian König Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

790d8e8e - drm/radeon: change cik_default_state table from global to static

Sat, 23 Apr 2022 13:44:02 +0000

drm/radeon: change cik_default_state table from global to staticSparse reports these issuescik_blit_shaders.c:31:11: warning: symbol 'cik_default_state' was not declared. Should it be static?cik_blit_shaders.c:246:11: warning: symbol 'cik_default_size' was not declared. Should it be static?cik_default_state and cik_default_size are only used in cik.c. Single file symbolsshould be static. So move their definitions to cik_blit_shaders.h and change theirstorage-class-specifier to static.Remove unneeded cik_blit_shader.cSigned-off-by: Tom Rix Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

79847f13 - drm/radeon/kms: change evergreen_default_state table from global to static

Sat, 16 Apr 2022 18:47:36 +0000

drm/radeon/kms: change evergreen_default_state table from global to staticevergreen_default_state and evergreen_default_size are onlyused in evergreen.c. Single file symbols should be static.So move their definitions to evergreen_blit_shaders.hand change their storage-class-specifier to static.Remove unneeded evergreen_blit_shader.cevergreen_ps/vs definitions were removed withcommit 4f8629675800 ("drm/radeon/kms: remove r6xx+ blit copy routines")So their declarations in evergreen_blit_shader.hare not needed, so remove them.Signed-off-by: Tom Rix Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

6866a60a - drm/radeon: remove r600_blit_shaders.[c|h]

Sat, 09 Apr 2022 17:11:31 +0000

drm/radeon: remove r600_blit_shaders.[c|h]The only use of the global variables in r600_blit_shaders.cwere in the old drivers/gpu/drm/radeon/r600_blit.cThis file was removed incommit 8333f607a631 ("drm/radeon: remove UMS support")So remove the r600_blit_shaders.[c|h] filesSigned-off-by: Tom Rix Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

02410693 - drm/radeon: change cayman_default_state table from global to static

Thu, 07 Apr 2022 21:46:59 +0000

drm/radeon: change cayman_default_state table from global to staticcayman_default_state and cayman_default_size are onlyused in ni.c. Single file symbols should be static.So move their definitions to cayman_blit_shaders.hand change their storage-class-specifier to static.Remove unneeded cayman_blit_shader.ccayman_ps/vs definitions were removed withcommit 4f8629675800 ("drm/radeon/kms: remove r6xx+ blit copy routines")So their declarations in cayman_blit_shader.hare not needed, so remove them.Signed-off-by: Tom Rix Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

b0778bb0 - drm/radeon: change si_default_state table from global to static

Mon, 04 Apr 2022 22:57:10 +0000

drm/radeon: change si_default_state table from global to staticSmatch reports these issuessi_blit_shaders.c:31:11: warning: symbol 'si_default_state' was not declared. Should it be static?si_blit_shaders.c:253:11: warning: symbol 'si_default_size' was not declared. Should it be static?Both symbols are only used in si.c. Single file symbolsshould be static. So move the definition ofsi_default_state and si_default_size to si_blit_shader.hand change their storage-class-specifier to static.Remove unneeded si_blit_shader.cReviewed-by: Christian König Signed-off-by: Tom Rix Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

1bd9c939 - drm/radeon: align short build log

Thu, 13 Feb 2020 15:39:27 +0000

drm/radeon: align short build logThis beautifies the build log.[Before] HOSTCC drivers/gpu/drm/radeon/mkregtable MKREGTABLE drivers/gpu/drm/radeon/r100_reg_safe.h MKREGTABLE drivers/gpu/drm/radeon/rn50_reg_safe.h CC [M] drivers/gpu/drm/radeon/r100.o MKREGTABLE drivers/gpu/drm/radeon/r300_reg_safe.h CC [M] drivers/gpu/drm/radeon/r300.o[After] HOSTCC drivers/gpu/drm/radeon/mkregtable MKREG drivers/gpu/drm/radeon/r100_reg_safe.h MKREG drivers/gpu/drm/radeon/rn50_reg_safe.h CC [M] drivers/gpu/drm/radeon/r100.o MKREG drivers/gpu/drm/radeon/r300_reg_safe.h CC [M] drivers/gpu/drm/radeon/r300.oSigned-off-by: Masahiro Yamada Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

a341deb9 - drm/radeon: use pattern rule to avoid code duplication in Makefile

Thu, 13 Feb 2020 15:39:26 +0000

drm/radeon: use pattern rule to avoid code duplication in MakefileThis Makefile repeats similar build rules. Use a pattern rule.Signed-off-by: Masahiro Yamada Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

4ed513b5 - drm/radeon: fix build rules of *_reg_safe.h

Wed, 25 Mar 2020 20:48:35 +0000

drm/radeon: fix build rules of *_reg_safe.hif_changed must have FORCE as a prerequisite, and the targets must beadded to 'targets'.Signed-off-by: Masahiro Yamada Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

4bc97748 - drm/radeon: remove unneeded header include path

Wed, 25 Mar 2020 20:47:16 +0000

drm/radeon: remove unneeded header include pathA header include path without $(srctree)/ is suspicious because it doesnot work with O= builds.You can build drivers/gpu/drm/radeon/ without this include path.Signed-off-by: Masahiro Yamada Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

5f2fb52f - kbuild: rename hostprogs-y/always to hostprogs/always-y

Sat, 01 Feb 2020 16:49:24 +0000

kbuild: rename hostprogs-y/always to hostprogs/always-yIn old days, the "host-progs" syntax was used for specifying hostprograms. It was renamed to the current "hostprogs-y" in 2004.It is typically useful in scripts/Makefile because it allows Kbuild toselectively compile host programs based on the kernel configuration.This commit renames like follows: always -> always-y hostprogs-y -> hostprogsSo, scripts/Makefile will look like this: always-$(CONFIG_BUILD_BIN2C) += ... always-$(CONFIG_KALLSYMS) += ... ... hostprogs := $(always-y) $(always-m)I think this makes more sense because a host program is always a hostprogram, irrespective of the kernel configuration. We want to specifywhich ones to compile by CONFIG options, so always-y will be handier.The "always", "hostprogs-y", "hostprogs-m" will be kept for backwardcompatibility for a while.Signed-off-by: Masahiro Yamada List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

b2441318 - License cleanup: add SPDX GPL-2.0 license identifier to files with no license

Wed, 01 Nov 2017 14:07:57 +0000

License cleanup: add SPDX GPL-2.0 license identifier to files with no licenseMany source files in the tree are missing licensing information, whichmakes it harder for compliance tools to determine the correct license.By default all files without license information are under the defaultlicense of the kernel, which is GPL version 2.Update the files which contain no license information with the 'GPL-2.0'SPDX license identifier. The SPDX identifier is a legally bindingshorthand, which can be used instead of the full boiler plate text.This patch is based on work done by Thomas Gleixner and Kate Stewart andPhilippe Ombredanne.How this work was done:Patches were generated and checked against linux-4.14-rc6 for a subset ofthe use cases: - file had no licensing information it it. - file was a */uapi/* one with no licensing information in it, - file was a */uapi/* one with existing licensing information,Further patches will be generated in subsequent months to fix up caseswhere non-standard license headers were used, and references to licensehad to be inferred by heuristics based on keywords.The analysis to determine which SPDX License Identifier to be applied toa file was done in a spreadsheet of side by side results from of theoutput of two independent scanners (ScanCode & Windriver) producing SPDXtag:value files created by Philippe Ombredanne. Philippe prepared thebase worksheet, and did an initial spot review of a few 1000 files.The 4.13 kernel was the starting point of the analysis with 60,537 filesassessed. Kate Stewart did a file by file comparison of the scannerresults in the spreadsheet to determine which SPDX license identifier(s)to be applied to the file. She confirmed any determination that was notimmediately clear with lawyers working with the Linux Foundation.Criteria used to select files for SPDX license identifier tagging was: - Files considered eligible had to be source code files. - Make and config files were included as candidates if they contained >5 lines of source - File already had some variant of a license header in it (even if <5 lines).All documentation files were explicitly excluded.The following heuristics were used to determine which SPDX licenseidentifiers to apply. - when both scanners couldn't find any license traces, file was considered to have no license information in it, and the top level COPYING file license applied. For non */uapi/* files that summary was: SPDX license identifier # files ---------------------------------------------------|------- GPL-2.0 11139 and resulted in the first patch in this series. If that file was a */uapi/* path one, it was "GPL-2.0 WITH Linux-syscall-note" otherwise it was "GPL-2.0". Results of that was: SPDX license identifier # files ---------------------------------------------------|------- GPL-2.0 WITH Linux-syscall-note 930 and resulted in the second patch in this series. - if a file had some form of licensing information in it, and was one of the */uapi/* ones, it was denoted with the Linux-syscall-note if any GPL family license was found in the file or had no licensing in it (per prior point). Results summary: SPDX license identifier # files ---------------------------------------------------|------ GPL-2.0 WITH Linux-syscall-note 270 GPL-2.0+ WITH Linux-syscall-note 169 ((GPL-2.0 WITH Linux-syscall-note) OR BSD-2-Clause) 21 ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) 17 LGPL-2.1+ WITH Linux-syscall-note 15 GPL-1.0+ WITH Linux-syscall-note 14 ((GPL-2.0+ WITH Linux-syscall-note) OR BSD-3-Clause) 5 LGPL-2.0+ WITH Linux-syscall-note 4 LGPL-2.1 WITH Linux-syscall-note 3 ((GPL-2.0 WITH Linux-syscall-note) OR MIT) 3 ((GPL-2.0 WITH Linux-syscall-note) AND MIT) 1 and that resulted in the third patch in this series. - when the two scanners agreed on the detected license(s), that became the concluded license(s). - when there was disagreement between the two scanners (one detected a license but the other didn't, or they both detected different licenses) a manual inspection of the file occurred. - In most cases a manual inspection of the information in the file resulted in a clear resolution of the license that should apply (and which scanner probably needed to revisit its heuristics). - When it was not immediately clear, the license identifier was confirmed with lawyers working with the Linux Foundation. - If there was any question as to the appropriate license identifier, the file was flagged for further research and to be revisited later in time.In total, over 70 hours of logged manual review was done on thespreadsheet to determine the SPDX license identifiers to apply to thesource files by Kate, Philippe, Thomas and, in some cases, confirmationby lawyers working with the Linux Foundation.Kate also obtained a third independent scan of the 4.13 code base fromFOSSology, and compared selected files where the other two scannersdisagreed against that SPDX file, to see if there was new insights. TheWindriver scanner is based on an older version of FOSSology in part, sothey are related.Thomas did random spot checks in about 500 files from the spreadsheetsfor the uapi headers and agreed with SPDX license identifier in thefiles he inspected. For the non-uapi files Thomas did random spot checksin about 15000 files.In initial set of patches against 4.14-rc6, 3 files were found to havecopy/paste license identifier errors, and have been fixed to reflect thecorrect identifier.Additionally Philippe spent 10 hours this week doing a detailed manualinspection and review of the 12,461 patched files from the initial patchversion early this week with: - a full scancode scan run, collecting the matched texts, detected license ids and scores - reviewing anything where there was a license detected (about 500+ files) to ensure that the applied SPDX license was correct - reviewing anything where there was no detection but the patch license was not GPL-2.0 WITH Linux-syscall-note to ensure that the applied SPDX license was correctThis produced a worksheet with 20 files needing minor correction. Thisworksheet was then exported into 3 different .csv files for thedifferent types of files to be modified.These .csv files were then reviewed by Greg. Thomas wrote a script toparse the csv files and add the proper SPDX tag to the file, in theformat that the file expected. This script was further refined by Gregbased on the output to detect more types of files automatically and todistinguish between header and source .c files (which need differentcomment types.) Finally Greg ran the script using the .csv files togenerate the patches.Reviewed-by: Kate Stewart Reviewed-by: Philippe Ombredanne Reviewed-by: Thomas Gleixner Signed-off-by: Greg Kroah-Hartman List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

f4fa88ab - drm/radeon: deprecate and remove KFD interface

Mon, 30 Oct 2017 13:16:21 +0000

drm/radeon: deprecate and remove KFD interfaceTo quote Felix: "For testing KV with current user mode stack, please useamdgpu. I don't expect this to work with radeon and I'm not planning tospend any effort on making radeon work with a current user mode stack."Only compile tested, but should be straight forward.Signed-off-by: Christian König Signed-off-by: Oded Gabbay List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

56d11d58 - drm/radeon: Use correct path to trace include

Fri, 01 Sep 2017 14:49:53 +0000

drm/radeon: Use correct path to trace includeThe header comment in include/trace/define_trace.h specifies that theTRACE_INCLUDE_PATH needs to be relative to the define_trace.h headerrather than the trace file including it. Most instances get that wrongand work around it by adding the $(src) directory to the include path.While this works, it is preferable to refer to the correct path to thetrace file in the first place and avoid any workaround.Reviewed-by: Christian König Signed-off-by: Thierry Reding Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

ff32d39b - radeon: take out dead compat ioctls

Sat, 03 Jun 2017 20:19:18 +0000

radeon: take out dead compat ioctls Compat wrappers in radeon_ioc32.c had been unreachable since"drm/radeon: remove UMS support" has removed radeon_driver_old_fops.Signed-off-by: Al Viro List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

64a9dfc4 - drm/radeon: fix include notation and remove -Iinclude/drm flag

Mon, 24 Apr 2017 04:50:31 +0000

drm/radeon: fix include notation and remove -Iinclude/drm flagInclude instead of relative path from include/drm, thenremove the -Iinclude/drm compiler flag.Signed-off-by: Masahiro Yamada Reviewed-by: Michel Dänzer Signed-off-by: Daniel Vetter Link: http://patchwork.freedesktop.org/patch/msgid/1493009447-31524-14-git-send-email-yamada.masahiro@socionext.com List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

8333f607 - drm/radeon: remove UMS support

Mon, 23 Nov 2015 18:13:45 +0000

drm/radeon: remove UMS supportIt's been deprecated behind a kconfig option for almosttwo years and hasn't really been supported for years beforethat. DDX support was dropped more than three years ago.Acked-by: Christian König Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile

9843ead0 - drm/radeon: add DisplayPort MST support (v2)

Mon, 23 Feb 2015 23:24:04 +0000

drm/radeon: add DisplayPort MST support (v2)This adds initial DP 1.2 MST support to radeon, on CAYMANand up in theory.This is off by default.v2: agd5f:- add UNIPHY3 offsets- move atom cmd table code into atombios_encoders.c- whitespace cleanup- replace some magic numbers with proper definesSigned-off-by: Dave Airlie Signed-off-by: Alex Deucher List of files: /linux-6.15/drivers/gpu/drm/radeon/Makefile