Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
From: Sean Paul <sean-p7yTbzM4H96eqtR555YLDQ@public.gmane.org>
To: Bruce Wang <bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	jcrouse-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	abhinavk-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org,
	jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: [PATCH v3 0/7] drm/msm/dpu: Clean up dpu code
Date: Fri, 28 Sep 2018 03:42:54 -0400	[thread overview]
Message-ID: <20180928074254.GY72545@art_vandelay> (raw)
In-Reply-To: <20180924162228.180689-1-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>

On Mon, Sep 24, 2018 at 12:22:21PM -0400, Bruce Wang wrote:
> Removes uneeded checks and unused variables. Changes some
> function that do not need return values to void.
> 
> Bruce Wang (7):
>   drm/msm/dpu: Remove unneeded checks in dpu_plane.c
>   drm/msm/dpu: Clean up plane atomic disable/update
>   drm/msm/dpu: Remove unneeded checks in dpu_crtc.c
>   drm/msm/dpu: Change _dpu_crtc_power_enable to void
>   drm/msm/dpu: Change _dpu_crtc_vblank_enable_no_lock to void
>   drm/msm/dpu: Make dpu_plane_danger_signal_ctrl void
>   drm/msm/dpu: Make _dpu_plane_get_aspace void


Applied to dpu-staging.

Congrats on your first patches!
                                   .''.       
       .''.      .        *''*    :_\/_:     . 
      :_\/_:   _\(/_  .:.*_\/_*   : /\ :  .'.:.'.
  .''.: /\ :   ./)\   ':'* /\ * :  '..'.  -=:o:=-
 :_\/_:'.:::.    ' *''*    * '.\'/.' _\(/_'.':'.'
 : /\ : :::::     *_\/_*     -= o =-  /)\    '  *
  '..'  ':::'     * /\ *     .'/.\'.   '
                   *..*         :

Sean


> 
>  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c  | 253 ++++---------------
>  drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 280 +++-------------------
>  2 files changed, 70 insertions(+), 463 deletions(-)
> 
> -- 
> 2.19.0.444.g18242da7ef-goog
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

  parent reply	other threads:[~2018-09-28  7:42 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-09-24 16:22 [PATCH v3 0/7] drm/msm/dpu: Clean up dpu code Bruce Wang
     [not found] ` <20180924162228.180689-1-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:22   ` [PATCH v3 1/7] drm/msm/dpu: Remove unneeded checks in dpu_plane.c Bruce Wang
     [not found]     ` <20180924162228.180689-2-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:45       ` Jordan Crouse
2018-09-24 16:22   ` [PATCH v3 2/7] drm/msm/dpu: Clean up plane atomic disable/update Bruce Wang
     [not found]     ` <20180924162228.180689-3-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:45       ` Jordan Crouse
2018-09-24 16:22   ` [PATCH v3 3/7] drm/msm/dpu: Remove unneeded checks in dpu_crtc.c Bruce Wang
     [not found]     ` <20180924162228.180689-4-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:45       ` Jordan Crouse
2018-09-24 16:22   ` [PATCH v3 4/7] drm/msm/dpu: Change _dpu_crtc_power_enable to void Bruce Wang
     [not found]     ` <20180924162228.180689-5-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:46       ` Jordan Crouse
2018-09-26  8:10       ` Sean Paul
2018-09-26 21:28         ` [PATCH v4 4/7] drm/msm/dpu: Remove _dpu_crtc_power_enable Bruce Wang
2018-09-24 16:22   ` [PATCH v3 5/7] drm/msm/dpu: Change _dpu_crtc_vblank_enable_no_lock to void Bruce Wang
2018-09-24 16:22   ` [PATCH v3 6/7] drm/msm/dpu: Make dpu_plane_danger_signal_ctrl void Bruce Wang
2018-09-24 16:22   ` [PATCH v3 7/7] drm/msm/dpu: _dpu_plane_get_aspace returns Bruce Wang
     [not found]     ` <20180924162228.180689-8-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 16:47       ` Jordan Crouse
     [not found]         ` <20180924164729.GE10761-9PYrDHPZ2Orvke4nUoYGnHL1okKdlPRT@public.gmane.org>
2018-09-24 16:57           ` Bruce Wang
2018-09-24 17:45           ` [PATCH v3 7/7] drm/msm/dpu: Make _dpu_plane_get_aspace void Bruce Wang
     [not found]             ` <20180924174555.11494-1-bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-09-24 17:53               ` Jordan Crouse
2018-09-24 18:01               ` Jeykumar Sankaran
     [not found]                 ` <41c32e7d920f1e9d5d80d82cdd3a2067-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2018-09-25 13:56                   ` Bruce Wang
2018-09-25 21:10                   ` [PATCH v3 7/7] drm/msm/dpu: Revise _dpu_plane_get_aspace Bruce Wang
2018-09-28  7:42   ` Sean Paul [this message]
2018-09-28 20:41     ` [PATCH v3 0/7] drm/msm/dpu: Clean up dpu code Bruce Wang

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=20180928074254.GY72545@art_vandelay \
    --to=sean-p7ytbzm4h96eqtr555yldq@public.gmane.org \
    --cc=abhinavk-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=bzwang-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
    --cc=freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=jcrouse-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=jsanka-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org \
    --cc=linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=robdclark-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox