[PATCH 0/7] DRM/Radeon Cleanup

Michael Witten mfwitten at gmail.com
Mon Sep 26 13:47:10 PDT 2011


On Wed, 21 Sep 2011 22:50:57 -0000, Michael Witten wrote:

> Whilst working on my failing Radeon GPU:
>
>   Subject: Re: Curious experiences with a Radeon on the fritz
>   Date: Wed, 21 Sep 2011 18:52:19 -0000
>   Message-ID: <c0efd8527ff34853a23f1d706be29c67-mfwitten at gmail.com>
>   http://lists.freedesktop.org/archives/dri-devel/2011-September/014506.html
>
> I made the following trivial improvements to the DRM/Radeon code
> along the way ([6] fixes a bug):
>
>   [1] DRM: comment: drm_crtc{=>_helper}_set_mode
>   [2] DRM: comment: halve -> half
>   [3] DRM: cleanup: Remove unused `gamma_size'
>   [4] DRM: comment: gdm_proc_lists -> drm_proc_lists
>   [5] DRM: cleanup: `size == 0' is never true
>   [6] DRM: bug: RADEON_DEBUGFS_MAX_{NUM_FILES => COMPONENTS}
>   [7] DRM: cleanup: Remove unsused `tmp'
>
> Here is the overall difference:
>
>  drivers/gpu/drm/drm_crtc_helper.c       |    2 +-
>  drivers/gpu/drm/drm_fb_helper.c         |    3 ---
>  drivers/gpu/drm/drm_irq.c               |    4 ++--
>  drivers/gpu/drm/drm_proc.c              |    2 +-
>  drivers/gpu/drm/radeon/radeon.h         |    2 +-
>  drivers/gpu/drm/radeon/radeon_bios.c    |    2 +-
>  drivers/gpu/drm/radeon/radeon_combios.c |    2 +-
>  drivers/gpu/drm/radeon/radeon_device.c  |   13 ++++++++-----
>  8 files changed, 15 insertions(+), 15 deletions(-)

When you get the chance, David, please apply all but [4] and [7].

  * Patch [4] is not really helpful, because it poorly changes a
    comment in what is deprecated, probably erroneous code anyway.

  * [7] was a mistake on my part.


More information about the dri-devel mailing list