From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [Intel-gfx] [PATCH v2 3/5] drm/dsi: add missing DSI DCS commands Date: Tue, 5 Nov 2019 10:18:01 +0100 Message-ID: <20191105091801.GA1607358@ulmo> References: <20191028150047.22048-1-jani.nikula@intel.com> <20191028150047.22048-3-jani.nikula@intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1517368421==" Return-path: In-Reply-To: <20191028150047.22048-3-jani.nikula@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Jani Nikula Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1517368421== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="KsGdsel6WgEHnImy" Content-Disposition: inline --KsGdsel6WgEHnImy Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 28, 2019 at 05:00:45PM +0200, Jani Nikula wrote: > Update from the DCS specification. >=20 > Cc: Vandita Kulkarni > Signed-off-by: Jani Nikula > --- > include/video/mipi_display.h | 10 ++++++++++ > 1 file changed, 10 insertions(+) >=20 > diff --git a/include/video/mipi_display.h b/include/video/mipi_display.h > index 6b6390dfa203..928f8c4b6658 100644 > --- a/include/video/mipi_display.h > +++ b/include/video/mipi_display.h > @@ -79,7 +79,9 @@ enum { > enum { > MIPI_DCS_NOP =3D 0x00, > MIPI_DCS_SOFT_RESET =3D 0x01, > + MIPI_DCS_GET_COMPRESSION_MODE =3D 0x03, > MIPI_DCS_GET_DISPLAY_ID =3D 0x04, > + MIPI_DCS_GET_ERROR_COUNT_ON_DSI =3D 0x05, > MIPI_DCS_GET_RED_CHANNEL =3D 0x06, > MIPI_DCS_GET_GREEN_CHANNEL =3D 0x07, > MIPI_DCS_GET_BLUE_CHANNEL =3D 0x08, > @@ -94,6 +96,8 @@ enum { > MIPI_DCS_EXIT_SLEEP_MODE =3D 0x11, > MIPI_DCS_ENTER_PARTIAL_MODE =3D 0x12, > MIPI_DCS_ENTER_NORMAL_MODE =3D 0x13, > + MIPI_DCS_GET_IMAGE_CHECKSUM_RGB =3D 0x14, > + MIPI_DCS_GET_IMAGE_CHECKSUM_CT =3D 0x15, > MIPI_DCS_EXIT_INVERT_MODE =3D 0x20, > MIPI_DCS_ENTER_INVERT_MODE =3D 0x21, > MIPI_DCS_SET_GAMMA_CURVE =3D 0x26, > @@ -105,6 +109,7 @@ enum { > MIPI_DCS_WRITE_LUT =3D 0x2D, > MIPI_DCS_READ_MEMORY_START =3D 0x2E, > MIPI_DCS_SET_PARTIAL_AREA =3D 0x30, > + MIPI_DCS_SET_PARTIAL_COLUMNS =3D 0x31, > MIPI_DCS_SET_SCROLL_AREA =3D 0x33, > MIPI_DCS_SET_TEAR_OFF =3D 0x34, > MIPI_DCS_SET_TEAR_ON =3D 0x35, > @@ -114,7 +119,10 @@ enum { > MIPI_DCS_ENTER_IDLE_MODE =3D 0x39, > MIPI_DCS_SET_PIXEL_FORMAT =3D 0x3A, > MIPI_DCS_WRITE_MEMORY_CONTINUE =3D 0x3C, > + MIPI_DCS_SET_3D_CONTROL =3D 0x3D, > MIPI_DCS_READ_MEMORY_CONTINUE =3D 0x3E, > + MIPI_DCS_GET_3D_CONTROL =3D 0x3F, > + MIPI_DCS_SET_VSYNC_TIMING =3D 0x40, > MIPI_DCS_SET_TEAR_SCANLINE =3D 0x44, > MIPI_DCS_GET_SCANLINE =3D 0x45, > MIPI_DCS_SET_DISPLAY_BRIGHTNESS =3D 0x51, /* MIPI DCS 1.3 */ > @@ -126,7 +134,9 @@ enum { > MIPI_DCS_SET_CABC_MIN_BRIGHTNESS =3D 0x5E, /* MIPI DCS 1.3 */ > MIPI_DCS_GET_CABC_MIN_BRIGHTNESS =3D 0x5F, /* MIPI DCS 1.3 */ > MIPI_DCS_READ_DDB_START =3D 0xA1, > + MIPI_DCS_READ_PPS_START =3D 0xA2, > MIPI_DCS_READ_DDB_CONTINUE =3D 0xA8, > + MIPI_DCS_READ_PPS_CONTINUE =3D 0xA9, > }; > =20 > /* MIPI DCS pixel formats */ Okay, found a copy of DCS v1.4 and the above matches the specification, so: Reviewed-by: Thierry Reding Does it perhaps make sense to add comments about the version number that these were introduced with? Thierry --KsGdsel6WgEHnImy Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl3BPj8ACgkQ3SOs138+ s6Gl5A//WP8vPygrsK6Psc7SQq6VaGta/c0tpKaBj7cqSXtcdRVzNvTMnkKFi55a maJkt4ZFnC2QMOwy6QEaOBcu+bor4cOGadU6u5YbuU2NUq+dwCVm2GaaEntPCSXR zuc+uaTwGWK53krKcKyAYe1vl8GllKUQ8U7JCMi4hHBVTQAbbrxuMQlCkW/m20ba iaX9g/9ieUbEsnU5tTTakMr0cqUTQg8oYgBCHyjUGXEqIHKwdp8bN5mXv7836Npr gd7F463d1Fsv9vafx6Vi2xanFeR56V7vKgMD2av5H8xNfWoMRP/6RBbuET86+kwX M1vmY/Ju7PHM/clxlX0veHftq2m3E3r48p7yb3Pmua+0Scpz+6IZUfeQKrBqdXFv ng0WOA+GdPkjy8w3ZZcru/lribjC6ofTMxIVjMj2Ximj7lhrTqUXoRaNlwd6sXVp Zl5My5gVDvim9rpfm/NbtXpaVel2J8asxoXqIhigJPcoRMK0z/PGwjv8bwXKBBYM XhWoSLljIan3feXWXARYveETBLeOs8lX12gUdKRv2z3vuWTuUJcMRjfIdZ3Rf670 aABp3kHvyx0bzx4iMA2vmuRUITajQ7dI9pKjt2xIubczcBLAVsE1Alu1vbqYKgMo 2ldG3Da5cihUQhmOhJw3A1Gh6NOjS8rCQDxRJWLRsCLD6PBIwAg= =zDRI -----END PGP SIGNATURE----- --KsGdsel6WgEHnImy-- --===============1517368421== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============1517368421==--