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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 A292CC5478C for ; Fri, 1 Mar 2024 14:47:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=dRJ/nSp/JssSVoCV0hIBCYKnNmnjvvjwQm9DHRla5Yk=; b=gKLn+bQUfgVuqF u7kMeb6zJn+qv92fzsom/FjAylZzKf0yHqTrRTyi44A9xpZ1U+FpxqmY4fK+KqzQsuXuIF7Kf5vi+ Av2Q7gbPZRnnyWFM/GvJwguZSGs8CVOSQczfCWo4gYNUUshLNCiJU1xGJoZMXTWSSBHzlUlbn8pEk 0vus8j73aeS8KFtdCGmtccv4Dig4LuqDa6cNEQCXv4IZBn3B7pg3/+ph0ocLHwK2VxLD3gy55ws3/ KZ639pDiENgHWpBo9Pt/BHhlZkOocPqGNd54w8zrXlTKpRGNuc7KRhyaAb5ikiNayNx4d4Rbt2Bpy v/f9DXmVGjby22ufelVw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rg4AX-00000000mRQ-2xAb; Fri, 01 Mar 2024 14:47:25 +0000 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rg2jX-00000000ME0-03pn for linux-arm-kernel@lists.infradead.org; Fri, 01 Mar 2024 13:15:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1709298925; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=j6JDdzMRW33EgNy8WVE2rwXJul+k7YILcoZp2XURKLI=; b=iI8lTj7syOyTuYUrra2JC9Ixh25zlm6lpJ5/XDSxvHHZ/6ckDatltHl+o0x+DBKt8i4e14 GWgGHG02FIHk3CvRUBtRNIt2Pb3jLghh+3m1ZOV1vVbydisna2iAiG8iA+MKLF1Y12RhUs cSNRD4SkeYAhCd1c/P3E/TVKYPX2EQo= Received: from mail-lf1-f71.google.com (mail-lf1-f71.google.com [209.85.167.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-680-jMbDxgq_NT2WEFJEBVqrwg-1; Fri, 01 Mar 2024 08:15:23 -0500 X-MC-Unique: jMbDxgq_NT2WEFJEBVqrwg-1 Received: by mail-lf1-f71.google.com with SMTP id 2adb3069b0e04-5131d042b1aso1634262e87.0 for ; Fri, 01 Mar 2024 05:15:23 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1709298922; x=1709903722; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=j6JDdzMRW33EgNy8WVE2rwXJul+k7YILcoZp2XURKLI=; b=a8ZSLyxxtjibg+zO65ziVaJ5KnIqABIjdM/iZzHAPEv8/D0KbbtnVgKBhZ8q31fmnB dY84F26FNncD1LdNehKtQEM10TWiHR3CyVGy4uyXl5a9h3QMbq2ZSbZkzjqVjd1S/ScT h0IKk2cE1g2SzfFz40Js2X+U4nPO/saEWyef11g1xIa/8u0uGmHGT7RxHf743WEaA+Mh E3lI9PHviDQGF5wvZx7KxPSdXIJSueXsyuhL3tj0by7XANp8N72n1N71whkfra7IqdgK mI6mbq/SbrNtdwH5Lmcfeyc+1fYDN5uX5soCaC6zWoJMLqRDSAHg/pzmHcr1VB3dih58 kTjA== X-Forwarded-Encrypted: i=1; AJvYcCVsYstoGRrlGd6Pud2JrZ+5oPld8OeO1z/MnPTVJKdgS3Hsp2P4gTerP/xsXtED4bqLsuK/HL5wrLorZY/m/oMF8NKIcKUZQND3faKG8x54T5X61kg= X-Gm-Message-State: AOJu0YxNrtJC5AMDF8kopgOt4p8tXX3zI9D0i2C5/SC+G7gasNgEo2qQ 3ZJN85S0zs9fGUmdVRn9Nu1Ek6Yzqwn+iWjAnRC6oum42I8e9URWGFV1E0H79r9Okvr69PuED6d JiV0hSi/WyLgQ8hgh8y/BS5MSuw5oAsS92Oydhq7xMlOt3hINpbomLWQ7uba9G8vTcex+3l+P X-Received: by 2002:ac2:599b:0:b0:512:f6d3:9998 with SMTP id w27-20020ac2599b000000b00512f6d39998mr1223724lfn.17.1709298922223; Fri, 01 Mar 2024 05:15:22 -0800 (PST) X-Google-Smtp-Source: AGHT+IFwlzt5HlgK0WAcZyh2ZsOJe95EyGGHjhxje8QxMH4nFVP4Pvs9Hg/dbyzZ0Awcz9X/EGacJg== X-Received: by 2002:ac2:599b:0:b0:512:f6d3:9998 with SMTP id w27-20020ac2599b000000b00512f6d39998mr1223689lfn.17.1709298921881; Fri, 01 Mar 2024 05:15:21 -0800 (PST) Received: from toolbox ([2001:9e8:89a0:c500:c65:1f3a:8c08:2a1d]) by smtp.gmail.com with ESMTPSA id bw1-20020a0560001f8100b0033db0c866f7sm4639850wrb.11.2024.03.01.05.15.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 05:15:21 -0800 (PST) Date: Fri, 1 Mar 2024 14:15:19 +0100 From: Sebastian Wick To: Maxime Ripard Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Jonathan Corbet , Sandy Huang , Heiko =?iso-8859-1?Q?St=FCbner?= , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Hans Verkuil , Ville =?iso-8859-1?Q?Syrj=E4l=E4?= , dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-sunxi@lists.linux.dev, Dave Stevenson Subject: Re: [PATCH v7 21/36] drm/connector: hdmi: Add Broadcast RGB property Message-ID: <20240301131519.GA10491@toolbox> References: <20240222-kms-hdmi-connector-state-v7-0-8f4af575fce2@kernel.org> <20240222-kms-hdmi-connector-state-v7-21-8f4af575fce2@kernel.org> <20240229194726.GB166694@toolbox> <20240301-light-impressive-grasshopper-adabeb@houat> <20240301112941.GE166694@toolbox> <20240301-loyal-cornflower-oxpecker-83ed59@houat> MIME-Version: 1.0 In-Reply-To: <20240301-loyal-cornflower-oxpecker-83ed59@houat> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Disposition: inline X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240301_051527_137603_E5A722E6 X-CRM114-Status: GOOD ( 30.73 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Mar 01, 2024 at 01:12:02PM +0100, Maxime Ripard wrote: > On Fri, Mar 01, 2024 at 12:29:41PM +0100, Sebastian Wick wrote: > > On Fri, Mar 01, 2024 at 11:30:56AM +0100, Maxime Ripard wrote: > > > On Thu, Feb 29, 2024 at 08:47:26PM +0100, Sebastian Wick wrote: > > > > > @@ -1708,6 +1731,39 @@ EXPORT_SYMBOL(drm_connector_attach_dp_subconnector_property); > > > > > /** > > > > > * DOC: HDMI connector properties > > > > > * > > > > > + * Broadcast RGB (HDMI specific) > > > > > + * Indicates the Quantization Range (Full vs Limited) used. The color > > > > > + * processing pipeline will be adjusted to match the value of the > > > > > + * property, and the Infoframes will be generated and sent accordingly. > > > > > + * > > > > > + * This property is only relevant if the HDMI output format is RGB. If > > > > > + * it's one of the YCbCr variant, it will be ignored and the output will > > > > > + * use a limited quantization range. > > > > > > > > Uh, maybe just say that the quantization range is selected automatically > > > > in case a YCbCr output format is in use. I'm not sure every YCbCr > > > > variant requires limited and even if it does, new formats could change > > > > this. > > > > > > I documented what i915 is doing: > > > https://elixir.bootlin.com/linux/latest/source/drivers/gpu/drm/i915/display/intel_hdmi.c#L2143 > > > > Sure, this is one valid strategy for the automatic behavior of YCbCr. > > Drivers could also always send an InfoFrame to ensure full range where > > possible. The point here is that this property shall not affect YCbCr > > output formats! > > > > Maybe it's even better to say "driver specific" instead of "automatic". > > Honestly, I'm not sure what you want from me here. Ville and you > insisted on the previous version to document what i915 is doing and to > follow whatever the behaviour was, and that we shouldn't spend time > improving the property. Fine, I did that. > > But now, you want me to ... improve the property? The property has a clear scope: quantization range for RGB output formats. What Intel does with things that are not in scope of the property is irrelevant. This isn't improving the property either but documenting the scope of the property. Sorry if this seems arbitrary but these details are so important to get right because the interactions between all the different things is already majorly broken. > Maxime > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel