From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Jyri Sarha <jsarha@ti.com>
Cc: Daniel Vetter <daniel.vetter@intel.com>,
Tomi Valkeinen <tomi.valkeinen@ti.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v3 09/15] drm: tilcdc: Replace drm_fb_get_bpp_depth() with drm_format_plane_cpp()
Date: Fri, 10 Jun 2016 16:25:47 +0300 [thread overview]
Message-ID: <20160610132547.GL4329@intel.com> (raw)
In-Reply-To: <575ABD96.4080502@ti.com>
On Fri, Jun 10, 2016 at 04:16:06PM +0300, Jyri Sarha wrote:
> On 06/10/16 16:08, Tomi Valkeinen wrote:
> > On 10/06/16 15:48, Tomi Valkeinen wrote:
> >> On 10/06/16 15:29, Ville Syrjälä wrote:
> >>
> >>>> The HW does not support any kind of blending to black
> >>>> or to anything else.
> >>>
> >>> Right. Then formats with alpha shouldn't be advertized.
> >>
> >> I don't think tilcdc does. It comes from the DRM core. With universal
> >> planes this will be fixed as the driver can define the formats.
> >
> > Actually, Jyri told that modetest only reports "RG16 RG24 XR24", which
> > is correct. I think it did contain AR24 at some point, but maybe I
> > remember wrong. I have no idea where those come from, though, as they
> > are not defined in tilcdc (at least not in that form)...
> >
>
> That was on top of my atomic modeset patches. The current mainline
> version reports "XR24 AR24".
Hmm. Oh, I guess we should kill ARGB8888 from safe_modeset_formats[]
and drm_mode_legacy_fb_format(). That should do it I think.
--
Ville Syrjälä
Intel OTC
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-06-10 13:25 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-08 23:32 [PATCH v3 00/15] Centralize format information Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 01/15] drm: Move format-related helpers to drm_fourcc.c Laurent Pinchart
2016-06-09 8:36 ` Daniel Vetter
2016-06-09 9:54 ` [PATCH v3.1 " Laurent Pinchart
2016-06-09 10:03 ` Daniel Vetter
2016-06-08 23:32 ` [PATCH v3 02/15] drm: Centralize format information Laurent Pinchart
2016-06-09 8:52 ` Daniel Vetter
2016-06-09 12:23 ` Ville Syrjälä
2016-06-09 12:40 ` Daniel Vetter
2016-06-09 13:05 ` Ville Syrjälä
2016-06-09 13:29 ` Daniel Vetter
2016-06-09 14:13 ` Ville Syrjälä
2016-09-08 13:49 ` Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 03/15] drm: Implement the drm_format_*() helpers as drm_format_info() wrappers Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 04/15] drm: Use drm_format_info() in DRM core code Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 05/15] drm: WARN when calling drm_format_info() for an unsupported format Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 06/15] drm: msm: Replace drm_fb_get_bpp_depth() with drm_format_plane_cpp() Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 07/15] drm: sti: " Laurent Pinchart
2016-06-09 7:52 ` Vincent ABRIOU
2016-06-09 9:17 ` Laurent Pinchart
2016-06-09 12:10 ` Vincent ABRIOU
2016-06-08 23:32 ` [PATCH v3 08/15] drm: hdlcd: " Laurent Pinchart
2016-06-09 9:01 ` Liviu Dudau
2016-07-25 11:10 ` Liviu Dudau
2016-09-08 14:45 ` Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 09/15] drm: tilcdc: " Laurent Pinchart
2016-06-10 11:51 ` Tomi Valkeinen
2016-06-10 12:05 ` Ville Syrjälä
2016-06-10 12:08 ` Tomi Valkeinen
2016-06-10 12:23 ` Ville Syrjälä
2016-06-10 12:26 ` Tomi Valkeinen
2016-06-10 12:29 ` Ville Syrjälä
2016-06-10 12:48 ` Tomi Valkeinen
2016-06-10 13:08 ` Tomi Valkeinen
2016-06-10 13:16 ` Jyri Sarha
2016-06-10 13:25 ` Ville Syrjälä [this message]
2016-06-10 14:21 ` Daniel Vetter
2016-06-10 12:07 ` Laurent Pinchart
2016-06-10 12:08 ` [PATCH v3.1 " Laurent Pinchart
2016-06-10 12:21 ` Tomi Valkeinen
2016-06-08 23:32 ` [PATCH v3 10/15] drm: cirrus: " Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 11/15] drm: gma500: Replace drm_fb_get_bpp_depth() with drm_format_info() Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 12/15] drm: amdgpu: Replace drm_fb_get_bpp_depth() with drm_format_plane_cpp() Laurent Pinchart
2016-06-09 1:42 ` Michel Dänzer
2016-06-09 9:18 ` Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 13/15] drm: radeon: " Laurent Pinchart
2016-06-08 23:32 ` [PATCH v3 14/15] drm: vmwgfx: Replace drm_fb_get_bpp_depth() with drm_format_info() Laurent Pinchart
2016-06-27 17:51 ` Sinclair Yeh
2016-06-08 23:32 ` [PATCH v3 15/15] drm: Don't export the drm_fb_get_bpp_depth() function Laurent Pinchart
-- strict thread matches above, loose matches on Subject: below --
2016-06-08 23:30 [PATCH v3 00/15] Centralize format information Laurent Pinchart
2016-06-08 23:31 ` [PATCH v3 09/15] drm: tilcdc: Replace drm_fb_get_bpp_depth() with drm_format_plane_cpp() Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160610132547.GL4329@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=daniel.vetter@intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jsarha@ti.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=tomi.valkeinen@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.