diff options
author | Harry Wentland <harry.wentland@amd.com> | 2017-11-23 10:31:50 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-12-14 11:01:07 -0500 |
commit | aff10cfe646b975b358b01f0f8469c76a9f153da (patch) | |
tree | 116dde5c16038948df05016c27c252ec5f600e61 | |
parent | 5b92d9d409e742ca09a4cb924e3f781eddbf7d81 (diff) |
drm/amd/display: Remove grph_object_id.c and move function to bios_parser
Signed-off-by: Harry Wentland <harry.wentland@amd.com>
Reviewed-by: Jordan Lazare <Jordan.Lazare@amd.com>
Acked-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
4 files changed, 47 insertions, 80 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/basics/Makefile b/drivers/gpu/drm/amd/display/dc/basics/Makefile index 6af8c8a9ad80..bca33bd9a0d2 100644 --- a/drivers/gpu/drm/amd/display/dc/basics/Makefile +++ b/drivers/gpu/drm/amd/display/dc/basics/Makefile | |||
@@ -24,7 +24,7 @@ | |||
24 | # It provides the general basic services required by other DAL | 24 | # It provides the general basic services required by other DAL |
25 | # subcomponents. | 25 | # subcomponents. |
26 | 26 | ||
27 | BASICS = conversion.o fixpt31_32.o fixpt32_32.o grph_object_id.o \ | 27 | BASICS = conversion.o fixpt31_32.o fixpt32_32.o \ |
28 | logger.o log_helpers.o vector.o | 28 | logger.o log_helpers.o vector.o |
29 | 29 | ||
30 | AMD_DAL_BASICS = $(addprefix $(AMDDALPATH)/dc/basics/,$(BASICS)) | 30 | AMD_DAL_BASICS = $(addprefix $(AMDDALPATH)/dc/basics/,$(BASICS)) |
diff --git a/drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c b/drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c deleted file mode 100644 index 147822545252..000000000000 --- a/drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c +++ /dev/null | |||
@@ -1,75 +0,0 @@ | |||
1 | /* | ||
2 | * Copyright 2012-15 Advanced Micro Devices, Inc. | ||
3 | * | ||
4 | * Permission is hereby granted, free of charge, to any person obtaining a | ||
5 | * copy of this software and associated documentation files (the "Software"), | ||
6 | * to deal in the Software without restriction, including without limitation | ||
7 | * the rights to use, copy, modify, merge, publish, distribute, sublicense, | ||
8 | * and/or sell copies of the Software, and to permit persons to whom the | ||
9 | * Software is furnished to do so, subject to the following conditions: | ||
10 | * | ||
11 | * The above copyright notice and this permission notice shall be included in | ||
12 | * all copies or substantial portions of the Software. | ||
13 | * | ||
14 | * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR | ||
15 | * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, | ||
16 | * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL | ||
17 | * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR | ||
18 | * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, | ||
19 | * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR | ||
20 | * OTHER DEALINGS IN THE SOFTWARE. | ||
21 | * | ||
22 | * Authors: AMD | ||
23 | * | ||
24 | */ | ||
25 | |||
26 | #include "dm_services.h" | ||
27 | #include "include/grph_object_id.h" | ||
28 | |||
29 | static bool dal_graphics_object_id_is_valid(struct graphics_object_id id) | ||
30 | { | ||
31 | bool rc = true; | ||
32 | |||
33 | switch (id.type) { | ||
34 | case OBJECT_TYPE_UNKNOWN: | ||
35 | rc = false; | ||
36 | break; | ||
37 | case OBJECT_TYPE_GPU: | ||
38 | case OBJECT_TYPE_ENGINE: | ||
39 | /* do NOT check for id.id == 0 */ | ||
40 | if (id.enum_id == ENUM_ID_UNKNOWN) | ||
41 | rc = false; | ||
42 | break; | ||
43 | default: | ||
44 | if (id.id == 0 || id.enum_id == ENUM_ID_UNKNOWN) | ||
45 | rc = false; | ||
46 | break; | ||
47 | } | ||
48 | |||
49 | return rc; | ||
50 | } | ||
51 | |||
52 | bool dal_graphics_object_id_is_equal( | ||
53 | struct graphics_object_id id1, | ||
54 | struct graphics_object_id id2) | ||
55 | { | ||
56 | if (false == dal_graphics_object_id_is_valid(id1)) { | ||
57 | dm_output_to_console( | ||
58 | "%s: Warning: comparing invalid object 'id1'!\n", __func__); | ||
59 | return false; | ||
60 | } | ||
61 | |||
62 | if (false == dal_graphics_object_id_is_valid(id2)) { | ||
63 | dm_output_to_console( | ||
64 | "%s: Warning: comparing invalid object 'id2'!\n", __func__); | ||
65 | return false; | ||
66 | } | ||
67 | |||
68 | if (id1.id == id2.id && id1.enum_id == id2.enum_id | ||
69 | && id1.type == id2.type) | ||
70 | return true; | ||
71 | |||
72 | return false; | ||
73 | } | ||
74 | |||
75 | |||
diff --git a/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c b/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c index 86e6438c5cf3..21fb78e8048d 100644 --- a/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c +++ b/drivers/gpu/drm/amd/display/dc/bios/bios_parser.c | |||
@@ -2254,6 +2254,52 @@ static enum bp_result get_gpio_i2c_info(struct bios_parser *bp, | |||
2254 | return BP_RESULT_OK; | 2254 | return BP_RESULT_OK; |
2255 | } | 2255 | } |
2256 | 2256 | ||
2257 | static bool dal_graphics_object_id_is_valid(struct graphics_object_id id) | ||
2258 | { | ||
2259 | bool rc = true; | ||
2260 | |||
2261 | switch (id.type) { | ||
2262 | case OBJECT_TYPE_UNKNOWN: | ||
2263 | rc = false; | ||
2264 | break; | ||
2265 | case OBJECT_TYPE_GPU: | ||
2266 | case OBJECT_TYPE_ENGINE: | ||
2267 | /* do NOT check for id.id == 0 */ | ||
2268 | if (id.enum_id == ENUM_ID_UNKNOWN) | ||
2269 | rc = false; | ||
2270 | break; | ||
2271 | default: | ||
2272 | if (id.id == 0 || id.enum_id == ENUM_ID_UNKNOWN) | ||
2273 | rc = false; | ||
2274 | break; | ||
2275 | } | ||
2276 | |||
2277 | return rc; | ||
2278 | } | ||
2279 | |||
2280 | static bool dal_graphics_object_id_is_equal( | ||
2281 | struct graphics_object_id id1, | ||
2282 | struct graphics_object_id id2) | ||
2283 | { | ||
2284 | if (false == dal_graphics_object_id_is_valid(id1)) { | ||
2285 | dm_output_to_console( | ||
2286 | "%s: Warning: comparing invalid object 'id1'!\n", __func__); | ||
2287 | return false; | ||
2288 | } | ||
2289 | |||
2290 | if (false == dal_graphics_object_id_is_valid(id2)) { | ||
2291 | dm_output_to_console( | ||
2292 | "%s: Warning: comparing invalid object 'id2'!\n", __func__); | ||
2293 | return false; | ||
2294 | } | ||
2295 | |||
2296 | if (id1.id == id2.id && id1.enum_id == id2.enum_id | ||
2297 | && id1.type == id2.type) | ||
2298 | return true; | ||
2299 | |||
2300 | return false; | ||
2301 | } | ||
2302 | |||
2257 | static ATOM_OBJECT *get_bios_object(struct bios_parser *bp, | 2303 | static ATOM_OBJECT *get_bios_object(struct bios_parser *bp, |
2258 | struct graphics_object_id id) | 2304 | struct graphics_object_id id) |
2259 | { | 2305 | { |
diff --git a/drivers/gpu/drm/amd/display/include/grph_object_id.h b/drivers/gpu/drm/amd/display/include/grph_object_id.h index 03a7a9ca95ea..c4197432eb7c 100644 --- a/drivers/gpu/drm/amd/display/include/grph_object_id.h +++ b/drivers/gpu/drm/amd/display/include/grph_object_id.h | |||
@@ -233,10 +233,6 @@ static inline struct graphics_object_id dal_graphics_object_id_init( | |||
233 | return result; | 233 | return result; |
234 | } | 234 | } |
235 | 235 | ||
236 | bool dal_graphics_object_id_is_equal( | ||
237 | struct graphics_object_id id1, | ||
238 | struct graphics_object_id id2); | ||
239 | |||
240 | /* Based on internal data members memory layout */ | 236 | /* Based on internal data members memory layout */ |
241 | static inline uint32_t dal_graphics_object_id_to_uint( | 237 | static inline uint32_t dal_graphics_object_id_to_uint( |
242 | struct graphics_object_id id) | 238 | struct graphics_object_id id) |