Re: [PATCH 2/4] drm/exynos: remove unused codes in hdmi and mixer driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 05, 2012 at 07:53:43PM +0900, Joonyoung Shim wrote:
> On 04/05/2012 07:38 PM, Semwal, Sumit wrote:
> >Hi Joonyoung,
> >
> >On Thu, Apr 5, 2012 at 3:53 PM, Joonyoung Shim<jy0922.shim@xxxxxxxxxxx>  wrote:
> >>Some defines and members in struct mixer_context aren't used, remove
> >>them.
> >>
> >>Signed-off-by: Joonyoung Shim<jy0922.shim@xxxxxxxxxxx>
> >>Signed-off-by: Kyungmin Park<kyungmin.park@xxxxxxxxxxx>
> >>---
> >>  drivers/gpu/drm/exynos/exynos_hdmi.c  |    1 -
> >>  drivers/gpu/drm/exynos/exynos_mixer.c |   21 +++++++++------------
> >>  2 files changed, 9 insertions(+), 13 deletions(-)
> >>
> >>diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c
> >>index 0a71317..340424f 100644
> >>--- a/drivers/gpu/drm/exynos/exynos_hdmi.c
> >>+++ b/drivers/gpu/drm/exynos/exynos_hdmi.c
> >>@@ -40,7 +40,6 @@
> >>
> >>  #include "exynos_hdmi.h"
> >>
> >>-#define HDMI_OVERLAY_NUMBER    3
> >>  #define MAX_WIDTH              1920
> >>  #define MAX_HEIGHT             1080
> >>  #define get_hdmi_context(dev)  platform_get_drvdata(to_platform_device(dev))
> >>diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c b/drivers/gpu/drm/exynos/exynos_mixer.c
> >>index 495a7af..563092e 100644
> >>--- a/drivers/gpu/drm/exynos/exynos_mixer.c
> >>+++ b/drivers/gpu/drm/exynos/exynos_mixer.c
> >>@@ -37,7 +37,8 @@
> >>  #include "exynos_drm_drv.h"
> >>  #include "exynos_drm_hdmi.h"
> >>
> >>-#define HDMI_OVERLAY_NUMBER    3
> >>+#define MIXER_WIN_NR           3
> >>+#define MIXER_DEFAULT_WIN      0
> >Patch description says 'remove unused codes...', but looks like you're
> >adding more codes here? I think you should do the addition / deletion
> >in separate patches.
> 
> "remove unused codes" is incorrect a bit, but please think this is just code
> cleanup to remove unnecessary codes.

Commit messages that flat-out lie in this way are bad. You need to at
least mention what additional stuff you're doing in the commit message,
besides removing unused code. Or split the patch up as Sumit suggested.

Cheers, Daniel
-- 
Daniel Vetter
Mail: daniel@xxxxxxxx
Mobile: +41 (0)79 365 57 48
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux