- 02 May, 2024 40 commits
-
-
Hersen Wu authored
[Why] Coverity reports RESOURCE_LEAK for some implemenations of clock_source_create. Do not release memory of clk_src if contructor fails. [How] Free clk_src if contructor fails. Reviewed-by:
Harry Wentland <harry.wentland@amd.com> Acked-by:
Tom Chung <chiahsuan.chung@amd.com> Signed-off-by:
Hersen Wu <hersenxs.wu@amd.com> Tested-by:
Daniel Wheeler <daniel.wheeler@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Hersen Wu authored
[Why] Coverity reports OVERRUN warning for CalculateSwathAndDETConfiguration_params->hw_debug5 = &s->dummy_boolean[2]. bool dummy_boolean[2] is defined within struct dml2_core_calcs_mode_support_locals. [How] Change array size from 2 to 3. Reviewed-by:
Aurabindo Pillai <aurabindo.pillai@amd.com> Acked-by:
-
Hersen Wu authored
[Why & How] Check return pointer of kzalloc before using it. Reviewed-by:
Alex Hung <alex.hung@amd.com> Acked-by:
-
Hersen Wu authored
[Why] For subtraction, coverity reports integer overflow warning message when variable type is uint32_t. [How] Change variable type to int32_t. Reviewed-by:
-
Hersen Wu authored
[Why] For substrcation, coverity reports integer overflow warning message when variable type is uint32_t. [How] Change varaible type to int32_t. Reviewed-by:
-
Hersen Wu authored
[Why] Coverity reports NULL_RETURN warning. [How] Add otg_master NULL check. Reviewed-by:
-
Alex Hung authored
[WHY & HOW] dc->links[] has max size of MAX_LINKS and NULL is return when trying to access with out-of-bound index. This fixes 3 OVERRUN and 1 RESOURCE_LEAK issues reported by Coverity. Reviewed-by:
-
Hersen Wu authored
[Why] Coverity reports RESOURCE_LEAK warning. State memory is not released if dm_create_color_properties fail. [How] Call kfree(state) before return. Reviewed-by:
-
Alex Hung authored
[WHY & HOW] HDCP_MESSAGE_ID_INVALID (-1) is not a valid msg_id nor is it a valid array index, and it needs checking before used. This fixes 4 OVERRUN issues reported by Coverity. Reviewed-by:
-
Alex Hung authored
This prevents accessing to negative index of link_encoders array. This fixes an OVERRUN issue reported by Coverity. Reviewed-by:
Rodrigo Siqueira <rodrigo.siqueira@amd.com> Acked-by:
-
Alex Hung authored
[WHY & HOW] num_valid_sets needs to be checked to avoid a negative index when accessing reader_wm_sets[num_valid_sets - 1]. This fixes an OVERRUN issue reported by Coverity. Reviewed-by:
-
Hersen Wu authored
[Why] Coverity reports OVERRUN warning. Do not check if array index valid. [How] Check msg_id valid and valid array index. Reviewed-by:
-
Hersen Wu authored
[Why & How] ASSERT if return NULL from kcalloc. Reviewed-by:
-
Alex Hung authored
aux_rd_interval has size of 7 and should be checked. This fixes 3 OVERRUN and 1 INTEGER_OVERFLOW issues reported by Coverity. Reviewed-by:
-
Alex Hung authored
[WHY] fe_clk_en has size of 5 but sizeof(fe_clk_en) has byte size 20 which is lager than the array size. [HOW] Divide byte size 20 by its element size. This fixes 2 OVERRUN issues reported by Coverity. Reviewed-by:
-
Hersen Wu authored
[Why] Coverity report OVERRUN warning. There are only max_links elements within dc->links. link count could up to AMDGPU_DM_MAX_DISPLAY_INDEX 31. [How] Make sure link count less than max_links. Reviewed-by:
-
Hersen Wu authored
[Why] Coverity reports OVERRUN warning. Should abort amdgpu_dm initialize. [How] Return failure to amdgpu_dm_init. Reviewed-by:
-
Hersen Wu authored
[WHY] Coverity reports OVERRUN issues within amdgpu_dm interrupt registers. Do not check index value before access array. Do not check NULL pointer. [HOW] Add index value check for array. Add check for pointer from amdgpu_dm_irq_register_interrupt. Reviewed-by:
-
Alex Hung authored
[WHY & HOW] GPIO_ID_UNKNOWN (-1) is not a valid value for array index and therefore should be checked in advance. This fixes 5 OVERRUN issues reported by Coverity. Reviewed-by:
-
Alex Hung authored
[WHY & HOW] tg_inst will be a negative if timing_generator_count equals 0, which should be checked before used. This fixes 2 OVERRUN issues reported by Coverity. Reviewed-by:
-
Alex Hung authored
[WHY] ENGINE_ID_UNKNOWN (-1) is not a valid eng_id and not a valid array index. [HOW] Check whether eng_id is unknown to avoid access array with negative array index. This fixes 4 OVERRUN issues reported by Coverity. Reviewed-by:
-
Revalla Hari Krishna authored
[why] cleaning up the code refactor requires dccg to be in its own component. [how] move all files under newly created dccg folder and fixing the makefiles. Reviewed-by:
Martin Leung <martin.leung@amd.com> Acked-by:
Revalla Hari Krishna <harikrishna.revalla@amd.com> Tested-by:
-
Leo Ma authored
[Why && How] Copyright notice failed in the Palamida scan and make changes to align with our guidelines. Acked-by:
Leo Ma <hanghong.ma@amd.com> Tested-by:
-
Gui Chengming authored
Populate pstate clock. Signed-off-by:
Gui Chengming <Jack.Gui@amd.com> Reviewed-by:
Kenneth Feng <kenneth.feng@amd.com> Signed-off-by:
-
Kenneth Feng authored
add tool log support on smu v14.0.2/3 Signed-off-by:
-
Kenneth Feng authored
enable mode1 reset on smu v14.0.2/v14.0.3 Signed-off-by:
Likun Gao <Likun.Gao@amd.com> Signed-off-by:
-
Kenneth Feng authored
support mode1 reset on smu_v14_0_3 Signed-off-by:
-
Alex Deucher authored
We can't use a shared fence location because each transaction should be considered independently. Signed-off-by: -
Alex Deucher authored
MES internally has a timeout allowance of 2 seconds. Increase driver timeout to 3 seconds to be safe. Signed-off-by: -
Alex Deucher authored
Makes it easier to review the logs when there are MES errors. Signed-off-by: -
Kenneth Feng authored
enable mmhub and athub cg on gc 12.0.1 Signed-off-by:
-
Kenneth Feng authored
Enable gfxoff on gc 12.0.1 Signed-off-by:
-
Likun Gao authored
Support to get clockgating state for gfx v12. Signed-off-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
-
Kenneth Feng authored
enable sram fgcg on gc 12.0.1 Signed-off-by:
-
Kenneth Feng authored
enable perfcounter mgcg and repeater fgcg on gc 12.0.1 Signed-off-by:
-
Kenneth Feng authored
enable 3D cgcg and 3D cgls on gc 12.0.1 Signed-off-by:
-
Kenneth Feng authored
enable mgcg on gfx 12.0.1 Signed-off-by:
-
Kenneth Feng authored
enable cgcg and cgls on gc 12.0.1 Signed-off-by:
-
David (Ming Qiang) Wu authored
Add VCN5 encode and decode capabilities support Signed-off-by:
David (Ming Qiang) Wu <David.Wu3@amd.com> Reviewed-by:
Leo Liu <leo.liu@amd.com> Signed-off-by:
-
Sonny Jiang authored
Enable DPG mode Signed-off-by:
Sonny Jiang <sonny.jiang@amd.com> Reviewed-by:
-
