All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabien DESSENNE <fabien.dessenne@st.com>
To: "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>
Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Subject: RE: [PATCH] drm/crtc-helper: use drm_framebuffer flags
Date: Thu, 10 Jul 2014 10:01:15 +0200	[thread overview]
Message-ID: <68B811F07A63BA48BF0F49F7D822D4190101283EC6C9@SAFEX1MAIL2.st.com> (raw)
In-Reply-To: <1404218442-10980-1-git-send-email-fabien.dessenne@st.com>

Hi,
Can anyone review this patch ?
Thanks for your time.
Fabien

> -----Original Message-----
> From: Fabien DESSENNE [mailto:fabien.dessenne@st.com]
> Sent: mardi 1 juillet 2014 14:41
> To: dri-devel@lists.freedesktop.org
> Cc: Benjamin Gaignard; Vincent ABRIOU; Fabien DESSENNE
> Subject: [PATCH] drm/crtc-helper: use drm_framebuffer flags
> 
> The "flags" parameter of the DRM_IOCTL_MODE_ADDFB2 ioctl must be
> propagated and used by the driver.
> The only possible value of flags is DRM_MODE_FB_INTERLACED.
> 
> Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com>
> Reviewed-by: Benjamin GAIGNARD <benjamin.gaignard@st.com>
> ---
>  drivers/gpu/drm/drm_crtc_helper.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/drm_crtc_helper.c
> b/drivers/gpu/drm/drm_crtc_helper.c
> index 23500c0..5974489 100644
> --- a/drivers/gpu/drm/drm_crtc_helper.c
> +++ b/drivers/gpu/drm/drm_crtc_helper.c
> @@ -966,6 +966,7 @@ int drm_helper_mode_fill_fb_struct(struct
> drm_framebuffer *fb,
>  	drm_fb_get_bpp_depth(mode_cmd->pixel_format, &fb->depth,
>  				    &fb->bits_per_pixel);
>  	fb->pixel_format = mode_cmd->pixel_format;
> +	fb->flags = mode_cmd->flags;
> 
>  	return 0;
>  }
> --
> 1.9.1

  reply	other threads:[~2014-07-10  8:15 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-01 12:40 [PATCH] drm/crtc-helper: use drm_framebuffer flags Fabien DESSENNE
2014-07-10  8:01 ` Fabien DESSENNE [this message]
2014-07-22 15:40   ` Benjamin Gaignard
2014-07-22 15:40     ` Benjamin Gaignard
2014-07-22 22:37     ` Dave Airlie
2014-07-22 22:37       ` Dave Airlie
  -- strict thread matches above, loose matches on Subject: below --
2014-07-01 12:28 Fabien DESSENNE

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=68B811F07A63BA48BF0F49F7D822D4190101283EC6C9@SAFEX1MAIL2.st.com \
    --to=fabien.dessenne@st.com \
    --cc=benjamin.gaignard@linaro.org \
    --cc=dri-devel@lists.freedesktop.org \
    /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.