From: Karthi Kandasamy <karthi.kandas...@amd.com>

[Why]
The functions read_ono_state are no longer in use and have been identified
as redundant.
Removing them helps streamline the codebase and improve maintainability by
eliminating unnecessary code.

[How]
These unused functions were removed from Hwss module, ensuring that no
functionality is affected, and the code is simplified.

Reviewed-by: Martin Leung <martin.le...@amd.com>
Signed-off-by: Karthi Kandasamy <karthi.kandas...@amd.com>
Signed-off-by: Tom Chung <chiahsuan.ch...@amd.com>
---
 .../amd/display/dc/hwss/dcn401/dcn401_hwseq.c | 85 -------------------
 .../amd/display/dc/hwss/dcn401/dcn401_hwseq.h |  2 -
 2 files changed, 87 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.c 
b/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.c
index d5bbd3531c61..555a9f590cd7 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.c
+++ b/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.c
@@ -127,91 +127,6 @@ void dcn401_program_gamut_remap(struct pipe_ctx *pipe_ctx)
        mpc->funcs->set_gamut_remap(mpc, mpcc_id, &mpc_adjust);
 }
 
-struct ips_ono_region_state dcn401_read_ono_state(struct dc *dc, uint8_t 
region)
-{
-       struct dce_hwseq *hws = dc->hwseq;
-       struct ips_ono_region_state state = {0, 0};
-
-       switch (region) {
-       case 0:
-               /* dccg, dio, dcio */
-               REG_GET_2(DOMAIN22_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 1:
-               /* dchubbub, dchvm, dchubbubmem */
-               REG_GET_2(DOMAIN23_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 2:
-               /* mpc, opp, optc, dwb */
-               REG_GET_2(DOMAIN24_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 3:
-               /* hpo */
-               REG_GET_2(DOMAIN25_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 4:
-               /* dchubp0, dpp0 */
-               REG_GET_2(DOMAIN0_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 5:
-               /* dsc0 */
-               REG_GET_2(DOMAIN16_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 6:
-               /* dchubp1, dpp1 */
-               REG_GET_2(DOMAIN1_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 7:
-               /* dsc1 */
-               REG_GET_2(DOMAIN17_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 8:
-               /* dchubp2, dpp2 */
-               REG_GET_2(DOMAIN2_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 9:
-               /* dsc2 */
-               REG_GET_2(DOMAIN18_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 10:
-               /* dchubp3, dpp3 */
-               REG_GET_2(DOMAIN3_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       case 11:
-               /* dsc3 */
-               REG_GET_2(DOMAIN19_PG_STATUS,
-                       DOMAIN_DESIRED_PWR_STATE, &state.desire_pwr_state,
-                       DOMAIN_PGFSM_PWR_STATUS, &state.current_pwr_state);
-               break;
-       default:
-               break;
-       }
-
-       return state;
-}
-
 void dcn401_init_hw(struct dc *dc)
 {
        struct abm **abms = dc->res_pool->multiple_abms;
diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.h 
b/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.h
index 17a06f299b68..17cea748789e 100644
--- a/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.h
+++ b/drivers/gpu/drm/amd/display/dc/hwss/dcn401/dcn401_hwseq.h
@@ -63,8 +63,6 @@ void dcn401_set_cursor_position(struct pipe_ctx *pipe_ctx);
 
 bool dcn401_apply_idle_power_optimizations(struct dc *dc, bool enable);
 
-struct ips_ono_region_state dcn401_read_ono_state(struct dc *dc,
-                                                 uint8_t region);
 void dcn401_wait_for_dcc_meta_propagation(const struct dc *dc,
                const struct pipe_ctx *top_pipe_to_program);
 
-- 
2.34.1

Reply via email to