aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarry Wentland <harry.wentland@amd.com>2017-11-23 10:31:50 -0500
committerAlex Deucher <alexander.deucher@amd.com>2017-12-14 11:01:07 -0500
commitaff10cfe646b975b358b01f0f8469c76a9f153da (patch)
tree116dde5c16038948df05016c27c252ec5f600e61
parent5b92d9d409e742ca09a4cb924e3f781eddbf7d81 (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>
-rw-r--r--drivers/gpu/drm/amd/display/dc/basics/Makefile2
-rw-r--r--drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c75
-rw-r--r--drivers/gpu/drm/amd/display/dc/bios/bios_parser.c46
-rw-r--r--drivers/gpu/drm/amd/display/include/grph_object_id.h4
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
27BASICS = conversion.o fixpt31_32.o fixpt32_32.o grph_object_id.o \ 27BASICS = 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
30AMD_DAL_BASICS = $(addprefix $(AMDDALPATH)/dc/basics/,$(BASICS)) 30AMD_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
29static 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
52bool 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
2257static 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
2280static 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
2257static ATOM_OBJECT *get_bios_object(struct bios_parser *bp, 2303static 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
236bool 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 */
241static inline uint32_t dal_graphics_object_id_to_uint( 237static inline uint32_t dal_graphics_object_id_to_uint(
242 struct graphics_object_id id) 238 struct graphics_object_id id)