From patchwork Wed Feb 13 14:45:34 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Francis, David" X-Patchwork-Id: 10810079 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 2DA996C2 for ; Wed, 13 Feb 2019 14:45:59 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1AA0128501 for ; Wed, 13 Feb 2019 14:45:59 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 0E15728981; Wed, 13 Feb 2019 14:45:59 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=unavailable version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 67C2E28501 for ; Wed, 13 Feb 2019 14:45:58 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id A84B289FD9; Wed, 13 Feb 2019 14:45:49 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from NAM01-BY2-obe.outbound.protection.outlook.com (mail-eopbgr810073.outbound.protection.outlook.com [40.107.81.73]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5944689F89; Wed, 13 Feb 2019 14:45:46 +0000 (UTC) Received: from BN4PR12CA0007.namprd12.prod.outlook.com (2603:10b6:403:2::17) by DM6PR12MB2969.namprd12.prod.outlook.com (2603:10b6:5:115::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Wed, 13 Feb 2019 14:45:44 +0000 Received: from CO1NAM03FT057.eop-NAM03.prod.protection.outlook.com (2a01:111:f400:7e48::202) by BN4PR12CA0007.outlook.office365.com (2603:10b6:403:2::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16 via Frontend Transport; Wed, 13 Feb 2019 14:45:44 +0000 Received-SPF: SoftFail (protection.outlook.com: domain of transitioning amd.com discourages use of 165.204.84.17 as permitted sender) Received: from SATLEXCHOV01.amd.com (165.204.84.17) by CO1NAM03FT057.mail.protection.outlook.com (10.152.81.124) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1580.10 via Frontend Transport; Wed, 13 Feb 2019 14:45:43 +0000 Received: from localhost.localdomain (10.180.168.240) by SATLEXCHOV01.amd.com (10.181.40.71) with Microsoft SMTP Server id 14.3.389.1; Wed, 13 Feb 2019 08:45:41 -0600 From: David Francis To: Subject: [PATCH 1/3] drm/i915: Move dsc rate params compute into drm Date: Wed, 13 Feb 2019 09:45:34 -0500 Message-ID: <20190213144536.21661-2-David.Francis@amd.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190213144536.21661-1-David.Francis@amd.com> References: <20190213144536.21661-1-David.Francis@amd.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:165.204.84.17; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(346002)(376002)(39860400002)(136003)(396003)(2980300002)(189003)(199004)(478600001)(50226002)(72206003)(8936002)(6916009)(81156014)(105596002)(1076003)(8676002)(2351001)(49486002)(81166006)(4326008)(86362001)(356004)(6666004)(106466001)(97736004)(76176011)(2906002)(77096007)(305945005)(446003)(476003)(2616005)(126002)(11346002)(53936002)(36756003)(51416003)(486006)(186003)(26005)(30864003)(68736007)(336012)(104016004)(316002)(54906003)(16586007)(50466002)(47776003)(426003)(48376002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM6PR12MB2969; H:SATLEXCHOV01.amd.com; FPR:; SPF:SoftFail; LANG:en; PTR:InfoDomainNonexistent; MX:1; A:1; X-Microsoft-Exchange-Diagnostics: 1; CO1NAM03FT057; 1:B9O/fTKk+jNIXODsWEVCrfp6h9OZaXV+3yz80HsmHntDV/3XFvcHT6mFv5aJP+ziZSx7wp+bC2I3AC7Sro6XEMNkyVj/ZC5idaNHf/RtCy9jhqS9I2fugMJRwnAbtke4md+irYhWMAkd7Nb8eapvag== X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: f6ff440d-09ab-405c-10d8-08d691c1ee92 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(2017052603328)(7153060); SRVR:DM6PR12MB2969; X-MS-TrafficTypeDiagnostic: DM6PR12MB2969: X-Microsoft-Exchange-Diagnostics: 1; DM6PR12MB2969; 20:etpv3eBlGhLdLJqKjuOuqIx42T5JJQzv0QxpkMEDZNDTQ2sW02e5ysGi8r4Rj/4Ve3gEJ4Kzd9IApCMhkEjZVrXS0WPYig8ZqRmt/TFLVQot7i+Vk5i8DS6g8l271Yes3Qi5sNDZbn2n+df2sY9OUwa0Rsd6SxgJmcp7JgD0V6nEX2bDuxtR4VpCe3kbJJ/cdXP1an1D4VHBnRhRExJDmx+SlgRYOS1YaGHJsaDQZ4CHFOkuZGNglagvZaRDLYzqcY8ECdiXXRlE9adecCoUmhNNGu3xHdg/Tp9knbK97D1JiZYYCGf+jzvByRCx0YRql103JYNmEvWpkOjchzMx+6YgF0Hs5w/oXzs29VDE9SSjIa22DOVLzBsjtUCXyGFxdfp1I6RSbTcLVWpDacdJYdniUI/DGgd9TUa8nHyDYnKaVxI4iMBqihPHMzoQ68VHPpBmtt7KPuwoAbvaV7k+nnJzwODBHeTFx1MlLdK/Nu5wNyKvBIDksEGQhYBAZ8AP X-Microsoft-Antispam-PRVS: X-Forefront-PRVS: 094700CA91 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DM6PR12MB2969; 23:AoJEKyafiCIMOBXf2JunjQHwA8ail6kKBHOajUFhG?= OTucn94oY7Xy90H3mtOT6l0ev/TsoYbrHp8yMbbJ8FOd7rOXhexW0QU/EFlhzyuuR+1CWlNmwSWKBxUD/eBs1/W5G0zWOdpAPB6ijVd2VuhrDVYp2uhoLmaK6/U3k8399FmWFQqHeQFJg71zE79c6ryoKWTsaemIVJYeghCDZp/TgQq/ZYNUuOsV62BO1fhUMTO6uWfNN5slMuITm2iwaFlzURTVCLX4zj5til9lMqqTg+Rbp8utLSSs7+LHTnX8oB8eRu2+56nZ024CcwcdXaJcfdtZtFLf3uBboNnEU7pOFD9aJ17NWYLUuI5NKQi5Z+IPOlO89gKl/RKBUrciS+8TCBshOhq0+eGluXGxReglcAB1uKFLSStNd72IA1JToCIXC5KB9JWpWaKx4Roob1Y4BEMFqR0sV9K73c87CJ0AY32igLNI043DGBVwxFIPZ/RrdF31jWKVyjlE1PBaDIgD+qNxheW3L9IYx8bBzLFZB4bpEiCt3iOjURPtJ5TZKLrVudG/I8l3ImnSKLtpi5Q4zy68s9ubZ/e7a8qtDtFv+j+c+FYLbBoFS9y8aDcGrU01QcZkltqmtmDjdasCDEclQti1heVJlWGzGfJlzRrvyZ//V9tTLE43Bj66ActmOrn5xvLntfTGq4YtNcbxI0h9WD70ju5dcd3VUrgup5hcp39balUDrwvtTZY2AC7T705yVAs2XFuI6GzumYarE6jLQ1wJgiLRfScJurN5QjJFkp8YahMfQRr8SfxRiCbPntdTYZ6aRmUhgx6zISvVTdG/XkHE9FjhXoy2VeCce2w/BCveBdLaAVtneHjeQ7+Uv7oC9I9qr4VH1YdfnS2OsumiKqZZ0lcQxYDaQHR5Oq7BYC/GeGoeHHTD+irzGGF3V1OLZDUqiTIr28y1ucukW5webI0933nqUHYQIy8f8894ahRZoqa1zUuWfg2Cb30LnnvRzIwjTyE7Niq7GNz8eTQkZ8zActZ3wm7LkZJ9z5yGodQP/6EEw2Qy78JXfvZ+K91uI+u9w4cnp9MKRmmM/8Fcv1Hwo5MFc0jr8MO42+fwJfv4rfLyaHdqemtwb7A9thaoX9NGteTQhewJAhSzbyy X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: dCMxv6WLqAg6XdCI7fNpcFwrA6K5jVLRAGiP+Xjq0M8ip4Pf+sQbtgckEcG5Oiz1+t/bhKtNQ43VyrJrBPd9upW3sk5klP3J3O4b5TUB0Gtz9HBKUoqrUBa+4W31Gk22XyPHN8V1z2Q2UL3zfJ8X47UFbc+L7IZq+UhfYLsdafVteE0pM2na+jB4Z0UELlAH4BGJo4VUt84/y7/8ZZ0n7fotXMLikC+GXm8efOgoFq+Wci0fI6hRUDkdD7Cq/WKEjiTqWC5IW28W8ctK+w6DR4RKwjgRsJornPsAl90YItWVH0EevtBm4g7upDKPDheMRa6mt+/LjsWomlXnMOceZt+TDJYsIRe4ZHVixdQJ3aTq+18WpY2P0iydsS82WV2P40JGftsTnE6wlRe/SpZJoRsQJwE6ArV4wv1BVxS5hso= X-Microsoft-Exchange-Diagnostics: 1; DM6PR12MB2969; 20:r6tIybALbaN6HvlSAxkuLPQ+kOnwMkQ9xgig9mpokaDLkuKfwMgio1jIc0UdBAJrKWE1Zbw09njM1KLkk8Kslp3K2cag9CoxRtk7UknOwv4J2WUEzLivWAb3xQ+H8A5CIGnOfVCukiv2HdGl8SP8BzWSivZX671wAVdGYinvBCcrD7sjB4paO7MAHlZyHudhvbERHgyO8LWa/2Je6jtDKalVPUIazaWNWYz2RUa4SsU80BUXeil6tkzU116ohyZ4 X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2019 14:45:43.3867 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f6ff440d-09ab-405c-10d8-08d691c1ee92 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXCHOV01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB2969 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector1-amd-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Z4f2j3Vdw27gcxwfvJh2PhwMcsnX+sXwzv/Qpszodh8=; b=S1sujK1GKzNxCXli05APEMk+u6womwG7vhjro3i0ey07vOsQpGzomnOFWAfut9KFHRaeVW57TrdtW4D1C+4xvJyfsYDmIUp7DD/S8dPNLfyRpF6k/Pp1aa1sxZyI9Rg4Db+07NZy24K3P3dLkaRy1UGntQHxhdysTwPoizonYjU= X-Mailman-Original-Authentication-Results: spf=softfail (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; lists.freedesktop.org; dkim=none (message not signed) header.d=none;lists.freedesktop.org; dmarc=permerror action=none header.from=amd.com; X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: intel-gfx@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, nikola.cornij@amd.com, manasi.d.navare@intel.com, David Francis Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP The function intel_compute_rc_parameters is part of the dsc spec and is not driver-specific. Other drm drivers might like to use it. The function is not changed; just moved and renamed. Signed-off-by: David Francis Reviewed-by: Harry Wentland --- drivers/gpu/drm/drm_dsc.c | 133 ++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_vdsc.c | 125 +--------------------------- include/drm/drm_dsc.h | 1 + 3 files changed, 135 insertions(+), 124 deletions(-) diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c index bc2b23adb072..4b0e3c9c3ff8 100644 --- a/drivers/gpu/drm/drm_dsc.c +++ b/drivers/gpu/drm/drm_dsc.c @@ -11,6 +11,7 @@ #include #include #include +#include #include #include @@ -226,3 +227,135 @@ void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, /* PPS 94 - 127 are O */ } EXPORT_SYMBOL(drm_dsc_pps_infoframe_pack); + +/** + * drm_dsc_compute_rc_parameters() - Write rate control + * parameters to the dsc configuration. Some configuration + * fields must be present beforehand. + * + * @dsc_cfg: + * DSC Configuration data partially filled by driver + */ +int drm_dsc_compute_rc_parameters(struct drm_dsc_config *vdsc_cfg) +{ + unsigned long groups_per_line = 0; + unsigned long groups_total = 0; + unsigned long num_extra_mux_bits = 0; + unsigned long slice_bits = 0; + unsigned long hrd_delay = 0; + unsigned long final_scale = 0; + unsigned long rbs_min = 0; + + /* Number of groups used to code each line of a slice */ + groups_per_line = DIV_ROUND_UP(vdsc_cfg->slice_width, + DSC_RC_PIXELS_PER_GROUP); + + /* chunksize in Bytes */ + vdsc_cfg->slice_chunk_size = DIV_ROUND_UP(vdsc_cfg->slice_width * + vdsc_cfg->bits_per_pixel, + (8 * 16)); + + if (vdsc_cfg->convert_rgb) + num_extra_mux_bits = 3 * (vdsc_cfg->mux_word_size + + (4 * vdsc_cfg->bits_per_component + 4) + - 2); + else + num_extra_mux_bits = 3 * vdsc_cfg->mux_word_size + + (4 * vdsc_cfg->bits_per_component + 4) + + 2 * (4 * vdsc_cfg->bits_per_component) - 2; + /* Number of bits in one Slice */ + slice_bits = 8 * vdsc_cfg->slice_chunk_size * vdsc_cfg->slice_height; + + while ((num_extra_mux_bits > 0) && + ((slice_bits - num_extra_mux_bits) % vdsc_cfg->mux_word_size)) + num_extra_mux_bits--; + + if (groups_per_line < vdsc_cfg->initial_scale_value - 8) + vdsc_cfg->initial_scale_value = groups_per_line + 8; + + /* scale_decrement_interval calculation according to DSC spec 1.11 */ + if (vdsc_cfg->initial_scale_value > 8) + vdsc_cfg->scale_decrement_interval = groups_per_line / + (vdsc_cfg->initial_scale_value - 8); + else + vdsc_cfg->scale_decrement_interval = DSC_SCALE_DECREMENT_INTERVAL_MAX; + + vdsc_cfg->final_offset = vdsc_cfg->rc_model_size - + (vdsc_cfg->initial_xmit_delay * + vdsc_cfg->bits_per_pixel + 8) / 16 + num_extra_mux_bits; + + if (vdsc_cfg->final_offset >= vdsc_cfg->rc_model_size) { + DRM_DEBUG_KMS("FinalOfs < RcModelSze for this InitialXmitDelay\n"); + return -ERANGE; + } + + final_scale = (vdsc_cfg->rc_model_size * 8) / + (vdsc_cfg->rc_model_size - vdsc_cfg->final_offset); + if (vdsc_cfg->slice_height > 1) + /* + * NflBpgOffset is 16 bit value with 11 fractional bits + * hence we multiply by 2^11 for preserving the + * fractional part + */ + vdsc_cfg->nfl_bpg_offset = DIV_ROUND_UP((vdsc_cfg->first_line_bpg_offset << 11), + (vdsc_cfg->slice_height - 1)); + else + vdsc_cfg->nfl_bpg_offset = 0; + + /* 2^16 - 1 */ + if (vdsc_cfg->nfl_bpg_offset > 65535) { + DRM_DEBUG_KMS("NflBpgOffset is too large for this slice height\n"); + return -ERANGE; + } + + /* Number of groups used to code the entire slice */ + groups_total = groups_per_line * vdsc_cfg->slice_height; + + /* slice_bpg_offset is 16 bit value with 11 fractional bits */ + vdsc_cfg->slice_bpg_offset = DIV_ROUND_UP(((vdsc_cfg->rc_model_size - + vdsc_cfg->initial_offset + + num_extra_mux_bits) << 11), + groups_total); + + if (final_scale > 9) { + /* + * ScaleIncrementInterval = + * finaloffset/((NflBpgOffset + SliceBpgOffset)*8(finalscale - 1.125)) + * as (NflBpgOffset + SliceBpgOffset) has 11 bit fractional value, + * we need divide by 2^11 from pstDscCfg values + */ + vdsc_cfg->scale_increment_interval = + (vdsc_cfg->final_offset * (1 << 11)) / + ((vdsc_cfg->nfl_bpg_offset + + vdsc_cfg->slice_bpg_offset) * + (final_scale - 9)); + } else { + /* + * If finalScaleValue is less than or equal to 9, a value of 0 should + * be used to disable the scale increment at the end of the slice + */ + vdsc_cfg->scale_increment_interval = 0; + } + + if (vdsc_cfg->scale_increment_interval > 65535) { + DRM_DEBUG_KMS("ScaleIncrementInterval is large for slice height\n"); + return -ERANGE; + } + + /* + * DSC spec mentions that bits_per_pixel specifies the target + * bits/pixel (bpp) rate that is used by the encoder, + * in steps of 1/16 of a bit per pixel + */ + rbs_min = vdsc_cfg->rc_model_size - vdsc_cfg->initial_offset + + DIV_ROUND_UP(vdsc_cfg->initial_xmit_delay * + vdsc_cfg->bits_per_pixel, 16) + + groups_per_line * vdsc_cfg->first_line_bpg_offset; + + hrd_delay = DIV_ROUND_UP((rbs_min * 16), vdsc_cfg->bits_per_pixel); + vdsc_cfg->rc_bits = (hrd_delay * vdsc_cfg->bits_per_pixel) / 16; + vdsc_cfg->initial_dec_delay = hrd_delay - vdsc_cfg->initial_xmit_delay; + + return 0; +} +EXPORT_SYMBOL(drm_dsc_compute_rc_parameters); diff --git a/drivers/gpu/drm/i915/intel_vdsc.c b/drivers/gpu/drm/i915/intel_vdsc.c index c56ba0e04044..c76cec8bfb74 100644 --- a/drivers/gpu/drm/i915/intel_vdsc.c +++ b/drivers/gpu/drm/i915/intel_vdsc.c @@ -318,129 +318,6 @@ static int get_column_index_for_rc_params(u8 bits_per_component) } } -static int intel_compute_rc_parameters(struct drm_dsc_config *vdsc_cfg) -{ - unsigned long groups_per_line = 0; - unsigned long groups_total = 0; - unsigned long num_extra_mux_bits = 0; - unsigned long slice_bits = 0; - unsigned long hrd_delay = 0; - unsigned long final_scale = 0; - unsigned long rbs_min = 0; - - /* Number of groups used to code each line of a slice */ - groups_per_line = DIV_ROUND_UP(vdsc_cfg->slice_width, - DSC_RC_PIXELS_PER_GROUP); - - /* chunksize in Bytes */ - vdsc_cfg->slice_chunk_size = DIV_ROUND_UP(vdsc_cfg->slice_width * - vdsc_cfg->bits_per_pixel, - (8 * 16)); - - if (vdsc_cfg->convert_rgb) - num_extra_mux_bits = 3 * (vdsc_cfg->mux_word_size + - (4 * vdsc_cfg->bits_per_component + 4) - - 2); - else - num_extra_mux_bits = 3 * vdsc_cfg->mux_word_size + - (4 * vdsc_cfg->bits_per_component + 4) + - 2 * (4 * vdsc_cfg->bits_per_component) - 2; - /* Number of bits in one Slice */ - slice_bits = 8 * vdsc_cfg->slice_chunk_size * vdsc_cfg->slice_height; - - while ((num_extra_mux_bits > 0) && - ((slice_bits - num_extra_mux_bits) % vdsc_cfg->mux_word_size)) - num_extra_mux_bits--; - - if (groups_per_line < vdsc_cfg->initial_scale_value - 8) - vdsc_cfg->initial_scale_value = groups_per_line + 8; - - /* scale_decrement_interval calculation according to DSC spec 1.11 */ - if (vdsc_cfg->initial_scale_value > 8) - vdsc_cfg->scale_decrement_interval = groups_per_line / - (vdsc_cfg->initial_scale_value - 8); - else - vdsc_cfg->scale_decrement_interval = DSC_SCALE_DECREMENT_INTERVAL_MAX; - - vdsc_cfg->final_offset = vdsc_cfg->rc_model_size - - (vdsc_cfg->initial_xmit_delay * - vdsc_cfg->bits_per_pixel + 8) / 16 + num_extra_mux_bits; - - if (vdsc_cfg->final_offset >= vdsc_cfg->rc_model_size) { - DRM_DEBUG_KMS("FinalOfs < RcModelSze for this InitialXmitDelay\n"); - return -ERANGE; - } - - final_scale = (vdsc_cfg->rc_model_size * 8) / - (vdsc_cfg->rc_model_size - vdsc_cfg->final_offset); - if (vdsc_cfg->slice_height > 1) - /* - * NflBpgOffset is 16 bit value with 11 fractional bits - * hence we multiply by 2^11 for preserving the - * fractional part - */ - vdsc_cfg->nfl_bpg_offset = DIV_ROUND_UP((vdsc_cfg->first_line_bpg_offset << 11), - (vdsc_cfg->slice_height - 1)); - else - vdsc_cfg->nfl_bpg_offset = 0; - - /* 2^16 - 1 */ - if (vdsc_cfg->nfl_bpg_offset > 65535) { - DRM_DEBUG_KMS("NflBpgOffset is too large for this slice height\n"); - return -ERANGE; - } - - /* Number of groups used to code the entire slice */ - groups_total = groups_per_line * vdsc_cfg->slice_height; - - /* slice_bpg_offset is 16 bit value with 11 fractional bits */ - vdsc_cfg->slice_bpg_offset = DIV_ROUND_UP(((vdsc_cfg->rc_model_size - - vdsc_cfg->initial_offset + - num_extra_mux_bits) << 11), - groups_total); - - if (final_scale > 9) { - /* - * ScaleIncrementInterval = - * finaloffset/((NflBpgOffset + SliceBpgOffset)*8(finalscale - 1.125)) - * as (NflBpgOffset + SliceBpgOffset) has 11 bit fractional value, - * we need divide by 2^11 from pstDscCfg values - */ - vdsc_cfg->scale_increment_interval = - (vdsc_cfg->final_offset * (1 << 11)) / - ((vdsc_cfg->nfl_bpg_offset + - vdsc_cfg->slice_bpg_offset) * - (final_scale - 9)); - } else { - /* - * If finalScaleValue is less than or equal to 9, a value of 0 should - * be used to disable the scale increment at the end of the slice - */ - vdsc_cfg->scale_increment_interval = 0; - } - - if (vdsc_cfg->scale_increment_interval > 65535) { - DRM_DEBUG_KMS("ScaleIncrementInterval is large for slice height\n"); - return -ERANGE; - } - - /* - * DSC spec mentions that bits_per_pixel specifies the target - * bits/pixel (bpp) rate that is used by the encoder, - * in steps of 1/16 of a bit per pixel - */ - rbs_min = vdsc_cfg->rc_model_size - vdsc_cfg->initial_offset + - DIV_ROUND_UP(vdsc_cfg->initial_xmit_delay * - vdsc_cfg->bits_per_pixel, 16) + - groups_per_line * vdsc_cfg->first_line_bpg_offset; - - hrd_delay = DIV_ROUND_UP((rbs_min * 16), vdsc_cfg->bits_per_pixel); - vdsc_cfg->rc_bits = (hrd_delay * vdsc_cfg->bits_per_pixel) / 16; - vdsc_cfg->initial_dec_delay = hrd_delay - vdsc_cfg->initial_xmit_delay; - - return 0; -} - int intel_dp_compute_dsc_params(struct intel_dp *intel_dp, struct intel_crtc_state *pipe_config) { @@ -575,7 +452,7 @@ int intel_dp_compute_dsc_params(struct intel_dp *intel_dp, vdsc_cfg->initial_scale_value = (vdsc_cfg->rc_model_size << 3) / (vdsc_cfg->rc_model_size - vdsc_cfg->initial_offset); - return intel_compute_rc_parameters(vdsc_cfg); + return drm_dsc_compute_rc_parameters(vdsc_cfg); } enum intel_display_power_domain diff --git a/include/drm/drm_dsc.h b/include/drm/drm_dsc.h index d03f1b83421a..ad43494f1cc8 100644 --- a/include/drm/drm_dsc.h +++ b/include/drm/drm_dsc.h @@ -481,5 +481,6 @@ struct drm_dsc_pps_infoframe { void drm_dsc_dp_pps_header_init(struct drm_dsc_pps_infoframe *pps_sdp); void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, const struct drm_dsc_config *dsc_cfg); +int drm_dsc_compute_rc_parameters(struct drm_dsc_config *vdsc_cfg); #endif /* _DRM_DSC_H_ */ From patchwork Wed Feb 13 14:45:35 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Francis, David" X-Patchwork-Id: 10810077 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 245796C2 for ; Wed, 13 Feb 2019 14:45:57 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 10E2028733 for ; Wed, 13 Feb 2019 14:45:57 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 04FFA29970; Wed, 13 Feb 2019 14:45:57 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 8C49028733 for ; Wed, 13 Feb 2019 14:45:56 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id D25A589FDB; Wed, 13 Feb 2019 14:45:49 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-eopbgr740058.outbound.protection.outlook.com [40.107.74.58]) by gabe.freedesktop.org (Postfix) with ESMTPS id D8F0B89FA6; Wed, 13 Feb 2019 14:45:47 +0000 (UTC) Received: from BN4PR12CA0004.namprd12.prod.outlook.com (2603:10b6:403:2::14) by MN2PR12MB2974.namprd12.prod.outlook.com (2603:10b6:208:c2::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Wed, 13 Feb 2019 14:45:46 +0000 Received: from CO1NAM03FT057.eop-NAM03.prod.protection.outlook.com (2a01:111:f400:7e48::203) by BN4PR12CA0004.outlook.office365.com (2603:10b6:403:2::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16 via Frontend Transport; Wed, 13 Feb 2019 14:45:45 +0000 Received-SPF: SoftFail (protection.outlook.com: domain of transitioning amd.com discourages use of 165.204.84.17 as permitted sender) Received: from SATLEXCHOV01.amd.com (165.204.84.17) by CO1NAM03FT057.mail.protection.outlook.com (10.152.81.124) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1580.10 via Frontend Transport; Wed, 13 Feb 2019 14:45:45 +0000 Received: from localhost.localdomain (10.180.168.240) by SATLEXCHOV01.amd.com (10.181.40.71) with Microsoft SMTP Server id 14.3.389.1; Wed, 13 Feb 2019 08:45:41 -0600 From: David Francis To: Subject: [PATCH 2/3] drm/dsc: Add native 420 and 422 support to compute_rc_params Date: Wed, 13 Feb 2019 09:45:35 -0500 Message-ID: <20190213144536.21661-3-David.Francis@amd.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190213144536.21661-1-David.Francis@amd.com> References: <20190213144536.21661-1-David.Francis@amd.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:165.204.84.17; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(346002)(396003)(376002)(136003)(39860400002)(2980300002)(189003)(199004)(47776003)(72206003)(53936002)(51416003)(76176011)(49486002)(126002)(1076003)(476003)(2616005)(486006)(6666004)(356004)(104016004)(2906002)(81166006)(186003)(11346002)(426003)(105596002)(446003)(50226002)(336012)(8936002)(2351001)(316002)(97736004)(54906003)(68736007)(16586007)(478600001)(4326008)(36756003)(50466002)(305945005)(48376002)(6916009)(8676002)(26005)(77096007)(81156014)(106466001)(86362001); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR12MB2974; H:SATLEXCHOV01.amd.com; FPR:; SPF:SoftFail; LANG:en; PTR:InfoDomainNonexistent; MX:1; A:1; X-Microsoft-Exchange-Diagnostics: 1; CO1NAM03FT057; 1:LLp4nFUCUjdRgO/V9LXQOAmPRTWlb7aWBg9eEF30Vezsh3l3svQZIWrOgUfS0nsgp6YQEBeL0us03NR84bjK4vTyWyLWBd7Mw9zb37jhZru47pRyb0B+5NJDkE/MVI9EiCgWnA2RKYgktKqfQq8/KA== X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 6052ca0a-3dbd-49a9-2885-08d691c1efb1 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(5600110)(711020)(4605077)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7153060); SRVR:MN2PR12MB2974; X-MS-TrafficTypeDiagnostic: MN2PR12MB2974: X-Microsoft-Exchange-Diagnostics: 1; MN2PR12MB2974; 20:NaTxk5THSujJ3pIszIcxRjMiI5zwRnqvnai5GgeWwqQD16j02MGwrXzIClOIp38+qjtYDBLSJpHPmEL0jlnlqWPJ+QFgIv0AmxeJL+ezlu4ipNUdZUVDQU9Qvk8cyfskEJpjr5G+KF9iwmc4bCabhzTSO4smra0HzFNYtaYFFZT9AlKiBCsz6Q485HFytfS4TOdQ8vci4OhwaT7Quxg51Fr5Jp3lXAGx5cajX/+Q4PNV4sVIoW4EjMdWnEg8pv7eD+awLRfoIp4ygX/L0BXMZ21DGyorZ//yltE1X/itHNJgCWajpVATSCo9CYnD4dLWqMJwEJS1WhCkJnL/uHgMH17kbajmEydzpx4s+MpMARODCHZcMYpmC+krdh+vVyFZwUdUC7hOoQGo7mKEJ8fVo7DWibGUTeGoxqvo8IIYRwygSO3UejRHWhUFTz5vfyTDi/xu7XgZT2eAXgCEUjbPMshrHEMXv8S2N+gBbawhRhJ+Q6QpHIoMOoY8ITC5AokM X-Microsoft-Antispam-PRVS: X-Forefront-PRVS: 094700CA91 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; MN2PR12MB2974; 23:Ju7QkOFuqeHOOTtfJhxsz+7TRbPYRmT6lXNwLd6vm?= RnGFMxqWy2gpjIT6vhg91RC2++/vDjFskCR3To9Zq5l2UjYNrpWMx6yjvzBicGtw7k24Z/rUrF2KnU7OD/iT1CTjj/G0RFyhC6OJlvtAFKbCynt30D2va5dRahUkMGsks4VFA/QN/PkZ1NX7uJZQIHhd3xnW9e5UYi05vI6L7Ot+qn1bqgeN7JT5i9PiWR59Y9vKC4VbJBUaRNThFGrzuDDnt28mlnU0aP0D37XEoYeKzG9FdPk8NYdwcp3MA7lDOuBxEUWc2g1Pxvzxgx4qzVSVipoc0QnScc2me+FCjX0+gevb0AQr0SQr9J30/XcmoiHX3tNqllrIFMiT80mOrtOus5IN9cA8yA0o9dMGY4OLQHfPb4E48TFC6LTtUoCg4dJx4pbZcPzwaXqbSwOvbkCI3G0hT0v7BgU/Q7qWHnh6DqBilUarzK3LgSoFh/j/Q6t7PtXz4AgkCvExcrOImu1ZXwUsY/GpM6zs+M7FxYgr/ZCvg5W6hXeb82lYinTVKo8bzcWLzCPkHG45rOrLFl5/Bs1Axqfc/V4BiFxOndgzQI0dfQXRKemFfhJEfYjywVVIxRHKIcC7rKES5jyT/EPeJWSI0L3PB4VhF+ek3BJcSwlMiAnNCf+VcHGyQmBx1a4tcsl+/NrV1CaifiHOS178CrXTpnMQ01AhGBjKeyjqhY+KUHtaxPq7dY4N6+RP5KZRRWD0rMmmK+Ag+0ZhMhlNQdKnLRMVrQLcmmjPRfPITI/YQEHhypPIwUSfrU7nR3tZIMVP1js1yrO9rD4eHIbdahL5GPSPDut7rlPk30chjCyAcL+4EVtx0ZZLhBv0DTJnwoWQ3kytxjy4bDxm7jskrcAKZJryxqJEjbb14xmODwQ46KBnBNZeJGfGA2S/gBNqcz+J+uNZbM0s2aDBhxapldIEEqswD1HkguK9t6xWIiMoeON7HvMZ+qK0IqTnouUq3cxZ1R0N88QEEHYPEwyMilbC97hw3edDKAm8bkpvuoOBJsICKHWYoxjcPw/4DCseKnsnXTydxygXRIzouvilkB3ONSywE4jgF20UPpRadPU71Tah6u3k1wGG/vptyU= X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: WAzv7BxO6er44fvDRtxIPtywUCRwNji4PWJfEZea0FSdXddjh3ub/Vw7LgAU67wUUVq5NgRsQ3MXv4Dg2LBYEf3pGOn2QLHGwLPzgxCV941KmPJyd9NgjN+EWIxgFdPIjcP2ERQmUIzZzyt+D4d35kZ2cQvV9+Sc7DlhG2g+7jUKPkxds2gnarlVtj0svn/UXQWuyo60jdLteR6pH/Gxz+MSwOIS+etemTqJ7eQ6xQvZpecWJGzXtW9jh5oIdgEwxf6v8OhpU79XebihtsMN3sFsuwRGXE1+6jgcPJKJarQrgRak4iiT2BSfEaOHqGGYzk8MomuLme+8yPQvaT9m7v++yjvIaCf6g6fVlBbmkwr7m+QSCevg3DwEJCb/hYVjpc/OaPzsxdEUKi2q86BgRvl61Vl8RDAEeQ5HYooLdpg= X-Microsoft-Exchange-Diagnostics: 1; MN2PR12MB2974; 20:q6pud+kDxN+kFhiecZqmR1iCwHBNG/x511rnplRW3UbnrzCBI9DIu0T1NfCg9f4iqtB5GD8ao72uMtPXC/WHP/Kb1UYxIMuEmU5meVGzxro34c8S/2qdRyBWG5FrWmZDJJNgiwDyACRUiwhAyH6p9C/nrA1DaKLO4WbAzFv/G1+mZknyLx3V++mdahwZvzYM6NVu6UvQ9thPfR3EHMI2PJkqaAJ8G+eqACzbl9W5s1vP8c+8hqGVRHCmCBWlXfBl X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2019 14:45:45.3442 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 6052ca0a-3dbd-49a9-2885-08d691c1efb1 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXCHOV01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB2974 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector1-amd-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=EuNQeFJ0Z6Os8u4d13LEKIx3JvohPUqVyBkaM0Y3ZJA=; b=gheqdoR18QVluQAg5rtsrSyiUkoqiCK8aeUFfISZKREAQArS9B+K81JNJrnt6kojzpx1TjCAqB6/QwOY135t+VfO12S6TbidoRe5Xnej16RQyLks+IixVb2IqBEcHpxtJeV9pB59XrIMoUa0V9vOl+0pzhyzsxYiVBAi+e4JDu4= X-Mailman-Original-Authentication-Results: spf=softfail (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; lists.freedesktop.org; dkim=none (message not signed) header.d=none;lists.freedesktop.org; dmarc=permerror action=none header.from=amd.com; X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: intel-gfx@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, nikola.cornij@amd.com, manasi.d.navare@intel.com, David Francis Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP Native 420 and 422 transfer modes are new in DSC1.2 In these modes, each two pixels of a slice are treated as one pixel, so the slice width is half as large (round down) for the purposes of calucating the groups per line and chunk size in bytes In native 422 mode, each pixel has four components, so the mux component of a group is larger by one additional mux word and one additional component Now that there is native 422 support, the configuration option previously called enable422 is renamed to simple_422 to avoid confusion Signed-off-by: David Francis Reviewed-by: Harry Wentland Reviewed-by: Manasi Navare --- drivers/gpu/drm/drm_dsc.c | 31 +++++++++++++++++++++++-------- drivers/gpu/drm/i915/intel_vdsc.c | 4 ++-- include/drm/drm_dsc.h | 4 ++-- 3 files changed, 27 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c index 4b0e3c9c3ff8..9e675dd39a44 100644 --- a/drivers/gpu/drm/drm_dsc.c +++ b/drivers/gpu/drm/drm_dsc.c @@ -77,7 +77,7 @@ void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, ((dsc_cfg->bits_per_pixel & DSC_PPS_BPP_HIGH_MASK) >> DSC_PPS_MSB_SHIFT) | dsc_cfg->vbr_enable << DSC_PPS_VBR_EN_SHIFT | - dsc_cfg->enable422 << DSC_PPS_SIMPLE422_SHIFT | + dsc_cfg->simple_422 << DSC_PPS_SIMPLE422_SHIFT | dsc_cfg->convert_rgb << DSC_PPS_CONVERT_RGB_SHIFT | dsc_cfg->block_pred_enable << DSC_PPS_BLOCK_PRED_EN_SHIFT; @@ -246,19 +246,34 @@ int drm_dsc_compute_rc_parameters(struct drm_dsc_config *vdsc_cfg) unsigned long final_scale = 0; unsigned long rbs_min = 0; - /* Number of groups used to code each line of a slice */ - groups_per_line = DIV_ROUND_UP(vdsc_cfg->slice_width, - DSC_RC_PIXELS_PER_GROUP); + if (vdsc_cfg->native_420 || vdsc_cfg->native_422) { + /* Number of groups used to code each line of a slice */ + groups_per_line = DIV_ROUND_UP(vdsc_cfg->slice_width / 2, + DSC_RC_PIXELS_PER_GROUP); - /* chunksize in Bytes */ - vdsc_cfg->slice_chunk_size = DIV_ROUND_UP(vdsc_cfg->slice_width * - vdsc_cfg->bits_per_pixel, - (8 * 16)); + /* chunksize in Bytes */ + vdsc_cfg->slice_chunk_size = DIV_ROUND_UP(vdsc_cfg->slice_width / 2 * + vdsc_cfg->bits_per_pixel, + (8 * 16)); + } else { + /* Number of groups used to code each line of a slice */ + groups_per_line = DIV_ROUND_UP(vdsc_cfg->slice_width, + DSC_RC_PIXELS_PER_GROUP); + + /* chunksize in Bytes */ + vdsc_cfg->slice_chunk_size = DIV_ROUND_UP(vdsc_cfg->slice_width * + vdsc_cfg->bits_per_pixel, + (8 * 16)); + } if (vdsc_cfg->convert_rgb) num_extra_mux_bits = 3 * (vdsc_cfg->mux_word_size + (4 * vdsc_cfg->bits_per_component + 4) - 2); + else if (vdsc_cfg->native_422) + num_extra_mux_bits = 4 * vdsc_cfg->mux_word_size + + (4 * vdsc_cfg->bits_per_component + 4) + + 3 * (4 * vdsc_cfg->bits_per_component) - 2; else num_extra_mux_bits = 3 * vdsc_cfg->mux_word_size + (4 * vdsc_cfg->bits_per_component + 4) + diff --git a/drivers/gpu/drm/i915/intel_vdsc.c b/drivers/gpu/drm/i915/intel_vdsc.c index c76cec8bfb74..7702c5c8b3f2 100644 --- a/drivers/gpu/drm/i915/intel_vdsc.c +++ b/drivers/gpu/drm/i915/intel_vdsc.c @@ -369,7 +369,7 @@ int intel_dp_compute_dsc_params(struct intel_dp *intel_dp, DSC_1_1_MAX_LINEBUF_DEPTH_BITS : line_buf_depth; /* Gen 11 does not support YCbCr */ - vdsc_cfg->enable422 = false; + vdsc_cfg->simple_422 = false; /* Gen 11 does not support VBR */ vdsc_cfg->vbr_enable = false; vdsc_cfg->block_pred_enable = @@ -496,7 +496,7 @@ static void intel_configure_pps_for_dsc_encoder(struct intel_encoder *encoder, pps_val |= DSC_BLOCK_PREDICTION; if (vdsc_cfg->convert_rgb) pps_val |= DSC_COLOR_SPACE_CONVERSION; - if (vdsc_cfg->enable422) + if (vdsc_cfg->simple_422) pps_val |= DSC_422_ENABLE; if (vdsc_cfg->vbr_enable) pps_val |= DSC_VBR_ENABLE; diff --git a/include/drm/drm_dsc.h b/include/drm/drm_dsc.h index ad43494f1cc8..4e55e37943d7 100644 --- a/include/drm/drm_dsc.h +++ b/include/drm/drm_dsc.h @@ -70,10 +70,10 @@ struct drm_dsc_config { /* Slice Height */ u16 slice_height; /* - * 4:2:2 enable mode (from PPS, 4:2:2 conversion happens + * Simple 4:2:2 mode (from PPS, 4:2:2 conversion happens * outside of DSC encode/decode algorithm) */ - bool enable422; + bool simple_422; /* Picture Width */ u16 pic_width; /* Picture Height */ From patchwork Wed Feb 13 14:45:36 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Francis, David" X-Patchwork-Id: 10810081 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 387B7139A for ; Wed, 13 Feb 2019 14:46:01 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 2619428733 for ; Wed, 13 Feb 2019 14:46:01 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 19BF32D2E0; Wed, 13 Feb 2019 14:46:01 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 68B5728733 for ; Wed, 13 Feb 2019 14:46:00 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 9B39089FE3; Wed, 13 Feb 2019 14:45:51 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-eopbgr780084.outbound.protection.outlook.com [40.107.78.84]) by gabe.freedesktop.org (Postfix) with ESMTPS id 2A51D89FA6; Wed, 13 Feb 2019 14:45:49 +0000 (UTC) Received: from BN4PR12CA0004.namprd12.prod.outlook.com (2603:10b6:403:2::14) by BN8PR12MB3459.namprd12.prod.outlook.com (2603:10b6:408:48::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16; Wed, 13 Feb 2019 14:45:47 +0000 Received: from CO1NAM03FT057.eop-NAM03.prod.protection.outlook.com (2a01:111:f400:7e48::203) by BN4PR12CA0004.outlook.office365.com (2603:10b6:403:2::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1622.16 via Frontend Transport; Wed, 13 Feb 2019 14:45:46 +0000 Received-SPF: SoftFail (protection.outlook.com: domain of transitioning amd.com discourages use of 165.204.84.17 as permitted sender) Received: from SATLEXCHOV01.amd.com (165.204.84.17) by CO1NAM03FT057.mail.protection.outlook.com (10.152.81.124) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.20.1580.10 via Frontend Transport; Wed, 13 Feb 2019 14:45:46 +0000 Received: from localhost.localdomain (10.180.168.240) by SATLEXCHOV01.amd.com (10.181.40.71) with Microsoft SMTP Server id 14.3.389.1; Wed, 13 Feb 2019 08:45:42 -0600 From: David Francis To: Subject: [PATCH 3/3] drm/dsc: Change infoframe_pack to payload_pack Date: Wed, 13 Feb 2019 09:45:36 -0500 Message-ID: <20190213144536.21661-4-David.Francis@amd.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190213144536.21661-1-David.Francis@amd.com> References: <20190213144536.21661-1-David.Francis@amd.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:165.204.84.17; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(2980300002)(189003)(199004)(16586007)(76176011)(54906003)(36756003)(4326008)(305945005)(2616005)(53936002)(426003)(126002)(476003)(72206003)(11346002)(51416003)(498600001)(486006)(446003)(104016004)(14444005)(49486002)(68736007)(336012)(2906002)(2351001)(6916009)(26005)(8936002)(106466001)(77096007)(1076003)(50226002)(97736004)(6666004)(105596002)(356004)(48376002)(8676002)(86362001)(81166006)(47776003)(186003)(81156014)(50466002); DIR:OUT; SFP:1101; SCL:1; SRVR:BN8PR12MB3459; H:SATLEXCHOV01.amd.com; FPR:; SPF:SoftFail; LANG:en; PTR:InfoDomainNonexistent; MX:1; A:1; X-Microsoft-Exchange-Diagnostics: 1; CO1NAM03FT057; 1:ViFy/ArEk8ClYOd99bVqt2XfIkhXoFJE8C5jqjkIlG6osMUaow9NXxSzHAoOC8nVxeCQlwJ4EkVf6yrLxBx7NpHB34lsQ26YMqYLIbytiwJDdRyZleNsA/b+B/0yprdQmXW/q7vRhn1NzzQ1rPtC1g== X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 8c76e2c7-69e0-4017-7e1e-08d691c1f067 X-Microsoft-Antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600110)(711020)(4605077)(2017052603328)(7153060); SRVR:BN8PR12MB3459; X-MS-TrafficTypeDiagnostic: BN8PR12MB3459: X-Microsoft-Exchange-Diagnostics: 1; BN8PR12MB3459; 20:kgxRoxq9B5Pn/dc7t2lXMRpnI9t39LxsFXRqJrEJXTfybr9GFq3G7GhC1IQcDvraxD2suQILLd1GLjbM6Y68800S/9noIMwdhvqeRVpCB8wVzOmFYN0VBkFZ+mKLkpULa/18HdJnayZY7wlpHv+ybiE+wc436woBzdVkBrHgQBubxtlKH8J5252JD7lyn6cheLYUlkjfS6eUfCQW7i97Uw2CTrMQRSg9evAyQiZi4kUzOAil2P/3U5hQ38zYAqbEwZIXULPFLFsgQs4tKyGbBZDNLTni/XMcBScg8ewBZxh7yqMt5UkzWfE8RbG+cVd6zs19g5+26YRxv/sJed7mFqmAeioS1kusm2WUvFBcpTdCmke0yTG1NeyRUHd2KPkTGTmf2tFu8iBXVVPifov90FuKlMUMY6GIDjpkrWb1vX+6DapU6RZ1cmDqhcFxvxM0YQcQXTAFEEPl9Ts3xBX9AbGXKNktFzyTFUHnECoPXAxJoxG8OoB67rWhEgkDLi95 X-Microsoft-Antispam-PRVS: X-Forefront-PRVS: 094700CA91 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN8PR12MB3459; 23:X1OADrAeW/HI04bHlOfnZO2wVO1J2gKh5+vEcra6b?= 7JzVXpSadID8K98XBfII+9myraZ7xS7ygv0/t43bEvGRdzc8hL26m03/CpW2RkuX48bGlI+6iBl/2NCtyr2s931FQEDaij8+ZTrNFS9Hm+yiobQeiujd038cUtQ80yUjxUw+xAtNEfRWDdQHE0lh/kWE+fFFVs+MZ9zsLn0h3Qh2yU+I1ULEBdhAM9bnCkmP3McPaLOi8JRBDrDLYNAxVQ0e/cyArOt0TimNHUnJIqX1BOLlCpB2ZRAOC2yPkRzPqzuLhriHYDMilXfe5NhJyVLHzHOHpjd7od/Rejm6nO0SqNNpvtWR0vr/6kRoQ7/a5JIUmb83TM/QcKvinKKlRlU2PwAezZqHqSVCOv9zTNv5IiiSsqxlNl0gsOdYhbas1JJ1N0nDE9KL/CspNM3Xw6RWOnm/wWs+hLyCUKglIazi0aUe5OkAymCtzqzHbIdjfBxWZREiBvDnScBi9DlMpng6mZMDJywuWywvWqs8nneDRTaDezP8WajYwQG/gsre9UbZGruUss+c9k8b27DCC4QRsAOEzlRAwS78ZJ3tTQDdf1x5kYWPOuOEZuWBzD9PrkBjcloILx/6Cyed8LgeCXxCIhzRTc+U/LYN4p82wLgzsbS877EvpZGvwdNYQn6sj37cqTqVLczy56GqEpYuyEy3RvCcD9ruXzzaYqQgKh5Jnik1MzhqjybFFdD4az9IRTPRzUmZ4h8gDhDYTCb/zNVCh3qv/Uy94RFogm2cT5MdzdhJaWfJH5FrkxrzSblxARbrsmq+yo4PQ9ZlrHEgB4g8euAMuqsbagECTBRr92ooQ+UXGza4r+FdV16mgAQcWRaZira287aR0zPUjvzm+UA3lDEbq0KRmAOEortbKfgGRT2kcs9+ExXEWQi4tWRkX7B5+tg12eVSBoK+qbYuIwwHHMpryi8GBWN4w4aWbzT3qAad22C1OPSuSjLO3otO85hKJPSY+UbM/Bq1sBkexSrHYqf4672geI29c79myQ+Cg== X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: yGiTZzzQpMpebX80J8tF7kJXrkvck9hpFKKCpJvbkesg/BOlb7HPnl7nkHHw5AQHubJ+U8PMNqjPa/UOVhTxjLXsvlaZ2xHvLjRbXhve+ZYyPMGjgMzvNq8i9raB6Sw5rf4qP/x2lsuEHu8302u+29vAzFnRcM0jY3M47izUB8OF9OIT3cwjcfwfZxTOYnOWECQplplsiko/P6h88+eAL5RWdRvM2PSaSqkh1DRkkSU7izGZQmyQzZHcc9Va+XAoxlXNAajUHqr5OpUbE8ouzIeKodfC2ZBvn4WkyYR7indgiaYJsUtvv3GnolRmcrL7XXR6Dl5++2f4OqDPvhP+XpTxfBwg1C/PkDPTr3RYapMLB1TNovYkjpsMnRkBSdNHKMKdmyLlEWAtTaa9AuBAeSHwqUNdy3AYK1meYipMoLQ= X-Microsoft-Exchange-Diagnostics: 1; BN8PR12MB3459; 20:NA8L7zqPWfUVWvVe9k9hnBt3U3jXbJOnn+XA/2X9Op7dikYhwo6IlKpmCqs4CN89puDjE5mmBBA53sMFt7DV+9r/pDmceV3Ti+I0GF7NTqwBctAujXjdLYGxU5TTMDOtK7S/PKQVk3I+jWyO56q9Qr27/VMAEcPJ+gmo0A7xPnGJVTpalEWWz7/gdoevq4g2o9zkL2uu4SpGxBkUkvTCZCB+JYuLOthBccJnwplwHdM+oADnV4Yd5f+YYbUKDHQI X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2019 14:45:46.4911 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 8c76e2c7-69e0-4017-7e1e-08d691c1f067 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXCHOV01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR12MB3459 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector1-amd-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=efzOyXnilIsXGIWVXvin+5TJwC3lip/UxVfajZfIFbU=; b=FSoXG5PmZ7xVt9dHt++VUkyhAbfHv0QAxFbTmWE6YoCM7wvy1upN2WSK4woEhVWSWymSYzE2rzqciSEV3BMfWR9pMEyB246JpvkKijGwmis/NVbLd5I9aXw89HBy7DEHlwmFjaLsYn345YsijwAvP83sPsFCw4Vikv3CCUfPs94= X-Mailman-Original-Authentication-Results: spf=softfail (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; lists.freedesktop.org; dkim=none (message not signed) header.d=none;lists.freedesktop.org; dmarc=permerror action=none header.from=amd.com; X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: intel-gfx@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, nikola.cornij@amd.com, manasi.d.navare@intel.com, David Francis Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP The function drm_dsc_pps_infoframe_pack only packed the payload portion of the infoframe. Change the input struct to the PPS payload to clarify the function's purpose and allow for drivers with their own handling of sdp. (e.g. drivers with their own struct for all SDP transactions) Signed-off-by: David Francis Reviewed-by: Harry Wentland --- drivers/gpu/drm/drm_dsc.c | 86 +++++++++++++++---------------- drivers/gpu/drm/i915/intel_vdsc.c | 2 +- include/drm/drm_dsc.h | 2 +- 3 files changed, 45 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/drm_dsc.c b/drivers/gpu/drm/drm_dsc.c index 9e675dd39a44..4ada4d4f59ac 100644 --- a/drivers/gpu/drm/drm_dsc.c +++ b/drivers/gpu/drm/drm_dsc.c @@ -38,42 +38,42 @@ void drm_dsc_dp_pps_header_init(struct drm_dsc_pps_infoframe *pps_sdp) EXPORT_SYMBOL(drm_dsc_dp_pps_header_init); /** - * drm_dsc_pps_infoframe_pack() - Populates the DSC PPS infoframe + * drm_dsc_pps_payload_pack() - Populates the DSC PPS payload * using the DSC configuration parameters in the order expected * by the DSC Display Sink device. For the DSC, the sink device * expects the PPS payload in the big endian format for the fields * that span more than 1 byte. * - * @pps_sdp: - * Secondary data packet for DSC Picture Parameter Set + * @pps_payload: + * DSC Picture Parameter Set * @dsc_cfg: * DSC Configuration data filled by driver */ -void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, +void drm_dsc_pps_payload_pack(struct drm_dsc_picture_parameter_set *pps_payload, const struct drm_dsc_config *dsc_cfg) { int i; /* Protect against someone accidently changing struct size */ - BUILD_BUG_ON(sizeof(pps_sdp->pps_payload) != + BUILD_BUG_ON(sizeof(*pps_payload) != DP_SDP_PPS_HEADER_PAYLOAD_BYTES_MINUS_1 + 1); - memset(&pps_sdp->pps_payload, 0, sizeof(pps_sdp->pps_payload)); + memset(pps_payload, 0, sizeof(*pps_payload)); /* PPS 0 */ - pps_sdp->pps_payload.dsc_version = + pps_payload->dsc_version = dsc_cfg->dsc_version_minor | dsc_cfg->dsc_version_major << DSC_PPS_VERSION_MAJOR_SHIFT; /* PPS 1, 2 is 0 */ /* PPS 3 */ - pps_sdp->pps_payload.pps_3 = + pps_payload->pps_3 = dsc_cfg->line_buf_depth | dsc_cfg->bits_per_component << DSC_PPS_BPC_SHIFT; /* PPS 4 */ - pps_sdp->pps_payload.pps_4 = + pps_payload->pps_4 = ((dsc_cfg->bits_per_pixel & DSC_PPS_BPP_HIGH_MASK) >> DSC_PPS_MSB_SHIFT) | dsc_cfg->vbr_enable << DSC_PPS_VBR_EN_SHIFT | @@ -82,7 +82,7 @@ void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, dsc_cfg->block_pred_enable << DSC_PPS_BLOCK_PRED_EN_SHIFT; /* PPS 5 */ - pps_sdp->pps_payload.bits_per_pixel_low = + pps_payload->bits_per_pixel_low = (dsc_cfg->bits_per_pixel & DSC_PPS_LSB_MASK); /* @@ -93,103 +93,103 @@ void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, */ /* PPS 6, 7 */ - pps_sdp->pps_payload.pic_height = cpu_to_be16(dsc_cfg->pic_height); + pps_payload->pic_height = cpu_to_be16(dsc_cfg->pic_height); /* PPS 8, 9 */ - pps_sdp->pps_payload.pic_width = cpu_to_be16(dsc_cfg->pic_width); + pps_payload->pic_width = cpu_to_be16(dsc_cfg->pic_width); /* PPS 10, 11 */ - pps_sdp->pps_payload.slice_height = cpu_to_be16(dsc_cfg->slice_height); + pps_payload->slice_height = cpu_to_be16(dsc_cfg->slice_height); /* PPS 12, 13 */ - pps_sdp->pps_payload.slice_width = cpu_to_be16(dsc_cfg->slice_width); + pps_payload->slice_width = cpu_to_be16(dsc_cfg->slice_width); /* PPS 14, 15 */ - pps_sdp->pps_payload.chunk_size = cpu_to_be16(dsc_cfg->slice_chunk_size); + pps_payload->chunk_size = cpu_to_be16(dsc_cfg->slice_chunk_size); /* PPS 16 */ - pps_sdp->pps_payload.initial_xmit_delay_high = + pps_payload->initial_xmit_delay_high = ((dsc_cfg->initial_xmit_delay & DSC_PPS_INIT_XMIT_DELAY_HIGH_MASK) >> DSC_PPS_MSB_SHIFT); /* PPS 17 */ - pps_sdp->pps_payload.initial_xmit_delay_low = + pps_payload->initial_xmit_delay_low = (dsc_cfg->initial_xmit_delay & DSC_PPS_LSB_MASK); /* PPS 18, 19 */ - pps_sdp->pps_payload.initial_dec_delay = + pps_payload->initial_dec_delay = cpu_to_be16(dsc_cfg->initial_dec_delay); /* PPS 20 is 0 */ /* PPS 21 */ - pps_sdp->pps_payload.initial_scale_value = + pps_payload->initial_scale_value = dsc_cfg->initial_scale_value; /* PPS 22, 23 */ - pps_sdp->pps_payload.scale_increment_interval = + pps_payload->scale_increment_interval = cpu_to_be16(dsc_cfg->scale_increment_interval); /* PPS 24 */ - pps_sdp->pps_payload.scale_decrement_interval_high = + pps_payload->scale_decrement_interval_high = ((dsc_cfg->scale_decrement_interval & DSC_PPS_SCALE_DEC_INT_HIGH_MASK) >> DSC_PPS_MSB_SHIFT); /* PPS 25 */ - pps_sdp->pps_payload.scale_decrement_interval_low = + pps_payload->scale_decrement_interval_low = (dsc_cfg->scale_decrement_interval & DSC_PPS_LSB_MASK); /* PPS 26[7:0], PPS 27[7:5] RESERVED */ /* PPS 27 */ - pps_sdp->pps_payload.first_line_bpg_offset = + pps_payload->first_line_bpg_offset = dsc_cfg->first_line_bpg_offset; /* PPS 28, 29 */ - pps_sdp->pps_payload.nfl_bpg_offset = + pps_payload->nfl_bpg_offset = cpu_to_be16(dsc_cfg->nfl_bpg_offset); /* PPS 30, 31 */ - pps_sdp->pps_payload.slice_bpg_offset = + pps_payload->slice_bpg_offset = cpu_to_be16(dsc_cfg->slice_bpg_offset); /* PPS 32, 33 */ - pps_sdp->pps_payload.initial_offset = + pps_payload->initial_offset = cpu_to_be16(dsc_cfg->initial_offset); /* PPS 34, 35 */ - pps_sdp->pps_payload.final_offset = cpu_to_be16(dsc_cfg->final_offset); + pps_payload->final_offset = cpu_to_be16(dsc_cfg->final_offset); /* PPS 36 */ - pps_sdp->pps_payload.flatness_min_qp = dsc_cfg->flatness_min_qp; + pps_payload->flatness_min_qp = dsc_cfg->flatness_min_qp; /* PPS 37 */ - pps_sdp->pps_payload.flatness_max_qp = dsc_cfg->flatness_max_qp; + pps_payload->flatness_max_qp = dsc_cfg->flatness_max_qp; /* PPS 38, 39 */ - pps_sdp->pps_payload.rc_model_size = + pps_payload->rc_model_size = cpu_to_be16(DSC_RC_MODEL_SIZE_CONST); /* PPS 40 */ - pps_sdp->pps_payload.rc_edge_factor = DSC_RC_EDGE_FACTOR_CONST; + pps_payload->rc_edge_factor = DSC_RC_EDGE_FACTOR_CONST; /* PPS 41 */ - pps_sdp->pps_payload.rc_quant_incr_limit0 = + pps_payload->rc_quant_incr_limit0 = dsc_cfg->rc_quant_incr_limit0; /* PPS 42 */ - pps_sdp->pps_payload.rc_quant_incr_limit1 = + pps_payload->rc_quant_incr_limit1 = dsc_cfg->rc_quant_incr_limit1; /* PPS 43 */ - pps_sdp->pps_payload.rc_tgt_offset = DSC_RC_TGT_OFFSET_LO_CONST | + pps_payload->rc_tgt_offset = DSC_RC_TGT_OFFSET_LO_CONST | DSC_RC_TGT_OFFSET_HI_CONST << DSC_PPS_RC_TGT_OFFSET_HI_SHIFT; /* PPS 44 - 57 */ for (i = 0; i < DSC_NUM_BUF_RANGES - 1; i++) - pps_sdp->pps_payload.rc_buf_thresh[i] = + pps_payload->rc_buf_thresh[i] = dsc_cfg->rc_buf_thresh[i]; /* PPS 58 - 87 */ @@ -198,35 +198,35 @@ void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, * are as follows: Min_qp[15:11], max_qp[10:6], offset[5:0] */ for (i = 0; i < DSC_NUM_BUF_RANGES; i++) { - pps_sdp->pps_payload.rc_range_parameters[i] = + pps_payload->rc_range_parameters[i] = ((dsc_cfg->rc_range_params[i].range_min_qp << DSC_PPS_RC_RANGE_MINQP_SHIFT) | (dsc_cfg->rc_range_params[i].range_max_qp << DSC_PPS_RC_RANGE_MAXQP_SHIFT) | (dsc_cfg->rc_range_params[i].range_bpg_offset)); - pps_sdp->pps_payload.rc_range_parameters[i] = - cpu_to_be16(pps_sdp->pps_payload.rc_range_parameters[i]); + pps_payload->rc_range_parameters[i] = + cpu_to_be16(pps_payload->rc_range_parameters[i]); } /* PPS 88 */ - pps_sdp->pps_payload.native_422_420 = dsc_cfg->native_422 | + pps_payload->native_422_420 = dsc_cfg->native_422 | dsc_cfg->native_420 << DSC_PPS_NATIVE_420_SHIFT; /* PPS 89 */ - pps_sdp->pps_payload.second_line_bpg_offset = + pps_payload->second_line_bpg_offset = dsc_cfg->second_line_bpg_offset; /* PPS 90, 91 */ - pps_sdp->pps_payload.nsl_bpg_offset = + pps_payload->nsl_bpg_offset = cpu_to_be16(dsc_cfg->nsl_bpg_offset); /* PPS 92, 93 */ - pps_sdp->pps_payload.second_line_offset_adj = + pps_payload->second_line_offset_adj = cpu_to_be16(dsc_cfg->second_line_offset_adj); /* PPS 94 - 127 are O */ } -EXPORT_SYMBOL(drm_dsc_pps_infoframe_pack); +EXPORT_SYMBOL(drm_dsc_pps_payload_pack); /** * drm_dsc_compute_rc_parameters() - Write rate control diff --git a/drivers/gpu/drm/i915/intel_vdsc.c b/drivers/gpu/drm/i915/intel_vdsc.c index 7702c5c8b3f2..223c7b6c6e78 100644 --- a/drivers/gpu/drm/i915/intel_vdsc.c +++ b/drivers/gpu/drm/i915/intel_vdsc.c @@ -885,7 +885,7 @@ static void intel_dp_write_dsc_pps_sdp(struct intel_encoder *encoder, drm_dsc_dp_pps_header_init(&dp_dsc_pps_sdp); /* Fill the PPS payload bytes as per DSC spec 1.2 Table 4-1 */ - drm_dsc_pps_infoframe_pack(&dp_dsc_pps_sdp, vdsc_cfg); + drm_dsc_pps_payload_pack(&dp_dsc_pps_sdp.pps_payload, vdsc_cfg); intel_dig_port->write_infoframe(encoder, crtc_state, DP_SDP_PPS, &dp_dsc_pps_sdp, diff --git a/include/drm/drm_dsc.h b/include/drm/drm_dsc.h index 4e55e37943d7..3bed1f119576 100644 --- a/include/drm/drm_dsc.h +++ b/include/drm/drm_dsc.h @@ -479,7 +479,7 @@ struct drm_dsc_pps_infoframe { } __packed; void drm_dsc_dp_pps_header_init(struct drm_dsc_pps_infoframe *pps_sdp); -void drm_dsc_pps_infoframe_pack(struct drm_dsc_pps_infoframe *pps_sdp, +void drm_dsc_pps_payload_pack(struct drm_dsc_picture_parameter_set *pps_sdp, const struct drm_dsc_config *dsc_cfg); int drm_dsc_compute_rc_parameters(struct drm_dsc_config *vdsc_cfg);