From 0e1339e03075380b45ec1b3ce3598bd5406f3a46 Mon Sep 17 00:00:00 2001 From: Bhanuprakash Modem Date: Fri, 10 Jun 2022 15:38:47 +0530 Subject: lib/igt_kms: Add a helper function to check Bigjoiner support Create a helper function to check that the system supports the given crtc/connector mode(s). Example: * Pipe-D won't support Bigjoiner, hence we can't use the connector modes greater than 5K on Pipe-D * To use 8K mode on a pipe, then the consecutive pipe must be free. The Kernel is expected to reject the invalid combo (which must be validated as a scenario separately). So, this helper function checks the validity of the combo to avoid failures. To use this helper, each individual subtest needs to set the @pipe to a specific @output by igt_output_set_pipe() and call this helper to check the validity of the combo. V2: * Rebase * Add support to handle fused pipes V3: * Reject the bigjoiner if the consecutive pipe is not available Signed-off-by: Bhanuprakash Modem Reviewed-by: Manasi Navare Reviewed-by: Karthik B S --- lib/igt_kms.c | 73 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 73 insertions(+) (limited to 'lib/igt_kms.c') diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 36dfcfcb..bd606548 100644 --- a/lib/igt_kms.c +++ b/lib/igt_kms.c @@ -5636,3 +5636,76 @@ bool igt_check_output_bpc_equal(int drmfd, enum pipe pipe, return (current == bpc); } + +/* + * igt_check_bigjoiner_support: + * @display: a pointer to an #igt_display_t structure + * + * Get all active pipes from connected outputs (i.e. pending_pipe != PIPE_NONE) + * and check those pipes supports the selected mode(s). + * + * Example: + * * Pipe-D can't support mode > 5K + * * To use 8K mode on a pipe then consecutive pipe must be free. + * + * Returns: true if a valid crtc/connector mode combo found, else false + */ +bool igt_check_bigjoiner_support(igt_display_t *display) +{ + uint8_t i, total_pipes = 0, pipes_in_use = 0; + enum pipe p; + struct { + enum pipe idx; + drmModeModeInfo *mode; + } pipes[IGT_MAX_PIPES]; + + /* Get total enabled pipes. */ + for_each_pipe(display, p) + total_pipes++; + + /* + * Get list of pipes in use those were set by igt_output_set_pipe() + * just before calling this function. + */ + for (i = 0 ; i < display->n_outputs; i++) { + igt_output_t *output = &display->outputs[i]; + + if (output->pending_pipe == PIPE_NONE) + continue; + + pipes[pipes_in_use].idx = output->pending_pipe; + pipes[pipes_in_use].mode = igt_output_get_mode(output); + pipes_in_use++; + } + + if (!pipes_in_use) { + igt_debug("We must set at least one output to pipe.\n"); + return true; + } + + /* + * if mode.hdisplay > 5120, then ignore + * - if the consecutive pipe is not available + * - last crtc in single/multi-connector config + * - consecutive crtcs in multi-connector config + * + * in multi-connector config ignore if + * - previous crtc mode.hdisplay > 5120 and + * - current & previous crtcs are consecutive + */ + for (i = 0; i < pipes_in_use; i++) { + if (((pipes[i].mode->hdisplay > MAX_HDISPLAY_PER_PIPE) && + ((pipes[i].idx >= (total_pipes - 1)) || + (!display->pipes[pipes[i].idx + 1].enabled) || + ((i < (pipes_in_use - 1)) && (abs(pipes[i + 1].idx - pipes[i].idx) <= 1)))) || + ((i > 0) && (pipes[i - 1].mode->hdisplay > MAX_HDISPLAY_PER_PIPE) && + ((!display->pipes[pipes[i - 1].idx + 1].enabled) || + (abs(pipes[i].idx - pipes[i - 1].idx) <= 1)))) { + igt_debug("Pipe/Output combo is not possible with selected mode(s).\n"); + + return false; + } + } + + return true; +} -- cgit v1.2.3