amdgpu/dc: inline dal grph object id functions.
Subsystems (3)
PATCH 1/1 amdgpu/dc: inline dal grph object id functions.
From: Dave Airlie <[email protected]>
This is worth 400 bytes.
Signed-off-by Dave Airlie <[email protected]>
--- .../gpu/drm/amd/display/dc/basics/grph_object_id.c | 61 +----------------- .../gpu/drm/amd/display/include/grph_object_id.h | 72 +++++++++++++++++----- 2 files changed, 56 insertions(+), 77 deletions(-)
drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c +1 -60
@@ -26,7 +26,7 @@#include "dm_services.h" #include "include/grph_object_id.h" -bool dal_graphics_object_id_is_valid (struct graphics_object_id id) +static bool dal_graphics_object_id_is_valid (struct graphics_object_id id) {bool rc = true;
@@ -72,63 +72,4 @@ bool dal_graphics_object_id_is_equal(return false; } -/* Based on internal data members memory layout */ -uint32_t dal_graphics_object_id_to_uint (struct graphics_object_id id) -{ -uint32_t object_id =0 ; - - object_id = id.id + (id.enum_id <<0x8 ) + (id.type <<0xc ); -return object_id; -} - -/* -* ******* get specific ID - internal safe cast into specific type ******* -*/ - -enum controller_id dal_graphics_object_id_get_controller_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CONTROLLER ) -return id.id; -return CONTROLLER_ID_UNDEFINED ; -} - -enum clock_source_id dal_graphics_object_id_get_clock_source_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CLOCK_SOURCE ) -return id.id; -return CLOCK_SOURCE_ID_UNDEFINED ; -} - -enum encoder_id dal_graphics_object_id_get_encoder_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_ENCODER ) -return id.id; -return ENCODER_ID_UNKNOWN ; -} - -enum connector_id dal_graphics_object_id_get_connector_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CONNECTOR ) -return id.id; -return CONNECTOR_ID_UNKNOWN ; -} - -enum audio_id dal_graphics_object_id_get_audio_id (struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_AUDIO ) -return id.id; -return AUDIO_ID_UNKNOWN ; -} - -enum engine_id dal_graphics_object_id_get_engine_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_ENGINE ) -return id.id; -return ENGINE_ID_UNKNOWN ; -}
drivers/gpu/drm/amd/display/include/grph_object_id.h +55 -17
79 lines not on this page · raw message
Signature · 6 lines
2.9.5 _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
A message that is not in the archive (sent off-list or not yet received): 20171003023654.4911-1-airlied-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
Harry Wentland
Reviewed-byQuoted 5 lines · Dave Airlie
From: Dave Airlie <[email protected]>
This is worth 400 bytes.
Signed-off-by Dave Airlie <[email protected]>
Reviewed-by Harry Wentland <[email protected]>
Harry
Quoted 157 lines
--- .../gpu/drm/amd/display/dc/basics/grph_object_id.c | 61 +----------------- .../gpu/drm/amd/display/include/grph_object_id.h | 72 +++++++++++++++++----- 2 files changed, 56 insertions(+), 77 deletions(-)drivers/gpu/drm/amd/display/dc/basics/grph_object_id.c +1
-60@@ -26,7 +26,7 @@#include "dm_services.h" #include "include/grph_object_id.h" -bool dal_graphics_object_id_is_valid (struct graphics_object_id id) +static bool dal_graphics_object_id_is_valid (struct graphics_object_id id) {bool rc = true;@@ -72,63 +72,4 @@ bool dal_graphics_object_id_is_equal(return false; } -/* Based on internal data members memory layout */ -uint32_t dal_graphics_object_id_to_uint (struct graphics_object_id id) -{ -uint32_t object_id =0 ; - - object_id = id.id + (id.enum_id <<0x8 ) + (id.type <<0xc ); -return object_id; -} - -/* -* ******* get specific ID - internal safe cast into specific type ******* -*/ - -enum controller_id dal_graphics_object_id_get_controller_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CONTROLLER ) -return id.id; -return CONTROLLER_ID_UNDEFINED ; -} - -enum clock_source_id dal_graphics_object_id_get_clock_source_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CLOCK_SOURCE ) -return id.id; -return CLOCK_SOURCE_ID_UNDEFINED ; -} - -enum encoder_id dal_graphics_object_id_get_encoder_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_ENCODER ) -return id.id; -return ENCODER_ID_UNKNOWN ; -} - -enum connector_id dal_graphics_object_id_get_connector_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_CONNECTOR ) -return id.id; -return CONNECTOR_ID_UNKNOWN ; -} - -enum audio_id dal_graphics_object_id_get_audio_id (struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_AUDIO ) -return id.id; -return AUDIO_ID_UNKNOWN ; -} - -enum engine_id dal_graphics_object_id_get_engine_id ( -struct graphics_object_id id) -{ -if (id.type ==OBJECT_TYPE_ENGINE ) -return id.id; -return ENGINE_ID_UNKNOWN ; -}drivers/gpu/drm/amd/display/include/grph_object_id.h +46
-17@@ -233,24 +233,62 @@ static inline struct graphics_object_id dal_graphics_object_id_init(return result; } -bool dal_graphics_object_id_is_valid ( -struct graphics_object_id id);bool dal_graphics_object_id_is_equal (struct graphics_object_id id1,struct graphics_object_id id2); -uint32_t dal_graphics_object_id_to_uint ( -struct graphics_object_id id); - -enum controller_id dal_graphics_object_id_get_controller_id ( -struct graphics_object_id id); -enum clock_source_id dal_graphics_object_id_get_clock_source_id ( -struct graphics_object_id id); -enum encoder_id dal_graphics_object_id_get_encoder_id ( -struct graphics_object_id id); -enum connector_id dal_graphics_object_id_get_connector_id ( -struct graphics_object_id id); -enum audio_id dal_graphics_object_id_get_audio_id ( -struct graphics_object_id id); -enum engine_id dal_graphics_object_id_get_engine_id ( -struct graphics_object_id id); + +/* Based on internal data members memory layout */ +static inline uint32_t dal_graphics_object_id_to_uint ( +struct graphics_object_id id) +{ +return id.id + (id.enum_id <<0x8 ) + (id.type <<0xc ); +} + +static inline enum controller_id dal_graphics_object_id_get_controller_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_CONTROLLER ) +return id.id; +return CONTROLLER_ID_UNDEFINED ; +} + +static inline enum clock_source_id dal_graphics_object_id_get_clock_source_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_CLOCK_SOURCE ) +return id.id; +return CLOCK_SOURCE_ID_UNDEFINED ; +} + +static inline enum encoder_id dal_graphics_object_id_get_encoder_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_ENCODER ) +return id.id; +return ENCODER_ID_UNKNOWN ; +} + +static inline enum connector_id dal_graphics_object_id_get_connector_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_CONNECTOR ) +return id.id; +return CONNECTOR_ID_UNKNOWN ; +} + +static inline enum audio_id dal_graphics_object_id_get_audio_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_AUDIO ) +return id.id; +return AUDIO_ID_UNKNOWN ;
drivers/gpu/drm/amd/display/include/grph_object_id.h static inline struct graphics_object_id dal_graphics_object_id_init( · line 294
Quoted 10 lines
drivers/gpu/drm/amd/display/include/grph_object_id.h +9
-0+} + +static inline enum engine_id dal_graphics_object_id_get_engine_id ( +struct graphics_object_id id) +{ +if (id.type ==OBJECT_TYPE_ENGINE ) +return id.id; +return ENGINE_ID_UNKNOWN ; +}#endif
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx