Date:   Mon, 22 Nov 2021 11:52:47 +0200
From:   Pekka Paalanen <ppaalanen@gmail.com>
To:     Hector Martin <marcan@marcan.st>
Cc:     Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Thomas Zimmermann <tzimmermann@suse.de>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        dri-devel@lists.freedesktop.org,
        Alyssa Rosenzweig <alyssa@rosenzweig.io>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/3] drm/format-helper: Add
 drm_fb_xrgb8888_to_xrgb2101010_dstclip()
Message-ID: <20211122115247.257f30fa@eldfell>
In-Reply-To: <20211117145829.204360-3-marcan@marcan.st>
References: <20211117145829.204360-1-marcan@marcan.st>
        <20211117145829.204360-3-marcan@marcan.st>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/eeC4l4N88GUSbcvOmvSRbh=";
 protocol="application/pgp-signature"; micalg=pgp-sha256
Precedence: bulk

--Sig_/eeC4l4N88GUSbcvOmvSRbh=
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

On Wed, 17 Nov 2021 23:58:28 +0900
Hector Martin <marcan@marcan.st> wrote:

> Add XRGB8888 emulation support for devices that can only do XRGB2101010.
>=20
> This is chiefly useful for simpledrm on Apple devices where the
> bootloader-provided framebuffer is 10-bit, which already works fine with
> simplefb. This is required to make simpledrm support this too.
>=20
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  drivers/gpu/drm/drm_format_helper.c | 64 +++++++++++++++++++++++++++++
>  include/drm/drm_format_helper.h     |  4 ++
>  2 files changed, 68 insertions(+)

Hi Hector,

I'm curious, since the bootloader seems to always set up a 10-bit mode,
is there a reason for it that you can guess? Is the monitor in WCG or
even HDR mode?


Thanks,
pq

--Sig_/eeC4l4N88GUSbcvOmvSRbh=
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAmGbaG8ACgkQI1/ltBGq
qqdRYQ//bM6tFPM2B6NM5OF0Pg8dajkciv+81+kA7QV1w7FrexCQS6acTPg9vtyX
J8EySl2TIx9XnBy5BQhp92iUWFuv2lex4/knF+7ZMKtrwAz4Oq33OgD8XKJ75gaq
AOYJCByth0q6Em/KG6DNCSE8yPyKGGQx5OAqTaRVqXTegsDHq9HtNJL300guTyPq
9YLkjrpp2j8WiysutvcpnfY4WvCqNdI5/TY9FynD3e3QkEEBnH9tGPv7HXIHQToV
J2kesFkXNF8nRYkGdaR05lEgfXsuWwLRZvH7IcAyKJr5dIDVY4dnUu/XhTBWr27X
mzyHxNi0/ZrSS5o+atrMVKqSPWdhvek0ce5ASx3FSN0nKNPIp3F+QKOKvocqOUxx
s3UOvwcgvHtB51h0RZBZwm/cAoSRvRraxgCHFkYL4Zb349YiYrcFpLxAEhK6Ujpt
6R70BAyVbtpDsWwIbBNk2T+uhNhafO6TegcWNp9cySSaowtqQn+BLSGpp6t92PAB
xqLrfOXRKjVGg9YmvINUP9bgy/HlhR4Mg6lT9AYe83vnoL+I4GvS98uWdibqigWt
cTXlQcVYKesq55GJ43YLhbbyjcj5P8ODXWmZoTQuYizh1a90VFj+eKQ2gzrnK8Kd
rVAkvy63wmFPgDLZPfSjP9WKAMUN7Z91x+kNa+SVw7toojTuKVQ=
=8F6L
-----END PGP SIGNATURE-----

--Sig_/eeC4l4N88GUSbcvOmvSRbh=--