From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C4881C636D4 for ; Fri, 3 Feb 2023 13:35:20 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 5043510E57F; Fri, 3 Feb 2023 13:35:20 +0000 (UTC) Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id EFE5210E57F; Fri, 3 Feb 2023 13:35:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1675431319; x=1706967319; h=date:from:to:cc:subject:message-id:references: mime-version:content-transfer-encoding:in-reply-to; bh=MLFA9sc6d6UkWzA6YjJyjSI+gD+WpHjmuvxAX7ugnKc=; b=CHHDLWQO7sk6XHhZW8XLkY+y04Mc40VKFop/9ARuoyMbtxsfV9CyqsV8 e22MH/t7l3bm530ueKaIBh4skDmoFVG7UzkfKElJ4ckhmgk1Q3tqyVdft B9+Y5ascE+0mGTQnFpDTPQMEPmwe6kZGoTfEjAGd2bMkv/w4ky4wcZhqZ a453iGx8QUlLaHp/5YgwfNx1K8peaJ8bgUFHirdJoZbOJ7ek9cpyXO63e 1O86tdbqFz+JwwP1dC+a/ZDM6P7elsInhyjgwEC1Ctixm+3qmL/dULMTJ TVBgNRS/ItgbzXsNrEhJLJCOkzCdf/mY7VWQPgm+4f/E1wU89D6LRIHiE w==; X-IronPort-AV: E=McAfee;i="6500,9779,10610"; a="316741957" X-IronPort-AV: E=Sophos;i="5.97,270,1669104000"; d="scan'208";a="316741957" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Feb 2023 05:35:18 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10609"; a="808362345" X-IronPort-AV: E=Sophos;i="5.97,270,1669104000"; d="scan'208";a="808362345" Received: from stinkpipe.fi.intel.com (HELO stinkbox) ([10.237.72.55]) by fmsmga001.fm.intel.com with SMTP; 03 Feb 2023 05:35:14 -0800 Received: by stinkbox (sSMTP sendmail emulation); Fri, 03 Feb 2023 15:35:14 +0200 Date: Fri, 3 Feb 2023 15:35:14 +0200 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Sebastian Wick Subject: Re: [PATCH 3/3] drm/connector: Deprecate split for BT.2020 in drm_colorspace enum Message-ID: References: <20230203020744.30745-1-joshua@froggi.es> <20230203020744.30745-3-joshua@froggi.es> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Patchwork-Hint: comment X-BeenThere: amd-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: amd-gfx@lists.freedesktop.org, Pekka Paalanen , Uma Shankar , dri-devel@lists.freedesktop.org, Joshua Ashton , Vitaly.Prosyak@amd.com Errors-To: amd-gfx-bounces@lists.freedesktop.org Sender: "amd-gfx" On Fri, Feb 03, 2023 at 01:59:07PM +0100, Sebastian Wick wrote: > On Fri, Feb 3, 2023 at 11:40 AM Ville Syrjälä > wrote: > > > > On Fri, Feb 03, 2023 at 02:07:44AM +0000, Joshua Ashton wrote: > > > Userspace has no way of controlling or knowing the pixel encoding > > > currently, so there is no way for it to ever get the right values here. > > > > That applies to a lot of the other values as well (they are > > explicitly RGB or YCC). The idea was that this property sets the > > infoframe/MSA/SDP value exactly, and other properties should be > > added to for use userspace to control the pixel encoding/colorspace > > conversion(if desired, or userspace just makes sure to > > directly feed in correct kind of data). > > I'm all for getting userspace control over pixel encoding but even > then the kernel always knows which pixel encoding is selected and > which InfoFrame has to be sent. Is there a reason why userspace would > want to control the variant explicitly to the wrong value? What do you mean wrong value? Userspace sets it based on what kind of data it has generated (or asked the display hardware to generate if/when we get explicit control over that part). -- Ville Syrjälä Intel