All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joonyoung Shim <jy0922.shim@samsung.com>
To: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>,
	linux-samsung-soc@vger.kernel.org
Cc: gustavo.padovan@collabora.co.uk, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 2/2] drm/exynos: cleanup exynos_drm_plane
Date: Fri, 24 Apr 2015 10:54:19 +0900	[thread overview]
Message-ID: <5539A24B.3080506@samsung.com> (raw)
In-Reply-To: <1429791146-4410-3-git-send-email-tjakobi@math.uni-bielefeld.de>

Hi Tobias,

On 04/23/2015 09:12 PM, Tobias Jakobi wrote:
> Remove the unused fields of struct exynos_drm_plane.
> 
> Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
> ---
>  drivers/gpu/drm/exynos/exynos_drm_drv.h | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.h b/drivers/gpu/drm/exynos/exynos_drm_drv.h
> index e12ecb5..c80331b 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_drv.h
> +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.h
> @@ -73,11 +73,6 @@ enum exynos_drm_output_type {
>   * @zpos: order of overlay layer(z position).
>   * @index_color: if using color key feature then this value would be used
>   *			as index color.
> - * @default_win: a window to be enabled.
> - * @color_key: color key on or off.
> - * @local_path: in case of lcd type, local path mode on or off.
> - * @transparency: transparency on or off.
> - * @activated: activated or not.
>   * @enabled: enabled or not.
>   * @resume: to resume or not.
>   *
> @@ -110,11 +105,6 @@ struct exynos_drm_plane {
>  	unsigned int zpos;
>  	unsigned int index_color;

How about remove also index_color?

>  
> -	bool default_win:1;
> -	bool color_key:1;
> -	bool local_path:1;
> -	bool transparency:1;
> -	bool activated:1;
>  	bool enabled:1;
>  	bool resume:1;
>  };
> 

Acked-by: Joonyoung Shim <jy0922.shim@samsung.com>

Thanks.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2015-04-24  1:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-23 12:12 drm/exynos: two small fixes (v2) Tobias Jakobi
2015-04-23 12:12 ` [PATCH 1/2] drm/exynos: mixer: cleanup pixelformat handling Tobias Jakobi
2015-04-24  1:48   ` Joonyoung Shim
2015-04-24  8:10     ` Tobias Jakobi
2015-04-24 18:14       ` Gustavo Padovan
2015-04-25 18:08         ` Tobias Jakobi
2015-04-27  6:33       ` Joonyoung Shim
2015-04-27 12:06         ` Tobias Jakobi
2015-04-23 12:12 ` [PATCH 2/2] drm/exynos: cleanup exynos_drm_plane Tobias Jakobi
2015-04-24  1:54   ` Joonyoung Shim [this message]
2015-04-24 18:13     ` Gustavo Padovan

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=5539A24B.3080506@samsung.com \
    --to=jy0922.shim@samsung.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gustavo.padovan@collabora.co.uk \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=tjakobi@math.uni-bielefeld.de \
    /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.