All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carlos Rafael Giani <dv@pseudoterminal.org>
To: Yuqing Zhu <carol.zhu@nxp.com>,
	"meta-freescale@yoctoproject.org"
	<meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-arm][PATCH 6/9] gstreamer1.0-plugins-bad: Change wayland default res to 1024x768
Date: Mon, 1 Feb 2016 08:26:10 +0100	[thread overview]
Message-ID: <56AF0892.40308@pseudoterminal.org> (raw)
In-Reply-To: <AM4PR04MB1635D541815779A6EFB245A7E9DB0@AM4PR04MB1635.eurprd04.prod.outlook.com>

I'd say this is OK then. It doesn't break anything and really is i.MX 
specific.

On 2016-01-29 07:41, Yuqing Zhu wrote:
> Hi Carlos,
>     
>        As in our release, the default display is full screen in LVDS, and 1024x768 resolution will be better in testing for our released platform.
>
>
>
> B.R.
> Yuqing Zhu
>
>
> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-bounces@yoctoproject.org] On Behalf Of Carlos Rafael Giani
> Sent: Wednesday, January 27, 2016 8:24 PM
> To: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-arm][PATCH 6/9] gstreamer1.0-plugins-bad: Change wayland default res to 1024x768
>
> Could you add an explanation to the commit message why you increase the default resolution?
>
> On 01/27/2016 10:41 AM, Yuqing Zhu wrote:
>> Change wayland backend default window resolution from 320x240 to
>> 1024x768.
>>
>> Signed-off-by: Yuqing Zhu <b54851@freescale.com>
>> ---
>>    ...in-Change-wayland-default-res-to-1024x768.patch | 41 ++++++++++++++++++++++
>>    .../gstreamer/gstreamer1.0-plugins-bad_%.bbappend  |  1 +
>>    2 files changed, 42 insertions(+)
>>    create mode 100755
>> recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0007-glplugin-Ch
>> ange-wayland-default-res-to-1024x768.patch
>>
>> diff --git
>> a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0007-glplugin-
>> Change-wayland-default-res-to-1024x768.patch
>> b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0007-glplugin-
>> Change-wayland-default-res-to-1024x768.patch
>> new file mode 100755
>> index 0000000..49a0a9d
>> --- /dev/null
>> +++ b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad/0007-glplu
>> +++ gin-Change-wayland-default-res-to-1024x768.patch
>> @@ -0,0 +1,41 @@
>> +From decc71a707d3b3596888d30a4c7ca25b57a686b2 Mon Sep 17 00:00:00
>> +2001
>> +From: Jian <Jian.Li@freescale.com>
>> +Date: Tue, 17 Nov 2015 14:39:07 +0800
>> +Subject: [PATCH 2/3] [glplugin] Change wayland default res to
>> +1024x768
>> +MIME-Version: 1.0
>> +Content-Type: text/plain; charset=UTF-8
>> +Content-Transfer-Encoding: 8bit
>> +
>> +Change wayland backend default window resolution from 320x240 to
>> +1024x768
>> +
>> +Upstream-Status: Inappropriate [i.MX specific]
>> +
>> +Signed-off-by: Jian <Jian.Li@freescale.com>
>> +---
>> + gst-libs/gst/gl/wayland/gstglwindow_wayland_egl.c |    4 ++--
>> + 1 file changed, 2 insertions(+), 2 deletions(-)
>> +
>> +diff --git a/gst-libs/gst/gl/wayland/gstglwindow_wayland_egl.c
>> +b/gst-libs/gst/gl/wayland/gstglwindow_wayland_egl.c
>> +index b34ad42..b557daa 100644
>> +--- a/gst-libs/gst/gl/wayland/gstglwindow_wayland_egl.c
>> ++++ b/gst-libs/gst/gl/wayland/gstglwindow_wayland_egl.c
>> +@@ -305,13 +305,13 @@ create_surfaces (GstGLWindowWaylandEGL * window_egl)
>> +   if (window_egl->window.window_width > 0)
>> +     width = window_egl->window.window_width;
>> +   else
>> +-    width = 320;
>> ++    width = 1024;
>> +   window_egl->window.window_width = width;
>> +
>> +   if (window_egl->window.window_height > 0)
>> +     height = window_egl->window.window_height;
>> +   else
>> +-    height = 240;
>> ++    height = 768;
>> +   window_egl->window.window_height = height;
>> +
>> +   if (!window_egl->window.native) {
>> +--
>> +1.7.9.5
>> +
>> diff --git
>> a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
>> b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
>> index b290423..40be87a 100644
>> --- a/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
>> +++ b/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_%.bbappend
>> @@ -16,6 +16,7 @@ IMX_PATCHES = " file://0001-PATCH-install-gstaggregator-and-gstvideoaggregator-h
>>                    file://0004-glplugin-Add-directviv-to-glimagesink-to-improve-playback-performance.patch \
>>                    file://0005-glplugin-support-video-crop-for-glimagesink.patch \
>>                    
>> file://0006-glplugin-Support-fb-backend-for-gl-plugins.patch \
>> +
>> + file://0007-glplugin-Change-wayland-default-res-to-1024x768.patch \
>>    "
>>    
>>    SRC_URI_append_mx6 = "${IMX_PATCHES}"
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale



  reply	other threads:[~2016-02-01  7:26 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-27  9:41 [meta-fsl-arm][PATCH 0/9] Add patches for gstreamer1.0-plugins-bad 1.6 Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 1/9] gstreamer1.0-plugins-bad: Modify the videoparsers rank down to avoid link them in Yuqing Zhu
2016-01-27 11:17   ` Carlos Rafael Giani
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 2/9] gstreamer1.0-plugins-bad: Fix for gl plugin not built in wayland backend Yuqing Zhu
2016-01-27 11:21   ` Carlos Rafael Giani
2016-01-27 12:05     ` Otavio Salvador
2016-01-28  7:29     ` Yuqing Zhu
2016-01-28 11:52       ` Otavio Salvador
2016-02-01  6:54         ` Yuqing Zhu
2016-02-01  7:25           ` Carlos Rafael Giani
2016-02-01  9:38             ` Yuqing Zhu
2016-02-01 12:43           ` Otavio Salvador
2016-02-02 13:57             ` Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 3/9] gstreamer1.0-plugins-bad: Add directviv to glimagesink to improve playback performance Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 4/9] gstreamer1.0-plugins-bad: Support video crop for glimagesink Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 5/9] gstreamer1.0-plugins-bad: Support fb backend for gl plugins Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 6/9] gstreamer1.0-plugins-bad: Change wayland default res to 1024x768 Yuqing Zhu
2016-01-27 12:24   ` Carlos Rafael Giani
2016-01-29  6:41     ` Yuqing Zhu
2016-02-01  7:26       ` Carlos Rafael Giani [this message]
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 7/9] gstreamer1.0-plugins-bad: Add fps print in glimagesink Yuqing Zhu
2016-01-27 12:23   ` Carlos Rafael Giani
2016-01-28  7:22     ` Yuqing Zhu
2016-01-28 17:10       ` Pushpal Sidhu
2016-01-29  6:28         ` Yuqing Zhu
2016-01-27  9:41 ` [meta-fsl-arm][PATCH 8/9] gstreamer1.0-plugins-bad: Fix gleffects fisheye shader compile error Yuqing Zhu
2016-01-27 12:11   ` Carlos Rafael Giani
2016-01-28  7:25     ` Yuqing Zhu
2016-02-01  7:27       ` Carlos Rafael Giani
2016-02-01  9:25         ` Yuqing Zhu
2016-01-27  9:42 ` [meta-fsl-arm][PATCH 9/9] gstreamer1.0-plugins-bad: Roll-back h264parse rank Yuqing Zhu

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=56AF0892.40308@pseudoterminal.org \
    --to=dv@pseudoterminal.org \
    --cc=carol.zhu@nxp.com \
    --cc=meta-freescale@yoctoproject.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.